Content
68%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, executable Membrane-CLI integration guide with concrete commands, a well-handled connection state machine, and good single-file organization. Its main weaknesses are a broken step numbering scheme (references to a nonexistent "Step 2"), an empty "Popular actions" section, and the absence of any verification or user-confirmation guidance before executing payment actions.
Suggestions
Fix the workflow numbering: either label the sections as Steps 1–4 (Install → Authenticate → Connect → Discover/Run) so the "skip to Step 2" references resolve, or remove the step references and the orphaned "1b" heading.
Add an explicit safety checkpoint before running payment or money-moving actions — e.g. confirm the action's inputSchema and get user approval before `membrane action run` on transaction/payment intents, and verify the `output` field afterward.
Populate the "Popular actions" section with 2–3 real example action invocations (e.g. listing transactions, creating a payment) with concrete connection IDs, or delete the section since it currently duplicates the "Searching for actions" command.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and command-first with no explanations of concepts Claude already knows, and the connection-state and proxy-flag material is reference-dense rather than padded. Minor instances could be trimmed — the rambling "Agent Types" aside ("claude, openclaw, codex, warp, windsurf, etc. Those will be used to adjust tooling..."), the near-duplicate `action list` command in "Popular actions", and the "Best practices" prose — so it is anchor 4, not the every-token-earns-its-place level of 5. | 4 / 5 |
Actionability | Concrete, executable CLI commands appear throughout (`npm install -g @membranehq/cli@latest`, `membrane connection ensure "https://banked.com/" --json`, `membrane action run <actionId> --input '{"key": "value"}'`, plus a complete proxy flag table), which is mostly copy-paste ready. It is not 5 because the "Popular actions" section contains no actual popular actions, `CONNECTION_ID` and `<actionId>` are never shown with a concrete worked example, and the Banked Overview's "Use action names and parameters as needed" is directionless. | 4 / 5 |
Workflow Clarity | The install → login → connect → discover → run sequence is present and the connection phase has a good polling/state checkpoint loop (READY / BUILDING / CLIENT_ACTION_REQUIRED with instructions to re-poll), but the flow is incoherent in places — it says "skip to **Step 2**" though no Step 2 exists, and a "1b" subsection appears with no "1a" or Step 1 header. More importantly, running Banked payment actions moves money, and there are no verification or confirmation steps before `action run`, so the destructive/batch-operations cap of 3 applies. | 3 / 5 |
Progressive Disclosure | This is a well-organized single-file skill: clear section headers, no nested or dead-end references, and at ~140 lines the workflow content appropriately lives inline (connection state handling, auth, and proxy usage are core, not auxiliary). It is not 5 because of minor organization gaps — the empty "Popular actions" section, the inconsistent step numbering, and the auth/proxy detail that could sit in one-level-deep reference files if the skill grows. | 4 / 5 |
Total | 15 / 20 Passed |