Content
67%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 strong, executable Membrane CLI guidance with a well-handled connection-state workflow, including a genuine feedback loop for CLIENT_ACTION_REQUIRED states. Its weaknesses are padding (an empty overview list, a duplicated section) and broken cross-references ('Step 2' is never defined). Tightening these would move it from good to excellent.
Suggestions
Remove the 'Centrifuge Overview' bullet list and the 'Popular actions' section that only repeats the discovery command — either list actual popular actions (query pool state, list transactions) or delete the section.
Fix the workflow numbering: define numbered steps (1. connect, 2. discover/run) so the 'skip to Step 2' reference and the '1b' subsection resolve, and add a short error-handling note for failed action runs (checking the output/error fields).
Clarify 'membrane login --tenant --clientName=<agentType>' — specify what value --tenant expects or drop it if none is needed.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient and command-focused, but contains unnecessary padding: the 'Centrifuge Overview' bullet list (Pool / Pool State / Transaction / Invoice plus 'Use action names and parameters as needed') conveys almost nothing, the opening paragraph explains what the platform is, and the 'Popular actions' section repeats the discovery command already given verbatim. This matches 'Mostly efficient but includes some unnecessary explanation or could be tightened' rather than anchor 4's minor-instances bar. | 3 / 5 |
Actionability | Concrete, copy-paste-ready commands cover the common cases — install, login (including headless code completion), connection ensure with polling, action list/run with --input JSON, and a full proxy flag table. It falls short of anchor 5 because the 'Popular actions' section promises actions but lists none (just a repeated discovery command) and 'membrane login --tenant --clientName=<agentType>' leaves --tenant's expected value unexplained. | 4 / 5 |
Workflow Clarity | The install → authenticate → connect → poll-until-ready → discover → run sequence is clear, and the state-branching guidance (READY / BUILDING / CLIENT_ACTION_REQUIRED / CONFIGURATION_ERROR) with re-polling forms an explicit feedback loop for connection setup. Minor gaps keep it below anchor 5: 'skip to Step 2' references a step that is never labeled (sections are '1b' and unnumbered), and there is no validation or error-handling guidance for action run results. | 4 / 5 |
Progressive Disclosure | No bundle files exist and all content is inline; at this size nearly everything present arguably belongs in SKILL.md, and the sections are clearly headed and easy to navigate. Organization gaps — the duplicated action-list instruction across 'Searching for actions' and 'Popular actions', and the inconsistent step numbering — match anchor 4 ('minor organization gaps') rather than 5's cleanly split, well-signaled structure. | 4 / 5 |
Total | 15 / 20 Passed |