dangernoodle-io / breadboard / 26004487197

100%

push

github

fix(bb_update_check): add bb_update_check_kick() non-blocking variant for HTTP handlers (#262)

PR #261 changed /api/ota/check to call bb_update_check_now(), which runs the
entire TLS handshake and manifest parsing synchronously on the HTTP handler's
stack. On tight-heap boards like bitaxe-650, the httpd worker task stack
(typically 4 KB, CONFIG_HTTPD_STACK_SIZE) overflows when the mbedTLS handshake
transient (5–8 KiB peak) runs on the caller's stack, corrupting heap metadata
and triggering a stack-overflow panic.

bb_update_check_now() remains synchronous and suitable for contexts with
≥8 KB stack (worker task, test harnesses). New non-blocking API:

  bb_err_t bb_update_check_kick(void)

On ESP-IDF, posts the semaphore that wakes the worker task to run the check
on its own dedicated 8 KiB stack (BB_HTTP_CLIENT_TASK_STACK). Returns
immediately; the check result lands in bb_update_check_get_status() after
the worker completes. On host/Arduino, provides a synchronous stub that calls
bb_update_check_now() (no worker task available).

Update /api/ota/check handler to use bb_update_check_kick() instead. Document
both functions in CLAUDE.md with stack requirements and blocking semantics.

Add host test that verifies bb_update_check_kick() returns BB_OK and drives
a check on the host stub.

Panic evidence on bitaxe-650:
  panic_reason: "***ERROR*** A stack overflow in task httpd has been detected."

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

960 of 960 branches covered (100.0%)

Branch coverage included in aggregate %.

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

1724 of 1724 relevant lines covered (100.0%)

1019.37 hits per line