Content
63%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 is highly actionable with executable code and specific CLI commands, and its operational guidance (preview vs run by intent, metadata sync with diff/dry-run feedback loops) is clear and high-signal. Its main weakness is structure: a large inlined API reference and no bundle files mean progressive disclosure is poor and the token budget is heavier than it needs to be.
Suggestions
Move the ~610-line windmill-client API reference (lines 172-783) into a separate REFERENCE.md bundle file and keep only a few key examples plus a pointer in SKILL.md.
Present the write -> preview -> generate-metadata -> deploy flow as a short numbered checklist with an explicit 'validate (preview) before deploy' checkpoint.
Give the deploy step a concrete command example (git push vs wmill sync push) instead of only a prose description.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The CLI/operational prose is tight and high-signal (no padding with concepts Claude already knows), but a roughly 610-line inlined windmill-client API reference with full JSDoc for every function is a substantial token cost that could be tightened or moved out, fitting 'mostly efficient but includes some unnecessary [bulk] or could be tightened' rather than the lean level above. | 3 / 5 |
Actionability | Code examples (//native main, RT.Stripe, fetch, S3 load/write) and CLI commands (wmill script preview, generate-metadata with --dry-run) are executable and copy-paste ready with specific flags and signatures, but the deploy step is only described abstractly ("via git push or wmill sync push") and some SDK entries lack usage examples, leaving minor gaps that cap it at 'mostly executable guidance'. | 4 / 5 |
Workflow Clarity | There is a clear sequence (write -> offer/preview -> generate-metadata -> deploy) with real validation checkpoints and a feedback loop (preview validates before deploy, generate-metadata --dry-run lists stale items, diff the .lock to catch unwanted version bumps), but it is delivered as prose decision rules rather than a crisp numbered checklist with explicit validate steps, so it does not reach the level-5 checklist anchor. | 4 / 5 |
Progressive Disclosure | Section headers (## / ###) give real structure and external references (AGENTS.wmail.md, rt.d.ts, the preview skill) are signaled, but there are no bundle files at all and the entire ~610-line SDK API reference is inlined in SKILL.md — exactly 'content that should be separate is inline' — so it sits at 'some structure but could be better organized' rather than the well-split level above. | 3 / 5 |
Total | 14 / 20 Passed |