Content
78%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 skill: dense imperative rules, exact commands and parameter names, and clean one-level-deep disclosure into verified reference files. The main gaps are inline version tables that will age, no copy-paste code for the common payment flows, and a core workflow that must be inferred from directives rather than followed as a sequence.
Suggestions
Move the 'Latest Stripe API version' line and the SDK version table into a dedicated versions/old-patterns section (or a reference file) so the body's time-sensitive data doesn't decay in place and bloat the overview.
Add one short copy-paste example of the mandated pattern — instantiating StripeClient and creating a Checkout Session (with integration_identifier and no payment_method_types) — since that is the most common flow the critical rules govern.
State the core workflow explicitly as a short numbered sequence (route via the table → read the matched reference → apply critical rules → confirm tax registration before enabling automatic_tax) instead of leaving it implied across sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and imperative with no explanation of concepts Claude already knows, but it embeds time-sensitive version data — 'Latest Stripe API version: 2026-08-26.dahlia' and the seven-row SDK version table — outside any deprecated/old-patterns section, which the guidelines say should penalize conciseness. It fits anchor 4 (efficient with minor instances that could be trimmed) rather than 5, and is well above 3 since everything else earns its place. | 4 / 5 |
Actionability | Guidance is largely executable: 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'), and a routing table mapping each integration type to a specific API and reference file. It falls short of anchor 5 because the most common integration flows (e.g., creating a Checkout Session with the mandated client-instance pattern) have no code example — they are deferred to references — and a few lines ('If stripe sandbox create is used, don't use MCP') are cryptic without surrounding context. | 4 / 5 |
Workflow Clarity | The core sequence is discernible — identify what you're building via the routing table, 'Read the relevant reference file before answering any integration question or writing code', then apply the critical rules — and the automatic_tax rule plus the 'stripe sandbox claim' check act as validation checkpoints. It sits at anchor 4 rather than 5 because the workflow is implied by scattered directives rather than stated as a sequence, and there is no error-recovery/feedback loop; it is not a 3 since checkpoints like the tax registration confirmation are explicit. | 4 / 5 |
Progressive Disclosure | The SKILL.md body is a clear overview that splits content appropriately: a routing table signals all six real, one-level-deep reference files (payments, connect, billing, tax, treasury, security — all verified to exist, none nesting further .md references), plus a 'Key documentation' section for cross-cutting cases. This matches the anchor-5 pattern of a concise overview with well-signaled one-level-deep references and easy navigation. | 5 / 5 |
Total | 17 / 20 Passed |