notes

Author: TomasVotruba

src/PhpParser/NodeTraverser/RectorNodeTraverser.php

@@ -19,12 +19,19 @@
 use Webmozart\Assert\Assert;
 
 /**
+ *  Based on native NodeTraverser class, but heavily customized for Rector needs.
+ *
+ *  The main differences are:
+ *  - no leaveNode(), as we do all in enterNode() that calls refactor() method
+ *  - cached visitors per node class for performance, e.g. when we find rules for Class_ node, they're cached for next time
+ *  - immutability features, register Rector rules once, then use; no changes on the fly
+ *
  * @see \Rector\Tests\PhpParser\NodeTraverser\RectorNodeTraverserTest
  */
 final class RectorNodeTraverser implements NodeTraverserInterface
 {
     /**
-     * @var list<NodeVisitor>
+     * @var RectorInterface[]
      */
     private array $visitors = [];
 
@@ -33,7 +40,7 @@ final class RectorNodeTraverser implements NodeTraverserInterface
     private bool $areNodeVisitorsPrepared = false;
 
     /**
-     * @var array<class-string<Node>, NodeVisitor[]>
+     * @var array<class-string<Node>, RectorInterface[]>
      */
     private array $visitorsPerNodeClass = [];
 
@@ -103,7 +110,9 @@ public function refreshPhpRectors(array $rectors): void
     }
 
     /**
-     * @return NodeVisitor[]
+     * @return RectorInterface[]
+     *
+     * @api used in tests
      */
     public function getVisitorsForNode(Node $node): array
     {