From 1d5672e4fed87ab4cbe64d866a74009c5de1f9b7 Mon Sep 17 00:00:00 2001 From: weichen Date: Sun, 11 Oct 2026 08:32:45 +0000 Subject: [PATCH] Cover the surface tables in storage-migrate's copy list Why: the copy-order coverage guard caught that backend PR #143's new tables (surfaces, surface_revisions, plugin_activation) were absent from the migration list - they would have been silently skipped on real migrations. Evidence: TestCopyOrderCoversAllFixtureTables failed on the merged main and passes with the addition; full migrator suite green on SQLite and real PostgreSQL. Impact: copyOrder list only; ordered after their referenced owners/projects. Refs: backend PR #143; Dev U4 task. --- cmd/storage-migrate/main.go | 1 + 1 file changed, 1 insertion(+) diff --git a/cmd/storage-migrate/main.go b/cmd/storage-migrate/main.go index 1016faa..e7f2a54 100644 --- a/cmd/storage-migrate/main.go +++ b/cmd/storage-migrate/main.go @@ -290,6 +290,7 @@ var copyOrder = []string{ "memory_entries", "audit_events", "pending_confirms", "llm_usage_daily", "secret_keys", "settings_snapshots", "regen_pending", "context_entries", "user_commands", "pending_notices", "recovery_holds", + "surfaces", "surface_revisions", "plugin_activation", } const controlDDL = `