feat: add metadata-ai-readiness skill and enrich mart model YAML#2
Open
aboyalejandro wants to merge 2 commits into
Open
feat: add metadata-ai-readiness skill and enrich mart model YAML#2aboyalejandro wants to merge 2 commits into
aboyalejandro wants to merge 2 commits into
Conversation
Add a Claude Code skill that audits and enriches dbt schema YAML for AI consumption. The skill automates dbt Agent Skills' writing-documentation and discovering-data standards via Postgres MCP. Enrich _marts.yml with structured descriptions using bracketed headers ([Business Purpose], [Data Grain], [Known Issues / Caveats]) on both campaign_performance and daily_summary models. Key caveats surfaced: COALESCE vs NULLIF inconsistency between models, composite grain testing gap, and averaged-averages in daily_summary aggregations. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…caveats Apply writing-documentation standards to all 46 columns across both mart models. Each description now includes [Business Purpose] context and [Known Issues / Caveats] discovered via database profiling (COALESCE masking, uniform session data, date gaps, misleading zero values on calculated KPIs). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
campaign_performanceanddaily_summarywith[Business Purpose]and[Known Issues / Caveats]sectionswriting-documentationstandards — no description restates its column nameAI Readiness Audit Report
campaign_performancedbt Schema
[Business Purpose]and caveatsnot_nullbut no compositeuniquetest — grain is unverifiedQuery Guidance
stg_campaigns_dailyavg_order_valueCOALESCE'd to 0 (misleading: 0 ≠ "no data")daily_summarydbt Schema
not_null+uniquepresentQuery Guidance
budget_utilization,overall_conversion_rate,overall_roas,overall_cpa— 0 NULLs currently (no divide-by-zero days exist, but would produce NULLs if they did)Summary
Remaining manual action
campaign_performanceneeds a composite uniqueness test on its grain. Add to_marts.yml:Test plan
_marts.ymlparses without errors:dbt parse --profiles-dir . --project-dir .🤖 Generated with Claude Code