Wire RichParser node visitors via DI factory instead of private property hack

Replace PHPStanContainerMemento reflection with a RichParserFactory service,
registered in config/phpstan/parser.neon. The factory builds RichParser with a
DirectExtensionsCollection of the two visitors Rector needs, so no private
property of PHPStan internals is touched.

A plain "nodeVisitors" argument in the config is not enough: PHPStan's
AutowiredExtensionsExtension::beforeCompile() overwrites that argument for every
service definition that instantiates the class directly. Factory-created
definitions are skipped, so the explicit visitor list survives.

Author: TomasVotruba

.github/workflows/code_analysis.yaml

@@ -53,7 +53,7 @@ jobs:
                     -
                         name: 'Active Classes'
                         run: |
-                            vendor/bin/class-leak check bin config src rules utils --skip-suffix "Rector" --skip-type="Rector\\Utils\\Compiler\\Unprefixer" --skip-type="Rector\\NodeCollector\\BinaryOpConditionsCollector" --skip-type="Rector\\Set\\Contract\\SetListInterface"
+                            vendor/bin/class-leak check bin config src rules utils --skip-suffix "Rector" --skip-type="Rector\\Utils\\Compiler\\Unprefixer" --skip-type="Rector\\NodeCollector\\BinaryOpConditionsCollector" --skip-type="Rector\\Set\\Contract\\SetListInterface" --skip-type="Rector\\DependencyInjection\\PHPStan\\RichParserFactory"
 
                     -
                         name: 'Compatible PHPStan versions'

config/phpstan/parser.neon

@@ -22,6 +22,22 @@ services:
 
     rectorParser:
         class: PHPStan\Parser\RichParser
+        factory: @rectorRichParserFactory::create()
+        arguments!: []
+        autowired: no
+
+    # RichParser used directly by Rector, see PHPStanServicesFactory::createPHPStanParser()
+    currentPhpVersionRichParser:
+        class: PHPStan\Parser\RichParser
+        factory: @rectorRichParserFactory::create()
+        arguments!: []
+        autowired: no
+
+    rectorRichParserFactory:
+        class: Rector\DependencyInjection\PHPStan\RichParserFactory
         arguments:
             parser: @currentPhpVersionPhpParser
+            nodeVisitors:
+                - @PHPStan\Parser\AnonymousClassVisitor
+                - @PHPStan\Parser\ArrayMapArgVisitor
         autowired: no

src/DependencyInjection/PHPStan/PHPStanContainerMemento.php

@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\DependencyInjection\PHPStan;
+
+use PhpParser\NodeVisitor;
+use PhpParser\NodeVisitor\NameResolver;
+use PhpParser\Parser;
+use PHPStan\Analyser\Ignore\IgnoreLexer;
+use PHPStan\DependencyInjection\DirectExtensionsCollection;
+use PHPStan\Parser\RichParser;
+
+/**
+ * Creates PHPStan RichParser with only the node visitors Rector needs,
+ * to avoid issues caused by node replacement, like @see https://github.com/rectorphp/rector/issues/9492
+ *
+ * The visitors have to be passed here, as PHPStan autowires them into every RichParser
+ * service definition, see PHPStan\DependencyInjection\AutowiredExtensionsExtension
+ */
+final readonly class RichParserFactory
+{
+    /**
+     * @param NodeVisitor[] $nodeVisitors
+     */
+    public function __construct(
+        private Parser $parser,
+        private NameResolver $nameResolver,
+        private IgnoreLexer $ignoreLexer,
+        private array $nodeVisitors
+    ) {
+    }
+
+    /**
+     * @api used by config/phpstan/parser.neon
+     */
+    public function create(): RichParser
+    {
+        return new RichParser(
+            $this->parser,
+            $this->nameResolver,
+            new DirectExtensionsCollection($this->nodeVisitors),
+            $this->ignoreLexer
+        );
+    }
+}

src/DependencyInjection/PHPStan/RichParserFactory.php

@@ -9,7 +9,6 @@
 use PhpParser\PhpVersion;
 use PHPStan\Parser\Parser;
 use PHPStan\Parser\RichParser;
-use Rector\DependencyInjection\PHPStan\PHPStanContainerMemento;
 use Rector\PhpParser\ValueObject\StmtsAndTokens;
 use Rector\Util\Reflection\PrivatesAccessor;
 
@@ -22,8 +21,6 @@ public function __construct(
         private Parser $parser,
         private PrivatesAccessor $privatesAccessor
     ) {
-
-        PHPStanContainerMemento::removeRichVisitors($parser);
     }
 
     /**

src/PhpParser/Parser/RectorParser.php

@@ -7,7 +7,6 @@
 use PhpParser\PrettyPrinter\Standard;
 use PHPStan\Parser\Parser;
 use PHPStan\Parser\RichParser;
-use Rector\DependencyInjection\PHPStan\PHPStanContainerMemento;
 use Rector\Testing\PHPUnit\AbstractLazyTestCase;
 use ReflectionProperty;
 
@@ -24,8 +23,6 @@ public function testAddingCommentOnSomeNodesFail(): void
         /** @var RichParser $phpstanParser */
         $phpstanParser = $this->make(Parser::class);
 
-        PHPStanContainerMemento::removeRichVisitors($phpstanParser);
-
         $stmts = $phpstanParser->parseFile(__DIR__ . '/Fixture/some_array_map.php');
 
         // get private property "parser"