dangernoodle-io / breadboard / 26205238989

100%

push

github

refactor: tune memory defaults and trim stack buffers (#296)

Group A — lower Kconfig defaults:
- bb_mdns BB_MDNS_EVT_POOL_SIZE: 16 → 8 (-4.7 KB BSS; ~588 bytes/slot)
- bb_event BB_EVENT_MAX_TOPICS: 16 → 8 (-2 KB BSS at default)
- bb_diag BB_DIAG_PANIC_BUF_SIZE: 2048 → 1024 (-1 KB RTC slow memory)

Group B — new Kconfig tunables (default preserves prior behavior):
- BB_LOG_TAG_REGISTRY_MAX (default 24, was hardcoded 32; -320 B BSS)
- BB_HTTP_ROUTE_REGISTRY_CAP (default 64, was hardcoded 64; no BSS change)
- BB_MDNS_TXT_PENDING_MAX (default 4, was hardcoded 8; -324 B BSS)
- BB_UPDATE_CHECK_CUSTOM_PARSER_BUF_BYTES (default 8192, was 16384; -8 KB heap per custom-parser fetch)

Group C — stack/buffer trims:
- bb_ota_pull s_releases_url[512] → [256] (-256 B BSS; GitHub URLs ~120-180 B)
- bb_update_check publish_state payload[512] → [256] (-256 B stack; payload ~150-200 B)
- bb_mdns_util bb_mdns_build_hostname: 4×256 B stack locals → single work[256] (-768 B peak stack)

Total BSS savings at default config: ~9.3 KB
Custom-parser heap saving per fetch: 8 KB (freed immediately after call)
Peak stack saving in bb_mdns_build_hostname: 768 B

Also: add overflow warning log to custom parser buffer path, add
bb_mdns_util.c to native test scaffold, and add 18 new tests to
achieve 100% branch coverage on the refactored bb_mdns_build_hostname.

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

1206 of 1206 branches covered (100.0%)

Branch coverage included in aggregate %.

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

2088 of 2088 relevant lines covered (100.0%)

921.2 hits per line