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.
Excellent token-efficient reference material: hard limits, gotchas, and exact API names with zero fluff, plus a deploy sequence with genuine validation checkpoints and a rollback loop. The recurring weakness is a dangling 'checklist' reference for breaking changes and the underspecified widen-migrate-narrow pattern, which keep actionability, workflow clarity, and disclosure just below top marks.
Suggestions
Define or inline the breaking-change migration checklist that the deploy table's 'checklist completed' checkpoint references, or point to where it lives (e.g. '.opencastle/stack/database-config.md').
Expand the widen-migrate-narrow rule with one concrete example (e.g. add the new optional field and deploy, backfill via internalMutation, then remove the old field) so the pattern is executable rather than a label.
Add deployment terms ('deploying a Convex backend', 'npx convex deploy') to the frontmatter description's capability list and trigger clause, since the Deploy Loop is a major section the description currently omits.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense tables and terse bullets with zero padding and no explanation of concepts Claude already knows — every line carries non-obvious, project-specific information ('Documents scanned | 32,000 — includes documents `.filter()` discards'; 'Components cannot access `ctx.auth` or `process.env`'). This matches anchor 5 ('lean and efficient; every token earns its place'); nothing rises to the 'minor instances of over-explanation' of anchor 4. | 5 / 5 |
Actionability | Exact commands and API identifiers appear throughout ('ctx.scheduler.runAfter(0, internal.x.batch, { cursor: result.continueCursor })', 'npx convex env set', 'v.null(), never v.undefined()'), giving mostly executable guidance per anchor 4. It falls short of anchor 5's copy-paste-ready completeness: the deploy checkpoint table says 'checklist completed' for breaking changes but no checklist exists anywhere in the skill, and widen-migrate-narrow is compressed to one line ('widen, migrate, then narrow') without concrete steps. | 4 / 5 |
Workflow Clarity | The Deploy Loop is a clear sequence ('npx convex dev' → verify → 'npx convex deploy') with an explicit error-recovery feedback loop (roll back via 'npx convex import' of the last good export, fix locally, re-deploy) and a five-row checkpoint table, satisfying anchor 4. It misses anchor 5 because the 'Breaking change | checklist completed' checkpoint references a checklist that is never shown, leaving a minor validation gap; the cap-at-3 rule for missing validation does not apply since verification steps are present. | 4 / 5 |
Progressive Disclosure | A single well-sectioned file (Hard limits, Rules by function type, Deploy Loop) with clearly signaled one-level-deep references to the project config ('.opencastle/stack/database-config.md') and official docs, matching anchor 4's 'good structure; most content appropriately placed'. It is not anchor 5: the body exceeds the under-50-lines simple-skill bar, and the migration checklist referenced in the deploy table points to no defined location, a minor navigation gap. | 4 / 5 |
Total | 17 / 20 Passed |