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 skill body: concrete two-mode workflow with real validation gates and feedback paths, and clean one-level-deep progressive disclosure to four genuine reference files. The main cost is token duplication between the Core Principles section and the Anti-patterns section, which repeats four principles and the entire safe-deploy-point failure-mode taxonomy twice.
Suggestions
Consolidate the overlap between Core Principles and Anti-patterns: keep the principles as one-line pointers (e.g. 'every wave boundary must be a safe deploy point — see the anti-patterns below') instead of restating the three failure modes and must-land-together rule in full in both places.
In the 'NEVER create an unsafe intermediate deploy state' anti-pattern, reference the failure-mode taxonomy once and point to references/dependency-analysis.md for detection guidance rather than re-explaining all three patterns already covered in Core Principles and Mode A step 5.
Merge 'NEVER share a working tree' and 'One branch per parallel workstream' into a single statement (the Branch Strategy table reference suffices for the mechanics), trimming a few further lines.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and skill-specific, but roughly 20 lines could be tightened: four of the five Core Principles bullets are restated nearly verbatim as anti-patterns ('Gates before advancing' vs 'NEVER advance to Wave N+1', 'Every wave boundary must be a safe deploy point' vs 'NEVER create an unsafe intermediate deploy state' — the latter even repeating the same three failure modes and the full 'must land together' rule a second time). Not 4: the duplication is systematic rather than minor; not 2: nothing explains concepts Claude already knows, and the bulk of the body earns its tokens. | 3 / 5 |
Actionability | Fully executable guidance for an instruction skill: exact output path ('.context/plans/<plan-slug>.md'), a precise wave-assignment rule ('Wave N = tasks whose all dependencies are in waves 1..N-1'), a concrete safe-boundary test question, real verification commands ('bun run test --coverage', 'bun run typecheck', 'git log --oneline main..HEAD'), a worked recognition example with expected output, and a copy-paste template reference. Not 4: no meaningful gaps remain — the scoring note confirms code absence is not penalized when guidance is this actionable. | 5 / 5 |
Workflow Clarity | Both modes are clearly sequenced with explicit validation: Mode A ends with a validate step ('every task appears in exactly one wave; no wave contains tasks that depend on each other'), Mode B runs the wave's Verification checklist before marking done, and feedback loops exist (Blocked status transition, 'name the failing check, hold Wave 2 as pending, and tell the user what must pass first'). Matches the anchor-5 pattern of validate-fix-retry with checkpoints. | 5 / 5 |
Progressive Disclosure | Clear overview body with well-signaled one-level-deep references: all four cited files (wave-format.md, dependency-analysis.md, status-tracking.md, wave-document.yaml) exist in references/ and match their stated purposes, each is linked inline with a purpose description at the point of need, and a References section recaps all four with one-line descriptions. No inlined content belongs in a separate file. | 5 / 5 |
Total | 18 / 20 Passed |