dangernoodle-io / breadboard / 29871071870

96%

push

github

refactor: route bbtool consumers through the discovery SSOT (#982)

- boards.py derive_component: derive includes/sources from the index's
  own comp_dir/plat_dir Path objects (relative to the owning root)
  instead of string-templating components/<name>/... and
  platform/<platform>/<name>/... — the templated form was the one site
  that would emit a functionally wrong -I flag under nesting, not
  merely a stale one
- header_annot.py primary_header: resolve the primary public header via
  the discovery index's component_dir, returning None (never a
  fabricated flat-convention path) when the index has no entry for the
  name — a name/root that discovery can't resolve gets a loud, accurate
  "not a discovered component" error from its callers rather than a
  silent plausible-but-wrong path
- commands/docs.py: _render_brief (DocsGenError) and the
  component-readme lint rule's brief-marker check now handle
  primary_header's None result explicitly, naming the real cause; the
  callers' own depth-1 iterdir() walks are untouched (PR3 of this lane)
- scripts/gen_components_readme.py: extract_purpose now guards the same
  None result — fails loud with a new ComponentIndexError naming the
  cause, caught cleanly in main() (no traceback, exit 1), instead of
  crashing with AttributeError on extract_brief(None). Chose fail-loud
  over skip/placeholder: this script's whole job is a components/
  README.md drift check (wired into make check as docs-index-check with
  zero prior test coverage — how this crash slipped through originally)
  — silently omitting or placeholder-ing an undiscovered directory would
  produce a WRONG index instead of a clean signal, the exact failure
  class (silent-but-wrong vs loud-and-right) this whole lane exists to
  remove. New scripts/tests/test_gen_components_readme.py (discovered by
  make test-py's `unittest discover -s scripts/tests -t scripts`, same
  runner as the existing coverage-tooling suite) covers extract_purpose,
  c... (continued)

7440 of 8036 branches covered (92.58%)

Branch coverage included in aggregate %.

12597 of 12759 relevant lines covered (98.73%)

1210.41 hits per line