fix(edit-flow): 补全 title_config 数据链路，修复编辑流程断裂 #1464

Merged

xiaoxia merged 4 commits from fix/edit-flow-data-chain into develop 2026-08-23 13:55:42 +08:00

Conversation 5 Commits 4 Files Changed 10

+255 -17

xiaoxia commented 2026-08-23 13:12:59 +08:00

Owner

Copy Link

Copy Source

改动

1. Schema 补字段

• CreateGenerationTaskRequest 新增 title_config: dict | None = None
• source_edit_plan_id 字段已存在，确认无需重复添加
• GenerationTaskResponse 新增 title_config 返回

2. 数据库

• generation_tasks 新增 title_config JSON 列，alembic 057 迁移
• ORM model / repository mapping 全链路支持

3. 创建任务后回写 plan.config

• 任务入队成功后，若请求传了 source_edit_plan_id，merge 更新对应 EditPlan.config
• 写入 generation_task_id 和 title_config，不整体覆盖
• 回写失败 catch 异常记日志，不影响任务创建

4. Worker 读取 title_config

• _load_task_info 读取 title_config
• _render_video 优先使用 title_config 构建标题图层
• custom_title 作为兜底，逻辑兼容

5. 旧路由标记废弃

• POST /generate 和 GET /generation-status 加 DEPRECATED 注释
• 不删除代码，不改动逻辑

兼容性

• 所有新增字段默认 None/{},不传时行为完全不变
• 215 个相关单元测试全通过

## 改动 ### 1. Schema 补字段 - CreateGenerationTaskRequest 新增 title_config: dict | None = None - source_edit_plan_id 字段已存在，确认无需重复添加 - GenerationTaskResponse 新增 title_config 返回 ### 2. 数据库 - generation_tasks 新增 title_config JSON 列，alembic 057 迁移 - ORM model / repository mapping 全链路支持 ### 3. 创建任务后回写 plan.config - 任务入队成功后，若请求传了 source_edit_plan_id，merge 更新对应 EditPlan.config - 写入 generation_task_id 和 title_config，不整体覆盖 - 回写失败 catch 异常记日志，不影响任务创建 ### 4. Worker 读取 title_config - _load_task_info 读取 title_config - _render_video 优先使用 title_config 构建标题图层 - custom_title 作为兜底，逻辑兼容 ### 5. 旧路由标记废弃 - POST /generate 和 GET /generation-status 加 DEPRECATED 注释 - 不删除代码，不改动逻辑 ## 兼容性 - 所有新增字段默认 None/{},不传时行为完全不变 - 215 个相关单元测试全通过

xiaoxia added 1 commit 2026-08-23 13:12:59 +08:00

fix(edit-flow): 补全 title_config 数据链路，修复编辑流程断裂 ... 06f68230af

1. Schema: CreateGenerationTaskRequest 增加 title_config(dict|None=None)
2. DB: generation_tasks 新增 title_config JSON 列 (alembic 057)
3. Domain/ORM/Repo/UseCase 全链路透传 title_config
4. API 创建任务成功后，merge 回写 EditPlan.config
   (generation_task_id + title_config)，失败不影响主流程
5. Worker 优先使用 title_config 构建标题图层，custom_title 兜底
6. GenerationTaskResponse 返回 title_config
7. 旧路由 /generate 和 /generation-status 标记 DEPRECATED

所有新字段默认 None/{}，不传时行为与现在完全一致。

gitea-actions bot commented 2026-08-23 13:15:24 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-23 13:15:42 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 410f672195

auto-approve-bot approved these changes 2026-08-23 13:25:34 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-23 13:25:34 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia added 1 commit 2026-08-23 13:35:10 +08:00

test: 补全 _writeback_edit_plan_config 单测，修复增量覆盖率不足 ... 817c6fa6a3

新增 9 个单元测试覆盖 generation_tasks.py 增量代码：
- 空 plan_id 直接返回
- plan 不存在 warning + 不 commit
- 正常写入（有/无 title_config）
- config 非 dict 兜底
- DB 异常 rollback + rollback 失败静默
- 空 title_config 不写入

auto-approve-bot added 1 commit 2026-08-23 13:39:30 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 0b000f96a6

auto-approve-bot commented 2026-08-23 13:41:09 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

1. [apps/api/app/api/routes/generation_tasks.py: 371-379] 批量生成任务时 EditPlan 关联数据丢失风险
   • 问题类型：逻辑bug
   • 问题描述：在 create_generation_task 函数中，针对批量创建任务（request.count > 1）的场景，代码在循环内部对同一个 source_edit_plan_id 调用 _writeback_edit_plan_config。该函数会直接覆盖 EditPlan.config 中的 generation_task_id 字段。这意味着当批量生成多个任务时，EditPlan 最终只会关联到最后一个生成的任务 ID，前序任务的关联信息会被覆盖丢失。如果业务依赖 EditPlan 反向查询生成的任务，这将导致数据不一致。
   • 修改建议：调整回写逻辑。如果是批量生成，应考虑将 generation_task_id 存储为数组，或者仅在生成第一个任务时回写，或者确认业务逻辑上只需关联最后一个任务（但通常批量任务需要保留所有关联）。建议修改 _writeback_edit_plan_config 逻辑，或者仅在循环结束后统一处理关联。

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

