Content
48%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 Membrane CLI half of this body is a solid, executable guide with a genuinely well-handled connection state machine. It is dragged down by a ~180-line meaningless keyword dump that dominates the token budget, plus small coherence defects (undefined "Step 2" references, an empty "Popular actions" section, inconsistent CLI invocation).
Suggestions
Delete the "Amplication Overview" bullet list (lines 23-202) entirely — it is duplicated verb keywords with no informational value and consumes over half the skill's tokens; if any of it matters, it belongs in a single short sentence about what Amplication resources exist.
Fix the broken step references: define Step 1/Step 2 (or renumber) so "skip to Step 2" and the orphaned "#### 1b" resolve, and either populate "Popular actions" with real examples or remove the section.
Standardize on one CLI invocation form (e.g. always "membrane ..." after the install step) so commands are consistently copy-paste ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Roughly 180 of the body's ~330 lines are a bare-verb bullet dump ("Scale", "Backup", "Monitor", "Automate"...) with many verbatim duplicates (Monitor appears 4 times, Alert 5 times, Build/Test/Deploy repeat), capped by the filler line "Use action names and parameters as needed." — over half the file is pure padding that earns no tokens. | 1 / 5 |
Actionability | The CLI instructions are concrete and executable: install, login (including headless completion), connection ensure, polling with --wait, action list/run with JSON --input examples, and a complete proxy flag table. Minor gaps: inconsistent invocation ("membrane" vs "npx @membranehq/cli" vs "npx @membranehq/cli@latest") and a "Step 2" that is referenced but never defined keep it below fully copy-paste-ready. | 4 / 5 |
Workflow Clarity | The connection flow has an explicit feedback loop — ensure, poll with --wait, branch on READY / CLIENT_ACTION_REQUIRED (with clientAction sub-fields explained) / CONFIGURATION_ERROR, re-poll after the user acts — but the step numbering is incoherent ("skip to Step 2" with no Step 2, a "1b" with no "1a") and the "Popular actions" section is an empty header, which are minor validation/organization gaps rather than missing checkpoints. | 4 / 5 |
Progressive Disclosure | No bundle files exist, so all content lives inline in a single ~330-line file. Sections are headed (not monolithic), but the giant overview keyword list is inline content that should be cut or split out, and there are no references to signal — matching the anchor for having some structure while content that should be separate stays inline. | 3 / 5 |
Total | 12 / 20 Passed |