Content
88%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 excellent, deeply knowledgeable skill body: executable code, explicit decision tables, error-code handling, and a thorough verification checklist. Its only weaknesses are mild repetition, inlined boilerplate helpers, and a lack of any reference-file split for the long single-file body.
Suggestions
Move the Supabase helper code and environment-variable setup into a reference file (e.g. references/helpers.md) and keep a one-line pointer in SKILL.md.
Consolidate the trialing/`do_not_bill` rule, currently restated in the proration table, flow table, status table, code comments, and pitfalls, into one authoritative section.
Trim the "What the user sees vs what gets billed" narrative, which overlaps with the webhook side already delegated to `subscription-sync`.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious Paddle-specific knowledge (status rules, error codes, renewal lock, credit-not-refund semantics) and assumes competence, but the ~50 lines of Supabase helper boilerplate and repeated restatements of the trialing/`do_not_bill` rule (4+ places) are minor over-explanation that could be trimmed. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready guidance: a complete Server Action with auth and ownership checks, a previewUpdate snippet with field-by-field comments, an onPaymentFailure override example, named API error codes mapped to UI messages, and a 9-step verification checklist covering the common cases. | 5 / 5 |
Workflow Clarity | The sequence is explicit — choose proration mode, check subscription status, preview before committing, execute with auth/ownership validation, verify via dashboard/webhook — with an explicit verification section, error-code feedback loops, and expected outcomes at each checkpoint for this billing operation. | 5 / 5 |
Progressive Disclosure | Well-organized single file with clear section headers and no nested references, but everything is inlined (Supabase helper code, related-docs lists) in a ~330-line body where splitting the helper/environment boilerplate into a reference file would improve the overview-to-detail structure. | 4 / 5 |
Total | 18 / 20 Passed |