Content
81%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 strong, terse rule-set that front-loads exactly the non-obvious Convex failure modes (import sources, index discipline, "use node" constraints) and closes with an unusually thorough verify-and-fix loop. The main improvements are structural: deduplicate the intro against the frontmatter, fix the dangling DATA ACCESS + IMPORTS header bullet, and add one short object-form function example.
Suggestions
Delete or drastically shorten the opening paragraph, which repeats the frontmatter description almost word-for-word and adds no new guidance.
Add a 5-10 line example of an object-form function with args and returns validators, since the body mandates this syntax repeatedly but never shows it.
Convert the dangling "DATA ACCESS + IMPORTS —" bullet into a proper subsection header (e.g., "### Data access and imports") so the Rules list reads as organized sections rather than one flat wall.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious, high-value rules (exact import table, "use node" placement, storage-id discipline) with no explanation of concepts Claude already knows, matching the efficient anchor. It falls short of lean/5 because the opening paragraph restates the frontmatter description nearly verbatim ("Knows the object-form function syntax, validator requirements, index naming rules...") and the SELF-VERIFY RULE bullet is a run-on sentence that could be tightened. | 4 / 5 |
Actionability | Mostly executable: exact imports ("`query`/`mutation`/`action`... come from `./_generated/server`") and copy-paste commands ("npx tsc --noEmit", "npx convex dev --once", "CONVEX_AGENT_MODE=anonymous npx convex dev --once") appear with concrete API names. The gap keeping it at 4 rather than 5: the headline pattern — object-form functions with args and returns validators — is mandated repeatedly ("Object form only", "args and returns validators on every registered function, no exceptions") but never shown with a short example, leaving the most common case demonstrated only in prose. | 4 / 5 |
Workflow Clarity | The 6-step Workflow is clearly sequenced (read schema first → write → verify push) and closes with an explicit validation checkpoint, while the SELF-VERIFY RULE adds the full feedback loop: run "npx tsc --noEmit" and push, "Fix every error it reports before finishing", with conditional branching on whether "npx convex whoami" succeeds. This matches the anchor with explicit validation steps and error-recovery loops. | 5 / 5 |
Progressive Disclosure | The skill is a single well-organized file under 50 lines with no bundle files (no references/, scripts/, or assets/), and its two sections (Workflow, Rules) are clearly signaled — per the simple-skill guideline this could reach 5. It sits at 4 because the Rules list opens with a dangling pseudo-header bullet ("DATA ACCESS + IMPORTS — read before writing any convex/*.ts (front-loaded, not a post-hoc lint):" followed by flat siblings), a minor organization gap suggesting an intended subsection that was flattened. | 4 / 5 |
Total | 17 / 20 Passed |