Content
72%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 well-structured, token-efficient overview tier that correctly delegates deep content to a real one-level reference file. Its main gaps are the absence of a validation feedback loop in the build workflow (capped at 3) and no executable model SQL or dbt commands in the top tier.
Suggestions
Sequence a validation loop into the workflow, e.g. 'Build staging → run `dbt test` on staging → fix and re-run before moving to intermediate/marts', so batch materialization has an explicit checkpoint.
Inline one minimal executable example (a small stg_ model SQL plus the `dbt run` / `dbt test` commands) in the Quick Start so the top tier is actionable without opening the reference.
Trim generic bullets Claude already knows ("Version control - dbt project in Git", "Don't test in prod") in favor of dbt-specific guidance like `--select state:modified` or dev-target usage.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is almost entirely operational (layer diagram, naming table, dbt_project.yml, project tree) with no padding or concept explanations, but a few bullets state things Claude already knows — "Version control - dbt project in Git" and "Don't test in prod - Use dev target" are generic advice that could be trimmed, matching the anchor for efficient-with-minor-over-explanation rather than the every-token-earns-its-place level. | 4 / 5 |
Actionability | The Quick Start provides a concrete, copy-paste dbt_project.yml and an exact directory tree, and the naming table gives real examples like `stg_stripe__payments` and `fct_orders`. It stops short of fully executable coverage: no inline model SQL and no dbt commands (`dbt run`, `dbt test`) appear in SKILL.md itself — those live entirely in references/details.md — so it fits 'mostly executable with minor gaps' rather than the copy-paste-ready-for-common-cases level. | 4 / 5 |
Workflow Clarity | The medallion flow (sources → staging → intermediate → marts) establishes build order, and testing is named as a practice ("Test aggressively - Not null, unique, relationships"), but no validation checkpoint is sequenced into the workflow (e.g. build staging, run `dbt test`, fix, then proceed). Since dbt materialization is a batch/database operation, the missing feedback loop caps this at the 'sequence present but checkpoints missing' level. | 3 / 5 |
Progressive Disclosure | SKILL.md is a clean navigation tier (concepts, naming, quick start, best practices) with a single well-signaled, one-level-deep reference — "Detailed pattern documentation lives in `references/details.md`. Read that file when the navigation tier above is insufficient." The bundle confirms references/details.md exists and contains the actual 447-line pattern content (7 worked patterns plus a commands section) rather than a pointer chain, so it matches the clear-overview/easy-navigation anchor; not below it because nothing that belongs in the overview is buried, and not above because 5 is the maximum. | 5 / 5 |
Total | 16 / 20 Passed |