dangernoodle-io / breadboard / 29974553672

96%

push

github

refactor: dissolve bb_log_http into bb_diag_http (#1012)

- folds GET/POST /api/log/level (already log-namespaced) into bb_diag_http,
  mirroring PR #1008's bb_storage_http dissolution (KB 1477): an SSOT
  component owns no routes, and its own HTTP surface is the sole route home
- removes the bb_log_http component entirely (CMakeLists, Kconfig, README);
  bb_log_http.c git-mv'd into platform/espidf/bb_diag_http/, keeping its
  file name; bb_log_http.h git-mv'd into components/bb_diag_http/include/
  (this component's own fourth public header)
- keeps CONFIG_BB_LOG_ROUTES and CONFIG_BB_LOG_SSE_KEEPALIVE_MS unchanged so
  any sdkconfig.defaults already setting them still works
- adds ZERO new CMakeLists dep edges: bb_http_server was already a public
  REQUIRES and bb_log already a PRIV_REQUIRES on bb_diag_http; bb_log itself
  gains no edge back onto bb_diag_http/bb_http_server, preserving the
  bb_http_server<->bb_log cycle-break bb_log_http existed to provide
  (components/bb_log/CMakeLists.txt only gets its precedent comment updated
  to name bb_diag_http instead of the now-dissolved bb_log_http)
- prunes the bb_log_http entry from the new_component fence baseline
  (shrink-only)
- folds bbtool.toml's bb_log_http smoke capability entry into the existing
  bb_diag_http entry (no addition needed, it was already listed)
- regenerates components/README.md and components/bb_diag_http/README.md;
  hand-updates bb_log_event's free-text "See also" cross-reference (outside
  the generated marker regions) and the bb_log_http.c path-comments in
  test_bb_openapi_validate.c/test_route_fidelity.c (no logic change)

Closes B1-1155


Claude-Session: https://claude.ai/code/session_01UwuRBPS8n5TkVkog1RQyVF

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

7386 of 7976 branches covered (92.6%)

Branch coverage included in aggregate %.

12530 of 12692 relevant lines covered (98.72%)

2497.66 hits per line