Content
61%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.
This is a solid, command-driven integration skill: it gives executable Membrane CLI instructions for auth, connection setup with a real state-polling feedback loop, action discovery, and raw proxy fallback. Its weaknesses are a padded auto-generated actions table and marketing boilerplate that waste tokens, broken "Step 2" cross-references, and no validation guidance for destructive actions like delete-event, which caps workflow clarity.
Suggestions
Cut the 18-row actions table where every row says "No description" (keep only a handful of representative actions), and remove the marketing intro and bare-noun Overview — replace with the concrete action discovery step the reader should run.
Add a validation step before destructive actions, e.g. "Before delete-event/delete-calendar, run get-event to confirm the target ID and show the user what will be deleted".
Fix the two "skip to **Step 2**" references by either labeling the "Searching for actions" section as Step 2 or rewording to "skip to Searching for actions"; also standardize the CLI invocation (`membrane` vs `npx @membranehq/cli`) used in the polling command.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The command-centric sections are lean, but there is real padding: a marketing-style intro ("Addevent is a platform that simplifies event scheduling and ticketing. It's used by event organizers, businesses, and individuals..."), an Overview list of bare nouns ("Event", "Ticket", "Order", "Attendee", "Account") that conveys almost nothing, and an 18-row "Popular actions" table where every Description cell literally reads "No description". This is 'mostly efficient but includes some unnecessary content that could be tightened' — not a 2, since most of the body is executable commands rather than explanation of concepts Claude already knows. | 3 / 5 |
Actionability | The body gives copy-paste-ready commands for the full lifecycle: `npm install -g @membranehq/cli@latest`, `membrane login --tenant --clientName=<agentType>`, `membrane connection ensure "https://www.addevent.com/" --json`, `membrane action list ... --intent "QUERY"`, `membrane action run <actionId> --input '{"key": "value"}'`, plus a proxy flag table. It stops short of anchor 5 because no Addevent-specific worked example is shown (every invocation uses placeholders like `<actionId>` and `'{"key": "value"}'`), and the CLI is invoked inconsistently (`membrane connection get` vs `npx @membranehq/cli connection get`), leaving minor gaps the reader must resolve. | 4 / 5 |
Workflow Clarity | The connect-and-wait flow has a genuine validation feedback loop — poll `connection get --wait`, branch on `READY` / `CLIENT_ACTION_REQUIRED` (with `clientAction.agentInstructions`) / `CONFIGURATION_ERROR` — but the sequence is undermined by two "skip to **Step 2**" references with no Step 2 heading anywhere, and there is zero verification guidance for the destructive operations the skill prominently lists (delete-event, delete-calendar, delete-calendar-subscriber, delete-rsvp-attendee). Per the rubric's cap, a workflow covering destructive/batch actions without validation steps cannot score above 3, and this sits exactly at anchor 3: sequence present, validation gaps elsewhere. | 3 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent) and the skill is a single ~160-line file with well-organized sections (Install, Authentication, Connecting, Searching, Running actions, Proxy requests, Best practices) and no nested or dangling file references. That matches 'good structure; most content appropriately placed; minor organization gaps' — the main gap is that the auto-generated action catalog table and proxy options could arguably live in a separate reference file, which keeps it from a 5. | 4 / 5 |
Total | 14 / 20 Passed |