Content
61%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 delivers a solid, executable CLI workflow with genuinely useful auth and connection-state guidance, and its structure is reasonable for a single-file skill. Its main weaknesses are a broken step numbering scheme that references a nonexistent 'Step 2', duplicate action-discovery sections, an empty entity-list overview, and a docs link that wrongly points to Apple Pay rather than CamPay documentation.
Suggestions
Fix the workflow numbering: either add explicit 'Step 1', 'Step 2', ... headers matching the 'skip to Step 2' and '1b' references, or replace those references with links to the actual section names.
Merge the 'Popular actions' section into 'Searching for actions' (they currently repeat the same `action list --intent` command), and either populate it with real common CamPay actions or remove it.
Correct 'Official docs: https://developer.apple.com/apple-pay/' to CamPay's actual documentation, settle on one CLI invocation style (`membrane` vs `npx @membranehq/cli`), and replace the content-free CamPay Overview bullet list with one sentence on what each entity (Payment, Invoice, Account) is used for.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — commands dominate and explanations like "Membrane handles authentication and credentials refresh automatically" add real value — but there is padding that could be trimmed: the "CamPay Overview" bullet list ("Payment / Recipient / Invoice / Account") conveys almost nothing, the sentence "Those will be used to adjust tooling to be used best with your harness" is garbled filler, and "This is the fastest way to get a connection" adds little. Fits the anchor 'Mostly efficient but includes some unnecessary explanation or could be tightened'; below 4 due to the empty overview section and redundant 'Popular actions' section repeating the 'Searching for actions' command. | 3 / 5 |
Actionability | Commands are concrete and copy-paste ready throughout: `npm install -g @membranehq/cli@latest`, `membrane connection ensure "https://www.campay.net/" --json`, `membrane action run <actionId> --connectionId=CONNECTION_ID --input '{"key": "value"}' --json`, plus a flag table for proxy requests. Fits 'Mostly executable guidance; concrete code or commands with minor gaps' — it falls short of 5 because no end-to-end example of an actual CamPay operation is shown (all examples use placeholders like QUERY/CONNECTION_ID), CLI invocation is inconsistently shown as `membrane` vs `npx @membranehq/cli`, and the docs link points to Apple Pay, not CamPay docs. | 4 / 5 |
Workflow Clarity | The sequence (install → login → connection ensure → poll for READY → search actions → run) is present and the connection-state handling (READY / CLIENT_ACTION_REQUIRED / CONFIGURATION_ERROR with re-poll after user action) is a genuine feedback loop, but the numbered flow is broken: the text says "skip to **Step 2**" and has a section "#### 1b. Wait for the connection to be ready" yet no 'Step 1' or 'Step 2' headings exist anywhere. Fits 'Steps listed but validation gaps; sequence present but checkpoints missing or implicit' — the step references point to nonexistent anchors, and there is no guidance on validating action output (e.g., inspecting `output` fields, handling errors from `action run`). | 3 / 5 |
Progressive Disclosure | There are no bundle files (no references/, scripts/, or assets/ directories), and the ~140-line body is organized with clear section headers (Install, Authentication, Connecting, Searching, Running, Proxy requests, Best practices), so all content sits appropriately in SKILL.md with no nested or buried references. Fits 'Good structure; most content is appropriately placed; minor organization gaps' — the gaps being the redundant 'Popular actions' section duplicating 'Searching for actions', the broken step numbering, and the proxy flag table that is acceptable inline but could live in a reference file if the skill grows. Below 5 because navigation between the duplicated discovery sections and the phantom 'Step 2' is confusing. | 4 / 5 |
Total | 14 / 20 Passed |