From cb67e71835778179f52669e1c5f8120f771bd8ca Mon Sep 17 00:00:00 2001
From: Michael Suchacz <203725896+ibetitsmike@users.noreply.github.com>
Date: Tue, 21 Apr 2026 13:10:17 +0200
Subject: [PATCH] fix(coderd/database): renumber duplicate MCP migration
 (#24552)

## Summary
- rename the `allow_in_plan_mode` migration pair from `000472` to
`000473`
- rename the matching fixture file and update its comment
- remove the duplicate migration version that broke containerized
database startup

## Testing
- `go test ./coderd/database/migrations -run '^TestMigrate$' -count=1
-timeout 15m`
- validated `iofs.New` for `coderd/database/migrations` and
`coderd/database/migrations/testdata/fixtures`

Closes coder/internal#1483

> Mux opened this PR on Mike's behalf.
---
 ...e.down.sql => 000473_mcp_server_allow_in_plan_mode.down.sql} | 0
 ..._mode.up.sql => 000473_mcp_server_allow_in_plan_mode.up.sql} | 0
 ..._mode.up.sql => 000473_mcp_server_allow_in_plan_mode.up.sql} | 2 +-
 3 files changed, 1 insertion(+), 1 deletion(-)
 rename coderd/database/migrations/{000472_mcp_server_allow_in_plan_mode.down.sql => 000473_mcp_server_allow_in_plan_mode.down.sql} (100%)
 rename coderd/database/migrations/{000472_mcp_server_allow_in_plan_mode.up.sql => 000473_mcp_server_allow_in_plan_mode.up.sql} (100%)
 rename coderd/database/migrations/testdata/fixtures/{000472_mcp_server_allow_in_plan_mode.up.sql => 000473_mcp_server_allow_in_plan_mode.up.sql} (78%)

diff --git a/coderd/database/migrations/000472_mcp_server_allow_in_plan_mode.down.sql b/coderd/database/migrations/000473_mcp_server_allow_in_plan_mode.down.sql
similarity index 100%
rename from coderd/database/migrations/000472_mcp_server_allow_in_plan_mode.down.sql
rename to coderd/database/migrations/000473_mcp_server_allow_in_plan_mode.down.sql
diff --git a/coderd/database/migrations/000472_mcp_server_allow_in_plan_mode.up.sql b/coderd/database/migrations/000473_mcp_server_allow_in_plan_mode.up.sql
similarity index 100%
rename from coderd/database/migrations/000472_mcp_server_allow_in_plan_mode.up.sql
rename to coderd/database/migrations/000473_mcp_server_allow_in_plan_mode.up.sql
diff --git a/coderd/database/migrations/testdata/fixtures/000472_mcp_server_allow_in_plan_mode.up.sql b/coderd/database/migrations/testdata/fixtures/000473_mcp_server_allow_in_plan_mode.up.sql
similarity index 78%
rename from coderd/database/migrations/testdata/fixtures/000472_mcp_server_allow_in_plan_mode.up.sql
rename to coderd/database/migrations/testdata/fixtures/000473_mcp_server_allow_in_plan_mode.up.sql
index 281117240f..9fa229f30d 100644
--- a/coderd/database/migrations/testdata/fixtures/000472_mcp_server_allow_in_plan_mode.up.sql
+++ b/coderd/database/migrations/testdata/fixtures/000473_mcp_server_allow_in_plan_mode.up.sql
@@ -1,4 +1,4 @@
--- Migration 472 adds allow_in_plan_mode with a default of false.
+-- Migration 473 adds allow_in_plan_mode with a default of false.
 -- Flip the existing fixture row to true here so fixture data exercises
 -- the non-default state only after the column exists.
 UPDATE mcp_server_configs