dangernoodle-io / breadboard / 31135574339

97%

push

github

refactor: collapse bb_http_serialize_stream_compose/_ex ladder onto a cfg struct (B1-1438) (#1245)

bb_http_serialize_stream_compose() now takes a single bb_http_serialize_stream_compose_cfg_t (groups/n_groups/f64_shortest), same idiom as B1-1437's bb_serialize_json_stream_compose_render_cfg_t; bb_http_serialize_stream_compose_ex() is retired, no second entry point survives.

bb_health_compose_and_stream() (the only f64_shortest=true caller) passes cfg.f64_shortest = true explicitly -- zero-init would silently flip /api/health's float formatting back to fixed-decimal.

Host tests exercise cfg.f64_shortest true/false/omitted (omitted proves zero-init reproduces the pre-collapse fixed-decimal default) plus a new null-cfg BB_ERR_INVALID_ARG case.

One coverage baseline marker relocated 73->72 in bb_http_serialize_stream.c -- pure line-shift from removing the thin wrapper function (same statement, 'if (render_err != BB_OK) return render_err;', at both line numbers), not a real coverage gap.


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

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

8536 of 9076 branches covered (94.05%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 2 files covered. (100.0%)

14281 of 14436 relevant lines covered (98.93%)

2647.44 hits per line