dangernoodle-io / breadboard / 25034618802

100%

push

github

refactor(bb_log): split bb_log_routes (HTTP/SSE) from bb_log core (B1-92) (#117)

bb_log used to REQUIRE bb_http (for the /api/logs SSE endpoint), which
transitively pulled bb_wifi, bb_system, esp_http_server, esp_wifi —
meaning any consumer of bb_log (which is most of them, including
bb_display) compiled the full HTTP/WiFi stack even when they only
needed logging.

- new component bb_log_routes owns the HTTP route registration:
  - platform/espidf/bb_log/{bb_log_routes,bb_log_http}.c moved under
    platform/espidf/bb_log_routes/
  - new public header components/bb_log_routes/include/bb_log_routes.h
    declares bb_log_stream_register_routes + bb_log_register_routes
  - REQUIRES bb_log bb_nv bb_http bb_json esp_http_server
- bb_log core REQUIRES drops to: esp_ringbuf freertos log bb_nv
  (added bb_nv for bb_err_t typedef)
- bb_log.h drops the two route-register prototypes — clean break, link
  error tells consumers to add bb_log_routes to REQUIRES

Unmasked latent missing REQUIRES (no longer transitively satisfied via
bb_log → bb_http → ...):
- bb_board: + bb_http
- bb_system: + esp_app_format
- bb_wifi: + bb_http (for bb_wifi_routes.c)

Verified: smoke-elecrow-p4-hmi7 SUCCESS, smoke-esp32-wroom-32 SUCCESS,
host tests 295/295. arduino-uno-cc3000 not built locally (avr-g++
arm64/x86_64 env issue); CI Linux runner exercises it.

Downstream impact: TaipanMiner and snugfeather currently call
bb_log_stream_register_routes / bb_log_register_routes via bb_log.h —
they will need to add `bb_log_routes` to REQUIRES and switch the
include to `bb_log_routes.h`. Follow-up items will be filed.

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

272 of 272 branches covered (100.0%)

Branch coverage included in aggregate %.

431 of 431 relevant lines covered (100.0%)

146.06 hits per line