Content
85%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-structured, opinionated skill body that sequences a multi-step post-merge memory workflow with explicit validation and feedback loops, and cleanly offloads detail to one-level-deep reference files. It is mostly lean, with some repeated framing and a few abstract steps keeping conciseness and actionability just short of perfect.
Suggestions
De-duplicate the 'lint is the highest-value memory / a rule the agent cannot ship past' framing — it appears in the intro, P6, and Hard nevers; state it once and reference it.
Make Step 4's shard and AGENTS.md updates more concrete (e.g., a one-line example of a pointer edit or shard diff) so 'apply diffs to the relevant shard(s)' is executable rather than abstract.
Tighten the philosophy section (P1–P8) by pruning editorializing adjectives; the principles are valuable but several could be stated in fewer tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence (no basic explanations of git/PRs/memory), but repeats the lint-value framing across the intro, P6, and 'Hard nevers' and includes some editorializing that could be trimmed. | 4 / 5 |
Actionability | Provides concrete scripts and commands (scripts/check-agents-md.sh, scripts/local-checks.sh, claude -p "/learn --since <sha> --sha <sha>", git ls-remote) with a sequenced Step 0–6 flow, but a few steps like 'apply diffs to the relevant shard(s)' stay abstract rather than copy-paste ready. | 4 / 5 |
Workflow Clarity | Steps 0–6 are clearly sequenced with explicit validation in Step 5 (run check-agents-md.sh, re-run drafted lint) and a real feedback loop — 'The drafted lint MUST pass against the just-merged code before you include it — run it; if it fails on current main it's wrong' — plus idempotency checks in Step 0. | 5 / 5 |
Progressive Disclosure | The SKILL.md is an overview that points to five one-level-deep references (routing-rules.md, expert-shards.md, agents-md-guidance.md, invariant-discovery.md, consensus.md), each clearly signaled with 'See references/X.md'; detailed rules are appropriately split into those files, all verified to exist. | 5 / 5 |
Total | 18 / 20 Passed |