From 436e12381dd79e419dce755718be17b66d73e17f Mon Sep 17 00:00:00 2001 From: nsfisis Date: Tue, 9 Jun 2026 00:09:08 +0900 Subject: feat(event-dispatcher): unify event dispatch via EventInterface trait Extract a superclass-trait EventInterface from the base Event. pool_builder's PrePoolCreateEvent stays deferred: its constructor needs owned, non-cloneable Request and repository boxes the builder only holds by reference (owned-payload blocker). The event is plugin-only, so its construction is re-tagged TODO(plugin). Co-Authored-By: Claude Opus 4.8 (1M context) --- crates/shirabe/src/dependency_resolver/pool_builder.rs | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) (limited to 'crates/shirabe/src/dependency_resolver') diff --git a/crates/shirabe/src/dependency_resolver/pool_builder.rs b/crates/shirabe/src/dependency_resolver/pool_builder.rs index 8647cef..b96dc79 100644 --- a/crates/shirabe/src/dependency_resolver/pool_builder.rs +++ b/crates/shirabe/src/dependency_resolver/pool_builder.rs @@ -299,8 +299,10 @@ impl PoolBuilder { } if self.event_dispatcher.is_some() { - // TODO(phase-b): PrePoolCreateEvent::new takes Request and Vec> - // by value but neither can be cloned (PHP class shared semantics). Needs Rc-based migration. + // TODO(plugin): PrePoolCreateEvent::new takes Request and Vec> + // by value but neither can be cloned (PHP class shared semantics). This event is purely + // plugin-facing and nothing in the no-plugin path reads it back, so it stays deferred + // until the plugin API drives an Rc-based migration of Request/repositories. let mut pre_pool_create_event = PrePoolCreateEvent::new( PluginEvents::PRE_POOL_CREATE.to_string(), todo!("share repositories with PrePoolCreateEvent without moving"), @@ -315,12 +317,15 @@ impl PoolBuilder { .cloned() .collect(), ); - // TODO(phase-b): EventDispatcher::dispatch expects an owned Event, not &mut PrePoolCreateEvent + let pre_pool_create_event_name = pre_pool_create_event.get_name().to_string(); self.event_dispatcher .as_ref() .unwrap() .borrow_mut() - .dispatch(Some(pre_pool_create_event.get_name()), None)?; + .dispatch( + Some(&pre_pool_create_event_name), + Some(&mut pre_pool_create_event), + )?; // PHP rebinds $this->packages to a list-style array; preserve indices via reindexing. // TODO(plugin)/TODO(phase-c): rebind self.packages from the (handle-based) event packages // once EventDispatcher::dispatch returns the mutated event. -- cgit v1.3.1