Content
71%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 a strong operational guide: fully executable commands and R examples with explicit validation checkpoints for every mutating operation. Its weaknesses are redundancy (the deploy and not-a-deploy rules are each restated several times) and organization (generic Windmill content inlined in the R skill, external references that are not part of any bundle, and inconsistent heading levels).
Suggestions
State each safety rule once: consolidate the deploy-only-on-explicit-request rule (currently at lines 15, 19, 26, and 45) and the 'generate-metadata is not a deploy' clarification (lines 14, 33, 45) into a single authoritative section.
Move the language-agnostic Windmill material (CLI commands, general principles, preprocessor scripts) into a shared reference file and keep only the R-specific sections inline in this skill.
Fix the heading hierarchy (## CLI Commands currently precedes two # H1s) and clearly signal which referenced files (AGENTS.wmill.md, the preview skill) live outside this skill bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The R-specific guide is lean and all content is Windmill-specific knowledge Claude does not already have, but the CLI/metadata half is noticeably redundant: the deploy-only-on-explicit-request rule appears four times (lines 15, 19, 26, 45) and the 'generate-metadata is not a deploy' clarification three times (lines 14, 33, 45). This fits anchor 3 ('mostly efficient but includes some unnecessary explanation or could be tightened'); the repetition of identical rules pushes it below anchor 4 but the absence of known-concept padding keeps it above anchor 2. | 3 / 5 |
Actionability | Fully executable throughout: concrete commands ('wmill script preview <script_path>', 'wmill generate-metadata --dry-run', 'wmill generate-metadata rehash', 'wmill resource-type list --schema'), complete copy-paste R examples, per-language placeholder syntax for args ('$1 for PostgreSQL, ? for MySQL/Snowflake, @P1 for MSSQL'), and decision rules keyed to exact user phrases. Only 'Place scripts in a folder' is underspecified, which is too minor to drop it from anchor 5. | 5 / 5 |
Workflow Clarity | Decision rules (preview vs run vs sync push) are explicit with concrete trigger phrases, and real validation checkpoints exist: --dry-run lists stale items with reasons before regenerating, regenerated locks are diffed and version bumps reported, and preview validates before any deploy. However, the workflow is organized topically rather than as a linear sequence, and the repeated rules scatter the same step across four sections — anchor 4 ('clear sequence with most checkpoints present; minor validation gaps') rather than the fully sequenced/checklisted form of anchor 5. Deploying mutates remote state, but it is gated behind explicit user request and prior preview validation, so no destructive-operation cap applies. | 4 / 5 |
Progressive Disclosure | No bundle files exist, so everything is inlined in SKILL.md. Section headers provide structure, but roughly half the body is language-agnostic Windmill material (CLI commands, general principles, TypeScript/Python preprocessors) inlined in an R-specific skill — content that belongs in a shared reference file. References to 'AGENTS.wmill.md' and the 'preview' skill point outside the skill without clear signaling, and the heading hierarchy is inconsistent (## sections precede two # H1s). Anchor 3 ('some structure but could be better organized; references present but not clearly signaled; content that should be separate is inline') is the best fit. | 3 / 5 |
Total | 15 / 20 Passed |