dangernoodle-io / breadboard / 25981766987

100%

push

github

perf(bb_update_check): lazy-allocate response body buffer (#243)

The 32 KB body buffer (CONFIG_BB_UPDATE_CHECK_BODY_SIZE) was previously
allocated once in bb_update_check_init() and held for the device lifetime.
On memory-tight boards (bitaxe650 with ~45 KB free pre-init) this consumed
65% of available headroom and pushed TLS handshakes during fetch into OOM.

Move the calloc into bb_update_check_run_one() so the buffer only exists
during the brief HTTP fetch + parse window (1-3 s, every poll interval),
freeing the 32 KB while idle. Fetch latency is unchanged; idle heap
recovers fully.

A streaming JSON parser would eliminate the transient spike too — filed
as a follow-up backlog item; lazy alloc is enough at production poll
cadences (1 h / 6 h).

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

752 of 752 branches covered (100.0%)

Branch coverage included in aggregate %.

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

1343 of 1343 relevant lines covered (100.0%)

1115.65 hits per line