fix(migrations): remove duplicate indexes

[waleedlatif1]

Summary

• remove duplicate indexes

Type of Change

• Bug fix

Testing

Tested manually

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

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

1 Skipped Deployment

Project	Deployment	Review	Updated (UTC)
docs	Skipped		Dec 20, 2025 9:52pm

[greptile-apps]

Greptile Summary

This PR removes duplicate index definitions that were already created in migration 0125 and adds defensive SQL patterns for idempotency.

• Removed 4 duplicate index creations (api_key_workspace_type_idx, api_key_user_type_idx, verification_expires_at_idx, workflow_blocks_type_idx) that were already defined in migration 0125
• Added IF NOT EXISTS to column addition to prevent errors on re-run
• Added conditional check to the UPDATE statement (AND wel."workspace_id" IS NULL) to avoid overwriting existing values
• Wrapped foreign key constraint creation in a DO block with existence check to prevent duplicate constraint errors
• Added IF NOT EXISTS to index creation for idempotency

These changes make the migration safer to run multiple times and eliminate database errors from duplicate index creation attempts.

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The changes are straightforward and correct - removing duplicate indexes that were already created in a previous migration (0125), while also adding defensive SQL patterns (IF NOT EXISTS checks) to make the migration idempotent and safer to run multiple times
• No files require special attention

Important Files Changed

Filename	Overview
packages/db/migrations/0127_flimsy_sister_grimm.sql	Removed duplicate indexes already created in migration 0125, added safety checks for column and constraint creation

Sequence Diagram

sequenceDiagram
    participant M as Migration Script
    participant DB as PostgreSQL Database
    participant WEL as workflow_execution_logs
    participant W as workflow
    participant WS as workspace

    Note over M,DB: Step 1: Add nullable column
    M->>DB: ALTER TABLE ADD COLUMN IF NOT EXISTS workspace_id
    DB-->>M: Column added or already exists

    Note over M,DB: Step 2: Backfill workspace_id
    M->>DB: UPDATE workflow_execution_logs<br/>SET workspace_id FROM workflow<br/>WHERE workspace_id IS NULL
    DB->>W: Join workflow table
    W-->>DB: Return workspace_id values
    DB->>WEL: Update rows with workspace_id
    DB-->>M: Rows updated

    Note over M,DB: Step 3: Delete orphaned logs
    M->>DB: DELETE WHERE workspace_id IS NULL
    DB->>WEL: Remove orphaned records
    DB-->>M: Orphaned records deleted

    Note over M,DB: Step 4: Add NOT NULL constraint
    M->>DB: ALTER COLUMN workspace_id SET NOT NULL
    DB-->>M: Constraint added

    Note over M,DB: Step 5: Add foreign key (with existence check)
    M->>DB: IF NOT EXISTS constraint check
    DB-->>M: Constraint status
    alt Constraint doesn't exist
        M->>DB: ADD CONSTRAINT FOREIGN KEY
        DB->>WS: Link to workspace table
        DB-->>M: Foreign key added
    else Constraint exists
        M->>M: Skip constraint creation
    end

    Note over M,DB: Step 6: Create index (idempotent)
    M->>DB: CREATE INDEX IF NOT EXISTS<br/>workflow_execution_logs_workspace_started_at_idx
    DB-->>M: Index created or already exists

Loading