Content
77%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 lean, single-purpose freshness skill with a clearly sequenced workflow, an explicit verification step, and a real error-recovery loop. Its weakness is redundancy — the freshness hierarchy and the no-memory rule are each stated multiple times across the intro, Workflow, and Rules sections, inflating token cost without adding information.
Suggestions
Remove the Rules bullets that restate the workflow verbatim — the freshness-hierarchy bullet and the "never write from memory" bullet duplicate step 2's (a)/(b)/(c) and its "Do NOT skip" line; keep each rule in exactly one place.
Trim the intro's narrative padding ("excellent for stable idioms, but it goes stale exactly where it hurts: ...") to one sentence stating the division of labor with convex-expert.
Make step 2(b) executable with one concrete fetch example (e.g., the exact WebFetch/curl form for a docs.convex.dev markdown page) so the primary path has a copy-paste-ready command.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient and assumes competence, but there is noticeable repetition: the freshness hierarchy appears in full in step 2 ("(a) if a served docs tool... (b) else fetch the specific docs page... (c) only then fall back") and again as a Rules bullet ("Follow the freshness hierarchy: served docs tool → page-as-markdown / pinned README → general web"), and "never write from memory" is stated three times (intro, step 2's "Do NOT skip to writing the API from memory", and Rules bullet 1). The intro also pads with narrative ("excellent for stable idioms, but it goes stale exactly where it hurts"). This fits the 3 anchor — could be tightened — better than the 4 anchor's "minor instances". | 3 / 5 |
Actionability | Concrete, executable guidance is present: a runnable version-pinning command ("node -p \"require('./node_modules/convex/package.json').version\""), a named MCP tool ("search_convex_docs"), a concrete URL pattern ("docs.convex.dev/<path>"), and a ground-truth check path ("node_modules/@convex-dev/<x>/" ... "its package.json exports, its .d.ts"). It is not a 5 because the actual fetch step (b) has no executable example — no curl/WebFetch command or worked URL — leaving a minor gap. | 4 / 5 |
Workflow Clarity | The 5-step sequence is clearly ordered with an explicit verification checkpoint (step 3, "VERIFY against the installed package") and a genuine error-recovery feedback loop (step 5, "On a version-mismatch build error... treat it as a currentness question — pin the version, fetch the current API, and correct"). This matches the 5 anchor's "clear sequence with explicit validation steps; feedback loops for error recovery"; operations here are read-only, so the destructive/batch cap does not apply. | 5 / 5 |
Progressive Disclosure | The skill is under 50 lines, single-purpose, and has no bundle files (no references/, scripts/, or assets/ exist), and the body makes no dangling file references — every pointer is to an external site or a sibling skill, which keeps content one level deep. Per the simple-skill exception, the well-organized Workflow/Rules sectioning earns the 5 anchor. | 5 / 5 |
Total | 17 / 20 Passed |