Content
38%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 usage section is executable and reasonably well-structured, but it is buried under an enormous, auto-generated entity list that is mostly noise and should not live in SKILL.md. The skill also lacks verification steps for destructive data operations.
Suggestions
Remove the ~370-line entity list from SKILL.md (or move genuine entities to a references/ file); the duplicated 'Workflow...View View View' entries are pure padding.
Add a validate/verify checkpoint after running actions or proxy mutations, e.g. re-query the record to confirm the change before reporting success.
Replace 'Use action names and parameters as needed.' with one or two concrete Rock RMS action examples showing real input JSON and expected output.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by a ~370-line entity list riddled with nonsensical duplicated entries (e.g. "Workflow Workflow Trigger Workflow Trigger... View View View View View"), which is severely padded noise; the intro also explains what Rock RMS is. This matches the 'severely verbose; heavily padded' anchor. | 1 / 5 |
Actionability | The CLI section provides concrete, executable, copy-paste commands (install, login, connection ensure, action list/run, request) plus a flags table, matching the 'mostly executable guidance with minor gaps' anchor. It is not 5 because Rock RMS-specific action examples use placeholders and the entity list adds no actionable guidance. | 4 / 5 |
Workflow Clarity | The connection setup is well-sequenced with polling and state feedback loops, but the core data-operation workflow (action run and proxy requests that can perform PUT/PATCH/DELETE) lacks any validate-then-verify step. Per the destructive/batch feedback-loop cap, workflow clarity cannot exceed 3. | 3 / 5 |
Progressive Disclosure | No bundle files exist and the massive entity reference is inlined directly in SKILL.md rather than split into a separate reference file, with no file-based navigation. This matches the anchor for content that clearly belongs in separate files being inlined. | 2 / 5 |
Total | 10 / 20 Passed |