[CodeQuality] Handle crash include first class callable on OptionalParametersAfterRequiredRector (#6552)

* [CodeQuality] Handle crash include first class callable on OptionalParametersAfterRequiredRector

* more fixture

* fix

* fix

Author: samsonasik

rules-tests/CodeQuality/Rector/ClassMethod/OptionalParametersAfterRequiredRector/Fixture/skip_include_first_class_callable.php.inc

@@ -0,0 +1,17 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\CodeQuality\Rector\ClassMethod\OptionalParametersAfterRequiredRector\Fixture;
+
+final class SkipIncludeFirstClassCallable
+{
+    public function run()
+    {
+        include $this->call(...);
+    }
+
+    public function call()
+    {
+    }
+}

rules-tests/CodeQuality/Rector/ClassMethod/OptionalParametersAfterRequiredRector/Fixture/skip_instanceof_first_class_callable.php.inc

@@ -0,0 +1,19 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\CodeQuality\Rector\ClassMethod\OptionalParametersAfterRequiredRector\Fixture;
+
+use stdClass;
+
+final class SkipInstanceofFirstClassCallable
+{
+    public function run($a)
+    {
+        $this->call(...) instanceof stdClass;
+    }
+
+    public function call()
+    {
+    }
+}

rules-tests/CodeQuality/Rector/ClassMethod/OptionalParametersAfterRequiredRector/Fixture/skip_isset_first_class_callable.php.inc

@@ -0,0 +1,17 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\CodeQuality\Rector\ClassMethod\OptionalParametersAfterRequiredRector\Fixture;
+
+final class SkipIssetFirstClassCallable
+{
+    public function run()
+    {
+        isset($this->call(...));
+    }
+
+    public function call()
+    {
+    }
+}

src/NodeTypeResolver/PHPStan/Scope/PHPStanNodeScopeResolver.php

@@ -28,6 +28,9 @@
 use PhpParser\Node\Expr\Eval_;
 use PhpParser\Node\Expr\Exit_;
 use PhpParser\Node\Expr\FuncCall;
+use PhpParser\Node\Expr\Include_;
+use PhpParser\Node\Expr\Instanceof_;
+use PhpParser\Node\Expr\Isset_;
 use PhpParser\Node\Expr\List_;
 use PhpParser\Node\Expr\Match_;
 use PhpParser\Node\Expr\MethodCall;
@@ -72,6 +75,7 @@
 use PhpParser\Node\Stmt\Switch_;
 use PhpParser\Node\Stmt\Trait_;
 use PhpParser\Node\Stmt\TryCatch;
+use PhpParser\Node\Stmt\Unset_;
 use PhpParser\Node\UnionType;
 use PhpParser\NodeTraverser;
 use PHPStan\Analyser\MutatingScope;
@@ -209,7 +213,9 @@ public function processNodes(
                 $node instanceof Eval_ ||
                 $node instanceof Print_ ||
                 $node instanceof Exit_ ||
-                $node instanceof ArrowFunction
+                $node instanceof ArrowFunction ||
+                $node instanceof Include_ ||
+                $node instanceof Instanceof_
             ) && $node->expr instanceof Expr) {
                 $node->expr->setAttribute(AttributeKey::SCOPE, $mutatingScope);
                 return;
@@ -356,6 +362,11 @@ public function processNodes(
                 $this->processYield($node, $mutatingScope);
                 return;
             }
+
+            if ($node instanceof Isset_ || $node instanceof Unset_) {
+                $this->processIssetOrUnset($node, $mutatingScope);
+                return;
+            }
         };
 
         $this->nodeScopeResolverProcessNodes($stmts, $scope, $nodeCallback);
@@ -383,6 +394,13 @@ private function processYield(Yield_ $yield, MutatingScope $mutatingScope): void
         }
     }
 
+    private function processIssetOrUnset(Isset_|Unset_ $node, MutatingScope $mutatingScope): void
+    {
+        foreach ($node->vars as $var) {
+            $var->setAttribute(AttributeKey::SCOPE, $mutatingScope);
+        }
+    }
+
     private function processMatch(Match_ $match, MutatingScope $mutatingScope): void
     {
         $match->cond->setAttribute(AttributeKey::SCOPE, $mutatingScope);