Skip to content

feat: add README tree drift check and wire into CI#5

Merged
CodeSigils merged 1 commit into
mainfrom
fix/readme-tree-check
Jul 17, 2026
Merged

feat: add README tree drift check and wire into CI#5
CodeSigils merged 1 commit into
mainfrom
fix/readme-tree-check

Conversation

@CodeSigils

Copy link
Copy Markdown
Owner

Changes

Add a script that prevents the README's repo-layout tree from silently drifting when files are added or removed.

  • Create scripts/check-readme-tree.py — parses the ## Repo Layout tree section from README.md, extracts file paths by tracking directory depth from ASCII tree indentation, resolves wildcard entries (*.md) by globbing the directory, and compares against git ls-files
  • Add scripts/check-version-consistency.py and scripts/check-readme-tree.py to the README tree (were missing after PR feat: add version metadata, consistency checker, and portability self-test #4)
  • Wire into CI and validate-ci.py required commands

Verification

  • Ruff clean
  • check-readme-tree.py passes against current state
  • validate-ci.py passes (required commands include new step)
  • test-validate-ci.py passes

what: create scripts/check-readme-tree.py that parses the ## Repo Layout tree from README.md, extracts file paths, and compares against git ls-files
why: the manually maintained repo tree in README silently drifts when files are added or removed; this check prevents that

- Parses ASCII tree format tracking directory depth via indentation
- Handles wildcard entries (*.md) by globbing the corresponding directory
- Flags files in the tree but not tracked (stale) and tracked but not in tree (omitted)
- Added scripts/check-version-consistency.py and scripts/check-readme-tree.py to README tree
- Wired check-readme-tree.py into CI and validate-ci.py required commands
@CodeSigils
CodeSigils merged commit 94e348a into main Jul 17, 2026
5 checks passed
@CodeSigils
CodeSigils deleted the fix/readme-tree-check branch July 17, 2026 08:51
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