[dx] immutable node visitor

Author: TomasVotruba

composer.json

@@ -133,8 +133,7 @@
                 "https://raw.githubusercontent.com/rectorphp/vendor-patches/main/patches/nikic-php-parser-lib-phpparser-node-stmt-if-php.patch",
                 "https://raw.githubusercontent.com/rectorphp/vendor-patches/main/patches/nikic-php-parser-lib-phpparser-node-stmt-case-php.patch",
                 "https://raw.githubusercontent.com/rectorphp/vendor-patches/main/patches/nikic-php-parser-lib-phpparser-node-stmt-elseif-php.patch",
-                "https://raw.githubusercontent.com/rectorphp/vendor-patches/main/patches/nikic-php-parser-lib-phpparser-node-stmt-namespace-php.patch",
-                "https://raw.githubusercontent.com/rectorphp/vendor-patches/main/patches/nikic-php-parser-lib-phpparser-nodetraverser-php.patch"
+                "https://raw.githubusercontent.com/rectorphp/vendor-patches/main/patches/nikic-php-parser-lib-phpparser-node-stmt-namespace-php.patch"
             ]
         },
         "composer-exit-on-patch-failure": true,

phpstan.neon

@@ -217,7 +217,6 @@ parameters:
 
         # optional as changes behavior, should be used explicitly outside PHP upgrade
         - '#Register "Rector\\Php73\\Rector\\FuncCall\\JsonThrowOnErrorRector" service to "php73\.php" config set#'
-        - '#Register "Rector\\Php81\\Rector\\ClassMethod\\NewInInitializerRector" service to "php81\.php" config set#'
         - '#Register "Rector\\Php80\\Rector\\NotIdentical\\MbStrContainsRector" service to "php80\.php" config set#'
         - '#Register "Rector\\Php85\\Rector\\StmtsAwareInterface\\SequentialAssignmentsToPipeOperatorRector" service to "php85\.php" config set#'
         - '#Register "Rector\\Php85\\Rector\\Expression\\NestedFuncCallsToPipeOperatorRector" service to "php85\.php" config set#'
@@ -356,6 +355,27 @@ parameters:
 
         - '#Method Rector\\Utils\\Rector\\RemoveRefactorDuplicatedNodeInstanceCheckRector\:\:getInstanceofNodeClass\(\) should return class\-string<PhpParser\\Node>\|null but returns class\-string#'
 
+        # helper classes for bin script
         -
-            path: bin/list-unused-rules.php
+            path: scripts
             identifier:  symplify.multipleClassLikeInFile
+
+        # copied from /vendor, to keep as original as possible
+        -
+            identifier: symplify.forbiddenNode
+            path: src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php
+        -
+            identifier: missingType.iterableValue
+            path: src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php
+        -
+            identifier: symplify.noDynamicName
+            path: src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php
+        -
+            identifier: offsetAccess.nonArray
+            path: src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php
+        -
+            message: '#Property PhpParser\\Node\\Stmt\\ClassMethod\:\:\$stmts \(array<PhpParser\\Node\\Stmt>\|null\) does not accept array<PhpParser\\Node>#'
+            path: scripts/create-immutable-node-visitor.php
+        -
+            message: '#Property Rector\\PhpParser\\NodeTraverser\\AbstractImmutableNodeTraverser\:\:\$visitors \(list<PhpParser\\NodeVisitor>\) does not accept array<int\|string, PhpParser\\NodeVisitor>#'
+            path: src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php

rector.php

@@ -28,12 +28,13 @@
     ->withPhpSets()
     ->withPaths([
         __DIR__ . '/bin',
+        __DIR__ . '/config',
         __DIR__ . '/src',
         __DIR__ . '/rules',
         __DIR__ . '/rules-tests',
         __DIR__ . '/tests',
         __DIR__ . '/utils',
-        __DIR__ . '/config',
+        __DIR__ . '/scripts',
         __DIR__ . '/build/build-preload.php',
     ])
     ->withRootFiles()

scripts/avoid-short-node-names-in-fixtures.php

@@ -9,19 +9,21 @@
 use Symfony\Component\Console\Input\ArrayInput;
 use Symfony\Component\Console\Output\ConsoleOutput;
 use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\Finder\Finder;
+use Symfony\Component\Finder\SplFileInfo;
 
 require __DIR__ . '/../vendor/autoload.php';
 
 $fixtureFiles = FixtureFinder::find([__DIR__ . '/../tests', __DIR__ . '/../rules-tests']);
 
 // get short node names
 
-$nodeFileFinder = \Symfony\Component\Finder\Finder::create()
+$nodeFileFinder = Finder::create()
     ->files()
     ->name('*.php')
     ->in(__DIR__ . '/../vendor/nikic/php-parser/lib/PhpParser/Node');
 
-/** @var \Symfony\Component\Finder\SplFileInfo[] $nodeFileInfos */
+/** @var SplFileInfo[] $nodeFileInfos */
 $nodeFileInfos = iterator_to_array($nodeFileFinder->getIterator());
 
 $shortNodeClassNames = [];
@@ -33,10 +35,10 @@
 
 $hasErrors = false;
 
-foreach ($fixtureFiles as $fixtureFileInfo) {
+foreach ($fixtureFiles as $fixtureFile) {
     $shortClassNameMatch = Strings::match(
-        $fixtureFileInfo->getContents(),
-        '/\b(?:class|interface)\s+(?<name>[A-Z][A-Za-z0-9_]*)/'
+        $fixtureFile->getContents(),
+        '/\b(?:class|interface)\s+(?<name>[A-Z]\w*)/'
     );
     if ($shortClassNameMatch === null) {
         continue;
@@ -52,7 +54,7 @@
         $fixtureClassName,
         PHP_EOL,
         PHP_EOL,
-        $fixtureFileInfo->getRealPath(),
+        $fixtureFile->getRealPath(),
         PHP_EOL
     ));
 

scripts/check-before-after-same-fixtures.php

@@ -3,6 +3,7 @@
 declare(strict_types=1);
 
 use Nette\Utils\Strings;
+use Rector\Scripts\Finder\FixtureFinder;
 
 require __DIR__ . '/../vendor/autoload.php';
 
@@ -27,7 +28,7 @@ public function __construct()
      */
     public function run(array $testDirectories): int
     {
-        $fixtureFiles = \Rector\Scripts\Finder\FixtureFinder::find($testDirectories);
+        $fixtureFiles = FixtureFinder::find($testDirectories);
 
         $invalidFixturePaths = [];
         foreach ($fixtureFiles as $fixtureFile) {