Content
65%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 tight, highly actionable guide to driving Chaport through the Membrane CLI, with executable commands and a well-documented connection state machine. Its main gaps are the missing validation guidance around destructive actions (delete-visitor/delete-operator), a broken "Step 2" cross-reference, and no progressive disclosure — the entire action catalog and CLI reference live inline in SKILL.md.
Suggestions
Add validation/confirmation steps before destructive actions — e.g., verify the target with `get-visitor` and confirm with the user before running `delete-visitor`, and state this in a short "Destructive operations" note.
Fix the dangling cross-reference: "skip to **Step 2**" points to a step label that doesn't exist; either label the sections as Steps 1/2 consistently or reference the section by name.
Split the 19-row action catalog and the proxy flags table into a references/ file (e.g., references/actions.md), keeping a handful of the most common actions inline in SKILL.md as a quick-start pointer.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every token is executable guidance Claude cannot know (Membrane CLI commands, connection state machine, action catalog, proxy flags). Only minor instances are trimmable — the bare "Chaport Overview" entity list, "This is the fastest way to get a connection…", and the rambling "Agent Types… will be used to adjust tooling" sentence. It is not 5 because those few sentences and the padding in "Best practices" ("This will burn less tokens") do not fully earn their place. | 4 / 5 |
Actionability | Concrete, copy-paste-ready commands cover install, login (including headless completion), connection lifecycle, action search/run with --input, and proxied API requests with a flags table. It is not 5 because no worked example ties an action key from the table (e.g., list-visitors or send-message) to a fully populated `membrane action run` invocation; placeholders like <actionId> and QUERY are never instantiated. | 4 / 5 |
Workflow Clarity | The install → login → connection ensure → poll → search → run sequence is clear, with state-based feedback loops (CLIENT_ACTION_REQUIRED → user acts → poll again). However, destructive operations (delete-visitor, delete-operator) ship with no validation or confirmation steps, which caps workflow clarity at 3 per the rubric; additionally, "skip to **Step 2**" references a step label that does not exist anywhere in the document. | 3 / 5 |
Progressive Disclosure | No bundle files exist, so everything — the 19-row action catalog, the proxy flag table, and the clientAction state documentation — is inlined in a single ~160-line SKILL.md. Structure is decent (clear section headers), but content that belongs in a references/ file (the full action catalog) is inline and the dangling "Step 2" cross-reference hurts navigation. It is not 2 because organization is clearly present; not 4 because reference material is not split out at all. | 3 / 5 |
Total | 14 / 20 Passed |