try to remove breaking service from container

Author: TomasVotruba

composer.json

@@ -23,7 +23,7 @@
         "nikic/php-parser": "^5.6.2",
         "ondram/ci-detector": "^4.2",
         "phpstan/phpdoc-parser": "^2.3",
-        "phpstan/phpstan": "2.1.19 as 2.1.32",
+        "phpstan/phpstan": "^2.1.32",
         "react/event-loop": "^1.6",
         "react/promise": "^3.3",
         "react/socket": "^1.17",

rules/CodingStyle/Rector/Enum_/EnumCaseToPascalCaseRector.php

@@ -208,9 +208,9 @@ private function convertToPascalCase(string $name): string
                 fn ($part): string =>
                 // If part is all uppercase, convert to ucfirst(strtolower())
                 // If part is already mixed or PascalCase, keep as is except ucfirst
-                ctype_upper($part)
-                    ? ucfirst(strtolower($part))
-                    : ucfirst($part),
+                ctype_upper((string) $part)
+                    ? ucfirst(strtolower((string) $part))
+                    : ucfirst((string) $part),
                 $parts
             )
         );

src/DependencyInjection/PHPStan/PHPStanContainerMemento.php

@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\DependencyInjection\PHPStan;
+
+use PHPStan\DependencyInjection\MemoizingContainer;
+use PHPStan\DependencyInjection\Nette\NetteContainer;
+use PHPStan\Parser\AnonymousClassVisitor;
+use PHPStan\Parser\RichParser;
+use Rector\Util\Reflection\PrivatesAccessor;
+
+/**
+ * Helper service to modify PHPStan container
+ * To avoid issues caused by node replacement, like @see https://github.com/rectorphp/rector/issues/9492
+ */
+final class PHPStanContainerMemento
+{
+    public static function removeRichVisitors(RichParser $richParser): void
+    {
+        // the only way now seems to access container early and remove unwanted services
+        // here https://github.com/phpstan/phpstan-src/blob/522421b007cbfc674bebb93e823c774167ac78cd/src/Parser/RichParser.php#L90-L92
+        $privatesAccessor = new PrivatesAccessor();
+
+        /** @var MemoizingContainer $container */
+        $container = $privatesAccessor->getPrivateProperty($richParser, 'container');
+
+        /** @var NetteContainer $originalContainer */
+        $originalContainer = $privatesAccessor->getPrivateProperty($container, 'originalContainer');
+
+        /** @var NetteContainer $originalContainer */
+        $deeperContainer = $privatesAccessor->getPrivateProperty($originalContainer, 'container');
+
+        // get tags property
+        $tags = $privatesAccessor->getPrivateProperty($deeperContainer, 'tags');
+
+        // keep only the anonymous class visitor
+        // remove all the rest, https://github.com/phpstan/phpstan-src/tree/1d86de8bb9371534983a8dbcd879e057d2ff028f/src/Parser
+        $tags[RichParser::VISITOR_SERVICE_TAG] = [
+            $container->findServiceNamesByType(AnonymousClassVisitor::class)[0] => true,
+        ];
+
+        $privatesAccessor->setPrivateProperty($deeperContainer, 'tags', $tags);
+    }
+}

src/PhpParser/Parser/RectorParser.php

@@ -8,15 +8,22 @@
 use PhpParser\ParserFactory;
 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;
 
 final readonly class RectorParser
 {
+    /**
+     * @param RichParser $parser
+     */
     public function __construct(
         private Parser $parser,
         private PrivatesAccessor $privatesAccessor
     ) {
+
+        PHPStanContainerMemento::removeRichVisitors($parser);
     }
 
     /**

tests/PhpParser/Printer/PHPStanPrinterTest.php

@@ -7,15 +7,15 @@
 use PhpParser\PrettyPrinter\Standard;
 use PHPStan\Parser\Parser;
 use PHPStan\Parser\RichParser;
+use Rector\DependencyInjection\PHPStan\PHPStanContainerMemento;
 use Rector\Testing\PHPUnit\AbstractLazyTestCase;
-<<<<<<< HEAD
 use ReflectionProperty;
-=======
->>>>>>> ea9a6ad8a5 ([issue 9492] create reproducer for modified array_map args, as creates changed args on print)
 
 /**
  * Test case for: https://github.com/rectorphp/rector/issues/9492
  * Most likely caused by https://github.com/phpstan/phpstan-src/pull/3763
+ *
+ * @see https://github.com/phpstan/phpstan-src/blob/2.1.x/src/Parser/ArrayMapArgVisitor.php
  */
 final class PHPStanPrinterTest extends AbstractLazyTestCase
 {
@@ -24,33 +24,22 @@ 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"
-<<<<<<< HEAD
         $parserReflectionProperty = new ReflectionProperty(RichParser::class, 'parser');
-=======
-        $parserReflectionProperty = new \ReflectionProperty(RichParser::class, 'parser');
->>>>>>> ea9a6ad8a5 ([issue 9492] create reproducer for modified array_map args, as creates changed args on print)
 
         /** @var \PhpParser\Parser $innerParser */
         $innerParser = $parserReflectionProperty->getValue($phpstanParser);
         $tokens = $innerParser->getTokens();
 
-<<<<<<< HEAD
         $standard = new Standard([]);
         $printerContents = $standard->printFormatPreserving($stmts, $stmts, $tokens);
 
         $newlineNormalizedContents = str_replace("\r\n", PHP_EOL, $printerContents);
 
         $this->assertStringEqualsFile(__DIR__ . '/Fixture/some_array_map.php', $newlineNormalizedContents);
-=======
-        $standardPrinter = new Standard([
-            'newline' => "\n",
-        ]);
-        $printerContents = $standardPrinter->printFormatPreserving($stmts, $stmts, $tokens);
-
-        $this->assertStringEqualsFile(__DIR__ . '/Fixture/some_array_map.php', $printerContents);
->>>>>>> ea9a6ad8a5 ([issue 9492] create reproducer for modified array_map args, as creates changed args on print)
     }
 }