dangernoodle-io / breadboard / 29871722431

96%

push

github

feat: add bb_mdns_cache wire descriptor (#983)

- adds bb_mdns_cache_entry_wire_desc/_fill (components/bb_mdns_cache/bb_mdns_cache_wire.c
  + bb_mdns_cache_wire_priv.h), a static bb_serialize_desc_t for a single mDNS peer entry
- mirrors bb_diag_boot_wire.c/bb_wifi_http_wire.c's additive/inert staging: no
  bb_data_bind() call, entry_serialize() and its bb_cache .serialize registration
  are untouched -- this is a second, parallel path
- shape deliberately diverges from the legacy entry_serialize() emitter: TXT fields
  render as a bounded "txt":[{"key","value"}, ...] array instead of being spliced flat
  into the top-level object, since TXT keys are runtime-configured by the consumer and
  cannot be named by a static field table (documented in bb_mdns_cache_wire_priv.h)
- fill()'s cap-overflow (more configured TXT fields than BB_MDNS_CACHE_WIRE_TXT_MAX)
  surfaces via a nullable `out_dropped` out-param rather than dropping silently; the
  fill itself stays pure/log-free, matching bb_mdns_cache.c's own bb_log_w precedent
  for the identical BB_MDNS_CACHE_REQUERY_TXT_MAX case, which now _Static_assert()s
  the two caps stay in lockstep
- TXT value reads are bounded by min(field->dest_len, sizeof(value)), not just
  sizeof(value), removing reliance on the unenforced NUL-within-dest_len invariant
- host tests pin the exact rendered JSON for both a TXT-less and a TXT-configured peer,
  plus the null-key/out-of-bounds/dest_len-bound/cap-overflow-and-drop-signal branches

7456 of 8052 branches covered (92.6%)

Branch coverage included in aggregate %.

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

12625 of 12787 relevant lines covered (98.73%)

1209.42 hits per line