P1-2: 剪辑计划统一渲染链路打通 - 收敛RenderAdapter + 补齐BGM和ASR自动字幕 #442

Merged

xiaoxia merged 2 commits from feature/p1-2-edit-plan-render-link into develop 2026-07-17 12:31:05 +08:00

Conversation 1 Commits 2 Files Changed 3

+711 -153

xiaoxia commented 2026-07-17 11:46:57 +08:00

Owner

Copy Link

Copy Source

改动说明

修复 Issue #408 P1-2 剪辑计划→渲染链路的 3 个 P0 断点：

1. BGM 未接入 ✅

• RenderAdapter 新增 _prepare_bgm 方法
• 从 plan.config.bgm 读取配置，支持 URL/素材库/预设库三种来源
• 优先级：audio_url > asset_id > preset_id
• 下载失败降级，不阻断主流程

2. ASR 自动字幕未接入 ✅

• RenderAdapter 新增 _get_asr_service 方法
• 调用 UnifiedRenderService 时传入 asr_service
• 初始化失败降级，不阻断主流程

3. 代码重复 ✅

• edit_plan_generation 的 unified 路径从直接调 UnifiedRenderService 改为通过 RenderAdapter
• 消除与 compose_video 的重复代码（素材下载+渲染+上传）
• compose_video 自动获得 BGM+ASR 能力

测试

• 新增/修改 31 个 RenderAdapter 单元测试，全部通过
• edit_plan_worker_failure 6 个测试通过
• unified_render_service 87 个测试通过
• 合计 124 个相关测试全绿

兼容性

• 默认关闭：BGM 需 plan.config.bgm.enabled=true 才启用
• 向后兼容：无 BGM/ASR 时行为与之前完全一致
• 失败降级：BGM 或 ASR 任一失败都不阻断渲染主流程

## 改动说明 修复 Issue #408 P1-2 剪辑计划→渲染链路的 3 个 P0 断点： ### 1. BGM 未接入 ✅ - RenderAdapter 新增 _prepare_bgm 方法 - 从 plan.config.bgm 读取配置，支持 URL/素材库/预设库三种来源 - 优先级：audio_url > asset_id > preset_id - 下载失败降级，不阻断主流程 ### 2. ASR 自动字幕未接入 ✅ - RenderAdapter 新增 _get_asr_service 方法 - 调用 UnifiedRenderService 时传入 asr_service - 初始化失败降级，不阻断主流程 ### 3. 代码重复 ✅ - edit_plan_generation 的 unified 路径从直接调 UnifiedRenderService 改为通过 RenderAdapter - 消除与 compose_video 的重复代码（素材下载+渲染+上传） - compose_video 自动获得 BGM+ASR 能力 ## 测试 - 新增/修改 31 个 RenderAdapter 单元测试，全部通过 - edit_plan_worker_failure 6 个测试通过 - unified_render_service 87 个测试通过 - 合计 124 个相关测试全绿 ## 兼容性 - 默认关闭：BGM 需 plan.config.bgm.enabled=true 才启用 - 向后兼容：无 BGM/ASR 时行为与之前完全一致 - 失败降级：BGM 或 ASR 任一失败都不阻断渲染主流程

xiaoxia added 1 commit 2026-07-17 11:46:57 +08:00

P1-2: 剪辑计划统一渲染链路打通 - 收敛RenderAdapter + 补齐BGM和ASR自动字幕 ... 6bdab42224

## 改动内容

### 1. RenderAdapter 增强（render_adapter.py）
- 新增 BGM 准备逻辑：支持 URL/素材库/预设库三种来源，失败降级不阻断
- 新增 ASR 服务初始化：自动字幕支持，初始化失败降级
- _download_assets 返回值扩展为 (asset_map, rendered_ids, failed_ids)
- RenderAdapterResult 增加 rendered_clip_ids / failed_clip_ids 字段
- render_plan 调用 UnifiedRenderService 时传入 bgm_path 和 asr_service

### 2. 剪辑计划渲染收敛（edit_plan_generation.py）
- _render_with_unified 从直接调 UnifiedRenderService 改为通过 RenderAdapter
- 消除与 compose_video 的重复代码（素材下载+渲染+上传）
- 主流程按引擎分支：unified 走 RenderAdapter，legacy 保留原逻辑
- 移除顶部不再使用的 UnifiedRenderService 直接导入

### 3. 单元测试（test_render_adapter.py）
- 修复 _download_assets 返回值变化的测试
- 新增 TestPrepareBgm：7 个用例覆盖三种来源+优先级+失败降级
- 新增 TestGetAsrService：3 个用例覆盖可用/导入失败/初始化失败
- 新增 TestRenderPlanWithBgmAsr：3 个集成用例
- 共 31 个测试全部通过

## 三个 P0 断点修复
✅ BGM 未接入 → RenderAdapter._prepare_bgm 自动从 plan.config.bgm 读取
✅ ASR 自动字幕未接入 → RenderAdapter._get_asr_service 自动初始化
✅ 代码重复 → edit_plan_generation 与 compose_video 统一走 RenderAdapter

xiaoxia added 1 commit 2026-07-17 12:21:05 +08:00

fix: black format code quality issues d1627d5aaf

auto-approve-bot reviewed 2026-07-17 12:24:03 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia merged commit 60699b0b0c into develop 2026-07-17 12:31:05 +08:00

xiaoxia referenced this pull request 2026-07-17 12:58:03 +08:00

【P1-2】打通剪辑计划→渲染链路 #408

xiaoxia referenced this pull request 2026-07-20 15:40:56 +08:00

【P1-1】统一渲染引擎效果层完整验证 #407

Sign in to join this conversation.

Reviewers

No Reviewers

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

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#442