lsm / neokai / 25470644606

81%

push

github

Add InternalCommandBus facade for semantic daemon commands (#1811)

* Add InternalCommandBus facade for semantic daemon commands

Implements InternalCommandBus with one-handler-per-command semantics,
structured CommandResult returns, and explicit error types for missing
and duplicate handlers. Follows the same facade pattern as InternalEventBus.

Includes the first canonical command contract: agent.message.inject.

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

* Guard unsubscribe against removing a newer handler

The unsubscribe closure now checks that the current handler at the
key is the same instance it registered before calling delete. This
prevents a stale unsubscribe from silently removing a re-registered
handler and causing confusing MissingCommandHandlerError failures.

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

* Normalize handler exceptions to CommandResult in dispatch

dispatch() now catches thrown/rejected errors from the handler and
returns a structured { ok: false, error } result instead of letting
the raw exception escape. This keeps the command API consistent —
callers only need to check result.ok rather than wrapping every
dispatch in try/catch.

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

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

8199 of 11342 branches covered (72.29%)

Branch coverage included in aggregate %.

54 of 54 new or added lines in 1 file covered. (100.0%)

62953 of 74989 relevant lines covered (83.95%)

287.93 hits per line