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 delivers strong operational content — executable Membrane CLI commands, a useful action catalog, and a well-handled connection-state feedback loop — undermined by corrupted artifacts: a stray LLM refusal paragraph, a dangling 'Step 2' reference, and a few vague filler lines. Cleaning those up would make this a highly usable skill.
Suggestions
Delete the pasted refusal paragraph ('I don't have enough information to do that. I'm a large language model...') — it is noise that actively confuses.
Fix the dangling navigation: 'skip to Step 2' has no defined Step 2, and the '#### 1b' heading implies a 'Step 1'/'Step 2' numbering scheme that doesn't exist. Introduce explicit numbered steps or reword the cross-references.
Replace the vague 'Use action names and parameters as needed' with a concrete worked example, e.g. running list-goals with a real --input payload and reading the output field, and add a brief validation note for create/update operations.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is efficient — terse commands, tables, and flag descriptions — but there is genuine noise: the pasted LLM refusal paragraph ('I don't have enough information to do that. I'm a large language model...'), the filler bullet 'Use action names and parameters as needed', and the loosely written 'Agent Types' line. This lands between anchor 4 (minor trimmable instances) and anchor 3 (some unnecessary content); the refusal paragraph is more than a trim, so anchor 3 is the better fit. | 3 / 5 |
Actionability | Guidance is concrete and mostly copy-paste ready: 'npm install -g @membranehq/cli@latest', 'membrane connection ensure "https://www.cascade.app/" --json', 'membrane action run <actionId> --connectionId=... --input {...}', plus a flags table and an actions table. Minor gaps keep it at anchor 4 rather than 5: no worked end-to-end example with real values, placeholder-only parameters, and an inconsistency between 'npx @membranehq/cli connection get' and 'membrane connection get'. | 4 / 5 |
Workflow Clarity | There is a clear sequence (install → login → connection ensure → wait/poll → search actions → run, with the proxy as fallback) and the connection-state section is a real feedback loop (poll --wait, handle CLIENT_ACTION_REQUIRED, check the error field on CONFIGURATION_ERROR/SETUP_FAILED). Anchor 4 fits: most checkpoints are present, but 'skip to Step 2' references a step that is never defined, the steps are only implicitly numbered (a '1b' exists with no 'Step 1'/'Step 2' headers), and create/update operations have no validation guidance. | 4 / 5 |
Progressive Disclosure | The body is well-sectioned (Overview, Install, Authentication, Connecting, Searching, Popular actions, Running, Proxy, Best practices) and easy to navigate in a single file with no bundle references — there are no nested or buried references to penalize. It sits at anchor 4 rather than 5 only because the 20-row popular-actions table is reference-style content that could be split into a separate file if the skill grows. | 4 / 5 |
Total | 15 / 20 Passed |