Skip to content

docs: reconcile agent/skill counts (53/20) and add drift guard#90

Merged
PAMulligan merged 2 commits into
mainfrom
83-align-readme-agentskill-counts-with-onboarding-docs-48-vs-53-17-vs-19
May 25, 2026
Merged

docs: reconcile agent/skill counts (53/20) and add drift guard#90
PAMulligan merged 2 commits into
mainfrom
83-align-readme-agentskill-counts-with-onboarding-docs-48-vs-53-17-vs-19

Conversation

@PAMulligan
Copy link
Copy Markdown
Collaborator

Problem

The README claimed 48 Custom Agents and 17 Development Skills, but the onboarding docs (and docs/onboarding/architecture.md) referenced 53 agents, 19 skills. They disagreed, which confuses first-time readers.

Ground truth on disk

Neither figure was fully correct:

  • .claude/agents/53 agent .md files
  • .claude/skills/20 skill directories (each with a SKILL.md)

The README also enumerated only 17 skills, omitting parallel-orchestration, screenshot-intake, and export-design-system. A stale # 51 specialized agents tree comment in CLAUDE.md was also caught.

Changes

Reconciled every live doc to 53 agents / 20 skills:

  • README.md — header, directory tree, agent category table (rebalanced to sum to 53), full 20-skill list, and documentation index
  • docs/onboarding/architecture.md — system diagram, skills heading, and added the 20th skill (export-design-system)
  • .claude/skills/README.md — count + documented the 3 previously-missing skills (renumbered 1–20)
  • CLAUDE.md, CONTRIBUTING.md, .claude/CUSTOM-AGENTS-GUIDE.md, docs/onboarding/README.md, docs/onboarding/quickstart.md, docs/react-development/README.md

CHANGELOG.md is intentionally not modified — it is an auto-generated historical release record.

New drift guard — scripts/check-doc-counts.sh:

  • Recounts .claude/agents/ and .claude/skills/, then scans all Markdown (excluding CHANGELOG.md and docs/plans/) for any N agents / N skills claim that disagrees. Human-readable output + --json; exits non-zero on drift.
  • Wired into the CI validate job (.github/workflows/ci.yml) and the .husky/pre-commit hook.
  • Covered by scripts/__tests__/check-doc-counts.test.js, which recomputes expected counts from the filesystem so the test never goes stale.

Verification

  • bash scripts/check-doc-counts.sh✓ All agent/skill counts in docs match the entries on disk (exit 0), ~2.6s
  • Planted capitalized bad counts → all four claim forms detected (exit 1)
  • New test suite: 5/5 passing

Out of scope (noted for a follow-up)

The CLAUDE.md footer says "33 scripts" but there are ~35 — pre-existing drift in the script count, which this issue and the new checker do not cover.

Closes #83

🤖 Generated with Claude Code

The README claimed 48 agents / 17 skills while the onboarding docs said
53 agents / 19 skills. The truth on disk is 53 agents (.claude/agents/)
and 20 skills (.claude/skills/ dirs with a SKILL.md). The README also
listed only 17 skills, omitting parallel-orchestration, screenshot-intake,
and export-design-system.

Update every live doc to 53/20 (README, onboarding/architecture.md and the
rest of the onboarding set, react-development, CONTRIBUTING, CLAUDE.md, and
the agent/skill catalogs). CHANGELOG.md is left untouched as a historical
release record. Also fixes a stale "51 agents" tree comment in CLAUDE.md.

Add scripts/check-doc-counts.sh: recounts .claude/agents/ and .claude/skills/
and fails on any documented count that disagrees (human-readable + --json).
Wire it into the CI validate job and the husky pre-commit hook, with a
matching test in scripts/__tests__/.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@PAMulligan PAMulligan linked an issue May 24, 2026 that may be closed by this pull request
3 tasks
Resolve lint-and-format CI failure:
- Reformat check-doc-counts.test.js per Prettier
- Remove unused readdirSync import in verify-test-coverage.test.js
- Remove unused fileName variable in generate-stories.js

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@PAMulligan PAMulligan merged commit 915b973 into main May 25, 2026
6 checks passed
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.

Align README agent/skill counts with onboarding docs (48 vs 53, 17 vs 19)

1 participant