Content
82%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: dense, concrete, multi-layered routing with genuine disambiguation logic for cross-domain overlaps. The main flaws are redundancy (dot graph duplicating the decision tree), an empty subagent-routing stub, and referenced files that are not present in the evaluated bundle to verify.
Suggestions
Delete the dot graph — it restates the numbered decision tree in ~30 lines of syntax for no additional routing value.
Replace the 'Cursor Subagent Routing' stub with an actual routing table listing the available subagents (iap-auditor, iap-implementation) and the conditions that trigger each, instead of burying them in decision-tree items 14–15.
Ship or verify the referenced bundle files — none of the ~40 'skills/*.md' paths exist in the evaluated bundle, leaving every routing target dangling.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, high-signal routing: a symptom→file table, a numbered decision tree, conflict resolution, and an anti-rationalization table, with essentially no explanation of concepts Claude already knows. Minor trimmable waste keeps it below anchor 5: the dot graph (~30 lines) fully duplicates the numbered decision tree, the 'Cursor Subagent Routing' section is a two-line stub, and line 9's 'You MUST use this skill for ANY iOS system integration... and more' banner repeats the frontmatter with a vague 'and more'. | 4 / 5 |
Actionability | Every routing instruction is concrete and executable: exact file paths ('skills/live-activities.md', 'skills/push-notifications-diag.md'), named APIs and error strings ('Missing required parameter', 'INPlayMediaIntent', 'mustReport false'), named subagents ('Delegate to the iap-auditor subagent'), and example invocations mapping real user phrasings to specific files ('My background task never runs' → 'skills/background-processing-diag.md'). As an instruction-only router skill, the guidance is fully actionable with specific examples covering common cases. | 5 / 5 |
Workflow Clarity | The routing workflow is unambiguous and layered: a numbered decision tree with sub-items (1a/1b/1c), cross-domain routing that splits every overlap into 'stay here' vs 'also invoke <sibling>', explicit conflict-resolution tie-breakers, and worked example invocations. It falls short of 5 because the 'Cursor Subagent Routing' section is a stub ('Delegate to the appropriate Cursor subagent when this router calls for a specialized auditor') with no list of available subagents or when to pick each — the subagent names only appear buried in decision-tree items 14–15. | 4 / 5 |
Progressive Disclosure | Textbook one-level-deep router structure: the SKILL.md keeps itself to routing only (no inlined API reference) with a symptom→file Quick Reference table and clear path-based references. Scored against the actual bundle as the guideline directs: no references/, scripts/, assets/ (or skills/) directory is present, so none of the ~40 referenced 'skills/*.md' files can be verified to exist — a real organization gap that keeps it below anchor 5's 'content appropriately split; easy navigation'. | 4 / 5 |
Total | 17 / 20 Passed |