1. [packages/adapters/sqlalchemy_impl/models.py: 301] SQLAlchemy 模型存在可变默认参数风险
   • 具体内容：title_config = Column(JSON, nullable=False, default=dict) 使用了 default=dict。在 Python 中，这会导致所有未显式设置该字段的新实例共享同一个 dict 对象引用。虽然在 SQLAlchemy 中通常通过 server_default 或 flush 机制规避了部分风险，但在 Python 对象层面操作时仍可能导致脏数据污染。建议改为 default={} 或使用 default_factory（如果框架支持），或者依赖 server_default 并移除 Python 侧的 default=dict。

✅ 良好实践

• 防御性编程：在 _writeback_edit_plan_config 中对 plan_model.config 进行了类型检查（isinstance(plan_model.config, dict)），防止因旧数据格式问题导致崩溃。
• 异常处理：_writeback_edit_plan_config 捕获了所有异常并回滚事务，确保回写失败不影响主流程（任务创建）。
• 数据兼容：在 Worker 渲染逻辑中，同时处理了新的 title_config 和旧的 custom_title，并做了字段名归一化（font_size -> size），保证了平滑过渡和向后兼容。
• 测试覆盖：新增了 test_writeback_edit_plan_config.py，覆盖了空值、异常、回滚等多种边界情况。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-23 05:41:09 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：1 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 1. **[apps/api/app/api/routes/generation_tasks.py: 371-379] 批量生成任务时 EditPlan 关联数据丢失风险** - 问题类型：逻辑bug - 问题描述：在 `create_generation_task` 函数中，针对批量创建任务（`request.count > 1`）的场景，代码在循环内部对同一个 `source_edit_plan_id` 调用 `_writeback_edit_plan_config`。该函数会直接覆盖 `EditPlan.config` 中的 `generation_task_id` 字段。这意味着当批量生成多个任务时，`EditPlan` 最终只会关联到最后一个生成的任务 ID，前序任务的关联信息会被覆盖丢失。如果业务依赖 `EditPlan` 反向查询生成的任务，这将导致数据不一致。 - 修改建议：调整回写逻辑。如果是批量生成，应考虑将 `generation_task_id` 存储为数组，或者仅在生成第一个任务时回写，或者确认业务逻辑上只需关联最后一个任务（但通常批量任务需要保留所有关联）。建议修改 `_writeback_edit_plan_config` 逻辑，或者仅在循环结束后统一处理关联。 ### 💡 改进建议（不阻塞合并） 1. **[packages/adapters/sqlalchemy_impl/models.py: 301] SQLAlchemy 模型存在可变默认参数风险** - 具体内容：`title_config = Column(JSON, nullable=False, default=dict)` 使用了 `default=dict`。在 Python 中，这会导致所有未显式设置该字段的新实例共享同一个 `dict` 对象引用。虽然在 SQLAlchemy 中通常通过 `server_default` 或 flush 机制规避了部分风险，但在 Python 对象层面操作时仍可能导致脏数据污染。建议改为 `default={}` 或使用 `default_factory`（如果框架支持），或者依赖 `server_default` 并移除 Python 侧的 `default=dict`。 ### ✅ 良好实践 - **防御性编程**：在 `_writeback_edit_plan_config` 中对 `plan_model.config` 进行了类型检查（`isinstance(plan_model.config, dict)`），防止因旧数据格式问题导致崩溃。 - **异常处理**：`_writeback_edit_plan_config` 捕获了所有异常并回滚事务，确保回写失败不影响主流程（任务创建）。 - **数据兼容**：在 Worker 渲染逻辑中，同时处理了新的 `title_config` 和旧的 `custom_title`，并做了字段名归一化（`font_size` -> `size`），保证了平滑过渡和向后兼容。 - **测试覆盖**：新增了 `test_writeback_edit_plan_config.py`，覆盖了空值、异常、回滚等多种边界情况。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-23 05:41:09 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 94caa63436 into develop 2026-08-23 13:55:42 +08:00

gitea-actions bot commented 2026-08-23 13:57:04 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia referenced this issue from a commit 2026-08-23 14:11:43 +08:00

fix(edit-flow): 批量生成时 plan.config 只回写一次，避免 generation_task_id 被循环覆盖

xiaoxia referenced this pull request 2026-08-23 14:11:54 +08:00

fix(edit-flow): 批量生成 plan.config 只回写一次 #1466

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#1464