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.
An information-dense, highly actionable router with excellent trigger-to-file mapping and one-level-deep reference signaling. Its main cost is token efficiency: the same routing decisions are repeated across Triage, Decision Tree, Critical Patterns, and Example Invocations sections, inflating the body by roughly a third.
Suggestions
Collapse the 'Critical Patterns' and 'Decision Tree' sections into the 'AI Approach Triage' table — they restate the same routes and cost ~30 lines each time they repeat.
State the Opus 4.7 Messages API migration warning (temperature/top_p/top_k/prefill removal) once, in 'External Resources', and drop the near-identical repeats in 'Anti-Rationalization' and the description of `claude-api` routing.
Move the adapter failure-mode symbol lists (compatibleAdapterNotFound, invalidAdapterName, etc.) out of the router body and into the referenced `skills/foundation-models-adapters-diag.md`, keeping only the routing cue in SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious version-specific facts (e.g., "runtime obsoleted in 27.0", "Apple's own sample judge starts at Cohen's kappa −0.037") and pads nothing with concepts Claude already knows, but routing decisions are restated across four sections (AI Approach Triage, Decision Tree, Example Invocations, Critical Patterns) and the Opus 4.7 migration warning appears three times. Above 2 because the redundancy is structural repetition rather than explanatory filler; below 4 because the duplication exceeds minor instances. | 3 / 5 |
Actionability | As a router skill the guidance is fully deterministic: exact file targets per situation ("Read: `skills/foundation-models-diag.md` (Pattern 5a...)"), exact symbols ("compatibleAdapterNotFound", "`provider(from:in:)`"), and explicit gates ("only after documented rung-1-4 failures"), with Example Invocations mapping natural user phrasings to files. The scoring-note allowance for instruction-only skills applies, and the guidance has no gaps for its purpose. | 5 / 5 |
Workflow Clarity | A clear triage-first sequence ("First, determine which kind of AI the developer needs"), a numbered 14-step Decision Tree, and gating rules for the adapter path provide a well-sequenced workflow. Below 5 because there is no validation/feedback checkpoint on the routing decision itself, and the four parallel routing sections leave mild ambiguity about the canonical entry point. | 4 / 5 |
Progressive Disclosure | SKILL.md acts as a router with ~20 one-level-deep `skills/*.md` references, each signaled with a concise content summary — good disclosure structure. Below 5 because none of the referenced files exist in the provided bundle to verify, and sections such as the adapter error-case lists inline detail that the referenced diag files presumably own. | 4 / 5 |
Total | 16 / 20 Passed |