feat: 统一预览与确认生成渲染路径 — 预览=正式品质+确认复用 #1339

Merged

xiaoxia merged 7 commits from feature/unify-preview-confirm-render-path into develop 2026-08-11 06:55:02 +08:00

Conversation 5 Commits 7 Files Changed 13

+659 -378

xiaoxia commented 2026-08-10 22:32:50 +08:00

Owner

Copy Link

Copy Source

渲染管线统一

• UnifiedRenderService: 移除 is_preview，统一 CRF 23 + medium preset
• RenderAdapter: 移除 is_preview，统一执行校验和缩略图生成
• Worker generation.py: 移除 480p+1M 码率覆盖逻辑
• generation_preview.py: 删除 PREVIEW_RESOLUTION 常量

确认生成复用预览产物

• Domain: 新增 mark_confirmed() 方法
• confirm_generation: 预览已完成时直接复用（秒出）
• Editor generate: plan未修改时复用预览产物

测试

• 重写 test_1280_preview_speedup（验证统一品质）
• 重写 test_confirm_generation（验证复用逻辑）

## 渲染管线统一 - UnifiedRenderService: 移除 is_preview，统一 CRF 23 + medium preset - RenderAdapter: 移除 is_preview，统一执行校验和缩略图生成 - Worker generation.py: 移除 480p+1M 码率覆盖逻辑 - generation_preview.py: 删除 PREVIEW_RESOLUTION 常量 ## 确认生成复用预览产物 - Domain: 新增 mark_confirmed() 方法 - confirm_generation: 预览已完成时直接复用（秒出） - Editor generate: plan未修改时复用预览产物 ## 测试 - 重写 test_1280_preview_speedup（验证统一品质） - 重写 test_confirm_generation（验证复用逻辑）

xiaoxia added 1 commit 2026-08-10 22:32:50 +08:00

feat: 统一预览与确认生成渲染路径 — 预览=正式品质+确认复用 ... 29994ffcbc

## 渲染管线统一
- UnifiedRenderService: 移除 is_preview 参数，统一 CRF 23 + medium preset
- RenderAdapter: 移除 is_preview 参数，统一执行校验和缩略图生成
- Worker generation.py: 移除 480p+1M 码率覆盖逻辑
- generation_preview.py: 删除 PREVIEW_RESOLUTION 常量和 _calc_preview_resolution()

## 确认生成复用预览产物
- Domain: 新增 GenerationTask.mark_confirmed() 方法
- confirm_generation: 预览已完成时直接复用产物（秒出），无需重新渲染
- Editor generate: 检查 plan 关联的预览任务，plan未修改时复用产物
- Schema: CreatePreviewGenerationTaskRequest 新增 source_edit_plan_id 字段

## 测试更新
- test_1280_preview_speedup: 重写为验证统一品质参数
- test_confirm_generation: 重写为验证复用逻辑

## 交付标准
- 预览渲染质量与确认生成一致（1080p, CRF 23, medium）
- 预览后确认生成直接复用产物（秒出）
- Worker 端无任何 is_preview 低质量渲染代码残留

gitea-actions bot commented 2026-08-10 22:34:19 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-08-10 22:41:32 +08:00

fix: 移除已删除的 PREVIEW_RESOLUTION 引用及未使用变量 ... 5924751bff

- test_generation_preview.py: 移除 PREVIEW_RESOLUTION 导入和 TestCalcPreviewResolution 测试类
- render_adapter.py: 移除未使用的 has_voice_id 变量

auto-approve-bot approved these changes 2026-08-10 22:50:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-10 22:50:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia added 1 commit 2026-08-10 22:51:45 +08:00

ci: retrigger CI 0be5bc1837

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

fix: 修复统一渲染后遗留的测试引用 ... 0213be0428

- test_1294_preview_voice_injection: 移除 _render_video() 调用中的 is_preview 参数
- test_generation_preview: 更新分辨率断言（预览不再设置自定义分辨率）

xiaoxia added 1 commit 2026-08-10 23:36:46 +08:00

test: 补充编辑器预览产物复用逻辑单测 ... 1156061ba3

- _find_reusable_preview_task: 6 个用例（正常复用/无任务/未完成/plan已修改/非预览/异常）
- _get_task_output_url: 3 个用例（正常/空视频/异常）
- mark_confirmed: 5 个用例（is_preview/cover/title/dimensions）

xiaoxia added 1 commit 2026-08-10 23:54:48 +08:00

fix: 修复 _get_task_output_url 导入路径并更新测试 ed80fd7205

auto-approve-bot commented 2026-08-11 00:01:11 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：需修改
• 建议级问题数量：2 个

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

1. [apps/api/app/api/routes/generation_tasks.py: 372] 确认生成时未校验分辨率一致性

   • 问题类型：逻辑bug
   • 问题描述：在 confirm_generation 函数的复用逻辑中，当预览任务已完成时，代码直接调用 mark_confirmed 并复用产物，但未校验用户在确认时请求的 output_width / output_height 是否与预览任务实际渲染的分辨率一致。
   • 后果：如果用户在预览后修改了分辨率设置（例如从默认的 1080p 改为 720p）再点击确认，系统会忽略新的分辨率设置，直接返回 1080p 的预览视频，但将数据库中的 output_width 更新为 720p，导致数据不一致（视频文件实际是 1080p，元数据却是 720p），造成用户获取到错误的视频文件。
   • 修改建议：在复用预览产物前，增加分辨率校验逻辑。如果 request.output_width 或 request.output_height 与 source_task 中的实际值不符，应跳过复用逻辑，走新建任务流程以重新渲染。

