[NodeTypeResolver] Handle crash after next exit() on no namespaced co…

…de after removal next attribute (#4074)

packages/NodeTypeResolver/PHPStan/Scope/PHPStanNodeScopeResolver.php

@@ -48,6 +48,8 @@
 use Rector\Core\Contract\PhpParser\Node\StmtsAwareInterface;
 use Rector\Core\Exception\ShouldNotHappenException;
 use Rector\Core\NodeAnalyzer\ClassAnalyzer;
+use Rector\Core\PhpParser\Node\CustomNode\FileWithoutNamespace;
+use Rector\Core\PhpParser\NodeTraverser\FileWithoutNamespaceNodeTraverser;
 use Rector\Core\Util\Reflection\PrivatesAccessor;
 use Rector\NodeNameResolver\NodeNameResolver;
 use Rector\NodeTypeResolver\Node\AttributeKey;
@@ -79,7 +81,8 @@ public function __construct(
         private readonly ScopeFactory $scopeFactory,
         private readonly PrivatesAccessor $privatesAccessor,
         private readonly NodeNameResolver $nodeNameResolver,
-        private readonly ClassAnalyzer $classAnalyzer
+        private readonly ClassAnalyzer $classAnalyzer,
+        private readonly FileWithoutNamespaceNodeTraverser $fileWithoutNamespaceNodeTraverser
     ) {
         $this->nodeTraverser = new NodeTraverser();
 
@@ -107,6 +110,14 @@ public function processNodes(
         Assert::allIsInstanceOf($stmts, Stmt::class);
         $this->nodeTraverser->traverse($stmts);
 
+        if (! $isScopeRefreshing) {
+            $stmts = $this->fileWithoutNamespaceNodeTraverser->traverse($stmts);
+
+            if (count($stmts) === 1 && $stmts[0] instanceof FileWithoutNamespace) {
+                $stmts = $stmts[0]->stmts;
+            }
+        }
+
         $scope = $formerMutatingScope ?? $this->scopeFactory->createFromFile($filePath);
 
         // skip chain method calls, performance issue: https://github.com/phpstan/phpstan/issues/254

rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/after_exit.php.inc

@@ -0,0 +1,29 @@
+<?php
+
+// no namespace on purpose to cover crash on FileWithoutNamespace
+exit();
+
+// this part is needed to produce crash
+echo 'some statement';
+
+function run2()
+{
+    preg_split('#a#', null);
+}
+
+?>
+-----
+<?php
+
+// no namespace on purpose to cover crash on FileWithoutNamespace
+exit();
+
+// this part is needed to produce crash
+echo 'some statement';
+
+function run2()
+{
+    preg_split('#a#', '');
+}
+
+?>

rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/after_exit_namespaced.php.inc

@@ -0,0 +1,29 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\FuncCall\NullToStrictStringFuncCallArgRector\Fixture;
+
+exit();
+
+echo 'some statement';
+
+function run2()
+{
+    preg_split('#a#', null);
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php80\Rector\FuncCall\NullToStrictStringFuncCallArgRector\Fixture;
+
+exit();
+
+echo 'some statement';
+
+function run2()
+{
+    preg_split('#a#', '');
+}
+
+?>

src/Kernel/RectorKernel.php

@@ -17,7 +17,7 @@ final class RectorKernel
     /**
      * @var string
      */
-    private const CACHE_KEY = 'v54';
+    private const CACHE_KEY = 'v55';
 
     private ContainerInterface|null $container = null;