fix(migrations): TEXT→UUID FK type mismatch blocking all E2E runs - #646
Merged
Conversation
Migrations 025 + 026 declared workspace_id/org_id as TEXT but workspaces.id is UUID — Postgres rejects the FK constraint, crashing every E2E run on main since these migrations were merged. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
|
Dev Lead clearance ✅ — URGENT merge Diff verified: TEXT→UUID on both FK columns that reference
+2/-2, no logic change, no test risk. E2E CI is blocked on main until this lands. Priority merge. |
HongmingWang-Rabbit
added a commit
that referenced
this pull request
Apr 17, 2026
…E2E CI Migration 028 declared workspace_id as TEXT with a FK to workspaces(id) which is UUID. Postgres rejects the FK: 'cannot be implemented' because the types don't match. Same class of bug as #646 (which fixed 025). This has been blocking ALL open PRs' E2E API Smoke Test for 5+ cycles (since 028 was introduced in #641 Cloudflare Artifacts). Every PR CI run applies all migrations from scratch → hits this → platform exits with log.Fatalf → /health never responds → 30s timeout → FAIL. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2 tasks
molecule-ai Bot
added a commit
that referenced
this pull request
Apr 21, 2026
Merge gate passed. +2/-2 FK type fix: workspace_id TEXT→UUID in 025, org_id TEXT→UUID in 026 — matches workspaces.id (UUID PK). Schema migration — CEO explicit authorization in chat (boot-blocker/urgent). UNSTABLE = known App token scope gap.
molecule-ai Bot
pushed a commit
that referenced
this pull request
Apr 21, 2026
…E2E CI Migration 028 declared workspace_id as TEXT with a FK to workspaces(id) which is UUID. Postgres rejects the FK: 'cannot be implemented' because the types don't match. Same class of bug as #646 (which fixed 025). This has been blocking ALL open PRs' E2E API Smoke Test for 5+ cycles (since 028 was introduced in #641 Cloudflare Artifacts). Every PR CI run applies all migrations from scratch → hits this → platform exits with log.Fatalf → /health never responds → 30s timeout → FAIL. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CRITICAL — unblocks all E2E CI
Migrations 025 (workspace_token_usage) and 026 (org_plugin_allowlist) declared FK columns as TEXT but workspaces.id is UUID. Postgres rejects the constraint, crashing the platform on boot. Every E2E run on main fails since these were merged.
Fix: TEXT → UUID on both FK columns.
🤖 Generated with Claude Code