Skip to content

Demo s029: Model description removed - #30

Open
anandgupta42 wants to merge 1 commit into
mainfrom
demo/s029-description-removed
Open

Demo s029: Model description removed#30
anandgupta42 wants to merge 1 commit into
mainfrom
demo/s029-description-removed

Conversation

@anandgupta42

Copy link
Copy Markdown
Contributor

Demo scenario

  • Scenario ID: s029
  • Branch: demo/s029-description-removed
  • Risk category: governance
  • Expected reviewer verdict: COMMENT
  • Expected deterministic evidence: rule-catalog:yml-description-removed

Customer walkthrough

Shows documentation drift for data consumers.

When presenting this PR, first show that dbt can build the project, then show the reviewer finding. The point is that the reviewer catches or ignores the change from deterministic dbt/SQL evidence, not from advisory AI text.

Validation command

rm -f demo.duckdb demo.duckdb.wal
dbt build --profiles-dir . --target dev
dbt compile --profiles-dir . --target dev
dbt docs generate --profiles-dir . --target dev
bun --conditions=browser /Users/anandgupta/codebase/altimate-code/packages/opencode/src/index.ts review \
  --cwd=/Users/anandgupta/codebase/altimate-code/demo/dbt-pr-review-demo \
  --base=main \
  --head=HEAD \
  --mode=gate \
  --manifest=/Users/anandgupta/codebase/altimate-code/demo/dbt-pr-review-demo/target/manifest.json \
  --json \
  --no-ai

Acceptance notes

  • DuckDB default warehouse.
  • AI disabled with --no-ai.
  • Included in the final local corpus matrix: 50/50 branches matched expected verdicts.
  • Real-world corpus floor held at 15/15 caught bad cases and 0/5 false positives.

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

✅ Approved — no findings (trivial tier)

No issues found in the changed dbt models. 🎉


altimate dbt-pr-review · verdict APPROVE · signed sha256:1b7498f03c3… · manifest cf4834f151

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Approved — no findings (trivial tier)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant