Content
38%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 Membrane CLI operating sections (auth, connection lifecycle, action discovery/run, proxy, best practices) are genuinely useful and executable, but they are buried under a massively duplicated 400-line keyword bullet list that wastes most of the skill's tokens. Destructive actions ship with no validation guidance, and nothing is offloaded to reference files.
Suggestions
Delete the 'Appcues Overview' bullet list entirely (or replace it with a ~10-line list of the actual API objects: flows, flow steps, users, events, accounts, segments, tags, themes, checklists), recovering ~75% of the skill's token budget.
Add validation guidance for destructive actions (delete-user-profile, delete-segment), e.g., confirm the target ID with the user and verify with get-user-profile / list-segments before deleting, and fix the dangling 'Step 2' reference by labeling the numbered workflow steps.
Provide one complete worked example with real values, e.g. `membrane action run update-user-profile --connectionId=<id> --input '{"userId": "u123", "properties": {"plan": "pro"}}' --json`, and move the full action catalog / proxy details to a references/ file if more coverage is needed.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 'Appcues Overview' section is roughly 400 lines of bullet items (GDPR, HIPAA, NFTs, Metaverse, Braille, DevOps, ...), with the entire list duplicated in reverse order after line ~205 — pure padding unrelated to the skill's instructions, dominating the ~540-line body. This matches anchor 1 ('heavily padded'); it is not a 2 because the padding is not a few unnecessary sections but the majority of the document. | 1 / 5 |
Actionability | Concrete, executable commands throughout: `npm install -g @membranehq/cli@latest`, `membrane login --tenant --clientName=<agentType>`, `membrane connection ensure "https://www.appcues.com/" --json`, `membrane action run <actionId> --connectionId=... --input '{"key": "value"}'`, plus a proxy flag table — matching anchor 4 (mostly executable with minor gaps). Not 5 because every example carries unresolved placeholders (CONNECTION_ID, QUERY, <actionId>) and there is no complete end-to-end worked example with real values. | 4 / 5 |
Workflow Clarity | The install → login → connection ensure → poll state → search actions → run sequence is present, and the connection state machine (READY / CLIENT_ACTION_REQUIRED / CONFIGURATION_ERROR) is a genuine feedback loop. However, the skill includes destructive operations ('Delete User Profile ... permanently, including all targeting properties', 'Delete Segment') with no validation or confirmation guidance, which caps workflow clarity at 3; the dangling 'skip to **Step 2**' reference (no Step 2 is labeled) also blurs the sequence. | 3 / 5 |
Progressive Disclosure | There are no bundle files at all (no references/, scripts/, or assets/), and a ~400-line keyword catalog — content that clearly belongs in a separate reference file or nowhere — is inlined in SKILL.md, matching anchor 2 ('content that clearly belongs in separate files is inlined'). Not 3 because beyond the one dead-weight list there is no reference mechanism or larger structure to organize. | 2 / 5 |
Total | 10 / 20 Passed |