Content
63%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 content is a solid, actionable guide to driving Burp Suite through the Membrane CLI, with real commands, an explicit connection-state checkpoint flow, and a sensible proxy fallback. Its weaknesses are wasted tokens (an intro explaining what Burp Suite is, a bare tool-name list, a duplicated command), broken step cross-references, and no progressive disclosure structure for detail that would fit reference files.
Suggestions
Cut the "Burp Suite is a popular set of tools..." intro paragraph and the ~20-item bare tool-name list — this is background Claude already knows and adds no guidance value.
Fix the broken "Step 2" cross-references (referenced twice but never defined) by numbering the workflow coherently, e.g. Step 1: connect, Step 2: discover and run actions.
De-duplicate the `action list` command shown in both "Searching for actions" and "Popular actions", and move the clientAction state-machine detail and `membrane request` flags table into a reference file linked from the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body opens by explaining what Burp Suite is ("Burp Suite is a popular set of tools used for web application security testing... It acts as a proxy, allowing users to intercept and manipulate HTTP traffic") — knowledge Claude already has — and includes a ~20-item bullet list of bare tool names with no guidance value. The `action list` command is also duplicated in both "Searching for actions" and "Popular actions". Anchor 3 fits; not 2 because the padding is localized and the CLI guidance itself is efficient. | 3 / 5 |
Actionability | Concrete, copy-pasteable commands throughout: `npm install -g @membranehq/cli@latest`, `membrane connection ensure "https://portswigger.net/burp" --json`, `membrane action run <actionId> --input '{"key": "value"}' --json`, plus a full flags table for `membrane request`. Not 5 because no Burp-specific action examples (e.g., listing scan issues or reading proxy history) are shown, so the app's common cases aren't covered; not 3 because the guidance is executable, not pseudocode. | 4 / 5 |
Workflow Clarity | A clear sequence (install → login → connection ensure → poll state → handle clientAction → discover/run actions → proxy fallback) with explicit state checkpoints (READY / CLIENT_ACTION_REQUIRED / CONFIGURATION_ERROR) and a poll-again loop after user actions. Not 5 because "skip to **Step 2**" is referenced twice but no Step 2 exists in the document (the only numbered heading is "1b"), and there is no validation guidance after running an action; not 3 because checkpoints are explicit rather than missing. | 4 / 5 |
Progressive Disclosure | No bundle files exist and the single-file body is ~140 lines with clear sections, but the clientAction state-machine detail, the proxy flags table, and the tool-name overview list are inline material that belongs in a reference file, and there is no navigation to deeper material at all. Anchor 3 (some structure, could be better organized); not 4 given the inlined detail and absence of any reference pointers. | 3 / 5 |
Total | 14 / 20 Passed |