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.
A well-designed router skill with concrete, specific routing directives and strong navigation structure, held back by significant redundancy — the same symptom-to-reference mapping is repeated in five different formats. The unverifiable external references and the underspecified Cursor Subagent Routing section are secondary weaknesses.
Suggestions
Consolidate redundant routing sections: the Quick Reference table, Decision Tree, numbered list, Critical Patterns, and Example Invocations all restate the same mappings. Keep the table and one decision aid, and fold Critical Patterns' per-file summaries into the table's Reference column.
Cut Anti-Rationalization rows that restate guidance already given elsewhere (e.g., the sleep() and fast-tests rows repeat Critical Patterns content), keeping only rows that add a new consideration.
Either specify the Cursor subagent mapping (which subagent handles which audit) or remove the 'Cursor Subagent Routing' section, since the numbered list already routes to named agents.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The same routing mappings (e.g., unit tests → skills/swift-testing.md) are restated across five sections — Quick Reference, Decision Tree, numbered list, Critical Patterns, and Example Invocations — and Anti-Rationalization rows like "sleep() makes tests slower AND flakier" duplicate guidance already given. This is more than anchor 4's minor trimmable instances but stops short of anchor 2's padded conceptual explanations, since no background concepts are over-explained. | 3 / 5 |
Actionability | Directives are concrete and executable for a router: exact file paths ("skills/testing-async.md"), named subagents (test-failure-analyzer, testing-auditor), a slash command ("/axiom-audit testing"), and specific APIs ("Issue.record(severity:)", "expectedCount: 0"). Not a 5 because the "Cursor Subagent Routing" section ("Delegate to the appropriate Cursor subagent when this router calls for a specialized auditor") never defines which subagents or when, leaving a vague directive. | 4 / 5 |
Workflow Clarity | The decision tree, numbered routing list, and Example Invocations give a clear conditional sequence with codebase checks ("Check codebase: XCUIApplication/XCUIElement? → skills/ui-testing.md") that function as checkpoints, matching anchor 4. Not a 5 because checkpoints are implicit in most routes (only one example shows the check-then-branch pattern) and the relationship between the 'Cursor Subagent Routing' section and the Agent routes in the numbered list is left unclear. | 4 / 5 |
Progressive Disclosure | The body is a clean index with well-signaled, one-level-deep references ("See skills/swift-testing.md"), each accompanied by a content preview in Critical Patterns — close to anchor 5's structure. Scored at anchor 4 because no bundle files are present and none of the referenced skills/*.md paths resolve within this bundle, so navigation cannot be verified and the cross-skill references (axiom-ai, axiom-build, axiom-xcode-mcp) point outside the skill entirely. | 4 / 5 |
Total | 15 / 20 Passed |