From 52b5e90d925a4f9ab2f461ee216a9c196d37a6d6 Mon Sep 17 00:00:00 2001 From: wangjia <809946525@qq.com> Date: Mon, 13 Jul 2026 07:08:26 +0800 Subject: [PATCH] =?UTF-8?q?fix(server/migrate):=20000024=20sqlite=20?= =?UTF-8?q?=E9=87=8D=E5=BB=BA=20subscriptions=20=E5=90=8E=E8=A1=A5?= =?UTF-8?q?=E5=9B=9E=20idx=5Fsubs=5Fuser=5Fexp=20=E7=B4=A2=E5=BC=95=20+=20?= =?UTF-8?q?=E5=9B=9E=E5=BD=92=E6=96=AD=E8=A8=80?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 000024 迁移重建 subscriptions 表(source CHECK 扩容)时未重新创建 idx_subs_user_exp(user_id, expires_at)索引,导致 SQLite(生产 pangolin1 实际驱动)上订阅按 user_id/expires_at 查询退化为全表扫描。up/down 均补回 CREATE INDEX,并在 TestSQLiteMigrateUpDown 里加断言防止再次静默丢失。 Co-Authored-By: Claude Sonnet 5 --- server/internal/store/sqlite_migrate_test.go | 13 +++++++++++++ .../sqlite/000024_invite_rewards.down.sql | 1 + .../migrations/sqlite/000024_invite_rewards.up.sql | 1 + 3 files changed, 15 insertions(+) diff --git a/server/internal/store/sqlite_migrate_test.go b/server/internal/store/sqlite_migrate_test.go index 7f37920..0323a91 100644 --- a/server/internal/store/sqlite_migrate_test.go +++ b/server/internal/store/sqlite_migrate_test.go @@ -50,6 +50,19 @@ func TestSQLiteMigrateUpDown(t *testing.T) { } } + // 2b. idx_subs_user_exp survives the 000024 subscriptions table rebuild + // (SQLite table-rebuild migrations silently drop indexes unless recreated; + // this guards against that regression — see 000024_invite_rewards.{up,down}.sql). + var idxCount int + if err := db.QueryRow( + `SELECT COUNT(*) FROM sqlite_master WHERE type='index' AND name='idx_subs_user_exp'`, + ).Scan(&idxCount); err != nil { + t.Fatalf("query idx_subs_user_exp: %v", err) + } + if idxCount != 1 { + t.Errorf("idx_subs_user_exp count = %d, want 1 (index lost on subscriptions rebuild?)", idxCount) + } + // 3. Seed: 3 plans + directory_version singleton. var plans int if err := db.QueryRow(`SELECT COUNT(*) FROM plans`).Scan(&plans); err != nil { diff --git a/server/migrations/sqlite/000024_invite_rewards.down.sql b/server/migrations/sqlite/000024_invite_rewards.down.sql index bc06792..b644e9c 100644 --- a/server/migrations/sqlite/000024_invite_rewards.down.sql +++ b/server/migrations/sqlite/000024_invite_rewards.down.sql @@ -13,6 +13,7 @@ INSERT INTO subscriptions_old SELECT id,user_id,plan_id,expires_at,source,create FROM subscriptions WHERE source IN ('trial','code','pay'); DROP TABLE subscriptions; ALTER TABLE subscriptions_old RENAME TO subscriptions; +CREATE INDEX idx_subs_user_exp ON subscriptions (user_id, expires_at); DROP TABLE reward_claims; DROP TABLE referrals; diff --git a/server/migrations/sqlite/000024_invite_rewards.up.sql b/server/migrations/sqlite/000024_invite_rewards.up.sql index 3088246..b66e5a6 100644 --- a/server/migrations/sqlite/000024_invite_rewards.up.sql +++ b/server/migrations/sqlite/000024_invite_rewards.up.sql @@ -47,3 +47,4 @@ INSERT INTO subscriptions_new (id,user_id,plan_id,expires_at,source,created_at) SELECT id,user_id,plan_id,expires_at,source,created_at FROM subscriptions; DROP TABLE subscriptions; ALTER TABLE subscriptions_new RENAME TO subscriptions; +CREATE INDEX idx_subs_user_exp ON subscriptions (user_id, expires_at);