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 strong, well-structured instruction-only skill: the 9-step workflow is explicitly sequenced with a real validation feedback loop, and external detail is correctly pushed to one-level-deep references. The weaknesses are minor — some duplicated rationale between steps and Anti-Patterns, and one referenced validation script (validate-context-frontmatter.sh) that does not exist in this bundle.
Suggestions
Fix the dead script reference in Step 8: ./scripts/validate-context-frontmatter.sh is not in this bundle (scripts/ contains only validate-debate-verdict.sh) — either point explicitly at the context-file skill's script by name or drop the local path.
Trim repetition: delete the 'Never spawn sequentially' paragraph in Step 6 (it restates the preceding paragraph) and let the ungrounded-debate rationale live in either Step 2 or the Anti-Patterns, not both.
Replace the comment-only parallel-spawn block in Step 6 with a minimal concrete example of the two Agent-call briefs, or fold it into references/role-design.md's sample instructions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly dense, non-obvious imperative guidance, but with minor repetition: "Never spawn sequentially — that triples wall-clock time for no benefit" restates the immediately preceding paragraph's point, the ungrounded-debate rationale appears in both Step 2 and the Anti-Patterns, and Step 8 triple-states its ALWAYS rule. Fits anchor 4 (efficient, minor instances that could be trimmed); not 5 because repeated tokens don't each earn their place, not 3 because the padding is minor rather than a broad unnecessary-explanation problem. | 4 / 5 |
Actionability | Concrete executable commands are present (wc -l, grep -rl | wc -l, ./scripts/validate-debate-verdict.sh with expected output, and explicit verdict enums), but Step 8 references ./scripts/validate-context-frontmatter.sh which does not exist in the bundle's scripts/ directory, and the Step 6 parallel-spawn block is comment-pseudocode. Fits anchor 4 (mostly executable with minor gaps); not 5 due to the dead script path and non-executable spawn example, not 3 because the commands that do exist are complete and runnable. | 4 / 5 |
Workflow Clarity | Nine clearly numbered steps with an explicit validation checkpoint showing both expected and error output ("OK: 1 file(s) validated..." vs "must set 'revisit_trigger'..."), error-recovery rules for "It depends" verdicts, and a Troubleshooting table of failure modes. Matches anchor 5 (clear sequence with explicit validation steps and feedback loops); not 4 because checkpoints are explicit rather than merely present. | 5 / 5 |
Progressive Disclosure | SKILL.md is a well-signaled overview: references/role-design.md and references/worked-example.md both exist, are one level deep (they contain no further bundle links), and are surfaced via a References table with a When-to-Use column alongside a Templates table. However, the body references ./scripts/validate-context-frontmatter.sh, which is absent from the bundle (scripts/ contains only validate-debate-verdict.sh) — a dead path among otherwise-valid references. Fits anchor 4 (good structure, minor organization gaps); not 5 because of the dead path, not 3 because structure and signaling are otherwise excellent. | 4 / 5 |
Total | 17 / 20 Passed |