Content
50%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 content is highly actionable with concrete Membrane CLI commands and a clear connection state machine, but it is dragged down by a bloated inline entity list and a lack of validation feedback loops for action execution. There are no external reference files despite content that would benefit from splitting.
Suggestions
Move the ~110-line entity enumeration into a references/ file (e.g. ENTITIES.md) and summarize it in SKILL.md with a one-level-deep link, cutting the bulk of the token cost.
Add a validation step after running actions, e.g. re-query or inspect the output field to confirm the operation succeeded before reporting completion to the user.
Prune generic entities from the overview (User, Address, Note, Tag, File, Image, Video, Audio, Link, Comment) that do not aid action discovery, or fold line-item sub-entities into their parents.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~110-line 'Tradogram Overview' entity enumeration (Purchase Order through Subscription, including generic noise like User, Address, Note, Tag, File, Image, Audio) is a noticeably padded section that inflates the token budget, fitting 'noticeably verbose; several unnecessary explanations or padded sections' better than the mostly-efficient anchor at 3. | 2 / 5 |
Actionability | Concrete, copy-paste-ready commands are provided throughout (npm install, membrane login, connection ensure, action list/run, request proxy) with a flags table, fitting 'mostly executable guidance; concrete code or commands with minor gaps'; it stops short of 5 only because identifiers are placeholders (CONNECTION_ID, actionId) and a worked end-to-end example is absent. | 4 / 5 |
Workflow Clarity | A reasonable sequence exists (install -> authenticate -> connect -> poll for READY -> search -> run/proxy) with good state-checkpointing on connection state, but running actions performs potentially batch/destructive data operations with no post-run verification step, so per the destructive/batch cap it cannot exceed the 'steps listed but validation gaps' anchor at 3. | 3 / 5 |
Progressive Disclosure | Sections are organized, but the large entity list is inlined in SKILL.md rather than split into a one-level-deep reference file, and no external references exist, matching 'some structure but could be better organized; content that should be separate is inline' rather than the well-split anchor at 4. | 3 / 5 |
Total | 12 / 20 Passed |