skip marker attribute in RemoveEmptyClassMethodRector

Author: TomasVotruba

rules-tests/DeadCode/Rector/ClassMethod/RemoveEmptyClassMethodRector/Fixture/skip_attribute_marker.php.inc

@@ -0,0 +1,14 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveEmptyClassMethodRector\Fixture;
+
+#[\Attribute()]
+final class SkipAttributeMarker
+{
+    /**
+     * @param class-string $className
+     */
+    public function __construct(string $className)
+    {
+    }
+}

rules/DeadCode/Rector/ClassMethod/RemoveEmptyClassMethodRector.php

@@ -4,10 +4,12 @@
 
 namespace Rector\DeadCode\Rector\ClassMethod;
 
+use PhpParser\Comment\Doc;
 use PhpParser\Node;
 use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\ClassMethod;
 use PHPStan\PhpDocParser\Ast\PhpDoc\DeprecatedTagValueNode;
+use PHPStan\Reflection\ClassReflection;
 use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfo;
 use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfoFactory;
 use Rector\Configuration\Parameter\FeatureFlags;
@@ -160,7 +162,17 @@ private function shouldSkipClassMethod(Class_ $class, ClassMethod $classMethod):
             return ! $classMethod->isPublic();
         }
 
-        return $this->isName($classMethod, MethodName::INVOKE);
+        if ($this->isName($classMethod, MethodName::INVOKE)) {
+            return true;
+        }
+
+        $classReflection = $scope->getClassReflection();
+        if (! $classReflection instanceof ClassReflection) {
+            return false;
+        }
+
+        // skip constructor in attributes as might be a marker parameter
+        return $classReflection->isAttributeClass() && $classMethod->getDocComment() instanceof Doc;
     }
 
     private function hasDeprecatedAnnotation(ClassMethod $classMethod): bool