Content
68%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 is highly actionable with concrete, executable Membrane CLI commands and a clear connection-state workflow, but it wastes tokens explaining Knack basics and omits validation safeguards for destructive record operations. A couple of trims and a pre-delete verification step would meaningfully raise quality.
Suggestions
Add an explicit validation/confirmation checkpoint before running destructive actions (e.g. verify the record ID and confirm with the user before delete-record, and check the output for success after update/create).
Remove the introductory paragraph explaining what Knack is and the vague 'When to use which actions' line; assume Claude's competence and keep only integration-specific guidance.
Fix the dangling 'skip to Step 2' reference — there is no labeled Step 2 — by renumbering the connecting/searching sections into an explicit step sequence.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is efficient CLI commands, but it opens by explaining what Knack is ('Knack is a no-code platform that allows users to build...') and includes vague filler ('When to use actions: Use action names and parameters as needed') that could be trimmed. | 3 / 5 |
Actionability | It provides copy-paste-ready, executable commands for every stage (npm install, membrane login, connection ensure, action list/run, request) plus a concrete popular-actions table and a proxy flag reference, covering the common cases fully. | 5 / 5 |
Workflow Clarity | The install→authenticate→connect→search→run sequence is present with state-based checkpoints, but destructive/batch actions (delete-record, update-record, list-records) are run with no validation or confirmation step, which caps workflow clarity at 3 per the destructive-operations rule. | 3 / 5 |
Progressive Disclosure | The single SKILL.md is well organized into clearly headed sections with no nested references, and for a CLI integration keeping it in one file is appropriate; the only gap is that the action/proxy tables could optionally live in reference files. | 4 / 5 |
Total | 15 / 20 Passed |