dangernoodle-io / breadboard / 29667935998

96%

push

github

fix: always compile bb_lifecycle async dispatch (drop CONFIG_BB_LIFECYCLE_ASYNC) (#932)

bb_lifecycle_observe_async() silently stubbed to BB_ERR_UNSUPPORTED when
CONFIG_BB_LIFECYCLE_ASYNC=n (the shipped default), so mdns/mqtt observers
registered against it never actually observed wifi lifecycle transitions
on the floor build. The async substrate (bb_bqueue + a lazily-spawned
drain task) is now always compiled in; the four sizing knobs
(_QUEUE_DEPTH/_TASK_STACK/_TASK_PRIORITY/_TASK_CORE) remain as
Kconfig-tunable but no longer depends-on the removed bool.

- drop the BB_LIFECYCLE_ASYNC Kconfig bool and its C-default bridge
- unwrap bb_lifecycle_async.c's #if/#else stub split into one unconditional path
- delete the now-redundant esp32-lifecycle-async smoke env/sdkconfig and its Makefile target
- update doc comments across bb_lifecycle.h/bb_lifecycle_priv.h/test header to drop the opt-in framing

7437 of 8072 branches covered (92.13%)

Branch coverage included in aggregate %.

12854 of 13068 relevant lines covered (98.36%)

2310.95 hits per line