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 dense, highly actionable reference: nearly every line is non-obvious framework policy backed by executable code, concrete commands, and genuine error-recovery loops, with a verified one-level-deep reference bundle. Its main cost is length and duplication — some rule detail (receipts, error plumbing, designSystem) belongs in action-fields.md and two Do/Don't bullets are redundant.
Suggestions
Merge the two near-duplicate bullets on fail() vs bare Error (the 'Do signal failure by throwing...' and 'Do use fail() rather than a bare Error...' items) into one, and fold the overlapping 'Don't reach for provider integrations, outputSchema...' bullet into the existing pointer to references/action-fields.md.
Move the write-receipt supersession/subject rules and the designSystem context plumbing detail into references/action-fields.md, keeping only the receipt contract and verified/changed semantics in SKILL.md.
Rewrite 'How to Create an Action' as a short numbered sequence (create actions/<name>.ts with default export → write schema with .describe() on every param → set http option → run via pnpm action) so the happy path is scannable without reading every policy paragraph.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Almost every sentence is framework-specific knowledge Claude cannot know (auto-refresh rules, receipt semantics, tool-loading behavior), so there is no filler — but there is real fat: two near-duplicate 'Do' bullets on fail() vs bare Error, an overlapping 'Don't' about provider integrations/outputSchema, and long passages (receipt supersession, designSystem plumbing) that could be tightened or deferred. Anchor 3 ('mostly efficient but could be tightened') fits better than 4 because the redundancy is more than minor. | 3 / 5 |
Actionability | Fully executable throughout: a copy-paste-ready defineAction example with imports, a concrete CLI invocation ('pnpm action log-meal --name "Salad" --calories 350'), a complete http-option table, executable hook imports, a worked WriteReceipt object, and a Troubleshooting section with concrete fixes. Covers the common cases the way the anchor-5 example does. | 5 / 5 |
Workflow Clarity | Clear decision order ('existing action → extend/create a defineAction → custom route as last resort'), an explicit stop trigger before adding files under server/routes/api/, and real feedback loops (verified:false receipts force a reconciliation retry; NoWriteError triggers argument-correcting retry; 405/parse-failure diagnostics). It falls short of anchor 5 because the create-an-action flow is implicit in prose rather than an explicitly numbered sequence. | 4 / 5 |
Progressive Disclosure | All three referenced files (references/action-fields.md, provider-apis.md, examples.md) exist, are one level deep, and are well signaled both inline ('examples in references/action-fields.md') and in a References section with per-file descriptions. Below anchor 5 because substantial detail (receipt supersession rules, designSystem context plumbing, MCP key-tool mechanics) is inlined in SKILL.md when the established references/action-fields.md pattern is the natural home for it. | 4 / 5 |
Total | 16 / 20 Passed |