Skip to content

docs: refresh README repository layout with drift guards - #13

Open
mattpartida wants to merge 1 commit into
mainfrom
docs/20260914-readme-layout-refresh
Open

mattpartida wants to merge 1 commit into
mainfrom
docs/20260914-readme-layout-refresh

Conversation

@mattpartida

Copy link
Copy Markdown
Owner

Summary

Backlog item 9 (stale README/docs). The README Repository layout tree had drifted from disk — it omitted docs/, examples/policies/, examples/schema-adapters/, the agent-security-prompt-sarif.yml and agent-security-compare-reports.yml workflow examples, the combined boundary report, scripts/, and package-skills.sh.

This PR refreshes the tree and adds tests/test_readme_repository_layout.py so it cannot drift again in either direction:

  • Existence guard: every concrete path named in the tree must exist on disk, so renamed/removed files can't linger as stale docs.
  • Coverage guard: every examples/ci/github-actions/*.yml workflow must be listed — new integration examples will fail CI until documented.
  • Core entries guard: skills/, docs/, tests/, scripts/, examples/, package-skills.sh, .github/workflows/ stay listed.
  • Shape guard: tree rows stay parseable (spaces-only, even indentation, glob-or-plain names).

Documentation-only: no scanner, rule, CLI, or packaging behavior changed; JSON output fields unchanged; dist/ artifacts remain byte-identical. Abbreviated glob entries (*.json, test_*.py, *.txt / *.json) are skipped by existence checks so the tree stays compact.

Deliberately scoped to avoid overlap with open PRs #8/#10/#12 (they touch README usage sections and CHANGELOG bullets, not the layout tree).

Test plan

  • New tests written first (TDD), confirmed failing on main (2 failures: missing workflows, missing core entries)
  • uv run --python 3.11 --with pytest pytest -q → 151 passed (147 on main + 4 new)
  • ruff check . → clean; ruff format --check on the new test → clean
  • python3.11 -m compileall -q skills tests scripts → OK
  • ./package-skills.sh + python3.11 scripts/package_skills.py --check → artifacts current and reproducible
  • git diff --check → clean

The README repository-layout tree had drifted from disk: it omitted
docs/, examples/policies/, examples/schema-adapters/, the prompt-sarif
and compare-reports CI workflow examples, the combined boundary report,
scripts/, and package-skills.sh.

Adds tests/test_readme_repository_layout.py so the tree cannot drift
again in either direction:

- every concrete path named in the tree must exist on disk
- every examples/ci/github-actions/*.yml workflow must be listed
- core top-level entries stay listed
- tree rows stay parseable (spaces-only indentation, glob-or-plain names)

Documentation-only change; no scanner, rule, or packaging behavior
changed. dist artifacts remain byte-identical.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant