Content
81%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 rigorous, highly actionable execution protocol with best-in-class validation checkpoints and feedback loops at every level (task, phase, final). Its main weakness is token efficiency: the honesty rule and failure-recovery instructions are repeated many times, and template blocks duplicate the bundled reference file. Tightening the repetition would raise an otherwise excellent skill.
Suggestions
State the honesty rule once (e.g., in Core Principle) and cut the restatements in §The Checklist System, §2e, §6c, the Anti-Patterns entry, and the Honesty Checklist, keeping only one cross-reference to it.
Replace the inlined baseline/checklist-entry templates in §1c, §2e, and §3e with a single pointer to references/checklist-templates.md, honoring that file's own instruction to 'Keep the SKILL.md procedure lean'.
Define the recovery protocol (Stop → Fix → Re-run → Record new output) once early in the Workflow section and have each gate's failure note reference it instead of repeating the four-step instruction.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient process detail Claude could not infer, but the honesty rule is restated at least six times ("A checked item without proof is a lie", "An [x] without Actual output is a lie", "Marked [x] but has no Actual output → LIE. Go back.", plus the Anti-Patterns and Honesty Checklist sections), and the "If it fails: Stop. Fix. Re-run." recovery instruction is repeated after nearly every gate. Baseline and checklist-entry templates in §1c/§2e/§3e also duplicate the fuller templates in references/checklist-templates.md, whose own header says "Keep the SKILL.md procedure lean". Not 2 because no section explains background concepts or pads with filler — every section carries skill-specific instruction; not 4 because the rule repetition and template duplication are more than minor trimmable instances. | 3 / 5 |
Actionability | Guidance is fully executable: exact checklist path (`.context/plans/<plan-slug>/CHECKLIST.md`), a fixed status vocabulary (PENDING/IN_PROGRESS/VALIDATED/BLOCKED), copy-paste commands (`grep -n "\[ \|IN_PROGRESS\|BLOCKED" CHECKLIST.md`), a concrete git commit template with evidence lines, and a real worked example in references/checklist-templates.md. Not 4: the placeholder gates (`<typecheck> && <build> && <test> && <lint>`) are not a gap — §1b explicitly mandates discovering the project's real commands from AGENTS.md, package.json, or CI config, which is justified flexibility for a project-agnostic skill. | 5 / 5 |
Workflow Clarity | The 8-step workflow is clearly sequenced with explicit validation checkpoints at every level — quick gate after 1-3 file ops, full gate at task end, phase gate suite, domain invariant, regression diff against a recorded baseline, final gates, and a checklist completeness audit — each with an explicit feedback loop ("If it fails: Stop. Fix the issue. Re-run the command... Do not proceed to the next task until this passes"). Divergence handling and between-phase dependency rules close the remaining gaps. This is exactly the anchor-5 pattern; not 4 because checkpoints and error-recovery loops are explicit at every stage rather than mostly present. | 5 / 5 |
Progressive Disclosure | The bundle has one reference file (references/checklist-templates.md, verified to exist) referenced one level deep and clearly signaled twice in the body ("See `references/checklist-templates.md` for the full CHECKLIST.md template and a worked example from a real phased run") plus a References section. Not 5 because template content that belongs solely in the reference file is inlined in SKILL.md (§1c baseline template, §2e checklist entry, §3e phase checklist) duplicating the reference's fuller templates; not 3 because references are clearly signaled and easy to navigate, which anchor 3 says they are not. | 4 / 5 |
Total | 17 / 20 Passed |