[phinor]

Fixes rectorphp/rector#9960

RemoveUnusedPrivateMethodParameterRector removed parameters from private methods used as callables. The code that receives a callable decides how many arguments it passes, so the callable then received arguments in the wrong positions. This PR fixes two separate causes.

self::method(...) and static::method(...)

#8793 moved the isFirstClassCallable() early return to the top of PHPStanNodeScopeResolver::processCallLike(). The class name of a first-class callable then got no scope, and NameTypeResolver needs the scope to resolve self and static. It returned an ObjectType for a class literally named self, so PrivateMethodParamRemover::resolveCallers() did not match the call and the isFirstClassCallable() guard never ran.

The early return only has to protect getArgs(), so this PR moves it back below the scope assignments. This is an engine change, not a rule change, so any rule that reads the type of such a class name now gets the real class. The full test suite passes with it.

Array callables

PrivateMethodParamRemover did not look at arrays at all. It now uses ArrayCallableMethodMatcher, as IsClassMethodUsedAnalyzer does, and keeps the parameter for [self::class, 'method'], [static::class, 'method'] and [$this, 'method'].

It also keeps the parameter when the matcher returns ArrayCallableDynamicMethod, for example [$this, $method]. This is deliberately cautious: the rule skips that class's method rather than risk removing a parameter in use.

The guard searches the whole class, so it also covers a callable in a property default.

Not fixed here

ArrayCallableMethodMatcher returns null for arrays passed to register_shutdown_function() and forward_static_call(), so this rule still removes the parameter in that case. The same exclusion makes RemoveUnusedPrivateMethodRector delete a private method that is only registered through register_shutdown_function([$this, 'method']). Changing the matcher also affects other rules, so I left it out of this PR.

Verification

• All nine new skip_* fixtures failed before the fix and pass now. remove_with_array_callable_of_other_class passes both ways: it stops the guard matching a same-named method on another class.
• composer complete-check equivalents pass: fastunit, ECS and PHPStan. bin/rector process --dry-run on this repository reports no changes. I did not run the e2e tests.
• On a real codebase, Rector 2.8.4 with only this rule flagged the file that originally broke at runtime. This branch flags nothing.

🤖 Generated with Claude Code

[samsonasik]

Thank you @phinor