[cleanup] Add unused public, remove unused methods and ignore rest

Author: TomasVotruba

composer.json

@@ -59,6 +59,7 @@
         "symplify/phpstan-rules": "^14.9.3",
         "symplify/vendor-patches": "^11.5",
         "tomasvotruba/class-leak": "^2.1",
+        "tomasvotruba/unused-public": "^2.1",
         "tracy/tracy": "^2.11"
     },
     "replace": {

phpstan.neon

@@ -14,7 +14,6 @@ parameters:
 
     # requires exact closure types
     checkMissingCallableSignature: true
-
     treatPhpDocTypesAsCertain: false
 
     paths:
@@ -47,6 +46,12 @@ parameters:
         narrow_param: true
         narrow_return: true
 
+    # see https://github.com/TomasVotruba/unused-public
+    unused_public:
+        methods: true
+        properties: true
+        constants: true
+
     ignoreErrors:
         # required generics on interface array, not helpful
         -
@@ -413,3 +418,18 @@ parameters:
         -
             identifier: method.parentMethodFinalByPhpDoc
             path: rules/TypeDeclaration/Rector/StmtsAwareInterface/DeclareStrictTypesRector.php
+
+        # deprecated
+        -
+            identifier: public.method.unused
+            paths:
+                - rules/Transform/ValueObject/ClassMethodReference.php
+                - rules/CodeQuality/ValueObject/KeyAndExpr.php
+
+        -
+            identifier: public.classConstant.unused
+            paths:
+                - rules/TypeDeclaration/Rector/StmtsAwareInterface/IncreaseDeclareStrictTypesRector.php
+                - rules/Php55/Rector/String_/StringClassNameToClassConstantRector.php
+                - rules/Php81/Enum/AttributeName.php
+

rules/Php80/NodeAnalyzer/PhpAttributeAnalyzer.php

@@ -23,7 +23,6 @@
 {
     public function __construct(
         private NodeNameResolver $nodeNameResolver,
-        private ReflectionProvider $reflectionProvider,
     ) {
     }
 
@@ -42,29 +41,6 @@ public function hasPhpAttribute(Property | ClassLike | ClassMethod | Param $node
         return false;
     }
 
-    /**
-     * @param AttributeName::* $attributeClass
-     */
-    public function hasInheritedPhpAttribute(Class_ $class, string $attributeClass): bool
-    {
-        $className = (string) $this->nodeNameResolver->getName($class);
-        if (! $this->reflectionProvider->hasClass($className)) {
-            return false;
-        }
-
-        $classReflection = $this->reflectionProvider->getClass($className);
-        $ancestorClassReflections = [...$classReflection->getParents(), ...$classReflection->getInterfaces()];
-
-        foreach ($ancestorClassReflections as $ancestorClassReflection) {
-            $nativeReflection = $ancestorClassReflection->getNativeReflection();
-            if ($nativeReflection->getAttributes($attributeClass) !== []) {
-                return true;
-            }
-        }
-
-        return false;
-    }
-
     /**
      * @param string[] $attributeClasses
      */

rules/TypeDeclaration/ValueObject/DataProviderNodes.php

@@ -34,11 +34,6 @@ public function __construct(
         Assert::allIsInstanceOf($phpDocTagNodes, PhpDocTagNode::class);
     }
 
-    public function isEmpty(): bool
-    {
-        return $this->getClassMethods() === [];
-    }
-
     /**
      * @return ClassMethod[]
      */

src/ChangesReporting/Output/GitHubOutputFormatter.php

@@ -23,10 +23,7 @@
  */
 final readonly class GitHubOutputFormatter implements OutputFormatterInterface
 {
-    /**
-     * @var string
-     */
-    public const NAME = 'github';
+    private const NAME = 'github';
 
     private const GROUP_NAME = 'Rector report';
 

src/ChangesReporting/Output/GitlabOutputFormatter.php

@@ -17,10 +17,7 @@
 
 final readonly class GitlabOutputFormatter implements OutputFormatterInterface
 {
-    /**
-     * @var string
-     */
-    public const NAME = 'gitlab';
+    private const NAME = 'gitlab';
 
     private const ERROR_TYPE_ISSUE = 'issue';
 

src/ChangesReporting/Output/JUnitOutputFormatter.php

@@ -18,7 +18,7 @@
 
 final readonly class JUnitOutputFormatter implements OutputFormatterInterface
 {
-    public const NAME = 'junit';
+    private const NAME = 'junit';
 
     private const XML_ATTRIBUTE_FILE = 'file';
 

src/NodeTypeResolver/Node/AttributeKey.php

@@ -261,12 +261,6 @@ final class AttributeKey
      */
     public const IS_USED_AS_ARG_BY_REF_VALUE = 'is_used_as_arg_by_ref_value';
 
-    /**
-     * @deprecated as never used
-     * @var string
-     */
-    public const ATTRIBUTE_COMMENT = 'attribute_comment';
-
     /**
      * @var string
      */

src/Reflection/ReflectionResolver.php

@@ -216,34 +216,6 @@ public function resolveFunctionLikeReflectionFromCall(
         return null;
     }
 
-    public function resolveMethodReflectionFromClassMethod(ClassMethod $classMethod, Scope $scope): ?MethodReflection
-    {
-        $classReflection = $scope->getClassReflection();
-        if (! $classReflection instanceof ClassReflection) {
-            return null;
-        }
-
-        $className = $classReflection->getName();
-        $methodName = $this->nodeNameResolver->getName($classMethod);
-
-        return $this->resolveMethodReflection($className, $methodName, $scope);
-    }
-
-    public function resolveFunctionReflectionFromFunction(Function_ $function): ?FunctionReflection
-    {
-        $name = $this->nodeNameResolver->getName($function);
-        if ($name === null) {
-            return null;
-        }
-
-        $functionName = new Name($name);
-        if ($this->reflectionProvider->hasFunction($functionName, null)) {
-            return $this->reflectionProvider->getFunction($functionName, null);
-        }
-
-        return null;
-    }
-
     public function resolveMethodReflectionFromNew(New_ $new): ?MethodReflection
     {
         $newClassType = $this->nodeTypeResolver->getType($new->class);