dangernoodle-io / breadboard / 29711421781

96%

push

github

feat: add bb_data ingress apply API, cut wifi-creds PATCH onto it (B1-1022) (#953)

Extends bb_data (B1-832) in place with the write-half mirror of
bb_data_render(): an optional apply hook on the binding plus
bb_data_apply(), which seeds a caller-owned dst_scratch (zero for POST,
via the binding's own gather() for PATCH), decodes wire bytes through
the bb_serialize format registry's typed .parse slot (B1-1030), scatters
via bb_serialize_populate(), then calls apply(). bb_data stays
HTTP-agnostic throughout.

Collapses PATCH /api/wifi's hand-rolled bb_json parse/validate onto
bb_data_apply() via a new "wifi" ingress binding, seeded from the
staged pending creds so an omitted password now genuinely keeps the
last-staged value (PATCH semantics the old handler never offered).
Preserves reject-on-oversize UX (vs bb_serialize_populate's truncate
contract) by oversizing the descriptor's ssid/pass buffers past the
real BB_WIFI_PENDING_SSID_MAX/PASS_MAX limits and re-checking with the
new bb_wifi_pending_validate_buf() (a truncation-safe strnlen variant),
a pure/host-testable helper alongside bb_wifi_pending_validate().
Response shaping (the 202 body) stays in the route; the downstream
bb_wifi_reconfigure() commit is unchanged.

7414 of 8032 branches covered (92.31%)

Branch coverage included in aggregate %.

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

12604 of 12778 relevant lines covered (98.64%)

1194.98 hits per line