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, highly actionable MCP-workflow skill: exact tool calls with concrete parameter values, a hard call budget, security redaction guidance, and three worked examples. Its weaknesses are minor — slight redundancy between the Core Concepts and Steps sections, and no recovery path for edge cases like an empty resolve-library-id result.
Suggestions
Merge the Core Concepts bullets for 'resolve-library-id' and 'query-docs' into Steps 1 and 3 to remove the duplicated definitions and tighten token usage.
Add a brief recovery branch for Step 1, e.g. 'If resolve-library-id returns no close match, retry once with a more specific library name; if still empty, answer from training data and say so' — this would give the workflow an explicit feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean, assumes Claude's competence (no explanation of what libraries or documentation are), and every section carries operational content. Minor duplication keeps it from 5: the Core Concepts bullet definitions of 'resolve-library-id' and 'query-docs' restate information given again in Steps 1 and 3, and could be merged. | 4 / 5 |
Actionability | Fully concrete, executable guidance: exact MCP tool names, exact parameter names and values ('libraryName: "Next.js"', 'libraryId: "/vercel/next.js"'), selection criteria, a hard 3-call limit, and three worked examples covering common cases (Next.js, Prisma, Supabase). This is copy-paste-ready instruction of the kind the score-5 anchor describes. | 5 / 5 |
Workflow Clarity | A clear four-step sequence with an explicit gate ('Do not call query-docs without a valid library ID'), selection criteria as a checkpoint, and a fallback when the answer stays unclear after 3 calls. Not 5: there is no true feedback/recovery loop — e.g. what to do when resolve-library-id returns no usable match is unaddressed — and no explicit verification that fetched snippets actually answer the question. | 4 / 5 |
Progressive Disclosure | The skill is a single self-contained SKILL.md with no references/, scripts/, or assets/ bundle, and nothing in it clearly belongs in a separate file; headers (Core Concepts, When to use, How it works, Examples, Best Practices) make navigation easy. Not 5: the body runs ~84 lines (above the under-50-line simple-skill exception), and Core Concepts/Examples sections could arguably be split out if the skill grows. | 4 / 5 |
Total | 17 / 20 Passed |