dangernoodle-io / breadboard / 27322454374

100%

push

github

refactor(bb_openapi): take bb_json_t in validate; keep cJSON private (#382)

bb_openapi.h exposed cJSON in its public API (`const cJSON *value` +
`#include <cJSON.h>`), leaking the json component's surface and only
building because consumers transitively pulled json through bb_json's
public REQUIRES — a latent breakage.

- bb_openapi_validate now takes `bb_json_t value`; drop <cJSON.h> from
  the public header. the impl casts bb_json_t (void* over cJSON*) back
  to const cJSON* internally.
- bb_openapi: declare json in PRIV_REQUIRES (impl uses cJSON directly).
- bb_json: move json from REQUIRES to PRIV_REQUIRES now that no public
  consumer relies on its transitive cJSON propagation.

verified: 1626 host tests pass; smoke green on esp32, esp32c3,
esp32-p4, tdongle-s3.

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

1637 of 1637 branches covered (100.0%)

Branch coverage included in aggregate %.

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

2755 of 2755 relevant lines covered (100.0%)

977.71 hits per line