feat(migrations): 对齐 staging 数据库结构与生产 #1627

Merged

auto-approve-bot merged 1 commits from feat/db-schema-alignment into develop 2026-09-02 15:30:38 +08:00

Conversation 5 Commits 1 Files Changed 3

+56

xiaoxia commented 2026-09-02 15:08:03 +08:00

Owner

Copy Link

Copy Source

问题

生产环境 vs Staging 数据库结构存在差异：

• template_categories 表缺少 sort_order 字段（生产已有）
• generation_tasks 表缺少 edit_plan_id 字段（生产已有，曾在 migration 011 中被删除）

修复

Migration 061

• template_categories 添加 sort_order 列（Integer, NOT NULL, default 0）

Migration 062

• generation_tasks 添加 edit_plan_id 列（String(36), nullable, 带索引）

Model 更新

• TemplateCategoryModel 增加 sort_order 字段
• GenerationTaskModel 增加 edit_plan_id 字段

部署后需执行 alembic upgrade head 应用 migration。

## 问题 生产环境 vs Staging 数据库结构存在差异： - `template_categories` 表缺少 `sort_order` 字段（生产已有） - `generation_tasks` 表缺少 `edit_plan_id` 字段（生产已有，曾在 migration 011 中被删除） ## 修复 ### Migration 061 - `template_categories` 添加 `sort_order` 列（Integer, NOT NULL, default 0） ### Migration 062 - `generation_tasks` 添加 `edit_plan_id` 列（String(36), nullable, 带索引） ### Model 更新 - `TemplateCategoryModel` 增加 `sort_order` 字段 - `GenerationTaskModel` 增加 `edit_plan_id` 字段 部署后需执行 `alembic upgrade head` 应用 migration。

xiaoxia added 1 commit 2026-09-02 15:08:03 +08:00

feat(migrations): align staging DB schema with production ... d35fcca447

1. Add sort_order (Integer, default 0) to template_categories
2. Re-add edit_plan_id (String(36), nullable) to generation_tasks

Also update corresponding ORM models to match.

auto-approve-bot commented 2026-09-02 15:12:25 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

• 是否存在阻塞级问题：是
• 阻塞级问题数量：1 个

📊 审查概览

• 整体评价：需修改
• 建议级问题数量：1 个

🔴 阻塞级问题（必须修复）

1. [alembic/versions/062_readd_edit_plan_id_to_generation_tasks.py:25 & packages/adapters/sqlalchemy_impl/models.py:292] 索引名称不匹配导致模型与数据库定义不一致
   • 问题类型：数据损坏风险 / 模式不一致
   • 问题描述：迁移脚本中创建的索引名称为 ix_generation_tasks_edit_plan_id_2，而 Model 定义中使用了 index=True，SQLAlchemy 会自动生成默认索引名 ix_generation_tasks_edit_plan_id。这种不一致会导致 Alembic autogenerate 检测到差异，可能会在后续迁移中错误地尝试创建重复索引或删除现有索引，导致数据库状态混乱。
   • 修改建议：将迁移脚本中的索引名称改为 ix_generation_tasks_edit_plan_id（如果生产环境确实存在该名称的残留索引，应先清理），或者在 Model 中显式指定 Index(name='ix_generation_tasks_edit_plan_id_2', 'edit_plan_id') 以保持一致。建议优先使用标准命名 ix_generation_tasks_edit_plan_id。

💡 改进建议（不阻塞合并）

1. [alembic/versions/061_add_sort_order_to_template_categories.py:9 & alembic/versions/062_readd_edit_plan_id_to_generation_tasks.py:10] 迁移脚本日期异常
   • 具体内容：两个迁移脚本的 Create Date 均为 2026-09-02，这看起来像是未来的日期，可能是笔误。建议修正为当前日期，以免造成版本时间线混淆。

✅ 良好实践

• 在添加非空列 sort_order 时，正确使用了 server_default="0"，保证了现有数据的兼容性，避免了全表更新。
• Model 定义与迁移脚本的数据类型（String(36), Integer）和可空性（nullable）保持一致。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-09-02 07:12:25 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：1 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 1. **[alembic/versions/062_readd_edit_plan_id_to_generation_tasks.py:25 & packages/adapters/sqlalchemy_impl/models.py:292] 索引名称不匹配导致模型与数据库定义不一致** - 问题类型：数据损坏风险 / 模式不一致 - 问题描述：迁移脚本中创建的索引名称为 `ix_generation_tasks_edit_plan_id_2`，而 Model 定义中使用了 `index=True`，SQLAlchemy 会自动生成默认索引名 `ix_generation_tasks_edit_plan_id`。这种不一致会导致 Alembic autogenerate 检测到差异，可能会在后续迁移中错误地尝试创建重复索引或删除现有索引，导致数据库状态混乱。 - 修改建议：将迁移脚本中的索引名称改为 `ix_generation_tasks_edit_plan_id`（如果生产环境确实存在该名称的残留索引，应先清理），或者在 Model 中显式指定 `Index(name='ix_generation_tasks_edit_plan_id_2', 'edit_plan_id')` 以保持一致。建议优先使用标准命名 `ix_generation_tasks_edit_plan_id`。 ### 💡 改进建议（不阻塞合并） 1. **[alembic/versions/061_add_sort_order_to_template_categories.py:9 & alembic/versions/062_readd_edit_plan_id_to_generation_tasks.py:10] 迁移脚本日期异常** - 具体内容：两个迁移脚本的 `Create Date` 均为 `2026-09-02`，这看起来像是未来的日期，可能是笔误。建议修正为当前日期，以免造成版本时间线混淆。 ### ✅ 良好实践 - 在添加非空列 `sort_order` 时，正确使用了 `server_default="0"`，保证了现有数据的兼容性，避免了全表更新。 - Model 定义与迁移脚本的数据类型（String(36), Integer）和可空性（nullable）保持一致。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-02 07:12:25 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

gitea-actions bot commented 2026-09-02 15:15:21 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#1627
预览链接	https://pr-1627.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1627 | | 预览链接 | [https://pr-1627.preview.xiaoxiajianji.com](https://pr-1627.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot approved these changes 2026-09-02 15:30:07 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-09-02 15:30:07 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit a633b64d4f into develop 2026-09-02 15:30:38 +08:00

auto-approve-bot deleted branch feat/db-schema-alignment 2026-09-02 15:30:38 +08:00

gitea-actions bot commented 2026-09-02 15:33:07 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1627 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1627 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

xiaoxia referenced this pull request 2026-09-02 16:33:49 +08:00

Release: 同步 develop 到 main（含前端素材库自动创建 + 数据库 migration） #1629

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1627