Content
92%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.
An exemplary conventions-reference skill: dense with project-specific, non-inferable knowledge, explicitly sequenced with a verification checklist, and appropriately sized with clean delegation to sibling skills. The only dimension short of full marks is actionability, where code appears as fragments and naming patterns rather than complete runnable examples.
Suggestions
Add one minimal copy-paste edge-function skeleton showing the CORS → authenticate() → handler chain, so the middleware convention is executable rather than descriptive.
Include a short example RLS policy snippet demonstrating the WITH CHECK column-whitelisting pattern (e.g. constraining status/type via CHECK or policy), since that security-critical rule is currently stated only as prose.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and dense with project-specific facts Claude could not know: named views ("contacts_summary", "companies_summary"), named shared utils, the middleware chain order, and a specific migration ID. It explains no general concepts, assumes competence throughout, and every line carries non-obvious information — matching 'every token earns its place'. Not a 4 because there are no trimmable over-explanations anywhere. | 5 / 5 |
Actionability | Mostly concrete, executable guidance: exact view names, file paths in `supabase/functions/_shared/`, the `verify_jwt = false` config setting, an inline call pattern (`httpClient(`${supabaseUrl}/functions/v1/<name>`)`), and named conventions (e.g. `mergeContacts()` → `/functions/v1/merge_contacts`). It stops short of anchor 5's 'copy-paste ready' bar because the code shown is fragments and naming patterns rather than complete, runnable examples (e.g. no full edge-function skeleton or RLS policy snippet), though for a conventions-reference skill the gaps are minor. | 4 / 5 |
Workflow Clarity | The entry instruction sequences the process explicitly — 'Read the relevant section before writing backend code, then check the Red Flags and Verification list' — and the skill closes with a 6-item verification checklist covering exactly its risky operations (view sync, RLS/trigger on new tables, SECURITY DEFINER counts, WITH CHECK column coverage, middleware order). Validation checkpoints are explicit and comprehensive, matching the anchor-5 pattern of clear sequence plus checklists; there are no missing-validation gaps that would cap it at 3 or 4. | 5 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), and none are needed: the ~55-line body is a genuinely lean overview with well-organized sections, and detailed material is delegated via clearly signaled one-level pointers to sibling skills (`Skill({skill: "writing-migrations"})`, `Skill({skill: "frontend-dev"})`). Under the rubric's guidance for short skills with no need for external references, well-organized sections alone warrant a 5, and nothing here should have been split out. | 5 / 5 |
Total | 19 / 20 Passed |