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.
The body delivers largely executable Membrane CLI guidance with a sensible connection-first workflow and clear error-state handling, making it a solidly actionable skill. Its main weaknesses are token waste in the 'No description' actions table, a dangling 'Step 2' reference with no matching heading, and the absence of any caution or validation around the destructive delete actions it advertises.
Suggestions
Fix the broken sequence labels: replace 'skip to Step 2' and '#### 1b' with an explicit numbered Step 1/Step 2 heading structure so the connection workflow reads coherently.
Remove or trim the 'Popular actions' table's empty 'No description' column (20 repeated no-value cells), and either populate descriptions or link the table's action Keys to the `action run` command's `<actionId>` with one worked example.
Add a validation checkpoint for destructive operations — e.g. instruct to run a get-credential or search action to confirm the target before delete-credential/delete-group, and confirm with the user before batch deletes.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Commands and explanations are mostly lean, but the 20-row 'Popular actions' table repeats 'No description' twenty times — pure token waste — and lines like 'Those will be used to adjust tooling to be used best with your harness' are rambling. This matches anchor 3 (mostly efficient but includes some unnecessary explanation or could be tightened) rather than 4, which requires only minor trimmable instances. | 3 / 5 |
Actionability | Copy-paste-ready bash commands cover install, login (including headless completion), connection setup with state polling, action search, action execution with JSON parameters, and proxy fallback, plus a flag table. It falls short of anchor 5 because the actions table lists 'Key' values while `membrane action run` takes an unspecified `<actionId>` with no mapping or worked example of a full command. | 4 / 5 |
Workflow Clarity | Install → authenticate → connect → search → run → proxy is sequenced, with explicit state handling (READY, BUILDING, CLIENT_ACTION_REQUIRED, CONFIGURATION_ERROR) and re-poll instructions. However, the text says 'skip to Step 2' but no Step 1/Step 2 headings exist (only '1b'), and destructive actions (delete-credential, delete-group, delete-evidence-item) are offered with no validation or caution — the destructive-operation cap holds workflow clarity at 3. | 3 / 5 |
Progressive Disclosure | The bundle is a single SKILL.md with clear, flat section headers (Overview, Install, Authentication, Connecting, Searching, Running, Proxy, Best practices), no nested references, and nothing buried. It matches anchor 4 (good structure, most content appropriately placed, minor organization gaps) rather than 5 because the 20-row actions table and entity overview are inline content that could live in a reference file at this length. | 4 / 5 |
Total | 14 / 20 Passed |