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 a concrete, executable Membrane CLI workflow with genuine validation feedback loops for connection setup. Its main weaknesses are prose padding in the intro and a serious internal inconsistency conflating the payroll product RollHQ with the tabletop product Roll20.
Suggestions
Trim the intro background paragraph and 'Think of it as a modern payroll solution' prose; Claude does not need Roll explained and the commands carry the content.
Resolve the Roll20/RollHQ confusion: the 'Roll Overview' lists Dice/Rolls and cites roll20.zendesk.com while the rest of the skill targets the payroll product at rollhq.com — pick one product and make the overview, docs link, and examples consistent.
Merge the redundant 'Popular actions' section into 'Searching for actions' and give the workflow explicit top-level numbered steps (1, 2, 3) instead of '1b' and an undefined 'Step 2'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient commands, but the intro paragraph over-explains what Roll is ('Think of it as a modern payroll solution...'), and several prose asides ('so you can focus on the integration logic', 'This is the fastest way to get a connection') pad beyond what Claude needs, fitting 'mostly efficient but includes some unnecessary explanation' at 3. | 3 / 5 |
Actionability | It provides concrete, executable commands throughout (install, login, connection ensure, action list/run with --input JSON, request with a full flag table), covering the common cases with only minor gaps like unresolved placeholders (CONNECTION_ID, actionId) and the vague 'Use action names and parameters as needed', placing it at 4 rather than fully-copy-paste at 5. | 4 / 5 |
Workflow Clarity | The connection flow has explicit validation checkpoints and a feedback loop (poll until not BUILDING, branch on READY/CLIENT_ACTION_REQUIRED/CONFIGURATION_ERROR, re-poll after user action), but step numbering is loose ('1b', references to an undefined 'Step 2') and the roll20/rollhq inconsistency muddies the path, so it sits at 4 rather than the checklist-grade anchor at 5. | 4 / 5 |
Progressive Disclosure | The skill is a single well-organized file with clear section headers and no nested references, and the CLI-guide content is appropriately kept inline; minor organization gaps (the contradictory Dice/Rolls overview, and 'Popular actions' duplicating 'Searching for actions') keep it at 4 rather than 5. | 4 / 5 |
Total | 15 / 20 Passed |