Content
45%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 workflow at the end is genuinely actionable and well-sequenced with state-handling feedback loops, but it is buried under ~1390 lines of duplicated, generic, Qwilr-irrelevant padding that destroys token efficiency and leaves the skill as a structureless monolith with no progressive disclosure. The actionable core is good; everything above it is the problem.
Suggestions
Delete the duplicated 'Qwilr Overview' bullet list (lines ~17-1409) entirely, or move a single deduplicated, Qwilr-specific glossary into a references/ file linked from the body.
Add an explicit numbered workflow (1. Install CLI -> 2. Authenticate -> 3. connection ensure -> 4. poll until READY -> 5. action list -> 6. action run) so the full sequence is a clear checklist rather than implied.
Provide at least one fully concrete end-to-end example with real values (e.g. a sample action list output and a run command with a resolved actionId) to lift actionability from mostly-executable to copy-paste ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Roughly 1390 of 1535 lines are a massively duplicated bullet list of generic business-document terms (e.g. 'Privacy Policy', 'Invoice', 'Disaster Recovery Plan') repeated verbatim many times with no Qwilr-specific value — textbook heavily-padded, severely-verbose content matching anchor 1, not the merely-verbose anchor 2. | 1 / 5 |
Actionability | The Membrane CLI section gives real, copy-paste-ready executable commands (e.g. 'npm install -g @membranehq/cli@latest', 'membrane connection ensure "https://qwilr.com" --json') plus a proxy flags table, matching anchor 4's 'mostly executable guidance' rather than 5 because parameters use placeholders and there is no fully concrete end-to-end example. | 4 / 5 |
Workflow Clarity | The connection flow has an explicit state-based feedback loop (poll with --wait until READY, handle CLIENT_ACTION_REQUIRED by re-polling, surface CONFIGURATION_ERROR), fitting anchor 4's clear sequence with most checkpoints present; it falls short of anchor 5 because the top-level install->auth->connect->search->run sequence is implied rather than a clean numbered checklist and 'Step 2' is referenced without a clear 'Step 1'. | 4 / 5 |
Progressive Disclosure | It is a single monolithic file with no external references (references/, scripts/, assets/ are empty) and the giant duplicated Qwilr Overview list is inlined content that clearly belongs in a separate reference file or should be removed, matching anchor 2's 'content that clearly belongs in separate files is inlined' rather than anchor 3 which requires at least some clearly-signaled references. | 2 / 5 |
Total | 11 / 20 Passed |