Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The content is highly actionable, packed with executable code across Great Expectations, dbt, and data contracts, but it is a long monolithic document that pads in concepts Claude already knows and does not split detail into reference files or present a single guided validation workflow with checkpoints.
Suggestions
Move the detailed code patterns (Great Expectations suites/checkpoints, dbt tests, data contracts, the pipeline class) into separate files under references/ (e.g. references/great-expectations.md, references/dbt-tests.md) and keep SKILL.md as a concise overview with one-level-deep links — this improves both progressive_disclosure and conciseness.
Remove or drastically shrink the "Core Concepts" section (the data-quality-dimensions table and the testing-pyramid ASCII art), which explains concepts Claude already knows and adds tokens without earning their place.
Add a single explicit end-to-end workflow with numbered steps and validation checkpoints (e.g. define expectations → run checkpoint → review failures → fix and re-validate → only then promote/merge) to lift workflow_clarity from a pattern catalog to a guided process.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most patterns are efficient concrete code, but the "Core Concepts" section explains data-quality dimensions in a table and a testing-pyramid ASCII diagram that Claude already knows, and the 583-line body could be tightened — matching the 'mostly efficient but includes some unnecessary explanation' anchor rather than the lean score-3. | 2 / 3 |
Actionability | The body is dominated by fully executable, copy-paste-ready examples (pip/bash setup, Python expectation suites, YAML checkpoints, dbt test YAML and SQL, data-contract YAML, a complete pipeline class with usage), matching the 'fully executable code/commands; copy-paste ready' anchor and clearly above the pseudocode/incomplete score-2. | 3 / 3 |
Workflow Clarity | The skill reads as a pattern catalog rather than a guided sequence; Pattern 6 does include a validation feedback loop (checking result.success and raising on failure), but there is no top-level numbered workflow with explicit validate-then-proceed checkpoints, so it sits at 'sequence present but checkpoints missing/implicit' rather than a 3. | 2 / 3 |
Progressive Disclosure | At 583 lines everything is inline in SKILL.md with no bundle reference files (only external doc links), and content that should be split (full suites, dbt tests, contracts, pipeline class) is inline; it is well-sectioned so it is above the score-1 wall-of-text, but at this size it does not meet the score-3 'appropriately split into one-level-deep references' bar, and the <50-line simple-skill exception does not apply. | 2 / 3 |
Total | 9 / 12 Passed |