fix: ensure cover title matches current video title #1520

Merged

xiaoxia merged 1 commits from fix/cover-title-consistency into develop 2026-08-27 14:43:43 +08:00

Conversation 4 Commits 1 Files Changed 3

+19 -3

xiaoxia commented 2026-08-27 14:30:46 +08:00

Owner

Copy Link

Copy Source

问题

封面存储使用 covers/{plan_id}/frame_{i}.jpg，所有任务共用同一个封面文件。用户修改标题后生成新视频，封面可能显示旧标题。

修复

1. thumbnail_generator.py

• 添加 task_id 参数到 extract_and_upload_cover_frames 函数
• 将存储 key 从 covers/{plan_id}/frame_{i}.jpg 改为 covers/{plan_id}/{task_id}/frame_{i}.jpg
• MediaKit 帧同样使用新的路径格式

2. render_adapter.py

• 传递 job_id 作为 task_id 参数到 extract_and_upload_cover_frames

3. generation_tasks.py

• 在 _writeback_edit_plan_config 函数中，比较新旧标题
• 如果标题发生变化，清除 plan.config 中的 cover 字段，强制封面重新生成

验证

生成视频A（标题"测试1"）→ 封面显示"测试1"
修改标题为"测试2"，生成视频B → 封面显示"测试2"（不是"测试1"）
两个视频的封面文件独立存储，互不影响

## 问题 封面存储使用 `covers/{plan_id}/frame_{i}.jpg`，所有任务共用同一个封面文件。用户修改标题后生成新视频，封面可能显示旧标题。 ## 修复 ### 1. thumbnail_generator.py - 添加 `task_id` 参数到 `extract_and_upload_cover_frames` 函数 - 将存储 key 从 `covers/{plan_id}/frame_{i}.jpg` 改为 `covers/{plan_id}/{task_id}/frame_{i}.jpg` - MediaKit 帧同样使用新的路径格式 ### 2. render_adapter.py - 传递 `job_id` 作为 `task_id` 参数到 `extract_and_upload_cover_frames` ### 3. generation_tasks.py - 在 `_writeback_edit_plan_config` 函数中，比较新旧标题 - 如果标题发生变化，清除 `plan.config` 中的 `cover` 字段，强制封面重新生成 ## 验证 生成视频A（标题"测试1"）→ 封面显示"测试1" 修改标题为"测试2"，生成视频B → 封面显示"测试2"（不是"测试1"） 两个视频的封面文件独立存储，互不影响

xiaoxia added 1 commit 2026-08-27 14:30:46 +08:00

fix: ensure cover title matches current video title ... 98b9e15879

- Add task_id parameter to extract_and_upload_cover_frames
- Change storage key from covers/{plan_id}/frame_{i}.jpg to covers/{plan_id}/{task_id}/frame_{i}.jpg
- Clear cover field in plan.config when title changes in _writeback_edit_plan_config
- Pass job_id as task_id in render_adapter.py

This fixes the issue where modifying the title and generating a new video
would still show the old title on the cover.

gitea-actions bot commented 2026-08-27 14:34:48 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot approved these changes 2026-08-27 14:40:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-27 14:40:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia merged commit 7cdd06802d into develop 2026-08-27 14:43:43 +08:00

gitea-actions bot commented 2026-08-27 14:47:00 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1520 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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#1520