feat: Support first-class callable syntax for proxy method/function invocations

[Copilot]

The framework previously relied on ReflectionMethod::invokeArgs / ReflectionFunction::invokeArgs and Closure::bind tricks inside invocation constructors to locate and call original methods. This adds first-class callable support, passing the original method/function reference directly into each joinpoint constructor as a required parameter.

Invocation classes

All $closureToCall parameters are now required (non-nullable) — this is a BC break for all invocation class constructors.

• AbstractMethodInvocation: gains a required Closure $closureToCall parameter (4th argument) stored as protected readonly. Child classes may wrap or rebind it as needed.
• DynamicTraitAliasMethodInvocation: stores the callable in the parent but dispatches via ReflectionMethod::invokeArgs($this->instance, $this->arguments) for reliability and performance. Internally resolves a private readonly ReflectionMethod $originalMethodToCall — the __aop__ alias for own methods, or getPrototype() for inherited ones. ReflectionMethod::invokeArgs is used because Closure::call() has known issues with pass-by-reference arguments (PHP bug #72326) and ReflectionMethod::invoke is surprisingly fast.
• StaticTraitAliasMethodInvocation: wraps the callable in a static fn(array $args) => forward_static_call($callable, ...$args) shim stored in $closureToCall; bindTo(null, $scope) forwards the correct LSB class per call.
• ReflectionFunctionInvocation: calls the callable directly via ($this->closureToCall)(...$this->arguments) in proceed().

InterceptorInjector

forMethod(), forStaticMethod(), and forFunction() each require a Closure $closureToCall parameter that is forwarded to the respective invocation constructor.

Proxy generators

Generated proxy method bodies now always pass a first-class callable as the 4th argument:

// Class-own method (has a __aop__ trait alias):
static $__joinPoint = InterceptorInjector::forMethod(self::class, 'getReport', [...], $this->__aop__getReport(...));

// Inherited method (no alias) — same pattern for both static and dynamic:
static $__joinPoint = InterceptorInjector::forMethod(self::class, 'getReport', [...], parent::getReport(...));

// Static method:
static $__joinPoint = InterceptorInjector::forStaticMethod(self::class, 'create', [...], self::__aop__create(...));

// Function proxy (leading backslash = global function, avoids calling the namespace-scoped proxy):
static $__joinPoint = InterceptorInjector::forFunction('file_get_contents', [...], \file_get_contents(...));

Tests

• testReflectionDispatchCallsEachInstanceCorrectly: verifies that the static singleton joinpoint correctly dispatches to each distinct caller instance via reflection.
• testDispatchInvokesOriginalMethodBody: asserts that dispatch routes through the __aop__ alias body, not the overridden public method.
• testInheritedMethodDispatchCallsEachInstanceCorrectly: verifies that inherited methods (no trait alias) dispatch correctly via getPrototype().
• testFirstClassCallableLsbWithSubclassScope: verifies LSB is forwarded correctly when the static callable is invoked from a subclass scope.
• ReflectionFunctionInvocationTest: covers the callable path for function interception.
• testGenerateProxyForInheritedMethodDoesNotCreateTraitAlias (extended): asserts that the generated proxy for an inherited instance method contains parent::method(...) as the first-class callable.
• testGenerateProxyForInheritedStaticMethodUsesParentCallable (new): asserts that the generated proxy for an inherited static method contains parent::method(...) as the first-class callable and no __aop__ alias, e.g.:

  static $__joinPoint = InterceptorInjector::forStaticMethod(self::class, 'staticSelfPublic', [...], parent::staticSelfPublic(...));

[lisachenko]

@copilot phpstan is failing. Also, I haven't seen any examples in generated proxies where parent::method(...) will be used, could you please generate a test scenario of interception of inherited public or protected method from parent class and ensure it works (at least for static methods where this syntax is useful)

[Copilot]

@copilot phpstan is failing. Also, I haven't seen any examples in generated proxies where parent::method(...) will be used, could you please generate a test scenario of interception of inherited pub...

Fixed in d26b543:

PHPStan: The error was a pre-existing nullCoalesce.offset warning in AopComposerLoader.php line 109. PHPStan correctly determined that $loader[0] always exists for array&callable types, so ?? null was redundant. Removed it — PHPStan now passes with no errors.

Generated proxy examples with parent::method(...): Added two tests to ClassProxyGeneratorTest:

• testGenerateProxyForInheritedMethodDoesNotCreateTraitAlias (extended with parent:: assertion) — verifies an inherited instance method like FirstStatic::publicMethod() (declared in First) generates parent::publicMethod(...) as the callable:

  static $__joinPoint = InterceptorInjector::forMethod(self::class, 'publicMethod', ['test'], parent::publicMethod(...));

• testGenerateProxyForInheritedStaticMethodUsesParentCallable (new) — verifies an inherited static method like FirstStatic::staticSelfPublic() generates parent::staticSelfPublic(...):

  static $__joinPoint = InterceptorInjector::forStaticMethod(self::class, 'staticSelfPublic', ['test'], parent::staticSelfPublic(...));