fix(worker): render_adapter分辨率读取 + ffmpeg错误捕获增强 #479

Merged

auto-approve-bot merged 3 commits from fix/render-resolution-from-config into develop 2026-07-17 17:31:26 +08:00

Conversation 2 Commits 3 Files Changed 4

+167 -5

xiaoxia commented 2026-07-17 17:01:21 +08:00

Owner

Copy Link

Copy Source

修复1：render_adapter从plan.config.export读取分辨率

问题

render_adapter 调用 UnifiedRenderService 时没有传 output_width/output_height，导致一直用默认 1280x720，忽略了 plan.config.export.resolution 里配置的 1080x1920。

根因

render_adapter.py 完全没有读取 plan.config.export 配置，UnifiedRenderService 默认值是 1280x720（横屏），与短视频竖屏场景不匹配。

修复

• 新增 _parse_resolution() 工具函数，解析 "1080x1920" 格式字符串
• render_plan() 从 plan.config.export.resolution 读取分辨率并传给 UnifiedRenderService
• 无配置时默认 1080x1920（与 config_schemas ExportConfig 默认值一致）

修复2：ffmpeg渲染失败时捕获stderr增强排障

问题

ffmpeg CalledProcessError 时，error_message 只有 exit code 和简短描述，没有 ffmpeg stderr，排障完全不知道具体错在哪。

修复

• RenderAdapterResult 新增 error_detail 字段存完整 stderr
• render_adapter 单独捕获 CalledProcessError，把 stderr 写进 error_message（前200字）和 error_detail（前3000字）
• edit_plan_generation 失败时把 stderr 拼进 gen_task.error_message，前端通过日志能看到
• 新增 test_ffmpeg_error_captures_stderr 测试

测试

• 35个 render_adapter 测试全绿
• black / isort 通过

## 修复1：render_adapter从plan.config.export读取分辨率 ### 问题 render_adapter 调用 UnifiedRenderService 时没有传 output_width/output_height，导致一直用默认 1280x720，忽略了 plan.config.export.resolution 里配置的 1080x1920。 ### 根因 render_adapter.py 完全没有读取 plan.config.export 配置，UnifiedRenderService 默认值是 1280x720（横屏），与短视频竖屏场景不匹配。 ### 修复 - 新增 _parse_resolution() 工具函数，解析 "1080x1920" 格式字符串 - render_plan() 从 plan.config.export.resolution 读取分辨率并传给 UnifiedRenderService - 无配置时默认 1080x1920（与 config_schemas ExportConfig 默认值一致） ## 修复2：ffmpeg渲染失败时捕获stderr增强排障 ### 问题 ffmpeg CalledProcessError 时，error_message 只有 exit code 和简短描述，没有 ffmpeg stderr，排障完全不知道具体错在哪。 ### 修复 - RenderAdapterResult 新增 error_detail 字段存完整 stderr - render_adapter 单独捕获 CalledProcessError，把 stderr 写进 error_message（前200字）和 error_detail（前3000字） - edit_plan_generation 失败时把 stderr 拼进 gen_task.error_message，前端通过日志能看到 - 新增 test_ffmpeg_error_captures_stderr 测试 ## 测试 - 35个 render_adapter 测试全绿 - black / isort 通过

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

fix(worker): render_adapter从plan.config.export读取分辨率，修复默认720p问题 ... e0bd51027d

- 新增_parse_resolution工具函数，解析1080x1920格式字符串
- render_plan从plan.config.export.resolution读取输出分辨率
- 无配置时默认1080x1920（与config_schemas默认值一致）
- 新增2个单元测试验证分辨率传递与默认值
- 34个render_adapter测试全绿

xiaoxia added 1 commit 2026-07-17 17:07:32 +08:00

feat(worker): ffmpeg渲染失败时捕获stderr写入error_detail，增强排障能力 ... 9b99c46d7d

- RenderAdapterResult新增error_detail字段存完整stderr
- render_adapter单独捕获CalledProcessError，把stderr写入error_message和error_detail
- edit_plan_generation失败时把error_detail拼进gen_task.error_message
- 新增test_ffmpeg_error_captures_stderr测试
- 35个render_adapter测试全绿

xiaoxia changed title from fix(worker): render_adapter从plan.config.export读取分辨率，修复默认720p问题 to fix(worker): render_adapter分辨率读取 + ffmpeg错误捕获增强 2026-07-17 17:07:46 +08:00

xiaoxia added 1 commit 2026-07-17 17:26:51 +08:00

perf(ci): ci-l1→ci-check 充分利用全部runner d365a0aa8f

auto-approve-bot approved these changes 2026-07-17 17:31:00 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-17 17:31:00 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 2a12532fb4 into develop 2026-07-17 17:31:26 +08:00

auto-approve-bot deleted branch fix/render-resolution-from-config 2026-07-17 17:31:26 +08:00

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

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#479