dangernoodle-io / breadboard / 29890783441

96%

push

github

refactor: delete dead bb_json-based JSON emit machinery from bb_http_server backends (#992)

B1-1138. The bb_http_resp_json_arr_begin/_emit/_end array-stream API had
zero production callers on any of the three platform backends (espidf,
host, arduino) — its only consumers were its own private send helper and
its own 231-line test file, both deleted here.

On espidf, json_stream_node()/json_child_emit() (the generic JSON-tree
walker used to render arbitrary bb_json_t subtrees) formed a mutually
recursive pair with no external entry point — each was reachable only
by calling the other, which is exactly the shape that hides deadness
from -Wunused-function: neither symbol individually looks unused
because each has a real caller, just not one reachable from outside
the pair.

Drops bb_json from PRIV_REQUIRES in bb_http_server's CMakeLists.txt and
the bb_json.h include on all three backends; removes the public
bb_http_resp_json_arr_* declarations, the bb_http_json_stream_t type,
and the header's local bb_json_t forward-typedef (all unused elsewhere).
bb_http_send_json_error and bb_http_json_obj (the newer non-bb_json
streaming object writer) are unrelated and untouched.

Verified unreachable via grep for the bare symbol names (calls,
&-references, struct/table assignments) across the whole repo including
examples/ — no dynamic dispatch or callback registration found for any
deleted symbol.

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

7426 of 8016 branches covered (92.64%)

Branch coverage included in aggregate %.

12558 of 12720 relevant lines covered (98.73%)

2431.56 hits per line