refactor(#1341): 删除 EditingPlanner 遗留封面生成器 + 清理 compose_video 封面代码 #1353

Merged

auto-approve-bot merged 1 commits from refactor/remove-legacy-cover-generator into develop 2026-08-13 01:16:00 +08:00

Conversation 5 Commits 1 Files Changed 5

-2092

xiaoxia commented 2026-08-13 01:02:15 +08:00

Owner

Copy Link

Copy Source

改动

删除旧封面生成器

• 删除 apps/worker/video_processing/cover_generator.py（431行）
• 这是 EditingPlanner 时代的封面逻辑，已被 Step6 → MediaKit 链路完全替代

清理 compose_video.py

• 移除 _compose_with_unified_engine() 中的封面生成代码块
• 删除 cover_url 变量和 result_data 中的 cover_url 字段
• compose_video 只负责视频渲染，封面由独立 API 处理

删除关联测试

• test_cover_generator_pure.py — 测试已删除的 cover_generator
• test_cover_reverse_sticker.py — 测试已删除的 cover_generator
• test_compose_video_cover_integration.py — 测试 compose_video 中已删除的封面集成

保留的唯一封面链路

Step6 前端 → POST /generate-cover → cover.py → ai_service.py → MediaKit AI 抽帧 → 返回封面图

测试

• 全量单测 13563 passed（排除无关的 version 测试）
• 净减 ~2100 行代码

## 改动 ### 删除旧封面生成器 - 删除 `apps/worker/video_processing/cover_generator.py`（431行） - 这是 EditingPlanner 时代的封面逻辑，已被 Step6 → MediaKit 链路完全替代 ### 清理 compose_video.py - 移除 `_compose_with_unified_engine()` 中的封面生成代码块 - 删除 `cover_url` 变量和 result_data 中的 `cover_url` 字段 - compose_video 只负责视频渲染，封面由独立 API 处理 ### 删除关联测试 - `test_cover_generator_pure.py` — 测试已删除的 cover_generator - `test_cover_reverse_sticker.py` — 测试已删除的 cover_generator - `test_compose_video_cover_integration.py` — 测试 compose_video 中已删除的封面集成 ### 保留的唯一封面链路 ``` Step6 前端 → POST /generate-cover → cover.py → ai_service.py → MediaKit AI 抽帧 → 返回封面图 ``` ## 测试 - 全量单测 13563 passed（排除无关的 version 测试） - 净减 ~2100 行代码

xiaoxia added 1 commit 2026-08-13 01:02:15 +08:00

refactor(#1341): 删除 EditingPlanner 时代遗留的封面生成器 ... b99146f353

- 删除 cover_generator.py（431行），旧封面逻辑已不再使用
- compose_video.py 移除封面生成代码块，智能剪辑封面由 Step6 前端
  通过 /generate-cover API → ai_service.py → MediaKit 独立处理
- 删除关联的 3 个测试文件（test_cover_generator_pure,
  test_cover_reverse_sticker, test_compose_video_cover_integration）
- 保留唯一链路: Step6 → /generate-cover → MediaKit AI 抽帧

auto-approve-bot commented 2026-08-13 01:05:23 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

1. [apps/worker/worker_app/tasks/compose_video.py:175] 接口兼容性确认
   • 具体内容：代码移除了 result_data 中的 cover_url 字段。请确认 job_service.complete_job 的下游消费者（如 API 接口、前端或其他 Worker）是否强依赖该字段。如果下游逻辑未同步修改，可能会导致 KeyError 或业务逻辑异常。建议确认此次变更为功能废弃，并已同步更新相关调用方。

✅ 良好实践

• 代码清理彻底：删除了不再使用的 cover_generator.py 模块及其相关单元测试，保持了代码库的整洁。
• 逻辑移除干净：compose_video.py 中移除了封面生成的调用逻辑及结果字段，避免了遗留的死代码。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无问题

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/worker/worker_app/tasks/compose_video.py:175] 接口兼容性确认** - 具体内容：代码移除了 `result_data` 中的 `cover_url` 字段。请确认 `job_service.complete_job` 的下游消费者（如 API 接口、前端或其他 Worker）是否强依赖该字段。如果下游逻辑未同步修改，可能会导致 `KeyError` 或业务逻辑异常。建议确认此次变更为功能废弃，并已同步更新相关调用方。 ### ✅ 良好实践 - 代码清理彻底：删除了不再使用的 `cover_generator.py` 模块及其相关单元测试，保持了代码库的整洁。 - 逻辑移除干净：`compose_video.py` 中移除了封面生成的调用逻辑及结果字段，避免了遗留的死代码。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-12 17:05:23 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

gitea-actions bot commented 2026-08-13 01:06:05 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot approved these changes 2026-08-13 01:10:29 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-13 01:10:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 52eb37472d into develop 2026-08-13 01:16:00 +08:00

auto-approve-bot deleted branch refactor/remove-legacy-cover-generator 2026-08-13 01:16:01 +08:00

gitea-actions bot commented 2026-08-13 01:17:09 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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