fix: capture dependencies for dynamic function calls

PHPStan's DependencyResolver resolves non-Name callees ($callable())
via the scope type and records classes referenced by the callable's
signature. Skipping them lost those dependency edges, so a change in
a file referenced only through a dynamic call signature could leave a
stale cache entry. New tripwire pins the PHPStan behavior.

Found by codex review.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: SanderMuller

src/NodeTypeResolver/PHPStan/Scope/PHPStanNodeScopeResolver.php

@@ -805,7 +805,6 @@ private function captureNodeDependencies(Node $node, MutatingScope $mutatingScop
      *   flows from body expressions that are captured by their own nodes
      * - foreach over a literal array: a literal is never an Iterator object,
      *   element classes are captured by the element nodes
-     * - dynamic function calls: PHPStan's branch only handles Name
      * - native PHP function calls: their reflections carry no file and the first
      *   resolution makes PHPStan load its entire function signature map (~100ms,
      *   otherwise never loaded), see NativeFunctionDependencyAssumptionTest
@@ -830,11 +829,7 @@ private function isDependencyFreeNode(Node $node): bool
             return $node->expr instanceof Array_;
         }
 
-        if ($node instanceof FuncCall) {
-            if (! $node->name instanceof Name) {
-                return true;
-            }
-
+        if ($node instanceof FuncCall && $node->name instanceof Name) {
             return $this->nativeFunctionCallAnalyzer->isNativeFunctionCall($node->name);
         }
 

tests/NodeTypeResolver/PHPStan/Scope/DependencyResolverNodeTypesTest.php

@@ -88,4 +88,23 @@ static function (string $className): string {
             ' PHPStanNodeScopeResolver::isDependencyResolvingNode() to match, then this list.'
         );
     }
+
+    public function testFuncCallBranchResolvesDynamicCallees(): void
+    {
+        $reflectionClass = new ReflectionClass(DependencyResolver::class);
+
+        /** @var string $fileName */
+        $fileName = $reflectionClass->getFileName();
+        $source = file_get_contents($fileName);
+        $this->assertIsString($source);
+
+        // the FuncCall branch resolves dynamic callees ($callable()) via the scope type,
+        // so isDependencyFreeNode() must not skip non-Name function calls
+        $this->assertStringContainsString(
+            'getCallableParametersAcceptors',
+            $source,
+            'PHPStan DependencyResolver no longer resolves dynamic callables, review' .
+            ' whether PHPStanNodeScopeResolver::isDependencyFreeNode() can skip non-Name FuncCalls.'
+        );
+    }
 }