Install
$ agentstack add skill-yeaight7-agent-powerups-semantic-layer-change-review ✓ scanned · ✓ verified — works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
About
Purpose
Changes to the semantic layer directly impact dashboards and business reporting. A silent drift in a metric definition destroys trust. Review every semantic-layer change for mathematical soundness and backwards compatibility before approval.
When to Use
- A PR modifies metric or semantic model YAML
- A metric's
expr, aggregation, or filters are changing - New dimensions or entities are being added to an existing semantic model
Inputs
- The semantic-layer diff (semantic models, metrics, saved queries)
- The underlying model's grain and entity keys
Workflow
- Enumerate what changed:
``bash git diff origin/main...HEAD -- '*.yml' '*.yaml' dbt ls --resource-type metric dbt ls --resource-type semantic_model ``
- Identify the change type:
- Addition — safe (adding a new metric or dimension).
- Deprecation — requires communication (removing a metric).
- Modification — high risk (changing the SQL expression, aggregation, or filters of an existing metric).
- Evaluate mathematical soundness:
- Are we averaging an average?
- Are we summing a distinct count?
- Does adding this dimension cause a fan-out that inflates the metric?
- Check backwards compatibility. If an existing metric's logic is changed, you MUST flag it. The recommended path is dbt's metric versioning or a new metric (e.g.,
revenue_v2) rather than silently altering historical numbers. Find consumers of the metric before judging impact:
``bash grep -rn "" --include="*.yml" --include="*.yaml" . ``
- Verify entity mapping. Ensure
entities(primary/foreign keys) match the granularity of the underlying semantic model.
- Confirm definitions still parse:
``bash dbt parse ``
Output
- Change classification (addition / deprecation / modification) per touched metric or semantic model
- Mathematical soundness findings
- Backwards-compatibility verdict, with consumers that need communication
- Entity/grain mismatches, if any
Verification
- [ ] Every touched metric and semantic model classified by change type
- [ ] Modifications to existing metrics explicitly flagged, never silently approved
- [ ] Consumers of modified metrics identified
- [ ]
dbt parsepasses on the changed project - [ ] Entity keys checked against the underlying model's grain
Failure Modes
- Silent restatement — approving a change to a core metric's
exprwithout explicitly confirming the business requested the restatement of historical data. - Treating modification like addition — additions are safe; modifications are high risk and need versioning or a new metric.
- Fan-out blindness — a new dimension join can inflate a metric while every individual definition still looks correct.
Source & license
This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.
- Author: yeaight7
- Source: yeaight7/agent-powerups
- License: Apache-2.0
Install and usage instructions live in the source repository linked above.
Reviews
No reviews yet — be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.