dangernoodle-io / breadboard / 30679710228

97%

push

github

fix: scan CMakeLists.txt for di_legacy AUTOREGISTER/AUTO_ATTACH usage (#1177)

The di_legacy fence's _config_usages scanner (backing
autoregister_usage/auto_attach_usage) globbed only *.c/*.h/*.cpp, so a
Kconfig-gated usage form in a build file was invisible to a
shrink-only fence -- a usage form that could be reintroduced without
ever tripping it.

This is not hypothetical: BB_SYSTEM_ROUTES_AUTOREGISTER's only usage
site was an if(CONFIG_BB_SYSTEM_ROUTES_AUTOREGISTER) gate around an
SRCS list in components/bb_system/CMakeLists.txt (removed in #1174),
and the fence carried only an autoregister_kconfig entry for its
declaration -- no autoregister_usage entry for the use. That same
form is also what causes B1-1334's codegen build breaks, since
codegen is Kconfig-blind.

_config_usages now also scans **/CMakeLists.txt (via the existing
_CMAKE_GLOBS/is_cmake_noise_line convention already used elsewhere in
this file), so a CONFIG_*_AUTOREGISTER/_AUTO_ATTACH reference inside
an if()/generator-expression gate is tracked the same as a C-source
usage, while a mere # comment mentioning the symbol is not.

The baseline did not grow: PR #1174 (B1-1332) removed the only live
CMakeLists gate, and the remaining CMakeLists hits for these families
are both comments (bb_log/CMakeLists.txt, examples/floor). di_legacy
stays at 15 markers, 0 new.

Refs B1-1341.


Claude-Session: https://claude.ai/code/session_0119nuE2NZME9y2aYEVJTPB5

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

8101 of 8674 branches covered (93.39%)

Branch coverage included in aggregate %.

13740 of 13895 relevant lines covered (98.88%)

2571.39 hits per line