fix(cover): E2兜底封面标题样式透传color/position/font_size #1467

Merged

xiaoxia merged 2 commits from fix/cover-title-style-e2 into develop 2026-08-23 14:57:06 +08:00

Conversation 5 Commits 2 Files Changed 5

+253 -9

xiaoxia commented 2026-08-23 14:44:37 +08:00

Owner

Copy Link

Copy Source

问题

封面E2兜底路径（从源素材抽帧）只用硬编码白色标题，未读取plan.config中的color/position/font_size，导致封面标题样式与预览视频不一致。

改动

1. packages/shared/title_overlay.py：apply_title_to_image 新增 color 参数，内部用 _parse_hex_color 解析 #RRGGBB/#RGB，替换硬编码白色
2. apps/api/.../generation_cover.py：E2路径从 plan.config.title 读取完整样式（color/font_color/position/font_size），_persist_cover_frame 新增 title_color/title_position/title_font_size 参数并透传
3. apps/worker/.../thumbnail_generator.py：apply_title_overlay 和 extract_and_upload_cover_frames 同步新增样式参数透传
4. 兼容前端 font_color 字段名
5. 新增6个单测，36个相关测试全通过

## 问题 封面E2兜底路径（从源素材抽帧）只用硬编码白色标题，未读取plan.config中的color/position/font_size，导致封面标题样式与预览视频不一致。 ## 改动 1. **packages/shared/title_overlay.py**：apply_title_to_image 新增 color 参数，内部用 _parse_hex_color 解析 #RRGGBB/#RGB，替换硬编码白色 2. **apps/api/.../generation_cover.py**：E2路径从 plan.config.title 读取完整样式（color/font_color/position/font_size），_persist_cover_frame 新增 title_color/title_position/title_font_size 参数并透传 3. **apps/worker/.../thumbnail_generator.py**：apply_title_overlay 和 extract_and_upload_cover_frames 同步新增样式参数透传 4. 兼容前端 font_color 字段名 5. 新增6个单测，36个相关测试全通过

xiaoxia added 1 commit 2026-08-23 14:44:37 +08:00

fix(cover): E2兜底封面标题样式透传color/position/font_size ... 6090412d26

- title_overlay.apply_title_to_image 新增 color 参数，支持自定义字体颜色
- 新增 _parse_hex_color 解析 #RRGGBB/#RGB，无效值 fallback 白色
- generation_cover E2路径从 plan.config.title 读取完整样式
  (color/font_color/position/font_size)，传给 _persist_cover_frame
- _persist_cover_frame 新增 title_color/title_position/title_font_size 参数
- Worker thumbnail_generator 的 apply_title_overlay 和
  extract_and_upload_cover_frames 同步透传样式参数
- 兼容前端 font_color 字段名（与 color 二选一）
- 新增6个单测：颜色解析、自定义颜色、E2完整样式透传、font_color兼容

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

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-23 14:47:29 +08:00

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

auto-approve-bot commented 2026-08-23 14:49:49 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1467

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

无

💡 建议（0个可选）

无

---

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

【阻塞级判定】

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

📊 审查概览

• 整体评价：通过
• 建议级问题数量：0 个

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

无

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

无

✅ 良好实践

1. 健壮的兼容性处理：在 generation_cover.py 中，同时兼容了 color 和 font_color 字段，确保新旧配置格式均能正常工作，降低了升级风险。
2. 完善的防御性编程：新增的 _parse_hex_color 函数对非法颜色格式（如空值、错误格式、非字符串）进行了完善的捕获和 Fallback 处理，避免了因配置错误导致程序崩溃。
3. 测试覆盖充分：针对新增的颜色解析、样式透传、字段回退逻辑均编写了对应的单元测试，确保了代码变更的可靠性。

---

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

## 代码审查结果 - PR #1467 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（0个可选） 无 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 ### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **健壮的兼容性处理**：在 `generation_cover.py` 中，同时兼容了 `color` 和 `font_color` 字段，确保新旧配置格式均能正常工作，降低了升级风险。 2. **完善的防御性编程**：新增的 `_parse_hex_color` 函数对非法颜色格式（如空值、错误格式、非字符串）进行了完善的捕获和 Fallback 处理，避免了因配置错误导致程序崩溃。 3. **测试覆盖充分**：针对新增的颜色解析、样式透传、字段回退逻辑均编写了对应的单元测试，确保了代码变更的可靠性。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-23 06:49:49 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia merged commit 4de4b8dc08 into develop 2026-08-23 14:57:06 +08:00

gitea-actions bot commented 2026-08-23 14:58:48 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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