Add PostRectorInterface instance class names to list of applied rectors

.github/workflows/e2e.yaml

@@ -24,6 +24,7 @@ jobs:
             matrix:
                 php_version: ['8.2']
                 directory:
+                    - 'e2e/applied-auto-import'
                     - 'e2e/applied-polyfill-php80'
                     - 'e2e/applied-rule-change-docblock'
                     - 'e2e/applied-rule-removed-node'

e2e/applied-auto-import/.gitignore

@@ -0,0 +1 @@
+/vendor

e2e/applied-auto-import/composer.json

@@ -0,0 +1,7 @@
+{
+    "require": {
+        "php": "^8.1"
+    },
+    "minimum-stability": "dev",
+    "prefer-stable": true
+}

e2e/applied-auto-import/expected-output.diff

@@ -0,0 +1,32 @@
+1 file with changes
+===================
+
+1) src/RenameDocblock.php:2
+
+    ---------- begin diff ----------
+@@ @@
+
+ namespace App;
+
+-use SomeUnusedClass;
++use DateTimeInterface;
+
+ /**
+- * @param \DateTime $someOldClass
++ * @param DateTimeInterface $someOldClass
+  */
+-function someFunction(\DateTime $someOldClass)
++function someFunction(DateTimeInterface $someOldClass)
+ {
+ }
+    ----------- end diff -----------
+
+Applied rules:
+ * RenameClassRector
+ * DocblockNameImportingPostRector
+ * NameImportingPostRector
+ * UnusedImportRemovingPostRector
+ * UseAddingPostRector
+
+
+ [OK] 1 file would have been changed (dry-run) by Rector

e2e/applied-auto-import/rector.php

@@ -0,0 +1,19 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Renaming\Rector\Name\RenameClassRector;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->paths([
+        __DIR__ . '/src',
+    ]);
+
+    $rectorConfig->ruleWithConfiguration(RenameClassRector::class, [
+        'DateTime' => 'DateTimeInterface'
+    ]);
+
+    $rectorConfig->importNames();
+    $rectorConfig->removeUnusedImports();
+};

e2e/applied-auto-import/src/RenameDocblock.php

@@ -0,0 +1,12 @@
+<?php
+
+namespace App;
+
+use SomeUnusedClass;
+
+/**
+ * @param \DateTime $someOldClass
+ */
+function someFunction(\DateTime $someOldClass)
+{
+}

phpstan.neon

@@ -238,6 +238,7 @@ parameters:
             paths:
                 # caching and message of specific child Rector rules
                 - src/Rector/AbstractRector.php
+                - src/PostRector/Rector/AbstractPostRector.php
                 # for cache
                 - src/Testing/PHPUnit/AbstractRectorTestCase.php
 

rules/CodingStyle/Application/UseImportsAdder.php

