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.
The body is a solid, command-driven guide with a well-sequenced connection workflow, real state-based checkpoints, and a sensible proxy escape hatch. Its weaknesses are a two-spot padding problem (the 'what is Adyen' intro and the empty overview bullets), a dangling 'Step 2' reference, and the absence of a single worked example with real Adyen values.
Suggestions
Delete the 'Adyen is a global payment platform...' intro paragraph and the bare 'Adyen Overview' bullet list with 'Use action names and parameters as needed.' — Claude already knows what Adyen is, and the bullets carry no information.
Fix the dangling "skip to **Step 2**" references (no Step 2 exists) by numbering the actual sections, and standardize on one CLI invocation (`membrane` vs `npx @membranehq/cli`) throughout.
Add one worked end-to-end example with concrete values (e.g. connect, find a refund action via `action list --intent "refund a payment"`, then run it) so the placeholders CONNECTION_ID/QUERY/actionId have a filled-in reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is command-focused and efficient, but it opens with a paragraph explaining what Adyen is ("Adyen is a global payment platform that allows businesses to accept payments online, in-app, and in-store... E-commerce businesses and retailers use Adyen to streamline their payment operations"), which Claude already knows, and includes the filler "Adyen Overview" bullet list with "Use action names and parameters as needed." This fits anchor 3 (mostly efficient but includes some unnecessary explanation); it is above anchor 2 because the padding is limited to two spots rather than pervading the document. | 3 / 5 |
Actionability | Concrete, copy-paste-ready commands throughout: `npm install -g @membranehq/cli@latest`, `membrane connection ensure "https://www.adyen.com/" --json`, `membrane action run <actionId> --input '{"key": "value"}' --json`, plus a complete proxy flag table. It falls short of anchor 5 because there is no worked end-to-end example with real values — every command uses placeholders (CONNECTION_ID, QUERY, actionId) and no Adyen-specific action is ever named or executed. | 4 / 5 |
Workflow Clarity | There is a clear sequence (install → login → connection ensure → poll state → search actions → run → proxy fallback) with genuine validation checkpoints: polling `--wait` until state leaves BUILDING, an explicit state machine for READY / CLIENT_ACTION_REQUIRED / CONFIGURATION_ERROR with recovery actions (show uiUrl, re-poll), and guidance to prefer pre-built actions. It misses anchor 5 due to a dangling instruction — "skip to **Step 2**" when no Step 2 exists in the document — and inconsistent CLI invocation (`membrane` vs `npx @membranehq/cli`). | 4 / 5 |
Progressive Disclosure | No bundle files (references/, scripts/, assets/) exist, and the single-file body (~140 lines) is organized into clear sections (Install, Authentication, Connecting, Searching, Running, Proxy, Best practices) with the bulkier reference-style material (the clientAction state-machine detail and the proxy flag table) appropriately set off. This matches anchor 4 (good structure, minor organization gaps); it misses anchor 5 because those reference-style blocks and the CLI setup boilerplate could live in separate reference files to keep SKILL.md a leaner overview. | 4 / 5 |
Total | 15 / 20 Passed |