Content
67%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-structured, genuinely actionable doctrine skill: concrete artifact requirements, a copy-paste subagent brief, explicit branch handling with error recovery, and a smells checklist. Its weaknesses are repetition of the same few rules across sections and total inlining of ~334 lines where a filled-in FEEDBACKS.md example and the subagent brief template would fit naturally in reference files.
Suggestions
Consolidate the repeated statements of 'contract first / no edit-in-tandem / never name the consumer' into their canonical sections (the Stage patterns) and let 'The short version' be the only recap.
Move the subagent brief (Step 2) into a template file under references/ and show a filled-in example FEEDBACKS.md so both are copy-paste ready and the body shrinks toward an overview.
Add an explicit pre-spawn checkpoint — a short list of criteria the FEEDBACKS.md must satisfy before spawning — to close the workflow's only validation gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~334-line body is mostly original doctrine (not concepts Claude already knows), so much of it earns its tokens, but the core rules are restated repeatedly — 'contract first', 'no edit-in-tandem', and 'just add the field' recur across 'The failure mode', the Stage sections, 'Why this is more than ceremony', and 'The short version' — and passages like 'This is the mechanism that keeps the contract unopinionated...' re-explain points already made. This is 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the padded verbosity of a 2. | 3 / 5 |
Actionability | For an instruction-only skill the guidance is concrete and near copy-paste ready: the five numbered MUST-contents and three MUST-NOTs for the FEEDBACKS.md artifact, the full verbatim subagent brief (accept/counter-propose/refuse with shipping instructions), the three-outcome verdict handling, and the four procedural patterns each with a named anti-pattern. Minor gaps keep it at 4: the FEEDBACKS.md is specified by requirements rather than shown as a filled-in example, and the worked example's contract fragments are illustrative rather than complete. | 4 / 5 |
Workflow Clarity | The multi-step process is clearly sequenced (write FEEDBACKS.md → spawn the profiled subagent → read the verdict and branch on accept/counter/refuse), with an explicit error-recovery path (refusal is absorbed or escalated, never overridden; don't re-spawn to get a different answer), test checkpoints ('Producer tests run; consumer tests run'), a smells checklist as a stop-and-reset signal, and a 'When NOT to use' gate. Not 5 because validation of the FEEDBACKS.md artifact itself before spawning is not checkpointed, and some steps (tool scoping, escalation rewrite) are described rather than sequenced. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), so structure rests on the body itself: well-ordered sections, a flow diagram, and one clearly signaled one-level cross-skill reference ('Companion to sdk-design'). The 50-line simple-skill exemption doesn't apply, and the ~334-line body inlines content that could arguably be split (the subagent brief template, the worked example), which keeps it at 4 ('most content appropriately placed; references mostly clear; minor organization gaps') rather than 5. | 4 / 5 |
Total | 15 / 20 Passed |