Content
65%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.
An exceptionally thorough and executable CLI reference — every command, flag, and non-obvious gotcha is documented in copy-paste-ready form. Its weaknesses are architectural: the whole reference is inlined in SKILL.md with no progressive disclosure or split files, destructive sync/instance push operations lack validate-first workflow guidance, and the sync pull/push flag duplication plus the repeated object-storage listing could be tightened.
Suggestions
Split the per-command reference into references/ files (e.g. references/commands.md or per-domain files) and keep SKILL.md as a concise overview plus the high-value non-obvious guidance (global options, key concepts, decision guides), loading detail only when needed.
Add a validate-then-apply workflow for destructive operations: e.g. 'wmill sync push --dry-run (or --lint) first, review the diff, then push --yes' — the flags exist but no workflow prescribes them.
Deduplicate the ~35 identical flags shared by `sync pull` and `sync push` (state shared flags once, e.g. 'both directions accept: --skip-*, --include-*, --keep-deleted...'), and have the Object Storage section link to its subcommands in the reference instead of re-listing them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Line-for-line the reference is dense and unpadded — flags, defaults, and gotchas Claude cannot know elsewhere (e.g. the $res/$var bare-string rule, the fork-parent resolution notes). However, `sync pull` and `sync push` duplicate ~35 near-identical flags verbatim, and the trailing Object Storage section re-documents subcommands already listed in the command reference. Fits 'mostly efficient but could be tightened'; not a 4 given the sizeable duplicated blocks. | 3 / 5 |
Actionability | Complete, copy-paste-ready command grammar: every subcommand with argument types, flags, defaults, and exact value formats (e.g. 'SCRIPT:PARAM=VALUE', '--upload SCRIPT[:PARAM]=SOURCE', '@<filename>' stdin conventions, port/behavior defaults). A user can construct any invocation directly from the text. | 5 / 5 |
Workflow Clarity | This is a lookup reference, so sequence is mostly per-command, and the Object Storage section adds a genuinely useful 'Choosing a subcommand' decision guide. But destructive/batch operations exist (`sync push` 'overrides any remote versions', `instance push` 'overwrite remote', `workspace merge` deploys, `object-storage delete`) with no prescribed validate-then-apply loop even though --dry-run/--diff flags exist — the rubric's cap for destructive/batch operations without validation applies. Not a 4: no checkpoint guidance anywhere for the overwriting commands. | 3 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), so the entire ~800-line command reference is inlined in SKILL.md rather than split into reference files behind a concise overview — 'content that should be separate is inline'. It stays at 3 rather than 2 because internal structure is good: per-command headers, consistent formatting, and the closing Key-concepts/Choosing-a-subcommand section model the right pattern; navigation is easy even though nothing is offloaded. | 3 / 5 |
Total | 14 / 20 Passed |