Skip to content

Add the Ahrefs Domain Rating columns (migrations only) - #132

Merged
RaviTejaKNTS merged 1 commit into
mainfrom
ad-network-dr-migration
Nov 14, 2026
Merged

RaviTejaKNTS merged 1 commit into
mainfrom
ad-network-dr-migration

Conversation

@RaviTejaKNTS

Copy link
Copy Markdown
Collaborator

What

The database part of pull request #131, released first, as stack127_docs/production-deployment.md describes for schema changes.

PR CI builds against the production database. #131's build failed with column p.domain_rating does not exist, because migrations only apply at release.

supabase/migrations/20261114130000_ad_network_domain_rating.sql:

  • Adds nullable domain_rating and domain_rating_checked_on to ad_network_publishers.
  • Adds a named NOT VALID range check (0 to 100), then validates it in a second transaction, so the 125,000-row table is not held under an exclusive lock while it is checked.
  • lock_timeout 10 s. Idempotent, with type guards.

Nothing reads the columns until #131 merges, so this release only adds the columns.

Review

Reviewed as part of #131 in three rounds. The migration's locking and idempotency findings were fixed and confirmed. The reviewer checked that Supabase CLI 2.122.1 runs the authored transaction boundaries without an outer transaction.

Checks

Applied twice in development; the second run was a no-op and all rows were preserved.

🤖 Generated with Claude Code

Migrations-only release ahead of pull request #131, so its production
build check can read the new columns. Nullable domain_rating and
domain_rating_checked_on, a named NOT VALID range check validated in a
second transaction, idempotent with type guards. No code reads the
columns until #131 merges.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@RaviTejaKNTS
RaviTejaKNTS merged commit f819685 into main Nov 14, 2026
4 checks passed

This branch was previously deployed

1 inactive deployment
production — ef8ef3a9 Deployed Nov 14, 2026 by RaviTejaKNTS via verify-production-build #124
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