Content
67%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 concise, well-structured instruction-style skill with a clear use/do-not-use boundary, a sensible workflow, and an explicit sandbox-validation checkpoint. Its two real weaknesses are a dangling reference to a nonexistent `resources/implementation-playbook.md` and instructions that stay at the level of named deliverables without executable specifics.
Suggestions
Create the referenced `resources/implementation-playbook.md` (with the promised patterns, checklists, and examples) or remove both references to it — as written, the skill points users to a file that does not exist.
Add concrete, executable detail to the instructions: e.g., a sample plan/proration data model, a dunning retry schedule (e.g., 3 retries over 14 days), specific tax API or rate-table handling, and the exact sandbox-validation commands.
Include an explicit validate → fix → retry loop around sandbox payment validation and ledger reconciliation, since billing changes are batch operations on customer money.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and directive with zero padding: sections are 'Use this skill when', 'Do not use this skill when', six one-line instructions, a safety section, and a resources pointer. No concepts Claude already knows are explained, and every token earns its place — matching the 5 anchor ('lean and efficient; assumes Claude's competence'). | 5 / 5 |
Actionability | The instructions enumerate concrete deliverables ("Define plans, pricing, billing intervals, and proration rules", "Implement invoicing, payments, retries, and dunning workflows", "Validate with sandbox payments and reconcile ledger outputs") but contain no executable specifics — no commands, schemas, library choices, or worked examples. This fits the 3 anchor ('some concrete guidance but incomplete; missing key details') rather than 4, since nothing is directly executable, and rather than 2, because the guidance goes beyond high-level hints to name specific artifacts. | 3 / 5 |
Workflow Clarity | The instructions form a sensible build sequence (define plans → map lifecycle → implement invoicing/dunning → model taxes → validate) and include an explicit validation checkpoint ("Validate with sandbox payments and reconcile ledger outputs") plus a safety guard ("Do not charge real customers in testing environments"). It stops short of the 5 anchor because there is no error-recovery feedback loop (validate → fix → retry) for what is a batch/financial operation, leaving a minor validation gap consistent with the 4 anchor. | 4 / 5 |
Progressive Disclosure | The body twice references `resources/implementation-playbook.md` ("If detailed templates are required, open `resources/implementation-playbook.md`" and the Resources section), but the bundle contains no `resources/` directory at all — the referenced file does not exist, so the disclosure path is broken. While the in-file structure and reference signaling are clean, a dangling reference to a nonexistent file leaves the skill without its promised detailed content, matching the 2 anchor's 'minimal structure / broken disclosure' level rather than 3, where references at least resolve. | 2 / 5 |
Total | 14 / 20 Passed |