2. [apps/api/app/api/routes/templates_editor/generation.py: 196] 时间比较可能引发空指针异常

   • 问题类型：逻辑bug
   • 问题描述：在 _find_reusable_preview_task 函数中，直接比较 plan_updated > completed_at。虽然 SQLAlchemy 通常会自动处理 updated_at，但如果 plan.updated_at 为 None（例如 Plan 模型配置允许为空或刚创建未刷新），此比较会抛出 TypeError。
   • 后果：导致模板编辑器生成逻辑在特定状态下崩溃，无法复用预览产物。
   • 修改建议：增加非空判断，修改为 if plan_updated and completed_at and plan_updated > completed_at:。

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

1. [apps/worker/video_processing/render_adapter.py: 508] ASR 服务初始化策略调整

   • 具体内容：代码移除了 is_preview 判断，现在无论是否有配音需求都会初始化 ASR 服务 (asr_service = self._get_asr_service())。如果 ASR 初始化涉及加载重型模型，这会导致所有渲染任务（包括无配音的纯视频任务）性能下降。建议保留原有的按需初始化逻辑，仅在检测到 voice_id 或 voiceover_audio_path 时才初始化 ASR。

2. [apps/api/app/api/routes/generation_tasks.py: 372] 复用产物前应校验视频文件存在性

   • 具体内容：在复用预览任务时，仅检查了 is_completed 状态。建议增加对 generated_videos 表的查询，确保任务确实有对应的视频输出文件 URL，避免因任务状态标记为“完成”但文件丢失（如存储故障）导致返回空链接给用户。

✅ 良好实践

• PR 的核心目标（预览与正式同质，实现秒出）逻辑清晰，通过移除 is_preview 分支统一了渲染路径，减少了维护成本。
• mark_confirmed 方法的设计合理，通过更新 is_preview 标志位有效防止了重复确认。
• generation_preview.py 中移除了硬编码的分辨率计算逻辑，改由 Worker 端根据模板配置决定，提高了灵活性。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-10 16:01:11 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：2 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 1. **[apps/api/app/api/routes/generation_tasks.py: 372] 确认生成时未校验分辨率一致性** - 问题类型：逻辑bug - 问题描述：在 `confirm_generation` 函数的复用逻辑中，当预览任务已完成时，代码直接调用 `mark_confirmed` 并复用产物，但未校验用户在确认时请求的 `output_width` / `output_height` 是否与预览任务实际渲染的分辨率一致。 - 后果：如果用户在预览后修改了分辨率设置（例如从默认的 1080p 改为 720p）再点击确认，系统会忽略新的分辨率设置，直接返回 1080p 的预览视频，但将数据库中的 `output_width` 更新为 720p，导致数据不一致（视频文件实际是 1080p，元数据却是 720p），造成用户获取到错误的视频文件。 - 修改建议：在复用预览产物前，增加分辨率校验逻辑。如果 `request.output_width` 或 `request.output_height` 与 `source_task` 中的实际值不符，应跳过复用逻辑，走新建任务流程以重新渲染。 2. **[apps/api/app/api/routes/templates_editor/generation.py: 196] 时间比较可能引发空指针异常** - 问题类型：逻辑bug - 问题描述：在 `_find_reusable_preview_task` 函数中，直接比较 `plan_updated > completed_at`。虽然 SQLAlchemy 通常会自动处理 `updated_at`，但如果 `plan.updated_at` 为 `None`（例如 Plan 模型配置允许为空或刚创建未刷新），此比较会抛出 `TypeError`。 - 后果：导致模板编辑器生成逻辑在特定状态下崩溃，无法复用预览产物。 - 修改建议：增加非空判断，修改为 `if plan_updated and completed_at and plan_updated > completed_at:`。 ### 💡 改进建议（不阻塞合并） 1. **[apps/worker/video_processing/render_adapter.py: 508] ASR 服务初始化策略调整** - 具体内容：代码移除了 `is_preview` 判断，现在无论是否有配音需求都会初始化 ASR 服务 (`asr_service = self._get_asr_service()`)。如果 ASR 初始化涉及加载重型模型，这会导致所有渲染任务（包括无配音的纯视频任务）性能下降。建议保留原有的按需初始化逻辑，仅在检测到 `voice_id` 或 `voiceover_audio_path` 时才初始化 ASR。 2. **[apps/api/app/api/routes/generation_tasks.py: 372] 复用产物前应校验视频文件存在性** - 具体内容：在复用预览任务时，仅检查了 `is_completed` 状态。建议增加对 `generated_videos` 表的查询，确保任务确实有对应的视频输出文件 URL，避免因任务状态标记为“完成”但文件丢失（如存储故障）导致返回空链接给用户。 ### ✅ 良好实践 - PR 的核心目标（预览与正式同质，实现秒出）逻辑清晰，通过移除 `is_preview` 分支统一了渲染路径，减少了维护成本。 - `mark_confirmed` 方法的设计合理，通过更新 `is_preview` 标志位有效防止了重复确认。 - `generation_preview.py` 中移除了硬编码的分辨率计算逻辑，改由 Worker 端根据模板配置决定，提高了灵活性。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-10 16:01:11 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia added 1 commit 2026-08-11 06:44:35 +08:00

fix: AI Code Review 阻塞级问题修复 — 分辨率一致性校验+空指针防护 142d4331cc

xiaoxia merged commit 1096a6a489 into develop 2026-08-11 06:55:02 +08:00

gitea-actions bot commented 2026-08-11 06:55:53 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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