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-built router skill: highly actionable routing to specific files and audit agents, a clear decision tree, and a genuinely useful anti-rationalization table. The main costs are token redundancy from stating the same routing three ways and references that point outside the skill's own bundle.
Suggestions
Consolidate the three parallel routing mechanisms: keep the Quick Reference table for symptom lookup and the Decision Tree for technology choice, but trim the Example Invocations to only cases not already covered, cutting ~40-70 lines of duplication.
If Example Invocations are kept for trigger matching, move them to a single reference file and summarize with a few representative examples inline.
State the migration validation gate once (in the Decision Tree or Critical Pattern section) instead of repeating it in the Quick Reference, Critical Pattern, and Example Invocations sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense routing signal with zero concept explanations, but the same routing facts are repeated across three parallel sections — e.g. `skills/grdb-performance.md` appears in the Quick Reference table, Decision Tree step 3a, and Example Invocations, while `skills/database-migration.md` appears five-plus times, and the ~70-line Example Invocations section largely duplicates the table. Mostly efficient, but a whole section of duplicated routing could be consolidated, fitting anchor 3 rather than 4. | 3 / 5 |
Actionability | Every entry gives a fully executable instruction: exact files to read ("Read: `skills/swiftdata.md`"), agents to launch, or copy-ready commands ("`/axiom:audit core-data`"). The Example Invocations map ~25 concrete user requests to exact actions, and per the scoring notes the absence of code in an instruction-only router skill is not penalized when guidance is this actionable. | 5 / 5 |
Workflow Clarity | The numbered Decision Tree (with 3a/4a/23a sub-branches) gives a clear, unambiguous routing sequence, and the one destructive-capable path is gated by an explicit checkpoint ("ALWAYS read `skills/database-migration.md` when adding/modifying database columns") with the failure modes it prevents, plus audit agents serving as validation. Below anchor 5 because there is no inline error-recovery feedback loop and the routing sequence is duplicated across three sections rather than stated once. | 4 / 5 |
Progressive Disclosure | The SKILL.md is a clean index: a symptom-to-file table with clearly signaled one-level-deep references and all detail delegated to separate files. It misses anchor 5 because the referenced `skills/*.md` files are not part of this bundle (no references/ directory; paths resolve against an external suite layout), and cross-suite indirection like "See axiom-security (skills/keychain.md)" adds navigation layers outside the skill itself. | 4 / 5 |
Total | 16 / 20 Passed |