Skip to content

fix: migration b1d72ad8a6ed - try/except 保证全新部署成功#8

Merged
Color2333 merged 4 commits intomainfrom
fix/migration-sqlite-safe
Mar 20, 2026
Merged

fix: migration b1d72ad8a6ed - try/except 保证全新部署成功#8
Color2333 merged 4 commits intomainfrom
fix/migration-sqlite-safe

Conversation

@Color2333
Copy link
Copy Markdown
Owner

Summary

  • b1d72ad8a6ed 的每个操作加 try/except 保证幂等
  • CREATE TABLE IF NOT EXISTS + DROP INDEX IF EXISTS + DROP TABLE IF EXISTS
  • 保证全新数据库首次部署也能成功运行 alembic upgrade head
  • 同样保证已手动建表的历史服务器不报错

问题

原 migration(以及上一个修复版本)不带 try/except:

  • 全新部署时,CREATE TABLE 成功后如果后续操作报错,整个 transaction rollback,两个新表也没了
  • 历史服务器如果已手动建表,下次 upgrade 还是报错

@Color2333 Color2333 merged commit a022a50 into main Mar 20, 2026
2 checks passed
@github-actions
Copy link
Copy Markdown

🔍 OpenCode PR Review Required

这是一个受保护的分支,merge 前需要进行 code review。

请运行以下命令进行 OpenCode review:

/oc review https://github.com/Color2333/PaperMind/pull/$PR_NUM

或者在 PR 页面评论 /oc 来触发 OpenCode review。


This is an automated reminder from PR Review Gate.

@Color2333 Color2333 deleted the fix/migration-sqlite-safe branch March 20, 2026 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant