Content
75%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 concrete wmill commands and executable Snowflake code covering the common cases, plus a sensible preview-first workflow with dry-run and lock-diff checkpoints. Its weaknesses are repetition of the deploy caution across sections, an inverted heading hierarchy, and general Windmill boilerplate mixed into a Snowflake-specific skill.
Suggestions
Consolidate the three repetitions of the "deploy only when explicitly asked" caution into one place and reference it from the other sections.
Fix the heading hierarchy — "CLI Commands" and its subsections appear before the first H1 ("Windmill Script Writing Guide"); promote or reorder so levels nest correctly.
Trim the general Windmill sections (Preprocessor Scripts, Return Values) that are not Snowflake-specific, or move them to a shared reference file, to cut tokens and sharpen the skill's focus.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean and concrete, but the "not a deploy" caution for preview/generate-metadata/deploy is repeated across three sections ("CLI Commands", "Preview vs run", "After writing"), and the metadata section carries dense parenthetical asides ("the same as deploying from the UI; expected, not a bug"). That is more than the "minor instances" of over-trimming tolerated at score 4, but well short of heavily padded. | 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`) and copy-paste-ready SQL blocks for ? parameter comments, PARSE_JSON + LATERAL FLATTEN on (s3object) inputs, and the `-- s3 prefix=... format=parquet` directive. The common Snowflake cases are all covered with real code. | 5 / 5 |
Workflow Clarity | A clear sequenced workflow emerges: write → preview with plausible args → generate-metadata (scoped, with --dry-run to inspect blast radius) → diff regenerated locks and report version changes → deploy only on explicit request. Checkpoints exist (preview validates before deploy, dry-run, lock diff), but the layout is topic-sectioned rather than linear and the deploy-is-destructive guard is scattered, leaving minor gaps versus the explicit feedback-loop structure of score 5. | 4 / 5 |
Progressive Disclosure | A single well-sectioned ~110-line file with no bundle files; guidance stays one level deep (external pointers like AGENTS.wmill.md are clearly signaled). Minor gaps: general Windmill material (General Principles, Return Values, Preprocessor Scripts) is inlined in a Snowflake-named skill, and the heading hierarchy is inverted (two H2 sections appear before the first H1), so it falls short of the cleanly organized score-5 case. | 4 / 5 |
Total | 16 / 20 Passed |