dangernoodle-io / breadboard / 28696097737

100%

push

github

feat: add bb_dispatch_cmd transport-neutral command registry (B1-543) (#707)

Second member of the bb_dispatch_ family (sibling of bb_dispatch_api,
part of the WS command-plane epic B1-542): a flat action-string keyed
registry dispatching bb_json args/result handlers with an optional
single authorizer checkpoint, no HTTP/transport concept in its public
header. Ships tested-but-unused, like bb_dispatch_api did initially —
no consumer wires it up yet; the reboot proof (B1-544) and envelope
parsing (B1-563, bb_dispatch_envelope) are separate later steps.

- components/bb_dispatch_cmd/ (header + CMakeLists + Kconfig,
  CONFIG_BB_DISPATCH_CMD_CAP default 32)
- platform/host/bb_dispatch_cmd/bb_dispatch_cmd.c — portable impl
  (pure C + bb_json, compiled host + espidf)
- wired into the host native_scaffold.py component map (including the
  bb_log dep the .c file's include actually requires — the flat host
  resolver doesn't propagate transitive REQUIRES like CMake does), the
  native test env's custom_bb_components, and the espidf smoke build
  graph (examples/smoke) to prove it compiles on both backends
- authorizer rejection now returns a distinct BB_ERR_UNAUTHORIZED
  (new bb_core.h code) instead of overloading BB_ERR_INVALID_STATE, so
  a WS consumer can tell "unauthorized" apart from a handler that
  legitimately returns BB_ERR_INVALID_STATE itself
- documented the pointer-lifetime contract on bb_dispatch_cmd_register
  (action) and bb_dispatch_api_add (path): both store the raw pointer,
  callers must pass static/registry-lifetime storage
- test/test_host/test_bb_dispatch_cmd.c — 100% branch/line coverage
  on the new file, including a test asserting the auth-reject code is
  distinct from a handler's own BB_ERR_INVALID_STATE return
- fix: introduce BB_ERR_BASE (0x10000) in bb_core.h's ESP branch so the
  three custom bb_err_t codes (VALIDATION/CONFLICT/UNAUTHORIZED) no
  longer numerically alias ESP-IDF NVS error codes. The prior encoding
  (ESP_ERR_I... (continued)

2262 of 2262 branches covered (100.0%)

Branch coverage included in aggregate %.

3793 of 3793 relevant lines covered (100.0%)

1207.23 hits per line