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.
A dense, highly actionable mapping skill: complete enum coverage, concrete copy-paste YAML, and a strong gotchas section that encodes real failure modes. The main costs are token duplication (the S3 example appears twice) and an Overview that re-explains Dagster basics Claude already knows.
Suggestions
Remove the duplicated S3 sensor example — keep it in either the mapping section or the Before/After section, not both, and let the Before/After example use a different sensor type (e.g. the Snowflake SQL one).
Trim the Overview's explanation of what Dagster sensors are; keep only the Orchestra side ('sensors live in the sensors: block at the pipeline root', 'cron window + polling interval with declarative checks') which is the non-obvious knowledge.
Consider moving the full SensorChecksEnum table and the extra mapping examples into a references/ file, keeping SKILL.md to the structure, one worked example, and the gotchas.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient, but the S3 @sensor Python block appears nearly verbatim twice (in 'S3-polling @sensor -> AWS_S3_FILE' and again in 'Before / After Example'), and the Overview re-explains what Dagster sensors are ('functions evaluated on a tick interval that inspect external state and yield RunRequest(...)') — concepts Claude already knows. Anchor 3 fits better than 4 given the duplication. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready YAML with real values (cron: '0 6 * * ? *', connection: snowflake_prod_12345, frequency_secs: 60), a complete SensorChecksEnum table, and per-pattern before/after examples covering the common cases (S3 polling, SQL polling, @asset_sensor, trigger_events). Nothing is pseudocode. | 5 / 5 |
Workflow Clarity | Clear per-pattern mapping structure with before/after pairs, and a Gotchas section that flags the real failure modes ('timeout_mins < cron interval', 6-field AWS cron syntax, 'SQL check semantics — passes when the query returns >= 1 row'). Not 5: the process is not sequenced (identify sensor type -> pick SensorChecksEnum value -> map fields -> verify) and there is no checkpoint for validating the generated YAML. | 4 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent), and all content is inline and well-sectioned (Structure -> enum table -> mapping -> before/after -> gotchas -> references) with clearly signaled external URLs. Good structure overall; not 5 because there is no overview-to-detail split — the enum table and several full mapping examples could live in reference files to keep SKILL.md leaner. | 4 / 5 |
Total | 16 / 20 Passed |