Content
75%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 skill body is highly actionable with copy-paste Membrane CLI commands and a state-driven connection workflow with feedback loops. Its main weakness is conciseness: a marketing-style intro and a few explanatory asides add tokens Claude does not need.
Suggestions
Cut the opening "EdgeDB is a graph-relational database..." paragraph and the "so you can focus on the integration logic rather than auth plumbing" aside; assume Claude knows what EdgeDB and auth plumbing are.
Label the section referenced as "Step 2" (e.g. rename "Searching for actions" / "Running actions" to "Step 2: Search and run actions") so the "skip to Step 2" pointer resolves.
Consider moving the proxy flags table and the full clientAction state breakdown into a reference file to keep SKILL.md as an overview, since the skill is over 50 lines.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient concrete commands, but the opening paragraph ("graph-relational database designed as a successor to SQL... those who find traditional ORMs cumbersome") and asides like "so you can focus on the integration logic rather than auth plumbing" explain concepts Claude already knows. | 3 / 5 |
Actionability | Commands are fully executable and copy-paste ready throughout — `membrane connection ensure`, `npx @membranehq/cli connection get <id> --wait --json`, `membrane action run ... --input '{...}'` — with a flag table covering the proxy use case. | 5 / 5 |
Workflow Clarity | The connection workflow has a clear state-based sequence with checkpoints (READY / BUILDING / CLIENT_ACTION_REQUIRED / CONFIGURATION_ERROR) and a poll-again feedback loop, but "skip to Step 2" references a header that is not labeled, leaving a navigation gap. | 4 / 5 |
Progressive Disclosure | Content is well-sectioned (Overview, Authentication, Connecting, Searching, Running, Proxy, Best practices) with no nested references, and no bundle files exist to verify; the inlined proxy flag table and connection-state detail could arguably live in separate files. | 4 / 5 |
Total | 16 / 20 Passed |