lsm / neokai / 25697314861

82%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(daemon): remove messageHub from StateProjectionService, kill state-manager.ts (M4/M5) (#1862)

* refactor(daemon): remove messageHub from StateProjectionService, kill state-manager.ts re-export (M4/M5)

StateProjectionService is now a pure projection service — no messageHub
dependency, no broadcast methods, no RPC handlers. All client-facing
delivery lives in ClientEventBridge:
- 5 versioned broadcast methods (system, settings, session, SDK messages)
- 7 RPC handler registrations (state snapshot queries)
- All DaemonHub event subscriptions and forwarding

The state-manager.ts compatibility shim is removed; app.ts constructs
ClientEventGateway directly and passes it to the bridge.

* fix(daemon): address review feedback on ClientEventBridge event sourcing

- Subscribe settings.updated and session.updated on InternalEventBus (not
  DaemonHub) to guarantee cache consistency before broadcasts
- Restore fallback broadcast from cached state when full getSessionState()
  fails during teardown races
- Track and clean up RPC handler unsubscribers in stop()
- Add getCachedSessionState() to StateProjectionService for fallback support
- Add InternalEventBus param to ClientEventBridge constructor

Coverage Stats

8347 of 11648 branches covered (71.66%)

Branch coverage included in aggregate %.

169 of 181 new or added lines in 3 files covered. (93.37%)

1 existing line in 1 file now uncovered.

66649 of 79135 relevant lines covered (84.22%)

290.14 hits per line