Skip to content

docs: clear leftovers from the sequential merges - #21

Merged
skateddu merged 1 commit into
mainfrom
docs/post-merge-cleanup
Aug 13, 2026
Merged

docs: clear leftovers from the sequential merges#21
skateddu merged 1 commit into
mainfrom
docs/post-merge-cleanup

Conversation

@skateddu

Copy link
Copy Markdown
Owner

What

A sweep over main after #18, #19 and #20 landed in sequence, looking for anything the chain left behind. Three items found and fixed.

Fixed

Where Problem
README.md project tree The ci.yml comment still read "Lint, config validation, shellcheck, hook tests". #20 removed the validator and updated the CI section's prose and table, but not this line
README.md project tree .github/ was shown as containing only workflows/, hiding ISSUE_TEMPLATE/ and pull_request_template.md — both present since long before this chain. The tree is exhaustive everywhere else, so this read as "there is nothing else here"
CHANGELOG.md A Fixed entry described restoring a workflow description that mentioned "configuration validation" — a dangling reference to something that, after #20, never shipped. Reworded to what it actually records

How it was checked

Rather than reading the tree, I re-derived every path from its indentation and resolved each against disk — 95 entries, all existing. That is what surfaced the .github/ gap, which eyeballing had missed three times.

Also verified clean:

  • No reference to validate_config, scripts/ or commands/code-review.md anywhere outside the changelog's historical Removed entry
  • No empty or orphaned directories
  • Every table matches the filesystem: 13 agents, 10 commands, 10 rules, 22 skills, 6 hooks, 7 test files
  • Every Python file the repo ships is inside ruff's include
  • The CI steps match the README's CI table and CONTRIBUTING.md's command, all three now naming the same checks
  • The declared test count (75) matches what pytest collects, in both README and CHANGELOG
  • .claude/hooks/lib/command-text.sh is present, sourced by both hooks, and no longer ignored by .gitignore
  • Every variable in .env.example appears in the README table

One deliberate omission

tests/__init__.py and tests/unit/__init__.py are not in the project tree. The tree is otherwise exhaustive, so this is an inconsistency — but they are package plumbing rather than components, and listing them costs two lines of noise in exchange for nothing a reader needs. Flagging it so it reads as a choice rather than an oversight.

🤖 Generated with Claude Code

Two references survived the validator's removal and one dangled in the
changelog. The tree also showed .github/ as if it held only workflows,
hiding the issue and pull request templates that were always there.

Found by re-deriving every path in the project tree from its indentation
and checking it against disk: 95 entries, all resolving.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@skateddu
skateddu merged commit 77e90ca into main Aug 13, 2026
1 check passed
@skateddu
skateddu deleted the docs/post-merge-cleanup branch August 13, 2026 14:20
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