fix: 预览渲染完成后回写 cover_url 到 GenerationTask #1412

Merged

xiaoxia merged 5 commits from fix/cover-url-in-finalize into develop 2026-08-18 00:07:49 +08:00

Conversation 5 Commits 5 Files Changed 2

+147

xiaoxia commented 2026-08-17 23:16:53 +08:00

Owner

Copy Link

Copy Source

问题

Step6 生成封面时接口返回 400 "封面尚未生成，请先重新生成预览视频以触发封面自动提取"。

根因： _finalize_render_success 更新 GenerationTask 时没有设置 cover_url 字段。封面生成接口 generation_cover.py 的 fallback 逻辑只读 GenerationTask.cover_url，全链路找不到就 400。

修复

• _finalize_render_success 函数签名新增 cover_candidates: list[dict] | None = None 参数
• 在 gen_task_repo.update(gen_task) 之前，取 cover_candidates[0] 的 image_url（fallback url）写入 gen_task.cover_url
• _render_with_unified 调用处传入 result.cover_candidates

测试

7 个单元测试：

• image_url 正确写入
• url 键 fallback
• 空列表不设置
• None 不设置
• 空字符串不设置
• gen_task 为 None 不崩溃
• image_url 优先于 url

## 问题 Step6 生成封面时接口返回 400 "封面尚未生成，请先重新生成预览视频以触发封面自动提取"。 **根因：** `_finalize_render_success` 更新 GenerationTask 时没有设置 `cover_url` 字段。封面生成接口 `generation_cover.py` 的 fallback 逻辑只读 `GenerationTask.cover_url`，全链路找不到就 400。 ## 修复 - `_finalize_render_success` 函数签名新增 `cover_candidates: list[dict] | None = None` 参数 - 在 `gen_task_repo.update(gen_task)` 之前，取 `cover_candidates[0]` 的 `image_url`（fallback `url`）写入 `gen_task.cover_url` - `_render_with_unified` 调用处传入 `result.cover_candidates` ## 测试 7 个单元测试： - `image_url` 正确写入 - `url` 键 fallback - 空列表不设置 - None 不设置 - 空字符串不设置 - gen_task 为 None 不崩溃 - `image_url` 优先于 `url`

xiaoxia added 1 commit 2026-08-17 23:16:53 +08:00

fix: 预览渲染完成后回写 cover_url 到 GenerationTask ... 8136760b4b

问题：_finalize_render_success 更新 GenerationTask 时没有设置 cover_url 字段，
导致封面生成接口 fallback 全链路找不到封面，返回 400。

修复：
- _finalize_render_success 新增 cover_candidates 参数
- 在 gen_task_repo.update 之前，取 cover_candidates[0] 的 image_url 写入 gen_task.cover_url
- _render_with_unified 调用处传入 result.cover_candidates

测试：7 个单元测试覆盖各种边界情况

auto-approve-bot added 1 commit 2026-08-17 23:20:43 +08:00

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

auto-approve-bot approved these changes 2026-08-17 23:30:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-17 23:30:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia added 1 commit 2026-08-17 23:38:08 +08:00

fix(test): 修复 MagicMock 属性自动创建导致的断言失败 ... 44efd776e9

使用 FakeGenTask 替代 MagicMock，通过追踪 _assigned 字典
准确判断 cover_url 是否被显式设置。

gitea-actions bot commented 2026-08-17 23:40:02 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-08-17 23:52:00 +08:00

fix(test): 修复 FakeGenTask 初始化顺序及 append_log 缺失问题 e52b63201a

auto-approve-bot commented 2026-08-17 23:55:21 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1412

⚠️ 问题（0个需要修改）

无

💡 建议（0个可选）

无

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-17 15:55:20 | 模型:}

## 代码审查结果 - PR #1412 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（0个可选） 无 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-17 15:55:20 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot added 1 commit 2026-08-17 23:57:01 +08:00

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

xiaoxia merged commit c3ac621ebd into develop 2026-08-18 00:07:49 +08:00

gitea-actions bot commented 2026-08-18 00:08:36 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

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