Content
86%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 routing skill: a scenario-to-API table with per-domain reference files, hard rules with exact parameter names and deprecated patterns, and concrete CLI commands. The only weaknesses are a slightly padded rationale sentence, version tables that will go stale, and rule-list structure that leaves feedback loops to the reference files.
Suggestions
Conciseness: trim the dynamic-payment-methods rationale ('dynamically display the most relevant eligible payment methods to each customer to maximize conversion') to just the rule — omit payment_method_types, use payment_method_configurations or excluded_payment_method_types — since the Dashboard-behavior explanation is not needed to follow it.
Conciseness: move the 'Latest SDK versions' table and pinned API version ('2026-09-30.endive') into a clearly labeled versions/deprecations reference (or the per-domain references) so stale version data is quarantined and the body stays evergreen.
Workflow clarity: state the verification loop inline for the riskiest rules, e.g. after 'confirm the user has an active registration', add the follow-up check (run a test Tax Calculation and check taxability_reason) rather than deferring it entirely to references/tax.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean and restricted to Stripe-specific facts Claude cannot know (latest API version, sandbox vs test-mode rules, exact parameter names), but two things could be trimmed or would rot: the conversion rationale 'dynamically display the most relevant eligible payment methods to each customer to maximize conversion', and time-sensitive version data ('2026-09-30.endive', the 7-row SDK version table) that is not placed in a deprecated/old-patterns section. Not 3, because there is no explanation of concepts Claude already knows and nearly every line is an imperative rule. | 4 / 5 |
Actionability | Fully executable guidance throughout: copy-paste commands ('npm i -g @stripe/cli', 'stripe sandbox create', 'stripe whoami --format json'), exact parameter rules ('Never include payment_method_types… use allowed_payment_method_types'), named deprecated patterns ('stripe.api_key = …', 'Stripe.setApiKey'), and a routing table mapping every common scenario to a specific API and reference file. Not 4, because no common case is left without concrete instruction. | 5 / 5 |
Workflow Clarity | The sequence is clear — classify the task via the routing table, then 'Read the relevant reference file before answering any integration question or writing code', then apply the critical rules — and pre-flight checkpoints are present ('confirm the user has an active registration', 'first make sure stripe sandbox claim is run'). Not 5, because the body presents rules as a list rather than an ordered workflow with explicit validate/fix/retry feedback loops (verification detail is deferred to the reference files); not 3, because ordering and preconditions are explicit and stated. | 4 / 5 |
Progressive Disclosure | A true overview that keeps only defaults, a routing table, and critical rules in SKILL.md, pushing domain detail to six real, well-signaled, one-level-deep reference files (references/payments.md, connect.md, billing.md, tax.md, treasury.md, security.md — all present in the bundle, none nesting further pointers). The 'Building… | Recommended API | Details' table makes navigation unambiguous; not 4, since there are no organization gaps to point to. | 5 / 5 |
Total | 18 / 20 Passed |