feat: Improve prefer_sliver_prefix to prefer_to_include_sliver_in_name

[k-nkmr]

🔗 Related Issues

• closes Improve prefer_sliver_prefix to prefer_to_include_sliver_in_name #36

🙌 What's Done

• Improve prefer_sliver_prefix to prefer_to_include_sliver_in_name
• update test

✍️ What's Not Done

• none

🖼️ Image Differences

Before	After
image	image

🤼 Desired Review Method

• Correction Commit
• Pair programming

Note

It is possible that a reviewer's will may cause a method to be implemented that is not selected.

📝 Additional Notes

Pre-launch Checklist

• I have reviewed my own code.
• I have added tests that prove my fix is effective or that my feature works.
• I updated/added relevant documentation (doc comments with ///).

[naipaka]

Thank you for the changes to the README.md as well!🚀

[naipaka]

The evaluation of returnSliverWidget could be done earlier, allowing for an early return if necessary!

Similarly, the checks for className and constructorNames could also be implemented as early returns, potentially reducing unnecessary member access.

Suggested change

	final className = node.name.lexeme;
	final constructorNames = node.members
	.whereType<ConstructorDeclaration>()
	.map((constructor) => constructor.name?.lexeme)
	.whereType<String>();
	final hasSliverInClassOrConstructor = className.contains('Sliver') ||
	constructorNames.any(
	(constructorName) =>
	constructorName.toLowerCase().contains('sliver'),
	);
	if (returnsSliverWidget && !hasSliverInClassOrConstructor) {
	reporter.atNode(node, _code);
	}
	if (!returnsSliverWidget) {
	return;
	}
	final className = node.name.lexeme;
	if (className.contains('Sliver')) {
	return;
	}
	final constructorNames = node.members
	.whereType<ConstructorDeclaration>()
	.map((constructor) => constructor.name?.lexeme)
	.nonNulls;
	final hasSliverConstructor = constructorNames.any(
	(name) => name.toLowerCase().contains('sliver'),
	);
	if (hasSliverConstructor) {
	return;
	}
	reporter.atNode(node, _code);

[naipaka]

I'd like you to write a test for the positive case as well, demonstrating that no warning is issued when the class name contains 'Sliver' and the named constructor includes 'sliver'. This would show the correct behavior in a valid scenario.