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.
The body is highly actionable with copy-paste Membrane CLI commands and a well-sequenced connection workflow including state validation, but it is somewhat verbose in its intro and is monolithic with no progressive disclosure into reference files.
Suggestions
Trim the marketing-style WeSupply intro and redundant 'fastest way to get a connection' sentences to improve token efficiency.
Fix the inconsistent step numbering (1b without 1a, 'Step 2' without a numbered parent) so the workflow sequence is unambiguous.
Move the detailed connection state machine and the full `membrane request` flag table into a references file and link to it from the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Quotes "WeSupply is a returns management platform that helps e-commerce businesses streamline and automate their returns process... improve customer satisfaction and reduce the costs associated with returns" and "This is the fastest way to get a connection" — mostly efficient CLI guidance, but the marketing-style intro, a few padded sentences, and a duplicated `action list` command could be trimmed. It does not reach 4 because several sentences add explanation rather than instruction. | 3 / 5 |
Actionability | Quotes copy-paste ready commands such as `npm install -g @membranehq/cli@latest`, `membrane connection ensure "https://wesupplylabs.com/" --json`, `membrane action run <actionId> --connectionId=CONNECTION_ID --input '{...}' --json`, plus a flag table for `membrane request` — fully executable guidance covering the common cases. No gaps justify a lower score. | 5 / 5 |
Workflow Clarity | Quotes a clear sequence (install → login → `connection ensure` → poll `connection get --wait` → `action list` → `action run`/`request`) with explicit state checks (READY, CLIENT_ACTION_REQUIRED, CONFIGURATION_ERROR) acting as validation checkpoints and a re-poll feedback loop. It stops at 4 because step labels are inconsistent ('#### 1b' with no 1a, references to 'Step 2' without a numbered parent structure). | 4 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ absent), and the body is a single monolithic document inlining the full connection state machine, proxy flag table, and best practices under clear headers. Structure is decent but content that could be split into reference files is inlined with no external references, fitting 'Some structure but could be better organized' rather than 4. | 3 / 5 |
Total | 15 / 20 Passed |