diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-08-04 04:07:41 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-08-04 05:43:24 +0900 |
| commit | 695365a0ad68e4534425c64b7a6a4b6598b68ef7 (patch) | |
| tree | a906972618f08e31012f721c717321bd08164cde /crates/shirabe/src/event_dispatcher/event_subscriber_interface.rs | |
| parent | 261516d5ce6f8b0d69cf9e3da7dd2f0ef1cdc36a (diff) | |
| download | php-shirabe-695365a0ad68e4534425c64b7a6a4b6598b68ef7.tar.gz php-shirabe-695365a0ad68e4534425c64b7a6a4b6598b68ef7.tar.zst php-shirabe-695365a0ad68e4534425c64b7a6a4b6598b68ef7.zip | |
feat(plugin): dispatch plugin event subscribers through the RPC worker
Wires the addPlugin subscriber branch end to end: EventSubscriberInterface
and Capable become fallible and dyn-compatible (their sole implementor is
the PHP plugin proxy, which answers getSubscribedEvents over RPC), listeners
register as Callable::PhpMethod and are invoked with a per-call event handle,
and the R table now drops entries when a child-side stub destructs. The R
table keeps its IndexMap with monotonically increasing handles, so released
handles are never reused and no generation counter is needed.
Upstream has no subscriber-plugin test, so the path is covered by a
Shirabe-owned fixture exercising all three getSubscribedEvents shapes.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/event_dispatcher/event_subscriber_interface.rs')
| -rw-r--r-- | crates/shirabe/src/event_dispatcher/event_subscriber_interface.rs | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/crates/shirabe/src/event_dispatcher/event_subscriber_interface.rs b/crates/shirabe/src/event_dispatcher/event_subscriber_interface.rs index 41d723dc..0a5d88d5 100644 --- a/crates/shirabe/src/event_dispatcher/event_subscriber_interface.rs +++ b/crates/shirabe/src/event_dispatcher/event_subscriber_interface.rs @@ -1,15 +1,26 @@ //! ref: composer/src/Composer/EventDispatcher/EventSubscriberInterface.php use indexmap::IndexMap; +use shirabe_php_rpc::PhpObjHandle; /// Represents one event's subscriber info: method name only, method+priority, or multiple handlers. +#[derive(Debug)] pub enum SubscribedEventEntry { Method(String), MethodWithPriority(String, Option<i64>), Methods(Vec<(String, Option<i64>)>), } +// The sole implementor is the PHP plugin proxy (plugins are the only subscribers in Composer +// itself), so the trait deviates from the PHP shape in two deliberate ways: the PHP-side static +// `getSubscribedEvents()` takes `&self` here (the receiver carries which PHP class to call, and +// an associated function would not be dyn-compatible), and it is fallible because the answer +// crosses the RPC boundary. pub trait EventSubscriberInterface { /// Returns an array of event names this subscriber wants to listen to. - fn get_subscribed_events() -> IndexMap<String, SubscribedEventEntry>; + fn get_subscribed_events(&self) -> anyhow::Result<IndexMap<String, SubscribedEventEntry>>; + + /// The subscriber as it crosses the wire: PHP's `[$subscriber, $method]` array callables + /// capture the subscriber object itself, represented here by its P-table handle. + fn subscriber_handle(&self) -> PhpObjHandle; } |
