dangernoodle-io / breadboard / 26265499482

100%

push

github

refactor(bb_http): convert internal callers to streaming json_obj helpers (B1-234) (#304)

* refactor(bb_http): convert internal callers to streaming json_obj helpers (B1-234)

- replace bb_json + bb_http_resp_send_json with bb_http_resp_json_obj_begin/set_*/end in all bb-internal handlers
- convert bb_http_resp_send_err error paths to bb_http_resp_set_status + obj stream + return BB_OK
- convert bb_http_resp_send / bb_http_resp_sendstr to bb_http_resp_send_chunk
- bb_info: keep bb_json tree (extender api takes bb_json_t); serialize + stream via send_chunk
- bb_manifest: keep bb_manifest_emit() (opaque tree); serialize + stream via send_chunk
- bb_diag sockets: snapshot pcb data under lwip lock into heap array; release lock before streaming
- bb_openapi: already using send_chunk directly; no change needed
- zero bb-internal callers of send_json / send_err / send remain in components/ and platform/espidf/
- all existing tests pass (1069/1069); branch coverage 100%

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(bb_http): use BB_ERR_NO_SPACE for OOM, not non-existent BB_ERR_NO_MEM

bb_core defines BB_ERR_NO_SPACE (aliased to ESP_ERR_NO_MEM) — there is no
BB_ERR_NO_MEM. Caught by CI on esp32c3 smoke.

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

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

1334 of 1334 branches covered (100.0%)

Branch coverage included in aggregate %.

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

2268 of 2268 relevant lines covered (100.0%)

900.49 hits per line