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 solid, command-driven integration guide: real CLI invocations, connection state-machine handling, and a proxy escape hatch. Its weaknesses are a destructive/batch action set with no validation guidance, a broken step numbering ('Step 2' is never defined), and a monolithic single-file layout where the action catalog should be a separate reference.
Suggestions
Add validation/verification guidance for destructive and batch operations — e.g. confirm a customer's UUID via `get-customer` before `delete-customer`, and state that deletes are irreversible.
Fix the workflow numbering: the text says 'skip to Step 2' and has a '#### 1b' heading, but no Step 2 exists — label the search/run sections as the numbered steps.
Move the 20-row Popular actions table into a references file (e.g. references/actions.md) and keep a handful of the most common actions inline, and trim the marketing-style intro about what Chartmogul the company does.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient command-first sections, but padded spots remain: 'Chartmogul is a subscription analytics platform. It helps SaaS businesses track and analyze key metrics...' explains the vendor's business rather than the task, the bare object-tree Overview ('Customers / Subscriptions / Invoices...') adds little, and editorial asides like 'This is the fastest way to get a connection' and 'This will burn less tokens' could be trimmed. Matches 'Mostly efficient but includes some unnecessary explanation or could be tightened'. | 3 / 5 |
Actionability | Concrete, executable CLI commands throughout: install (`npm install -g @membranehq/cli@latest`), login including the headless completion flow, `membrane connection ensure`, polling with `--wait`, `action run` with a real `--input` JSON example, and a proxy flag table. Not a 5 because the search example uses an unresolved placeholder (`--intent "QUERY"`) with no filled-in real intent example, and there is no complete end-to-end run (search → run with actual parameters → read output) tying the pieces together. | 4 / 5 |
Workflow Clarity | The install → login → connect → poll → search → run sequence is present with explicit state handling (READY, BUILDING, CLIENT_ACTION_REQUIRED, CONFIGURATION_ERROR) and poll-again guidance, which is good. However the body references 'skip to Step 2' and a '#### 1b' heading with no Step 2 ever labeled, and — decisive for the cap — destructive operations (delete-customer, delete-plan) and batch list operations ship with no validation or verification guidance, which per the rubric caps workflow_clarity at 3. It is not a 2 because the sequence and error-state recovery are substantially better defined than a rough outline. | 3 / 5 |
Progressive Disclosure | There is no bundle at all (no references/, scripts/, or assets/), so everything lives inline in a ~160-line SKILL.md. Section headers and tables give it reasonable navigation, but content that belongs in a separate reference — the 20-row Popular actions table and the proxy flag options — is inlined, matching 'Some structure but could be better organized... content that should be separate is inline'. It is not a 2 because the structure is genuinely present and navigable, not a wall of text. | 3 / 5 |
Total | 13 / 20 Passed |