@@ -31,15 +31,14 @@ public function __construct(
      * @param array<FullyQualifiedObjectType|AliasedObjectType> $useImportTypes
      * @param array<FullyQualifiedObjectType|AliasedObjectType> $constantUseImportTypes
      * @param array<FullyQualifiedObjectType|AliasedObjectType> $functionUseImportTypes
-     * @return Stmt[]
      */
     public function addImportsToStmts(
         FileWithoutNamespace $fileWithoutNamespace,
         array $stmts,
         array $useImportTypes,
         array $constantUseImportTypes,
         array $functionUseImportTypes
-    ): array {
+    ): bool {
         $usedImports = $this->usedImportsResolver->resolveForStmts($stmts);
         $existingUseImportTypes = $usedImports->getUseImports();
         $existingConstantUseImports = $usedImports->getConstantImports();
@@ -57,7 +56,7 @@ public function addImportsToStmts(
 
         $newUses = $this->createUses($useImportTypes, $constantUseImportTypes, $functionUseImportTypes, null);
         if ($newUses === []) {
-            return [$fileWithoutNamespace];
+            return false;
         }
 
         $stmts = array_values(array_filter($stmts, static function (Stmt $stmt): bool {
@@ -94,7 +93,7 @@ public function addImportsToStmts(
             $fileWithoutNamespace->stmts = $stmts;
             $fileWithoutNamespace->stmts = array_values($fileWithoutNamespace->stmts);
 
-            return [$fileWithoutNamespace];
+            return true;
         }
 
         $this->mirrorUseComments($stmts, $newUses);
@@ -103,7 +102,7 @@ public function addImportsToStmts(
         $fileWithoutNamespace->stmts = array_merge($newUses, $this->resolveInsertNop($fileWithoutNamespace), $stmts);
         $fileWithoutNamespace->stmts = array_values($fileWithoutNamespace->stmts);
 
-        return [$fileWithoutNamespace];
+        return true;
     }
 
     /**
@@ -116,7 +115,7 @@ public function addImportsToNamespace(
         array $useImportTypes,
         array $constantUseImportTypes,
         array $functionUseImportTypes
-    ): void {
+    ): bool {
         $namespaceName = $this->getNamespaceName($namespace);
 
         $existingUsedImports = $this->usedImportsResolver->resolveForStmts($namespace->stmts);
@@ -141,13 +140,15 @@ public function addImportsToNamespace(
         $newUses = $this->createUses($useImportTypes, $constantUseImportTypes, $functionUseImportTypes, $namespaceName);
 
         if ($newUses === []) {
-            return;
+            return false;
         }
 
         $this->mirrorUseComments($namespace->stmts, $newUses);
 
         $namespace->stmts = array_merge($newUses, $this->resolveInsertNop($namespace), $namespace->stmts);
         $namespace->stmts = array_values($namespace->stmts);
+
+        return true;
     }
 
     /**

rules/CodingStyle/Application/UseImportsRemover.php

@@ -4,8 +4,9 @@
 
 namespace Rector\CodingStyle\Application;
 
-use PhpParser\Node\Stmt;
+use PhpParser\Node\Stmt\Namespace_;
 use PhpParser\Node\Stmt\Use_;
+use Rector\PhpParser\Node\CustomNode\FileWithoutNamespace;
 use Rector\Renaming\Collector\RenamedNameCollector;
 
 final readonly class UseImportsRemover
@@ -16,35 +17,40 @@ public function __construct(
     }
 
     /**
-     * @param Stmt[] $stmts
      * @param string[] $removedUses
-     * @return Stmt[]
      */
-    public function removeImportsFromStmts(array $stmts, array $removedUses): array
+    public function removeImportsFromStmts(FileWithoutNamespace|Namespace_ $node, array $removedUses): bool
     {
         $hasRemoved = false;
-        foreach ($stmts as $key => $stmt) {
+        foreach ($node->stmts as $key => $stmt) {
             if (! $stmt instanceof Use_) {
                 continue;
             }
 
-            $stmt = $this->removeUseFromUse($removedUses, $stmt);
+            if ($this->removeUseFromUse($removedUses, $stmt)) {
+                $node->stmts[$key] = $stmt;
+                $hasRemoved = true;
+            }
 
             // remove empty uses
             if ($stmt->uses === []) {
-                unset($stmts[$key]);
-                $hasRemoved = true;
+                unset($node->stmts[$key]);
             }
         }
 
-        return $hasRemoved ? array_values($stmts) : $stmts;
+        if ($hasRemoved) {
+            $node->stmts = array_values($node->stmts);
+        }
+
+        return $hasRemoved;
     }
 
     /**
      * @param string[] $removedUses
      */
-    private function removeUseFromUse(array $removedUses, Use_ $use): Use_
+    private function removeUseFromUse(array $removedUses, Use_ $use): bool
     {
+        $hasChanged = false;
         foreach ($use->uses as $usesKey => $useUse) {
             $useName = $useUse->name->toString();
             if (! in_array($useName, $removedUses, true)) {
@@ -56,8 +62,13 @@ private function removeUseFromUse(array $removedUses, Use_ $use): Use_
             }
 
             unset($use->uses[$usesKey]);
+            $hasChanged = true;
+        }
+
+        if ($hasChanged) {
+            $use->uses = array_values($use->uses);
         }
 
-        return $use;
+        return $hasChanged;
     }
 }

src/Bridge/SetRectorsResolver.php

@@ -46,7 +46,7 @@ public function resolveFromFilePathIncludingConfiguration(string $configFilePath
     {
         $rectorConfig = $this->loadRectorConfigFromFilePath($configFilePath);
 
-        $rectorClassesWithOptionalConfiguration = $rectorConfig->getRectorClasses();
+        $rectorClassesWithOptionalConfiguration = $rectorConfig->getMainRectorClasses();
 
         foreach ($rectorConfig->getRuleConfigurations() as $rectorClass => $configuration) {
             // remove from non-configurable, if added again with better config

src/ChangesReporting/ValueObject/RectorWithLineChange.php

@@ -5,6 +5,7 @@
 namespace Rector\ChangesReporting\ValueObject;
 
 use Rector\Contract\Rector\RectorInterface;
+use Rector\PostRector\Contract\Rector\PostRectorInterface;
 use Symplify\EasyParallel\Contract\SerializableInterface;
 use Webmozart\Assert\Assert;
 
@@ -21,26 +22,16 @@
     private const KEY_LINE = 'line';
 
     /**
-     * @var class-string<RectorInterface>
-     */
-    private string $rectorClass;
-
-    /**
-     * @param class-string<RectorInterface>|RectorInterface $rectorClass
+     * @param class-string<RectorInterface|PostRectorInterface> $rectorClass
      */
     public function __construct(
-        string|RectorInterface $rectorClass,
+        private string $rectorClass,
         private int $line
     ) {
-        if ($rectorClass instanceof RectorInterface) {
-            $rectorClass = $rectorClass::class;
-        }
-
-        $this->rectorClass = $rectorClass;
     }
 
     /**
-     * @return class-string<RectorInterface>
+     * @return class-string<RectorInterface|PostRectorInterface>
      */
     public function getRectorClass(): string
     {
@@ -63,7 +54,7 @@ public static function decode(array $json): self
     }
 
     /**
-     * @return array{rector_class: class-string<RectorInterface>, line: int}
+     * @return array{rector_class: class-string<RectorInterface|PostRectorInterface>, line: int}
      */
     public function jsonSerialize(): array
     {

src/Config/RectorConfig.php

@@ -439,7 +439,7 @@ public function getRuleConfigurations(): array
      * @internal Used only for bridge
      * @return array<class-string<RectorInterface>>
      */
-    public function getRectorClasses(): array
+    public function getMainRectorClasses(): array
     {
         return $this->tags[RectorInterface::class] ?? [];
     }

src/Configuration/RectorConfigBuilder.php

@@ -250,7 +250,7 @@ public function __invoke(RectorConfig $rectorConfig): void
         }
 
         // log rules from sets and compare them with explicit rules
-        $setRegisteredRectorClasses = $rectorConfig->getRectorClasses();
+        $setRegisteredRectorClasses = $rectorConfig->getMainRectorClasses();
         SimpleParameterProvider::addParameter(Option::SET_REGISTERED_RULES, $setRegisteredRectorClasses);
 
         if ($this->paths !== []) {

src/PostRector/Rector/AbstractPostRector.php

@@ -4,8 +4,10 @@
 
 namespace Rector\PostRector\Rector;
 
+use PhpParser\Node;
 use PhpParser\Node\Stmt;
 use PhpParser\NodeVisitorAbstract;
+use Rector\ChangesReporting\ValueObject\RectorWithLineChange;
 use Rector\PostRector\Contract\Rector\PostRectorInterface;
 use Rector\ValueObject\Application\File;
 use Webmozart\Assert\Assert;
@@ -33,4 +35,12 @@ public function getFile(): File
 
         return $this->file;
     }
+
+    protected function addRectorClassWithLine(Node $node): void
+    {
+        Assert::isInstanceOf($this->file, File::class);
+
+        $rectorWithLineChange = new RectorWithLineChange(static::class, $node->getStartLine());
+        $this->file->addRectorClassWithLine($rectorWithLineChange);
+    }
 }

src/PostRector/Rector/ClassRenamingPostRector.php

@@ -44,7 +44,9 @@ public function beforeTraverse(array $nodes): array
         foreach ($nodes as $node) {
             if ($node instanceof FileWithoutNamespace || $node instanceof Namespace_) {
                 $removedUses = $this->renamedClassesDataCollector->getOldClasses();
-                $node->stmts = $this->useImportsRemover->removeImportsFromStmts($node->stmts, $removedUses);
+                if ($this->useImportsRemover->removeImportsFromStmts($node, $removedUses)) {
+                    $this->addRectorClassWithLine($node);
+                }
 
                 break;
             }

src/PostRector/Rector/DocblockNameImportingPostRector.php

@@ -39,6 +39,7 @@ public function enterNode(Node $node): Node|null
             return null;
         }
 
+        $this->addRectorClassWithLine($node);
         $this->docBlockUpdater->updateRefactoredNodeWithPhpDocInfo($node);
         return $node;
     }

src/PostRector/Rector/NameImportingPostRector.php

@@ -5,6 +5,7 @@
 namespace Rector\PostRector\Rector;
 
 use PhpParser\Node;
+use PhpParser\Node\Name;
 use PhpParser\Node\Name\FullyQualified;
 use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\GroupUse;
@@ -36,13 +37,19 @@ public function beforeTraverse(array $nodes): array
         return $nodes;
     }
 
-    public function enterNode(Node $node): Node|null
+    public function enterNode(Node $node): Name|null
     {
         if (! $node instanceof FullyQualified) {
             return null;
         }
 
-        return $this->nameImporter->importName($node, $this->getFile(), $this->currentUses);
+        $name = $this->nameImporter->importName($node, $this->getFile(), $this->currentUses);
+        if (! $name instanceof Name) {
+            return null;
+        }
+
+        $this->addRectorClassWithLine($node);
+        return $name;
     }
 
     /**

src/PostRector/Rector/UnusedImportRemovingPostRector.php

@@ -93,6 +93,7 @@ public function enterNode(Node $node): ?Node
             return null;
         }
 
+        $this->addRectorClassWithLine($node);
         $node->stmts = array_values($node->stmts);
         return $node;
     }