PHP 8.0–8.5 native reflection coverage: close API gaps found by audit

[lisachenko]

Summary

Closes the PHP 8.0–8.5 native reflection coverage gaps found by a full audit of the library against native reflection (mechanical API diff + runtime probes against PHP 8.4.19, PHP 8.5 grammar via php-parser 5.8). One commit per audit ticket:

• [PHP8.5] ReflectionEngine pins parser grammar to host PHP version — PHP 8.5 sources (pipe operator, etc.) fail to parse on PHP 8.4 runtime #210 — ReflectionEngine parses with the newest supported PHP grammar instead of the host version, so PHP 8.5 sources (pipe operator, etc.) are reflectable on a PHP 8.4 runtime; an explicit PhpVersion can still be pinned via init().
• [PHP8.4] ReflectionProperty: getHooks() fatals despite hook support; isDynamic() fatals; raw-value/lazy methods don't delegate #207 — ReflectionProperty::getHooks() composed statically from AST hook nodes; isDynamic() answers false; isLazy/getRawValue/setRawValue/setRawValueWithoutLazyInitialization/skipLazyInitialization initialize the internal reflection and delegate.
• [PHP8.1-8.4] Enum case reflection fatals on getModifiers(), hasType()/getType(), isDeprecated() #205/[PHP8.4] Lazy object API (newLazyGhost/newLazyProxy/...) fatals on ReflectionClass and ReflectionEnum instead of delegating #208 — enum cases answer getModifiers()/hasType()/getType()/isDeprecated() statically (and isFinal() now matches native, which reports false for enum cases); the PHP 8.4 lazy-object API (newLazyGhost, newLazyProxy, resetAsLazyGhost/Proxy, initializeLazyObject, isUninitializedLazyObject, markLazyObjectAsInitialized, getLazyInitializer) delegates to the native parent for classes and enums.
• [PHP8.5] final promoted constructor properties: ReflectionProperty::isFinal() returns false #212 — final on promoted constructor properties (PHP 8.5) reflected in isFinal()/getModifiers().
• [PHP8.1-8.3] Function/method introspection added in PHP 8.1–8.3 fatals: tentative return types, closure info, isAnonymous(), isStatic(), createFromMethodName() #204/[PHP8.4] #[\Deprecated] attribute not reflected: isDeprecated() returns false for functions/methods, fatals for class constants #206 — tentative return type accessors, getClosureUsedVariables(), getClosureCalledClass(), ReflectionFunction::isAnonymous()/isStatic(), ReflectionMethod::createFromMethodName() answer statically instead of fataling; isDeprecated() honors #[\Deprecated] (PHP 8.4) for functions, methods and class constants.
• [PHP8.1] NodeExpressionResolver::resolveExprNew() triggers autoloading via class_exists($className) #203/[PHP8.5] Closures and first-class callables in constant expressions are unsupported by NodeExpressionResolver #211 — expression resolver no longer triggers implicit autoloading (class sources are located explicitly through the registered locator); PHP 8.5 closures/arrow functions in constant expressions are resolved to real static closures, and first-class callables of undefined functions fail descriptively.
• [PHP8.0] getAttributes() ignores $flags argument — ReflectionAttribute::IS_INSTANCEOF filtering not supported #202 — getAttributes() honors ReflectionAttribute::IS_INSTANCEOF, resolving attribute-class ancestry without autoloading (native reflection for loaded classes, parser-based otherwise), and validates flags with the native ValueError.
• [PHP8.4/8.5] Missing ReflectionConstant support for global constants (incl. PHP 8.5 attributes on constants and #[\Deprecated]) #209 — new Go\ParserReflection\ReflectionConstant mirrors the native PHP 8.4 API for const-declared global/namespaced constants (the internal class is final, so it cannot be extended), including the PHP 8.5 getAttributes() extension and #[\Deprecated]; exposed via ReflectionFileNamespace::getReflectionConstant(s)().

All static behavior operates purely on the AST — no class_exists()-style calls that can trigger autoloading; methods that inherently need runtime state initialize the internal reflection and delegate to the native parent.

Known residual limitations (documented deliberately)

• Exact-name getAttributes($name) filtering (flags = 0) remains case-sensitive as before, while native is case-insensitive; the new IS_INSTANCEOF path is case-insensitive.
• A constant that is genuinely unresolvable still aborts a whole getConstants() batch (ReflectionFileNamespace::findConstants() / collectSelfConstants() have no per-constant isolation); with the new closure handlers the previously reported symptom no longer occurs.
• ReflectionConstant does not pass instanceof \ReflectionConstant (native class is final); define()-style constants are out of scope for static analysis.

Test plan

• Full PHPUnit suite green after every commit; final state: 13,743 tests, 15,379 assertions, 0 failures, 0 errors (146 skipped, 1 pre-existing incomplete, Inherited methods should contain inherits section for the method __toString #55).
• Each fix ships targeted tests: static-path assertions proving the reflected classes are never loaded (class_exists(..., false) spies, autoload-spy for the resolver) plus native-parity assertions where stubs are loadable on PHP 8.4; PHP 8.5 syntax stubs are parse-only and never included.

---

🤖 Generated with Claude Code

https://claude.ai/code/session_019wBszKXQG1s3UFRqpaRSF1