Content
71%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 well-structured routing-style skill: concrete commands and parameter-level rules, a clear domain-to-reference routing table, and a verified one-level-deep bundle. The main costs are time-sensitive version tables and SDK rows inlined in the body, plus slightly padded rationale in the payment-methods rule.
Suggestions
Move the 'Latest API version' line and the SDK version table into a dedicated reference file or a 'current versions' section so stale numbers don't burden the always-loaded body.
Trim the rationale in the payment_method_types rule ('to maximize conversion', 'dynamically display the most relevant eligible payment methods') to a one-line directive.
Add a one-line correct-pattern example of instantiating a StripeClient, since the rule currently only shows the deprecated anti-patterns.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient directive prose, but time-sensitive version data sits in the main body rather than a deprecated/old-patterns section ('Latest Stripe API version: 2026-08-26.dahlia' plus a 7-row SDK version table), and a few passages explain rationale Claude doesn't need (the payment_method_types rule spends several clauses justifying dynamic payment methods and conversion). This fits 'Mostly efficient but includes some unnecessary explanation or could be tightened'. | 3 / 5 |
Actionability | Highly concrete throughout: exact commands ('npm i -g @stripe/cli', 'stripe sandbox create', 'stripe whoami --format json'), exact parameter names ('payment_method_types', 'allowed_payment_method_types', 'integration_identifier', 'automatic_tax: { enabled: true }'), and named deprecated patterns. It falls just short of fully copy-paste ready because the StripeClient rule says what not to do ('Do not use the deprecated global/module-level API key pattern') without showing the correct instantiation example. | 4 / 5 |
Workflow Clarity | The routing table gives a clear decision flow, 'Read the relevant reference file before answering any integration question or writing code' sets the sequence, and the tax rule includes an explicit pre-flight checkpoint ('read the tax reference and confirm the user has an active registration') with the failure consequence. It matches 'Clear sequence with most checkpoints present; minor validation gaps' — there is no explicit error-recovery/feedback loop, but no destructive or batch operation requires one. | 4 / 5 |
Progressive Disclosure | The body is a lean overview that routes each domain to a real one-level-deep bundle file via the integration routing table (payments, connect, billing, tax, treasury, security — all verified present in references/), with a mandatory 'read the relevant reference' directive and no nested references inside the bundle. This matches 'Clear overview with well-signaled one-level-deep references; content appropriately split; easy navigation'. | 5 / 5 |
Total | 16 / 20 Passed |