Content
76%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.
A highly actionable, well-structured CLI integration guide whose commands are uniformly copy-paste ready. Its main weaknesses are workflow polish — a dangling 'Step 2' reference, inconsistent CLI invocation, and no validation guidance for the destructive actions it exposes — and a single-file layout with a sizable inline action reference rather than progressive disclosure.
Suggestions
Add validation/verification steps for destructive actions (Cancel Appointment, Delete Client, Delete Block): e.g., run get-appointment/get-client to confirm the target, then confirm with the user before running delete/cancel, and verify the result afterward.
Fix the broken cross-references and inconsistent commands: either label the next section 'Step 2' or reword to 'skip to Searching for actions', and use the same invocation form for `connection get` (`membrane` vs `npx @membranehq/cli`).
Consider moving the Popular actions table (and the detailed connection-state semantics) into a references/ file so SKILL.md stays a lean overview with one-level-deep, clearly signaled pointers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by executable commands and compact tables, with only minor trimmable filler ("Use action names and parameters as needed.", the intro sentence about salons/therapists/consultants, and the awkward "Agent Types" aside). This fits 'efficient; minor instances of over-explanation that could be trimmed' better than score 3, since the unnecessary material is only a few short lines across ~180 lines. | 4 / 5 |
Actionability | Every stage has copy-paste-ready commands: `npm install -g @membranehq/cli@latest`, `membrane login complete <code>`, `membrane connection ensure "https://www.acuityscheduling.com/" --json`, `membrane action list --connectionId=CONNECTION_ID --intent "QUERY" --limit 10 --json`, and `membrane action run <actionId> --input '{"key": "value"}' --json`, plus a 20-row popular-actions table and a proxy fallback with a full flag table. Fully executable guidance covering the common cases — matches the anchor 5 example; nothing significant is missing for score 4. | 5 / 5 |
Workflow Clarity | The connection workflow has a clear sequence with genuine feedback loops (poll `--wait` while BUILDING, handle `clientAction`, re-poll after user auth, check `error` on CONFIGURATION_ERROR/SETUP_FAILED). However, "If the returned connection has `state: "READY"`, skip to **Step 2**" references a step label that doesn't exist, the heading "1b" belongs to a numbering scheme never established, and `membrane connection get` vs `npx @membranehq/cli connection get` are used inconsistently. Most importantly, destructive operations in the actions table (Cancel Appointment, Delete Client, Delete Block) have no validation/verification steps, which per the rubric caps this dimension at 3. | 3 / 5 |
Progressive Disclosure | This is a single-file skill with no bundle files present; sections are well-organized (Install, Authentication, Connecting, Searching, Popular actions, Running, Proxy, Best practices) and easy to navigate. The ~20-row Popular actions table and the detailed connection-state handling are inline reference material that could move to a separate file, which keeps it below anchor 5, but placement is defensible for a CLI skill of this size — matching 'good structure; most content is appropriately placed; minor organization gaps' (4) rather than 'content that should be separate is inline' (3). | 4 / 5 |
Total | 16 / 20 Passed |