dangernoodle-io / breadboard / 29718212175

96%

push

github

feat: bb_http_section dispatch helper + bb_data parse/commit split (B1-828) (#960)

- components/bb_data: split bb_data_apply() into bb_data_parse() (slot/
  format lookup + wire-body decode) and bb_data_commit() (seed/populate/
  apply); bb_data_apply() is now a thin wrapper composing the two, kept
  behaviorally identical for every existing caller (bb_wifi_http,
  bb_storage_http, bb_ota_check on jae/ota-check-migrate)
- components/bb_http_server: new bb_http_section helper -- a shared
  method+prefix dispatch (portable registry + ESP-IDF glue, mirrors
  bb_diag_section) that drives bb_data_parse()/bb_data_commit() directly
  so its err->status mapping (bb_http_section_status_for_apply) is correct
  by construction from the parse/commit stage tag, not guessed from the
  bb_err_t alone
- mechanism only: no existing route is converted onto the helper in this
  PR; host tests cover the full status-mapping matrix plus an end-to-end
  bb_data-backed apply() to prove it is not dead code
- bb_wifi_http_routes.c and bb_storage_http_routes.c now share
  bb_http_req_recv_body_stack(); this is an intentional error-text
  unification, not drift -- their 400 body errors changed from
  "missing or oversized body" / "read failed" to the shared
  "missing, oversized, or unreadable body", and a request body of
  exactly the documented max size is now accepted in full instead of
  being silently truncated by one byte
- test/test_host/test_bb_http_section.c: add 3 count_path_segments()
  edge-case tests (double-slash prefix, trailing double slash, interior
  double slash) proving consecutive slashes never create phantom
  segments (review LOW finding)


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

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

7521 of 8140 branches covered (92.4%)

Branch coverage included in aggregate %.

115 of 115 new or added lines in 6 files covered. (100.0%)

12725 of 12899 relevant lines covered (98.65%)

1188.25 hits per line