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-built router: dense lookup tables, real API names, and clear conditional dispatch to per-topic files and agents. The main costs are redundant repetition of the same routing information across four sections and persuasion padding, plus the fact that every referenced skills/*.md target is absent from the bundle as provided.
Suggestions
Consolidate the four routing structures (Quick Reference table, dot-graph Decision Tree, numbered Decision Tree, Example Invocations) into one canonical routing table — the graph and numbered list duplicate the same branches, and Example Invocations mostly restate Quick Reference rows.
Cut or drastically trim the Anti-Rationalization table; at most fold the two generative-AI rows into the routing table as trigger phrases, since the rest is persuasion rather than routing information.
Ship the referenced skills/*.md files alongside this SKILL.md (or fix the paths) — as provided, every routing target the body points to is a dangling path, which undermines the skill's entire navigation function.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The routing decisions are stated four times — the Quick Reference table, the dot-graph Decision Tree, the numbered Decision Tree (same branches again), Critical Patterns, and Example Invocations largely restate one another — and the Anti-Rationalization table ("I'll add tests later | Tests written after implementation miss edge cases") is persuasion padding. It avoids explaining concepts Claude already knows and the tables are dense, so it is above anchor 2's padded verbosity but below anchor 4's minor-trim level: the duplication is structural, not incidental. | 3 / 5 |
Actionability | Directives are concrete and specific: exact targets ("See skills/ui-testing.md Pattern 7"), real APIs ("Issue.record(severity:)", "Test.cancel", "expectedCount: 0", ".serialized"), and conditional dispatch with codebase checks ("Check codebase: XCUIApplication/XCUIElement? → skills/ui-testing.md"). It falls short of anchor 5 because the router contains no executable code or commands itself and the routing targets are dangling paths in this bundle, but it is well above anchor 3's pseudocode-level guidance. | 4 / 5 |
Workflow Clarity | Dispatch sequencing is explicit via the decision tree plus the Example Invocations' branching demonstrations ("My UI tests are flaky in CI" → check for XCUIApplication vs @Test/#expect to pick the route). Not anchor 5: the dot graph routes "flaky tests" to skills/ui-testing.md while the numbered list routes Swift Testing flakiness to an agent — an inconsistency resolved only by reading the examples — and there are no explicit checkpoints; not anchor 3: decision points and routes are explicit throughout. | 4 / 5 |
Progressive Disclosure | The body is an overview-only router pointing one level deep, with every reference clearly signaled as a path in a labeled table row — the right shape for a router skill. It misses anchor 5 because none of the referenced skills/*.md files exist in the provided bundle (every routing target is a dangling path here) and navigation is split across four overlapping structures rather than one canonical map; it is above anchor 3 since references are prominent and unburied. | 4 / 5 |
Total | 15 / 20 Passed |