Content
71%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 tight, well-sequenced guide for a simple single-purpose skill: both seeding paths are distinguished, idempotency and a safety rule about secrets/PII are called out, and a verification step exists. The main gap is that the primary workflow (write an internalMutation and run it) and the verification step remain at the hint level rather than copy-paste executable.
Suggestions
Add a minimal re-runnable internalMutation example with the exact `npx convex run <functionName>` invocation so the primary fixtures path is copy-paste executable.
Specify how to verify row counts (e.g., the Convex dashboard table view or a count function via `npx convex run`) and what to do when counts don't match.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is very lean with no background on Convex or seeding concepts ("Populate tables via an internalMutation seed function (re-runnable) or `npx convex import`, matching the schema"), assuming Claude's competence. However, the Rules section duplicates the workflow — "Make seeding idempotent" restates step 3 verbatim and "Seed via internalMutation or convex import, matching validators" restates steps 1-2 — so a few tokens could be trimmed, keeping it below anchor 5. | 4 / 5 |
Actionability | Real commands and mechanisms are named (`npx convex run`, `npx convex import`, internalMutation, "clear-then-insert or upsert"), but the primary path lacks executable detail: no example internalMutation seed function, no exact `npx convex run <functionName>` invocation syntax, and "Verify row counts" specifies no method (dashboard, count query, or CLI). Missing key details on the main path fits anchor 3 rather than anchor 4's mostly-executable guidance. | 3 / 5 |
Workflow Clarity | A clear 4-step sequence distinguishes both paths ("For fixtures: write an internalMutation...", "For bulk import: shape the data to the schema and use `npx convex import`") and ends with an explicit validation checkpoint ("Verify row counts") plus a safety mechanism ("clear-then-insert or upsert so re-running is safe"), matching anchor 4. Not 5: there is no failure-handling feedback loop (what to do if counts are wrong) and the verification method is unspecified. | 4 / 5 |
Progressive Disclosure | The skill is under 50 lines, single-purpose, and well organized into a summary line, Workflow, and Rules sections with no external references needed or referenced — and no bundle files (references/, scripts/, assets/) exist. Per the guideline for simple skills under 50 lines, well-organized sections alone merit a 5. | 5 / 5 |
Total | 16 / 20 Passed |