lsm / neokai / 25712881253

81%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(daemon): migrate session/agent/query/config events to InternalEventBus (#1866)

* refactor(daemon): migrate session, agent, query, and config events from DaemonHub to InternalEventBus

Moves all session lifecycle, agent, message, model, question, rewind, and
config-related event emissions from DaemonHub to InternalEventBus. Updates
ClientEventBridge to subscribe to both buses, removes redundant bridge
subscriptions in DaemonApp, and wires internalEventBus through all handler
constructors and RPC handler call sites.

* fix(daemon): add namespaceId to migrated event payloads, pass internalEventBus to SessionManager

Rebase on dev introduced namespaceId requirement for all InternalEventBus
payloads. This commit adds namespaceId to all publish/publishAsync calls
in the migrated publishers and updates test assertions accordingly.

Also fixes:
- Pass internalEventBus to SessionManager constructor in app.ts
- Update daemon-app-cleanup test thresholds for session cleanup delays
- Add timeout to slow daemon-app-cleanup test
- Fix syntax error in query-lifecycle-manager test assertion

* fix(daemon): clean up rebase conflict artifacts in test files

Remove stray commit message fragments and duplicate parameters left by
git conflict resolution during rebase on dev.

* fix(daemon): revert events with DaemonHub subscribers back to daemonHub.emit

Events that have unmigrated DaemonHub subscribers must remain on
DaemonHub because the app.ts bridge only forwards DaemonHub →
InternalEventBus, not the reverse direction. Events with all
subscribers already on InternalEventBus stay there.

Key changes:
- Revert session.created/deleted/updated, session.error/errorClear,
  api.connection, commands.updated, context.updated, agent.interruptRequest
  back to daemonHub.emit (DaemonHub subscribers exist)
- Keep messages.statusChanged, sdk.message, message.sent, session.reset,
  query.trigger on InternalEventBus (subscribers already migrated)
- Fix namespaceId: remove f... (continued)

Coverage Stats

8397 of 11700 branches covered (71.77%)

Branch coverage included in aggregate %.

241 of 290 new or added lines in 23 files covered. (83.1%)

888 existing lines in 6 files now uncovered.

66366 of 79536 relevant lines covered (83.44%)

289.4 hits per line