Content
57%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-designed router: dense, concrete symptom-to-reference routing with strong guardrails and anti-rationalization tables. Its weaknesses are redundancy across three overlapping routing structures, no in-body validation steps for the destructive database-migration workflows it gates, and references to skills/*.md files that are absent from the bundle.
Suggestions
Add explicit validation checkpoints for migration work in the body (e.g. "run both migration paths and verify zero data loss before shipping") instead of only routing to `skills/database-migration.md`, satisfying the feedback-loop requirement for database operations.
Ensure every referenced `skills/*.md` file ships in the bundle — none are present here — and disambiguate cross-skill paths like "See axiom-security (skills/keychain.md)" with full paths.
Collapse the three overlapping routing structures: the Example Invocations section duplicates most of the Quick Reference table; keep the table plus Decision Tree and trim examples to cases that add new routing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line is actionable routing with no concept over-explanation, but the same routing is repeated across three parallel structures: the Quick Reference table, the Decision Tree, and a ~70-line Example Invocations section that duplicates most table entries (e.g. GRDB performance and Keychain routing each appear three times). This fits anchor 3 — mostly efficient but could be tightened by collapsing to two structures — rather than anchor 2, since none of it is fluff explanation. | 3 / 5 |
Actionability | Guidance is concrete and executable in form: exact file targets ("Read: `skills/database-migration.md`"), exact error strings ("errSecDuplicateItem", "FOREIGN KEY constraint failed"), and named auditors per task. Minor gaps keep it below anchor 5: the Decision Tree and Example Invocations say "Launch: `core-data-auditor` agent" while the Automated Scanning section and inline note use different skill names (`axiom-audit-core-data`), requiring a translation step, and none of the referenced `skills/*.md` targets exist in the provided bundle. | 4 / 5 |
Workflow Clarity | The Decision Tree is a well-sequenced routing procedure with explicit guardrails ("ALWAYS read `skills/database-migration.md`", "CRITICAL: every local directory is purgeable"), but this skill's context is database operations, where the rubric requires validate→fix→retry feedback loops — and the body contains no validation or verification steps for its destructive-adjacent workflows (schema migrations that risk data loss), capping workflow_clarity at 3 per the rubric's feedback-loops rule. | 3 / 5 |
Progressive Disclosure | The design is a clean one-level-deep index (symptom → file table plus a decision tree), which would merit a 4-5, but no bundle files are provided: the 30+ referenced `skills/*.md` paths do not exist in this skill's directory, so none of the references can be verified to resolve. Cross-skill references like "See axiom-security (skills/keychain.md)" are also ambiguous about which bundle the path lives in, fitting anchor 3 (references present and signaled, but organization/resolvability is questionable) rather than anchor 4. | 3 / 5 |
Total | 13 / 20 Passed |