Content
46%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 actionable Membrane CLI guidance is strong and executable, but the body is crippled by a massive inlined list of irrelevant entities that destroys token efficiency and structure. Destructive actions also lack validation checkpoints.
Suggestions
Delete the entire 'CompanyCam Overview' entity list (lines 21-897); it is almost entirely unrelated to CompanyCam and wastes ~85% of the file's tokens.
If a real entity/data-model overview is needed, keep only CompanyCam-relevant entities (User, Company, Group, Project, Photo, Document, Tag) and move detail to a references file.
Add a validation/confirmation step before destructive actions (delete-project, delete-photo, archive-project), e.g. 'Confirm the project ID with the user before running delete-project'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 'CompanyCam Overview' is a ~870-line dump of entity names, the overwhelming majority irrelevant to CompanyCam (Greek gods, planets, stars, hundreds of 'Kepler-NNNb' exoplanets, foundations, legal concepts), making the body severely verbose and heavily padded. Not a 2 because the padding is extreme and pervasive, not just several unnecessary explanations. | 1 / 5 |
Actionability | The working sections provide fully executable, copy-paste-ready commands (npm install, membrane login, connection ensure, action list/run, request proxy) plus a concrete Popular actions table covering common cases. Not below 5 because the common cases are specifically covered with real commands. | 5 / 5 |
Workflow Clarity | The connection flow is sequenced with state checks (READY/BUILDING/CLIENT_ACTION_REQUIRED) and a poll loop, but destructive/batch actions (delete-project, delete-photo, archive-project) are listed with no validation or confirmation checkpoints. Per the rubric cap, missing validation for destructive operations caps this at 3. | 3 / 5 |
Progressive Disclosure | No bundle files exist and the file is monolithic; the giant inlined overview list is content that clearly should be removed or split out rather than placed inline in SKILL.md. Not a 3 because the inlined noise dominates the structure rather than being an isolated organizational gap. | 2 / 5 |
Total | 11 / 20 Passed |