lsm / neokai / 21810677840

82%

Build Type

push

github

Committed by web-flow

Commit Message

refactor: simplify MessageHub protocol from 11 to 6 message types (#88)

* refactor: simplify MessageHub protocol from 11 to 6 message types

Replace WAMP-inspired RPC+pub/sub protocol with room-based command/query/event model.

- Remove CALL/RESULT/ERROR/SUBSCRIBE/SUBSCRIBED/UNSUBSCRIBE/UNSUBSCRIBED message types
- Add COMMAND (CMD), QUERY (QRY), RESPONSE (RSP) message types alongside EVENT/PING/PONG
- New API: command/onCommand, query/onQuery, event/onEvent, joinRoom/leaveRoom
- Add RoomManager for room-based event delivery (global + session rooms)
- Migrate all 86 daemon RPC handlers to onQuery/onCommand
- Migrate all client stores, hooks, and components to new API
- Delete cache.ts (~200 lines) and subscription-manager.ts (~150 lines)
- Reduce message-hub.ts from ~1500 to ~860 lines
- Reduce router.ts from ~718 to ~375 lines
- Rewrite all shared package tests for new protocol

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: update test assertions for room-based MessageHub API

- Unit tests: sessionId → room in event spy assertions (6 files)

- Integration tests: Add event() to MockMessageHub, fix RSP vs EVENT

  ordering in WebSocket tests, waitForWebSocketMessageType helper

- RPC handlers: question.respond/saveDraft/cancel and settings.session.update

  changed from onCommand to onQuery (return values needed)

- Components: sessionId → room in sdk-message-handler assertions

- Lint: remove unused startTime variable

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: add self-delivery for server-side event handlers

Server-side event() calls now also invoke local onEvent() handlers
(self-delivery). Previously, events were only routed to WebSocket
clients via the router, so server-side listeners (e.g., test helpers,
state observers) never received events.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: add room joins in online test helpers for event delivery

Online test helpers (waitForProcessingState... (continued)

Coverage Stats

462 of 553 new or added lines in 31 files covered. (83.54%)

1044 existing lines in 14 files now uncovered.

11628 of 13546 relevant lines covered (85.84%)

371.8 hits per line