From 6ae7e0d4b3eaf20e2d2cd3d000cf61ab0c9b6e83 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Thu, 11 Jun 2026 02:39:35 +0900 Subject: feat(console): resolve phase-b TODOs in doRun and IO wiring Wire up ConsoleIO with HelperSet/QuestionHelper, register the ErrorHandler with the IO instance, and fall back to a default output in run(). Replace resolved phase-b TODOs across the console, command, io, factory, installer, dependency_resolver, and util modules; reclassify the remaining blockers (typed Symfony command registry, stdin resource caching) as phase-c. Co-Authored-By: Claude Opus 4.8 (1M context) --- crates/shirabe/src/event_dispatcher/event_dispatcher.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'crates/shirabe/src/event_dispatcher') diff --git a/crates/shirabe/src/event_dispatcher/event_dispatcher.rs b/crates/shirabe/src/event_dispatcher/event_dispatcher.rs index 729979d..2124093 100644 --- a/crates/shirabe/src/event_dispatcher/event_dispatcher.rs +++ b/crates/shirabe/src/event_dispatcher/event_dispatcher.rs @@ -267,10 +267,11 @@ impl EventDispatcher { } // other newly appeared prepended autoloaders should be appended instead to ensure Composer loads its classes first + // PHP: spl_autoload_unregister($cb); spl_autoload_register($cb, true, $prepend); // TODO(plugin): ClassLoader detection via instanceof — currently treat all callbacks uniformly - // TODO(phase-b): `cb` is a PhpMixed; spl_autoload_*/register expect a typed - // Box PhpMixed + Send + Sync> callback. Bridging requires - // exposing the underlying callable from PhpMixed. + // TODO(phase-c): `cb` is a PhpMixed holding a callable; spl_autoload_register/unregister + // (php-shims that stay todo!()) need a typed Box PhpMixed> callback. + // Bridging requires the callable model to expose the underlying closure from PhpMixed. let _ = &cb; let _ = spl_autoload_unregister; let _ = spl_autoload_register; -- cgit v1.3.1