Repository navigation
feat(core): observe emissions across an actor system - #5820
Conversation
🦋 Changeset detectedLatest commit: 1f62a03 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Addresses review finding: "Canceled emission listeners remain in memory".
Why
Analytics attached to a root actor miss child emissions. Systems can now observe actor emissions with source identity without forwarding them into machine event handlers.
What changed
createSystem({ onEmitted })and removablesystem.onEmitted(listener)subscriptions.Scope
Observes actor emission requests before runtime delivery across all actors sharing a system. Direct low-level runtime delivery calls remain delivery-only.
Blast Radius
Changes emission execution and lazy system initialization. Regression tests cover constructor and startup emissions, snapshot views, listener failures, unsubscribe, multiple roots, and runtime overrides.
Verification
pnpm test:core --maxWorkers=3passed. 165 files, 2,764 tests.pnpm typecheck,pnpm lint,pnpm format:check,pnpm check:exports, andpnpm buildpassed.