Skip to content

docs: align PR template checklist with the count-free index convention#81

Merged
mxriverlynn merged 1 commit into
testdouble:mainfrom
taminomara:docs/pr-template-count-free-checklist
Jun 18, 2026
Merged

docs: align PR template checklist with the count-free index convention#81
mxriverlynn merged 1 commit into
testdouble:mainfrom
taminomara:docs/pr-template-count-free-checklist

Conversation

@taminomara

@taminomara taminomara commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR updates the contributor PR-template checklist to stop chasing entity counts that no longer exist, so that the template matches the "indexes stay complete, not counted" convention.

  • Rewrites the index-maintenance checklist item: instead of telling contributors to update a count line in CLAUDE.md plus hardcoded counts in docs/concepts.md and README.md, it now asks them to confirm the two indexes, the CLAUDE.md catalog, and docs/concepts.md each list every current skill and agent.
  • Drops "counts" from the list of things /han-update-documentation syncs, since there are no longer any counts to sync.
  • Scope is two lines in one file. Pure process/documentation text, with no skill, agent, or runtime behavior change.
  • Follow-up to a triage that flagged this drift; the adjacent path-fix branch (PR docs: fix stale long-form doc paths after the plugin-rename reorg #80) deliberately left this count-convention drift out of its scope.

What to look at first

  • The replacement checklist item (line 20): does it correctly name the four places that must each list every entity, and does the instruction read clearly to a contributor who has never seen the convention? It states the rule prospectively, with no reference to the old count-based wording (that history lives in the commit message, not the checklist).
  • Whether the targets it points at are real: the old line referenced a "Counts to verify when editing indexes" line in CLAUDE.md and counts in docs/concepts.md and README.md that have already been removed. Worth a quick confirm that the new wording does not point anyone back at a count that no longer lives in those files.
  • The /han-update-documentation paragraph (line 26): removing "counts" should leave the sentence accurate about what that skill still syncs (long-form docs, indexes, cross-references) and not imply it dropped real work.

Files of interest

  • .github/pull_request_template.md — the only file changed; two checklist lines realigned to the count-free index convention.

The "Before requesting review" checklist still told contributors to
update entity counts and a "Counts to verify when editing indexes" line
in CLAUDE.md that no longer exists. The suite moved to count-free indexes
("Indexes stay complete, not counted"), and docs/concepts.md and
README.md carry no hardcoded counts.

- Replace the count-update checklist item with a completeness check:
  confirm the indexes, the CLAUDE.md catalog, and docs/concepts.md each
  list every current skill and agent.
- Drop "counts" from the /han-update-documentation refresh paragraph.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@taminomara taminomara force-pushed the docs/pr-template-count-free-checklist branch from 053c1a7 to 2e3a6f9 Compare June 18, 2026 03:28
@taminomara taminomara marked this pull request as ready for review June 18, 2026 03:29
@mxriverlynn mxriverlynn merged commit c267f67 into testdouble:main Jun 18, 2026
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.

2 participants