[Feature] AOP-unaware container: aspects as typical services, engine-probed lazy proxies

[lisachenko]

Summary

Makes Go\Core\Container a genuinely generic, AOP-unaware DI container and removes the hot-path overhead flagged in review: the per-class isLazyProxyCompatible() reflection walk on the service read path, and validateAspectRegistration() running twice (plus a partial third time in debug) per aspect initialization.

Aspects are typical services

registerAspect(), materializeAspect(), validateAspectRegistration(), the $factoryValidators map and isDebug() are removed from the container and its interface. Aspects register through the existing generic API from configureAop():

$container->add(MonitorAspect::class, new MonitorAspect());                       // eager
$container->addLazyService(LoggingAspect::class, static fn() => new LoggingAspect($logger)); // deferred

Aspect enumeration on the weaving path is unchanged — getServicesByInterface(Aspect::class) already finds deferred aspects by id. Since a factory closure is now always present, the old "default-constructible without factory" validation is moot; a factory returning the wrong type is caught once by the container's instanceof guard on first use.

Interface-keyed registration listeners (onRegistration)

The one thing that genuinely could not wait for materialization — debug-mode tracking of aspect source files for cache freshness — becomes a generic container feature: onRegistration(Aspect::class, $listener) fires with the id (never the value, so laziness is fully preserved) whenever a matching deferred service is registered. The kernel arms a single listener in debug mode only; in production no listener exists and addLazyService() is a pure array write with zero autoloading. addResource() joins the interface as public for this.

Engine-probed lazy proxies (Go\Core\NativeLazyProxy)

The 30-line isLazyProxyCompatible() predicate (property enumeration + parent-chain reflection walk, run on first touch of every service, every request) is deleted. The engine already knows every incompatibility: newLazyProxy() throws for classes PHP cannot make lazy (memoized for worker runtimes), and one isUninitializedLazyObject() call detects property-less classes whose initializer would never run. In the common compatible case the whole check is one engine-level boolean. Interceptor::createLazily() now uses the same helper via its trusted create() path, unifying the two previously diverged lazy-proxy sites. The probe also retires the stale readonly-before-8.5 version gate — current PHP 8.4.x already supports readonly lazy proxies.

Also

• Framework service wiring moved out of the container constructor into FrameworkServices::register(), called by the kernel during init() — new Container() registers nothing.
• getService() delegates to getValue() + instanceof guard instead of duplicating the materialize-or-throw block.
• Container throws SPL exceptions (InvalidArgumentException, UnexpectedValueException) instead of Go\Aop\AspectException — no Go\Aop imports remain in Container.

BC breaks (4.0-dev)

1. AspectContainer::registerAspect() removed — use add() / addLazyService().
2. AspectContainer gains onRegistration() and public addResource(); custom containerClass implementations must follow.
3. Registration-time aspect validation errors are gone; wrong factories fail on first use.
4. Bare Container construction registers no framework services — call FrameworkServices::register() (the kernel does).
5. Container exception types changed to SPL.

Validation

• composer cs — clean (PER-CS)
• composer analyze — clean (PHPStan level 10)
• composer test under PHP 8.4 — 2666 tests, 3312 assertions, 0 failures (10 pre-existing skips). The functional suite fails under this environment's PHP 8.5.10 identically on master (pointcut parser issue unrelated to this change).

🤖 Generated with Claude Code

https://claude.ai/code/session_01W6rjmxdekGoJdXwwxDjhD1

---

Generated by Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.40%. Comparing base (57221fa) to head (e672fa2).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #658      +/-   ##
============================================
+ Coverage     85.32%   85.40%   +0.08%     
+ Complexity     1656     1636      -20     
============================================
  Files            97       99       +2     
  Lines          4545     4509      -36     
============================================
- Hits           3878     3851      -27     
+ Misses          667      658       -9     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.