Content
57%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 concrete, largely copy-paste-ready CLI commands with a sensible overall flow and a genuinely useful connection-state handling loop. Its weaknesses are padding (the Revit explainer paragraph), a workflow whose step references point to unlabeled steps, and no use of reference files to separate detail (proxy options, clientAction schema) from the core path.
Suggestions
Delete the 'Autodesk Revit is a BIM software...' overview paragraph and the 'Use action names and parameters as needed.' filler — Claude already knows what Revit is.
Number the workflow steps explicitly (1. Install, 2. Authenticate, 3. Connect, 4. Search actions, 5. Run) so the existing 'skip to Step 2' reference resolves, and add a brief error-handling/validation note for action runs (e.g. inspect the output/error fields and retry guidance).
Move the proxy request options table and the clientAction field-by-field documentation into a references/ file (e.g. REFERENCES.md) and link to it from the main flow, keeping SKILL.md as a lean overview.
Consolidate the duplicated action-discovery guidance in 'Searching for actions' and 'Popular actions' into a single section with one realistic filled-in example (actual connection id, intent, and a sample response).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The opening paragraph ('Autodesk Revit is a BIM (Building Information Modeling) software... Revit helps streamline workflows from concept to construction') explains a concept Claude already knows, and filler like 'Use action names and parameters as needed.' adds nothing. Otherwise the body is mostly efficient commands, matching the anchor 'Mostly efficient but includes some unnecessary explanation or could be tightened' — not level 2 since only one section is clearly padded, and not level 4 because the overview paragraph and filler lines should be trimmed. | 3 / 5 |
Actionability | Commands are concrete and executable throughout: 'npm install -g @membranehq/cli@latest', 'membrane connection ensure "https://www.autodesk.com/products/revit/overview" --json', 'membrane action run <actionId> --connectionId=... --input {...}'. This matches 'Mostly executable guidance; concrete code or commands with minor gaps' — placeholders (CONNECTION_ID, QUERY, actionId) are never shown with realistic values and there is no example of parsing a response, which keeps it below level 5. | 4 / 5 |
Workflow Clarity | The install → login → connection → search → run sequence is present, and the connection state machine (READY / BUILDING / CLIENT_ACTION_REQUIRED / CONFIGURATION_ERROR) gives explicit checkpoints with a poll-again feedback loop. However, 'skip to **Step 2**' references a step that is never labeled (no steps are numbered), and there is no validation or error-handling guidance after running an action. This matches 'Steps listed but validation gaps; sequence present but checkpoints missing or implicit' rather than level 4, whose clear navigation is broken by the dangling step references. | 3 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are all absent) and nothing is offloaded: ~140 lines are fully inline, including the proxy flag options table and the detailed clientAction state-machine documentation that belong in a reference file. Section headers provide decent structure, so this is above level 2, but it matches 'Some structure but could be better organized... content that should be separate is inline' — the 'Popular actions' heading even duplicates the earlier 'Searching for actions' guidance. | 3 / 5 |
Total | 13 / 20 Passed |