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 copy-paste-ready Membrane CLI commands with a mostly clear connection-to-action workflow and useful state-based validation guidance. Its weaknesses are padded explanatory content (the CRM intro paragraph, an empty 'Popular actions' section) and a dangling 'Step 2' reference that breaks the otherwise good step sequencing.
Suggestions
Cut the introductory paragraph explaining what AmoCRM/CRM is and delete the 'Popular actions' section (or populate it with actual action names) — it currently just repeats the action-discovery command.
Replace the dangling 'skip to Step 2' and '1b' references with an explicit numbered sequence (Step 1: connect, Step 2: discover actions, Step 3: run actions) so the workflow reads as one coherent process.
Add one complete proxy request example with a real AmoCRM endpoint (e.g. /api/v4/leads) alongside the flag table.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The opening paragraph explains what AmoCRM is ('customer relationship management platform geared towards sales teams... helps businesses track leads') — knowledge Claude already has — and 'Use action names and parameters as needed.' is a filler line. The 'Popular actions' section duplicates the earlier action-discovery command without listing any actions. Mostly efficient with some unnecessary explanation matches anchor 3 rather than anchor 4's 'minor instances'. | 3 / 5 |
Actionability | Concrete, executable commands throughout: 'npm install -g @membranehq/cli@latest', 'membrane connection ensure "https://www.amocrm.com/" --json', 'membrane action run <actionId> --input ... --json', plus a complete proxy flag table. Minor gaps keep it below anchor 5: the proxy section shows flags but no full example with a real endpoint, and the 'Popular actions' heading promises actions but delivers only a discovery command. | 4 / 5 |
Workflow Clarity | The sequence is followable (install CLI, login including headless variant, connection ensure, poll with --wait, handle clientAction states, search and run actions) with validation via state polling and an explicit error branch for CONFIGURATION_ERROR. However 'skip to **Step 2**' dangles — no Step 1/Step 2 headings exist, only an unnumbered '1b' — a minor gap that fits anchor 4 rather than anchor 5. | 4 / 5 |
Progressive Disclosure | A single SKILL.md with well-organized, clearly headed sections and no nested references, so navigation is easy. At roughly 140 lines it exceeds the under-50-line simple-skill exception, and content like the proxy flag table and clientAction schema detail could arguably live in a reference file, which fits anchor 4's 'minor organization gaps' rather than anchor 5. | 4 / 5 |
Total | 15 / 20 Passed |