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.
Highly actionable with a well-sequenced, guarded workflow, but its token efficiency suffers from a massive inlined SDK reference and padded prose, and its progressive disclosure is undermined by keeping that reference inline rather than in a separate file.
Suggestions
Move the ~600-line windmill-client SDK signature block into a separate `references/windmill-client-api.md` and link to it from SKILL.md to improve both conciseness and progressive disclosure.
Tighten the 'Keep metadata in sync' and 'After writing' prose by trimming inferred rationale (e.g. why pushing-to-test is bad) to the essential rule and the command.
If the SDK reference must stay inline, add a short navigational table of contents at the top of that section so callers can jump to the relevant helper instead of scanning 600 lines.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~600-line verbatim windmill-client SDK signature block is inlined directly into SKILL.md and competes heavily with context, and several metadata-sync prose blocks are padded with rationale Claude can infer, landing 'noticeably verbose' rather than merely 'could be tightened'. | 2 / 5 |
Actionability | Concrete commands (`wmill script preview <path>`, `wmill generate-metadata --dry-run`, `wmill resource-type list --schema`), complete executable code samples, and full typed API signatures make the guidance copy-paste ready across the common cases. | 5 / 5 |
Workflow Clarity | The write -> sync-metadata -> preview -> deploy-only-on-explicit-request sequence is clear with explicit guardrails and checkpoints (`--dry-run` to inspect stale items, diff `.lock` files to report dependency bumps before deploy, preview validates before deploy), with only minor validation gaps keeping it off the top anchor. | 4 / 5 |
Progressive Disclosure | Section headers and inline signals to `AGENTS.wmill.md` and sibling skills give some structure, but no bundle files exist and the large SDK API reference is inlined in SKILL.md rather than split into a separate one-level-deep reference file, fitting the 'structure present but content that should be separate is inline' anchor. | 3 / 5 |
Total | 14 / 20 Passed |