fix: List postrectors in the list of applied rules

Resolves: rectorphp/rector#9373

Author: simonschaufi

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,26 @@
     private const KEY_LINE = 'line';
 
     /**
-     * @var class-string<RectorInterface>
+     * @var class-string<RectorInterface|PostRectorInterface>
      */
     private string $rectorClass;
 
     /**
-     * @param class-string<RectorInterface>|RectorInterface $rectorClass
+     * @param class-string<RectorInterface|PostRectorInterface>|RectorInterface|PostRectorInterface $rectorClass
      */
     public function __construct(
-        string|RectorInterface $rectorClass,
+        string|RectorInterface|PostRectorInterface $rectorClass,
         private int $line
     ) {
-        if ($rectorClass instanceof RectorInterface) {
+        if ($rectorClass instanceof RectorInterface || $rectorClass instanceof PostRectorInterface) {
             $rectorClass = $rectorClass::class;
         }
 
         $this->rectorClass = $rectorClass;
     }
 
     /**
-     * @return class-string<RectorInterface>
+     * @return class-string<RectorInterface|PostRectorInterface>
      */
     public function getRectorClass(): string
     {
@@ -63,7 +64,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/PostRector/Rector/AbstractPostRector.php

@@ -12,7 +12,7 @@
 
 abstract class AbstractPostRector extends NodeVisitorAbstract implements PostRectorInterface
 {
-    private File|null $file = null;
+    protected File $file;
 
     /**
      * @param Stmt[] $stmts
@@ -29,8 +29,6 @@ public function setFile(File $file): void
 
     public function getFile(): File
     {
-        Assert::isInstanceOf($this->file, File::class);
-
         return $this->file;
     }
 }

src/PostRector/Rector/ClassRenamingPostRector.php

@@ -8,6 +8,7 @@
 use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\Namespace_;
 use PhpParser\NodeVisitor;
+use Rector\ChangesReporting\ValueObject\RectorWithLineChange;
 use Rector\CodingStyle\Application\UseImportsRemover;
 use Rector\Configuration\Option;
 use Rector\Configuration\Parameter\SimpleParameterProvider;
@@ -44,6 +45,10 @@ public function beforeTraverse(array $nodes): array
                 $removedUses = $this->renamedClassesDataCollector->getOldClasses();
                 $node->stmts = $this->useImportsRemover->removeImportsFromStmts($node->stmts, $removedUses);
 
+                // notify this rule changing code
+                $rectorWithLineChange = new RectorWithLineChange(self::class, $node->getStartLine());
+                $this->file->addRectorClassWithLine($rectorWithLineChange);
+
                 break;
             }
         }

src/PostRector/Rector/DocblockNameImportingPostRector.php

@@ -9,6 +9,7 @@
 use PhpParser\Node\Stmt;
 use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfo;
 use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfoFactory;
+use Rector\ChangesReporting\ValueObject\RectorWithLineChange;
 use Rector\Comments\NodeDocBlock\DocBlockUpdater;
 use Rector\NodeTypeResolver\PhpDoc\NodeAnalyzer\DocBlockNameImporter;
 use Rector\PostRector\Guard\AddUseStatementGuard;
@@ -39,6 +40,10 @@ public function enterNode(Node $node): Node|null
             return null;
         }
 
+        // notify this rule changing code
+        $rectorWithLineChange = new RectorWithLineChange(self::class, $node->getStartLine());
+        $this->file->addRectorClassWithLine($rectorWithLineChange);
+
         $this->docBlockUpdater->updateRefactoredNodeWithPhpDocInfo($node);
         return $node;
     }

src/PostRector/Rector/NameImportingPostRector.php

@@ -9,6 +9,7 @@
 use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\GroupUse;
 use PhpParser\Node\Stmt\Use_;
+use Rector\ChangesReporting\ValueObject\RectorWithLineChange;
 use Rector\CodingStyle\Node\NameImporter;
 use Rector\Naming\Naming\UseImportsResolver;
 use Rector\PostRector\Guard\AddUseStatementGuard;
@@ -42,6 +43,10 @@ public function enterNode(Node $node): Node|null
             return null;
         }
 
+        // notify this rule changing code
+        $rectorWithLineChange = new RectorWithLineChange(self::class, $node->getStartLine());
+        $this->file->addRectorClassWithLine($rectorWithLineChange);
+
         return $this->nameImporter->importName($node, $this->getFile(), $this->currentUses);
     }
 

src/PostRector/Rector/UnusedImportRemovingPostRector.php

@@ -18,6 +18,7 @@
 use PhpParser\Node\UseItem;
 use PhpParser\NodeVisitor;
 use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfoFactory;
+use Rector\ChangesReporting\ValueObject\RectorWithLineChange;
 use Rector\NodeTypeResolver\Node\AttributeKey;
 use Rector\PhpDocParser\NodeTraverser\SimpleCallableNodeTraverser;
 use Rector\PhpParser\Node\CustomNode\FileWithoutNamespace;
@@ -93,6 +94,10 @@ public function enterNode(Node $node): ?Node
             return null;
         }
 
+        // notify this rule changing code
+        $rectorWithLineChange = new RectorWithLineChange(self::class, $node->getStartLine());
+        $this->file->addRectorClassWithLine($rectorWithLineChange);
+
         $node->stmts = array_values($node->stmts);
         return $node;
     }

src/Testing/PHPUnit/ValueObject/RectorTestResult.php

@@ -5,6 +5,7 @@
 namespace Rector\Testing\PHPUnit\ValueObject;
 
 use Rector\Contract\Rector\RectorInterface;
+use Rector\PostRector\Contract\Rector\PostRectorInterface;
 use Rector\ValueObject\ProcessResult;
 
 /**
@@ -24,7 +25,7 @@ public function getChangedContents(): string
     }
 
     /**
-     * @return array<class-string<RectorInterface>>
+     * @return array<class-string<RectorInterface|PostRectorInterface>>
      */
     public function getAppliedRectorClasses(): array
     {

src/ValueObject/Reporting/FileDiff.php

@@ -8,6 +8,7 @@
 use Rector\ChangesReporting\ValueObject\RectorWithLineChange;
 use Rector\Contract\Rector\RectorInterface;
 use Rector\Parallel\ValueObject\BridgeItem;
+use Rector\PostRector\Contract\Rector\PostRectorInterface;
 use Symplify\EasyParallel\Contract\SerializableInterface;
 use Webmozart\Assert\Assert;
 
@@ -81,7 +82,7 @@ public function getRectorShortClasses(): array
     }
 
     /**
-     * @return array<class-string<RectorInterface>>
+     * @return array<class-string<RectorInterface|PostRectorInterface>>
      */
     public function getRectorClasses(): array
     {