Content
65%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 highly actionable, code-rich skill that gives copy-paste-ready guidance for Great Expectations, dbt tests, data contracts, and pipeline orchestration. Its weaknesses are conciseness (conceptual padding Claude already knows), missing fix-retry feedback loops for batch validation, and a monolithic structure with no progressive disclosure into separate files.
Suggestions
Trim the 'Core Concepts' section: drop the testing-pyramid ASCII diagram and the definitions of completeness/uniqueness/validity that Claude already knows, keeping only the expectation-name mappings if useful.
Add an explicit validate-fix-retry loop to the automated pipeline pattern (e.g., on failure, log failed expectations, attempt remediation, then re-run the checkpoint before failing the pipeline).
Split the six Patterns into separate reference files (e.g., references/great_expectations.md, references/dbt_tests.md, references/data_contracts.md) and keep SKILL.md as a concise overview with one-level-deep links.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by valuable executable code, but the 'Core Concepts' section pads with concepts Claude already knows (the testing-pyramid ASCII diagram and a dimensions table defining completeness/uniqueness/validity) and the opening line restates the description. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready examples cover the common cases across all three tools: a Great Expectations suite builder, checkpoint YAML, dbt test schema, custom SQL tests, a data-contract YAML, and a complete orchestration pipeline class. | 5 / 5 |
Workflow Clarity | The Quick Start and automated-pipeline pattern give a usable sequence with fail-fast checks (result.success / raise ValueError), but there are no explicit validate-then-fix-retry feedback loops for these batch operations, and checkpoints are implicit rather than spelled out. | 3 / 5 |
Progressive Disclosure | Section headers are clear and well-organized, but everything lives in a single ~580-line SKILL.md with no bundle files and no one-level-deep references; the six pattern blocks are exactly the kind of content that would benefit from being split into separate reference files. | 3 / 5 |
Total | 14 / 20 Passed |