Content
75%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, actionably written skill body with a clear dispatch workflow, a real validation guard for the AskUserQuestion bug, and appropriate use of one-level-deep reference files that all exist. Main gaps are mild padding in the examples and anti-patterns sections, a duplicated when-to-use guide between body and reference.md, and a small pattern-count inconsistency with the reference file.
Suggestions
Cut the three bash 'Usage Examples' blocks or compress them into two lines each — they are commentary, not executable guidance, and cost tokens without adding instructions.
Deduplicate the when-to-use guidance: keep a two-line summary in the body and point to reference.md's When-To-Use Guide for the full symptom table.
Fix the pattern-count mismatch: reference.md's header says 'v1: 8 patterns' while both the body and the catalog table describe 9; and give the `deep` sub-agent an explicit prompt template instead of 'target description + relevant file paths'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient: a dispatch table, exact fallback prompts, and pointed references, with essentially no explanation of concepts Claude already knows. It could still be trimmed — the three bash 'Usage Examples' blocks are illustrative comments rather than instructions, and each Anti-Pattern carries a "Why:" rationale that is somewhat padded. Not 5 because of those padded sections; not 3 because nothing here is redundant teaching, and the AskUserQuestion guard is genuinely novel operational knowledge. | 4 / 5 |
Actionability | Concrete, executable guidance throughout: a dispatch table mapping each subcommand to a specific protocol file ("Read references/protocols/anchor.md → execute"), an exact AskUserQuestion prompt with enumerated options A-D, explicit sub-agent spawn instructions, and a 4-element structured output format. Not 5 because the `deep` sub-agent prompt is only sketched ("target description + relevant file paths") rather than given as a ready-to-use prompt, and the 'Usage Examples' are commentary rather than runnable invocations. | 4 / 5 |
Workflow Clarity | Sequencing is clear: parse subcommand → dispatch table → execute protocol; no-subcommand fallback routes through an explicit question; and the AskUserQuestion guard is a genuine validation checkpoint with error recovery (empty answers → numbered text list → wait for reply). Not 5 because the "Wait for user reply" loop has no timeout/escalation path and per-subcommand execution details are deferred to referenced files without an explicit completion check in the body. | 4 / 5 |
Progressive Disclosure | Good structure: the body stays an overview (dispatch, guard, output format pointer) while pattern catalogs and protocol steps live in four real, one-level-deep reference files, all listed with content descriptions in the References section (all verified to exist). Not 5 due to minor organization gaps: the body's 'When to Use' section substantially duplicates reference.md's When-To-Use Guide, and the body states "9 patterns" while reference.md's header reads "v1: 8 patterns" (its table actually lists 9), a small cross-file inconsistency. | 4 / 5 |
Total | 16 / 20 Passed |