Content
57%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 largely executable guide to the Membrane CLI workflow for Amadeus, with strong command coverage (install, auth, connection lifecycle, action discovery/execution, proxy fallback) and useful error-state handling. Its weaknesses are structural: a dangling 'Step 2' reference with no numbered steps, a duplicated/empty 'Popular actions' section, and buzzword filler (company description, generic action-name list) that spends tokens without adding guidance. Splitting connection-state and proxy detail into reference files would tighten the main file considerably.
Suggestions
Fix the broken step numbering: either add explicit '### Step 1' / '### Step 2' headings so 'skip to **Step 2**' resolves, or rewrite the references to point at the actual section names ('Searching for actions').
Remove the duplicate 'Popular actions' section (or populate it with 3-5 concrete action examples) and delete the company-description paragraph and 'Use action names and parameters as needed' filler line.
Move the clientAction state semantics and the proxy flags table into a references/ file, keeping SKILL.md as a lean overview with one-line links.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — CLI commands, a flags table, and connection-state handling — but padded with unnecessary material: a corporate-description opener ('Amadeus is a technology company providing IT solutions...'), a bare feature-name bullet list ending in the filler line 'Use action names and parameters as needed', and the same 'action list' command duplicated under both 'Searching for actions' and 'Popular actions'. Fits 'Mostly efficient but includes some unnecessary explanation or could be tightened'; not 4 because the redundancy and boilerplate are more than minor. | 3 / 5 |
Actionability | Concrete, copy-paste-ready commands throughout: 'npm install -g @membranehq/cli@latest', 'membrane connection ensure "https://amadeus.com" --json', 'membrane action run <actionId> --connectionId=... --input', plus a complete proxy-options table and headless login completion flow. Minor gaps keep it at anchor 4 rather than 5: the 'Popular actions' section contains no actual example actions or intents. | 4 / 5 |
Workflow Clarity | The install → login → connect → poll → act → proxy sequence is present with real error-handling guidance (clientAction handling, CONFIGURATION_ERROR → check 'error' field, re-poll loop). But the flow says 'skip to **Step 2**' when no Step 2 (or Step 1) heading exists — only an orphaned '#### 1b' — leaving the sequence with genuine gaps. Steps listed with sequence gaps → anchor 3, not 4. | 3 / 5 |
Progressive Disclosure | No bundle files exist, and the ~160-line body has reasonable section headers but inlines content that belongs in reference files (the clientAction state semantics, the proxy options table), plus a duplicated section and an empty 'Popular actions' header. Being well over the 50-line simple-skill threshold with these organization issues fits 'Some structure but could be better organized' — anchor 3 rather than 4. | 3 / 5 |
Total | 13 / 20 Passed |