feat(db): contracts/05 — add inbound_webhook_events.contract_version column by adm01-debug · Pull Request #56 · adm01-debug/promo-gifts-v4

adm01-debug · 2026-05-22T00:35:59Z

Sumário

Adiciona a coluna contract_version à tabela inbound_webhook_events.

Por quê

O handler webhook-inbound migrado em #45 grava contract_version: '1'|'2' em todo INSERT. Sem essa coluna no Postgres, o handler vai falhar com column "contract_version" does not exist quando #45 for mergeado.

O que faz

ALTER TABLE public.inbound_webhook_events
  ADD COLUMN IF NOT EXISTS contract_version text NOT NULL DEFAULT '1';

comentário descritivo na coluna + sanity check no fim da migration (raise exception se a coluna não existir após o ALTER, evita silent partial-apply).

Risco

Baixo. ALTER TABLE com DEFAULT em coluna nova é metadata-only no PG 12+: não rewrite, não lock pesado, instantâneo mesmo em tabelas grandes. Default '1' garante que linhas pré-migração ganham valor válido automaticamente.

Validação

IF NOT EXISTS torna a migration idempotente
Sanity check no fim levanta exception se a coluna sumir
Compatível com o INSERT do handler migrado (testado em sandbox no feat(contracts): Zod schemas, formato único 422, versionamento v1/v2 e testes de contrato #45)

Closes #49

Summary by cubic

Add contract_version to inbound_webhook_events to store the negotiated contract version used by the webhook-inbound handler. Prevents missing-column errors when #45 lands; closes #49.

Migration
- Adds contract_version (text, NOT NULL, default '1') to public.inbound_webhook_events.
- Idempotent via IF NOT EXISTS with a sanity check that raises if the column is missing.
- Metadata-only on Postgres 12+, avoiding table rewrite and heavy locks.

^{Written for commit 9475684. Summary will update on new commits. Review in cubic}

Summary by CodeRabbit

Notas de Lançamento

Novas Funcionalidades
- Adicionado suporte completo a versionamento de contrato para webhooks de entrada. Permite melhor compatibilidade com diferentes versões da API de forma totalmente transparente. Nenhuma alteração é necessária nas integrações existentes. Compatibilidade total com payloads antigos é mantida automaticamente, garantindo uma transição suave e absolutamente sem impacto em ambientes de produção.

… '1') Required by webhook-inbound handler migrated in #45. Stores the contract version negotiated via accept-version header. Default '1' for compat with all existing rows and unversioned clients. Closes #49

supabase · 2026-05-22T00:36:05Z

This pull request has been ignored for the connected project doufsxqlfjyuvxuezpln due to reaching the limit of concurrent preview branches.
Go to Project Integrations Settings ↗︎ if you wish to update this limit.

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

vercel · 2026-05-22T00:36:05Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
we-dream-big	Ready	Preview, Comment	May 22, 2026 12:36am

coderabbitai · 2026-05-22T00:36:09Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3aaf0bdb-93db-4533-bcc6-6cae2b8edf2b

📥 Commits

Reviewing files that changed from the base of the PR and between 4383445 and 9475684.

📒 Files selected for processing (1)

supabase/migrations/20260522001000_add_contract_version_to_inbound_webhook_events.sql

Walkthrough

Migração que adiciona coluna contract_version à tabela public.inbound_webhook_events, permitindo rastrear a versão de contrato negociada via header accept-version. O DEFAULT '1' preserva compatibilidade com eventos legados. Inclui sanity check para confirmar criação dentro de transação.

Changes

Migração: contract_version para inbound_webhook_events

Layer / File(s)	Summary
Migração e validação de contract_version `supabase/migrations/20260522001000_add_contract_version_to_inbound_webhook_events.sql`	Adiciona coluna `contract_version` (text, NOT NULL, DEFAULT '1') com comentário descritivo e bloco `DO $$` para sanity check pós-ALTER dentro de transação explícita.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	O título descreve com precisão a alteração principal: adicionar a coluna contract_version à tabela inbound_webhook_events.
Linked Issues check	✅ Passed	A migração atende todos os critérios técnicos: cria a coluna com tipo text, DEFAULT '1', adiciona COMMENT descritivo, e inclui sanity check para garantir a aplicação.
Out of Scope Changes check	✅ Passed	Nenhuma alteração fora do escopo. A migração está focada exclusivamente em adicionar a coluna contract_version conforme especificado na issue `#49`.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch feat/contracts-05-inbound-contract-version-column

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

cubic-dev-ai

No issues found across 1 file

_{Re-trigger cubic}

Copilot

Pull request overview

Adds database support for contract versioning used by the migrated webhook-inbound handler (PR #45) by introducing a contract_version column on public.inbound_webhook_events, preventing runtime INSERT failures due to a missing column (closes #49).

Changes:

Add contract_version text NOT NULL DEFAULT '1' to public.inbound_webhook_events (idempotent via IF NOT EXISTS).
Add a descriptive COMMENT ON COLUMN for contract_version.
Add a post-migration sanity check that raises if the column is still missing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+COMMENT ON COLUMN public.inbound_webhook_events.contract_version IS
+  'Versão de contrato negociada via header accept-version. Default 1 (compat com payloads legados).';
+
+-- Sanity check: deve existir após o ALTER
+DO $$
+BEGIN
+  IF NOT EXISTS (
+    SELECT 1 FROM information_schema.columns
+    WHERE table_schema = 'public'
+      AND table_name = 'inbound_webhook_events'
+      AND column_name = 'contract_version'
+  ) THEN
+    RAISE EXCEPTION 'contracts/05 migration: contract_version column was not created';
+  END IF;


feat(db): add inbound_webhook_events.contract_version column (default…

9475684

… '1') Required by webhook-inbound handler migrated in #45. Stores the contract version negotiated via accept-version header. Default '1' for compat with all existing rows and unversioned clients. Closes #49

Copilot AI review requested due to automatic review settings May 22, 2026 00:36

Copilot started reviewing on behalf of adm01-debug May 22, 2026 00:36 View session

github-actions Bot mentioned this pull request May 22, 2026

contracts/05 — Adicionar coluna inbound_webhook_events.contract_version #49

Closed

5 tasks

vercel Bot deployed to Preview May 22, 2026 00:36 View deployment

cubic-dev-ai Bot reviewed May 22, 2026

View reviewed changes

Copilot AI reviewed May 22, 2026

View reviewed changes

adm01-debug merged commit 385522e into main May 22, 2026
28 of 32 checks passed

adm01-debug deleted the feat/contracts-05-inbound-contract-version-column branch May 24, 2026 18:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(db): contracts/05 — add inbound_webhook_events.contract_version column#56

feat(db): contracts/05 — add inbound_webhook_events.contract_version column#56
adm01-debug merged 1 commit into
mainfrom
feat/contracts-05-inbound-contract-version-column

adm01-debug commented May 22, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

supabase Bot commented May 22, 2026

Uh oh!

vercel Bot commented May 22, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented May 22, 2026 •

edited

Loading

Uh oh!

cubic-dev-ai Bot left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

adm01-debug commented May 22, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Sumário

Por quê

O que faz

Risco

Validação

Summary by cubic

Summary by CodeRabbit

Notas de Lançamento

Uh oh!

supabase Bot commented May 22, 2026

Uh oh!

vercel Bot commented May 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented May 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

adm01-debug commented May 22, 2026 •

edited by coderabbitai Bot

Loading

vercel Bot commented May 22, 2026 •

edited

Loading

coderabbitai Bot commented May 22, 2026 •

edited

Loading