dangernoodle-io / breadboard / 29976580596

96%

push

github

refactor: group bb_storage* under components/storage/ (B1-1162) (#1014)

Moves 4 components into components/storage/:

- bb_storage, bb_storage_nvs, bb_storage_ram, bb_storage_rtc

bb_storage_http was already dissolved into bb_diag_http on main (#1008) --
not in scope. Each moved component's CMakeLists.txt relative platform/ path
bumped one level (../../ -> ../../../, 4 lines across bb_storage,
bb_storage_nvs, bb_storage_ram, bb_storage_rtc). Scaffolds
components/storage/README.md (group index) and regenerates the top-level
components/README.md index.

platform/{espidf,host}/bb_storage*/ dirs stay FLAT and untouched -- nesting
platform dirs is unsupported and breaks discovery (B1-980).

Fixes a docs-generator bug this move surfaced: `_dep_role_and_link` resolved
a dependency's directory via a flat `root/components/<dep>` lookup instead
of the discovery-index-aware resolution the rest of the pipeline already
uses, so any consumer of a now-grouped bb_storage (bb_config, bb_settings,
bb_diag_http) rendered a broken/blank Dependencies row. Routes the lookup
through `discovery.build_index()` (matching `_resolve_component_dir`) and
computes the rendered link relative to the CONSUMING component's own
directory (realpath-normalized, to survive a symlinked tmp/root spelling
e.g. macOS's /var -> /private/var) rather than assuming one level under
components/ -- correct for both a flat consumer and a consumer that is
itself grouped. Threads `comp_dir` through `_render_deps`/`_gen_component_readme`
instead of a bare name. New test covers a dep resolving across group
nesting from both a flat and a grouped consumer.

Closes B1-1162


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

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

7386 of 7976 branches covered (92.6%)

Branch coverage included in aggregate %.

12530 of 12692 relevant lines covered (98.72%)

2498.54 hits per line