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.
The body is a well-engineered interactive workflow: concrete scripts, mandatory validation with correction loops, and a clean one-level reference bundle that all checks out on disk. Its main weakness is token efficiency — repeated checklist blocks, triplicated public-preview instructions, and a few wall-of-text guardrail paragraphs that could be compressed or pushed into the references.
Suggestions
Replace the five full checklist reproductions with a single 'update the checklist' instruction plus one canonical checklist block in Step 0.
State the public-preview disclosure requirements once (e.g., in a short subsection or in the compatibility-matrix reference) and have the two Step 4a bullets and the Step 3 guardrail reference it instead of restating all four requirements three times.
Break the Step 3 SaaS direct-charge guardrail into a short decision list and move the Express preview configuration detail into a reference file, mirroring how discovery questions are already handled.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is dense, non-obvious Stripe policy (compatibility tuples, fee-economics rules) that earns its tokens, but there is real padding: the 4-line progress checklist is reproduced in full five times, the public-preview disclosure instructions are stated three times (Step 3 guardrail and both Step 4a bullets), and the Step 3 SaaS guardrail is a single ~200-word run-on paragraph. This fits 'mostly efficient but could be tightened'; a 2 would require several sections of explanation Claude already knows, which is not the case. | 3 / 5 |
Actionability | The guidance is copy-paste ready throughout: exact user-facing question scripts and option sets ('Does this recommendation look right?' with three defined responses), exact warning copy with links, concrete grep patterns for the codebase scan ('transfers.create', 'account.updated', 'transfer_data'), and specific API field names (dashboard, fees_collector, losses_collector, application_fee_amount) with defined behavior for each option. As an instruction-only skill, the absence of code is not penalized and the specificity fully covers common cases. | 5 / 5 |
Workflow Clarity | Steps 0-6 are explicitly numbered and sequenced, each step ends with a checklist update, and there are genuine validation checkpoints with feedback loops: a mandatory compatibility validation before presenting any recommendation, BLOCKED configs trigger 're-run the recommendation with the corrected configuration', a fee-structure checkpoint gates Step 4, and 'Change something' loops back to re-ask the relevant question. This matches the top anchor including error-recovery loops. | 5 / 5 |
Progressive Disclosure | All 8 referenced files (terminology-rules, company-researcher, decision-matrix, discovery-questions, account-types, charge-patterns, compatibility-matrix, recommendation-template) exist in references/, are one level deep, and are clearly signaled at the point of need ('Read <references/decision-matrix.md> and apply it'), with bulk content correctly split out (38KB decision matrix, 20KB discovery questions). Not a 5: SKILL.md itself runs ~360 lines and inlines compatibility and guardrail detail that overlaps the reference files, and there is no upfront index of the reference bundle. | 4 / 5 |
Total | 17 / 20 Passed |