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, highly actionable single-file skill: complete executable code, a safe-default decision table, an explicit event-vs-DB-state model, and a verification checklist with negative tests. The only real weaknesses are the inlined Supabase boilerplate (both a conciseness and organization cost) and two small executable gaps (the unshown `getPaddleInstance` helper and the abbreviated undo action).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The effectiveFrom table, event timeline, and pitfalls are dense with every token earning its place; the one padded zone is ~45 lines of standard `@supabase/ssr` boilerplate helper code plus explanation of what the service-role client does, which the skill itself marks as optional ("if your project already has equivalent helpers, use those instead"). Not 3 — only one trimmable section; not 5 — that section is genuine over-explanation of library setup Claude already knows. | 4 / 5 |
Actionability | The full Server Action is nearly copy-paste ready with auth, ownership check, `paused` branching, error mapping, revalidation, and a slim DTO, plus env vars and a 7-step verification checklist. Two minor gaps keep it below 5: `getPaddleInstance` is imported but never shown, and `undoCancelSubscription` abbreviates its auth/ownership steps as "Run the same auth and ownership checks" rather than being standalone. | 4 / 5 |
Workflow Clarity | Numbered steps inside the action (authenticate → verify ownership → cancel → revalidate → DTO → error handling), a clear 3-stage webhook event timeline with the warning "Don't depend on which arrives first", feedback loops for recoverable errors (409 lock → "Try again after it renews"), and an explicit verification checklist including negative tests (logged-out and forged `subscriptionId`). This is a destructive operation and validation is emphatically present. | 5 / 5 |
Progressive Disclosure | Single-level structure with well-organized sections and clearly signaled cross-skill pointers (`subscription-sync`, `webhooks`, `customer-portal`) plus a Related docs list. Not 5: the ~50-line Supabase client setup and the Retain/portal alternative details are inlined in SKILL.md when they could live in a reference file. | 4 / 5 |
Total | 17 / 20 Passed |