fix: phpstan level 5 errors #84

[Morgy93]

This pull request primarily includes refactoring and minor behavior adjustments across several files to improve code clarity, error handling, and dependency management. The most important changes involve relaxing static analysis requirements, refining command return values, and cleaning up unused dependencies.

Static Analysis and CI Adjustments:

• Lowered the PHPStan static analysis level from 6 to 5 in phpstan.neon, making static analysis less strict.
• Removed continue-on-error: true from the PHPStan GitHub Actions workflow, so the workflow will now fail on PHPStan errors.

Command and Return Value Handling:

• Updated TokensCommand.php and InspectorCommand.php to return null instead of Cli::RETURN_FAILURE in some cases, aligning return values with expected behaviors and possibly improving error handling. [1] [2] [3]

Dependency and Constructor Cleanup:

• Removed unused dependencies from constructors in ThemePath and CompatibilityChecker, simplifying these classes and reducing coupling. [1] [2]

Logic and Behavior Refinements:

• Changed StaticContentCleaner to return a boolean based only on files being cleaned, not on the existence of static files, clarifying its intent.
• Added a guard in InspectorHints to skip block hint injection when not enabled, preventing unnecessary processing.

Miscellaneous Cleanup:

• Removed the unused PACKAGE_NAME constant from VersionCommand.php.