dangernoodle-io / breadboard / 28690154309

100%

push

github

fix(bb_mdns): skip browse recreate on delete-enqueue failure to avoid orphaned browse under queue pressure (B1-539) (#696)

- under mDNS action-queue pressure, mdns_browse_delete's enqueue can fail
  (ESP_ERR_NO_MEM/queue full) transiently while the existing browse is still
  live; the periodic refresh previously recreated unconditionally, issuing
  mdns_browse_new against a browse that was never actually torn down and
  orphaning the old handle
- browse_refresh_work_fn now maps the delete's esp_err_t to a pure
  bb_mdns_refresh_delete_rc_t and only recreates when
  bb_mdns_refresh_should_recreate() says it's safe (delete OK or browse
  already gone); on a NO_MEM enqueue failure it increments a throttled skip
  counter, logs once per 16 skips, and retries on the next tick
- happy-path (delete enqueue OK) behavior is unchanged
- new pure, host-tested decision seam (bb_mdns_refresh_decision.{h,c}),
  mirroring sse_pool_reclaim_decision — 100% branch coverage, wired into
  both the device CMakeLists and the native test scaffold

2260 of 2260 branches covered (100.0%)

Branch coverage included in aggregate %.

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

3784 of 3784 relevant lines covered (100.0%)

1201.61 hits per line