Content
88%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.
The body is a well-sequenced, highly actionable multi-phase workflow with strong validation and feedback loops. It could improve by de-duplicating recurring guidance and splitting some material into reference files now that the skill is long.
Suggestions
Consolidate the commit-shape guidance and the 'ask only when product decision/destructive action/broad reshape/conflict' rule so each appears once, then reference it, to reduce repetition.
Move the sub-agent prompt template and anti-patterns into a reference file under references/ and link to it, improving progressive disclosure for a skill of this length.
Consider enumerating discrete capabilities (preflight, wave planning, execution, final review/ADR harvest) more explicitly in the description to lift specificity above 3.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly lean and efficient, referencing other skills (git, standalone-commits, post-implementation-review) instead of re-explaining them, but the commit-shape guidance and the 'ask only when product decision/destructive/broad reshape' rule each recur multiple times, fitting the 'minor instances that could be trimmed' anchor 4 rather than the every-token-earns-its-place anchor 5. | 4 / 5 |
Actionability | Provides concrete executable commands (bun typecheck, bun test, bun scripts/check-doc-hygiene.ts, git rm), copy-paste templates (wave plan, commit format, spec checkoff markdown), and specific response tables, matching the fully-executable anchor 5. | 5 / 5 |
Workflow Clarity | Clear Phase 0-4 sequence with explicit validation checkpoints (type-check/tests after each wave, hygiene check at final review), feedback loops ('fix before moving on', 'Don't carry broken state'), and checklists (Preflight, Wave Planning), matching anchor 5; destructive ops (git rm spec) are guarded by verification so the batch/destructive cap does not apply. | 5 / 5 |
Progressive Disclosure | Well-organized into clear sections with one-level-deep references to other skills and repo docs (specs/README.md, docs/adr/README.md), but for a complex skill all content lives inline in SKILL.md with no bundle files splitting sub-agent prompt templates or anti-patterns, fitting the good-structure anchor 4 rather than the split-and-signaled anchor 5. | 4 / 5 |
Total | 18 / 20 Passed |