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 content is a solid, command-rich guide to driving Mattermost via the Membrane CLI with a genuine connection-state feedback loop, but it pads in a concept-explaining intro, mixes invocation styles, and lacks validation guidance around destructive proxy/action calls. Structure is clean for a single-file skill.
Suggestions
Delete the 'Mattermost is an open-source, self-hosted team collaboration platform similar to Slack...' intro paragraph; Claude already knows what Mattermost is.
Add a verification/feedback step after destructive operations, e.g. after `membrane request ... -X DELETE` re-fetch the resource to confirm it is gone, to lift workflow clarity above the destructive-ops cap.
Resolve the dangling 'Step 2' reference by numbering the workflow explicitly (Step 1 install/auth, Step 2 connect, Step 3 discover/run) and standardize on either `membrane` or `npx @membranehq/cli` consistently.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly concrete CLI commands, but opens with an encyclopedia-style paragraph explaining what Mattermost is ('Mattermost is an open-source, self-hosted team collaboration platform similar to Slack...') that Claude already knows, plus minor marketing prose, fitting 'mostly efficient but includes some unnecessary explanation' rather than the lean anchor at 4. | 3 / 5 |
Actionability | It provides many concrete, executable commands (install, login, connection ensure, action list/run, request) plus a proxy flag table, but mixes `membrane` and `npx @membranehq/cli` without explanation and gives no worked end-to-end Mattermost example, fitting 'mostly executable with minor gaps' over the fully-copy-paste anchor at 5. | 4 / 5 |
Workflow Clarity | A real polling feedback loop exists for connection state (BUILDING/READY/CLIENT_ACTION_REQUIRED/CONFIGURATION_ERROR), but the destructive proxy (DELETE/PUT) and action-run steps have no verification guidance, and a dangling 'Step 2' reference has no matching numbered step, so the destructive-operations-without-validation cap and sequence gaps hold this at 3 rather than 4. | 3 / 5 |
Progressive Disclosure | The single-file content is well organized into clear sections (Overview, Authentication, Connecting, Searching, Running actions, Proxy, Best practices) with no nested references, and for a self-contained CLI integration this is appropriately placed; it stops short of 5 only because some reference-grade detail (e.g. the proxy flag table, connection-state catalog) is inlined rather than split out. | 4 / 5 |
Total | 14 / 20 Passed |