Content
75%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-structured advisory workflow with genuinely actionable mappings from runtime evidence to code-level fixes and strong guardrails (read-only discipline, no invented findings, explicit scope routing). The main improvement opportunities are de-duplicating rules restated in the workflow and adding a concrete fix snippet or example finding.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean, dense prose that assumes competence ("Static review guesses; the deployment KNOWS") with no concept explanations. Not 5 because the severity mapping, read-only guard, and scope routing each appear twice (Workflow steps 4/1/6 and again in Rules) and could be consolidated. | 4 / 5 |
Actionability | Concrete, executable guidance for an instruction-only skill: named MCP tools (`status`, `insights`, `tables`, `functionSpec`), typed event kinds, and specific fixes (`.withIndex`, `.take(n)`, `.paginate`, `@convex-dev/sharded-counter`). Not 5 because fixes are named rather than shown — no copy-paste snippets for the common cases. | 4 / 5 |
Workflow Clarity | A clear 6-step sequence with a guard step, an explicit fallback when insights are unavailable ("do NOT invent findings"), and a verify loop after applying fixes ("re-run `insights` after traffic to verify the trend"). Not 5 because the verification step is offered rather than mandated and there is no explicit error-recovery path if the flagged function's code cannot be read. | 4 / 5 |
Progressive Disclosure | At ~28 lines with well-organized Workflow/Rules sections and no bundle files, the structure is good and appropriately sized. Not 5 because pointers like "specs/finding.schema.json" and "convex-expert patterns" reference paths outside the skill that are not resolvable from the bundle, a minor navigation gap. | 4 / 5 |
Total | 16 / 20 Passed |