dangernoodle-io / breadboard / 30520226531

97%

push

github

feat: register wifi bb_lifecycle service in smoke composition (B1-1037 PR-1) (#1149)

Adds a second // bbtool:init manifest entry to examples/smoke/main/bb_wire.h
registering the "wifi" bb_lifecycle service at tier=early via
bb_lifecycle_register(), using out= to declare the file-scope
bb_lifecycle_svc_t handle. Lands in bb_wire.h rather than a new manifest
file because bbtool codegen --consumer-manifest accepts exactly one path.

This is deliberately INERT: registered-but-unstarted (BB_LIFECYCLE_STOPPED)
is a harmless no-op, and it does NOT by itself restore bb_mdns/
bb_mqtt_client wifi-service auto-start -- the service only reaches
RUNNING once bb_wifi_set_emit() is wired to drive it, which is PR-4
(out of scope here).

Verified: examples/smoke generated bb_app_init.c emits the registration
call + static declaration in bb_app_init_early(); examples/floor generated
output (handwired, no consumer-manifest) is byte-identical before/after.
make check/test/coverage pass with 0 new fence markers and 0 new coverage
markers; smoke-esp32 and smoke-esp32c3 build clean.

Size impact (clean smoke-esp32/smoke-esp32c3 builds, measured against
this commit's rebased parent origin/main acc8e13d -- an earlier revision
of this message measured against the pre-rebase parent 312a544a and is
stale, see PR discussion): esp32 total flash +68 B (.text +324 B,
.rodata/.appdesc -256 B), DRAM .bss -8 B, DRAM .data unchanged; esp32c3
total flash +166 B (.text +430 B, .rodata/.appdesc -264 B), DRAM .bss
+8 B, DRAM .data unchanged. nm --size-sort -S confirms bb_lifecycle.c
s_services array (0x120/288 B BSS) is already linked pre-PR on both
targets -- bb_lifecycle_autoinit(), already wired before this change,
walks it regardless of registrations, so bb_lifecycle_register() does
not newly pull it in. The only real addition is the out= handle (static
bb_lifecycle_svc_t s_smoke_wifi_svc, a 4 B BSS pointer) emitted into
generated bb_app_init.c on both targets; the remaining swings are... (continued)

7973 of 8576 branches covered (92.97%)

Branch coverage included in aggregate %.

13540 of 13695 relevant lines covered (98.87%)

2611.2 hits per line