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.
Highly actionable CLI guidance with concrete commands and tables, but it carries a padded intro and a large inline catalog, has incoherent workflow numbering, and omits validation guidance around destructive operations.
Suggestions
Add a confirmation/validation checkpoint before destructive runs, e.g. 'Before delete-file/delete-user/delete-group, verify the target with a get-* call and confirm with the user'.
Fix the broken step numbering: replace '1b' with a proper '1a/1b' pair or renumber the connect → search → run sections as explicit numbered steps including the missing 'Step 2'.
Move the 45-item entity list and/or the full action catalog into a reference file (e.g. references/actions.md) and keep SKILL.md as a lean overview that links to it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The marketing intro ("Files.com is a secure file management and automation platform. It's used by businesses of all sizes...") and the ~45-item inline entity bullet list explain context Claude can infer and consume tokens that could be trimmed or offloaded. | 3 / 5 |
Actionability | Copy-paste-ready commands throughout — `npm install -g @membranehq/cli@latest`, `membrane connection ensure`, `membrane action list/run`, `membrane request` — plus a flags table and a 20-row action catalog covering the common cases. | 5 / 5 |
Workflow Clarity | A sequence exists (install → auth → connect → search → run) with polling checkpoints for connection state, but step numbering is incoherent (a referenced "Step 2" and a "1b" with no "1a"/"Step 2"), and destructive `delete-file`/`delete-user`/`delete-group` actions are listed with no confirmation/validation checkpoint, capping this at 3. | 3 / 5 |
Progressive Disclosure | Sections are well-organized, but at ~200 lines with a 45-item entity list and a 20-row action table all inlined and zero external references (no references/scripts/assets bundle), content that could be split into a reference file is not, so it falls short of the well-split anchor 4. | 3 / 5 |
Total | 14 / 20 Passed |