dangernoodle-io / breadboard / 29032180773

95%

push

github

refactor: dissolve the failed DI/autowire attempt onto codegen composition (#795)

* refactor: drop floor autowire discovery-fragment gimmick for static COMPONENTS

examples/floor/CMakeLists.txt hard-failed unless a `bbtool
autowire`-generated main/generated/bb_autowire_components.cmake fragment
existed, and only used it to set the ESP-IDF discovery COMPONENTS
allowlist -- vestigial, since main/CMakeLists.txt already hand-declares
the real link truth (REQUIRES bb_log bb_meminfo bb_timer). Replace the
FATAL_ERROR + generated-fragment include with a static COMPONENTS
allowlist verified against the true transitive closure (bb_core, bb_str,
bb_registry, bb_task, bb_mem_arena, bb_wdt), so floor builds with no
codegen/autowire pre-step.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* build: plumb bbtool codegen path at the floor level

Prove `bbtool codegen` mirrors floor's hand-wired composition
(bb_log/bb_meminfo/bb_timer) end-to-end: `make floor-codegen` regenerates
generated/bb_app_init.c from bb_log's `// bbtool:init` markers and rebuilds
floor so it compiles against the real bb_log.h prototypes. main/CMakeLists.txt
appends the generated file to SRCS only when present (gitignored, absent by
default), so floor's primary handwired build and app_main entry point are
unaffected -- bb_app_init() is compiled but never called.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* docs: declare codegen+handwire as the only composition paths; mark bb_init walker legacy

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* refactor: delete the failed bbtool autowire command; keep the resolver codegen needs

- delete commands/autowire.py's CLI surface (NAME/HELP/add_arguments/register/run,
  argparse plumbing) -- the dead link-set-spike command
- relocate the shared resolver (resolve_composition/render_cmake_fragment/
  DEFAULT_PLATFORM/DEFAULT_OUT_REL) into a new neutral scripts/bbtool/composition.py,
  which has no CLI surface; bbtool co... (continued)

3371 of 3371 branches covered (100.0%)

Branch coverage included in aggregate %.

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

5359 of 5359 relevant lines covered (100.0%)

9985.3 hits per line