xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 11 Pull Requests 3 Actions Packages Projects Releases 7 Wiki Activity

【架构清理】删除legacy渲染引擎，统一走unified引擎 #599

New Issue
Closed
opened 2026-07-19 17:15:09 +08:00 by xiaoxia · 0 comments
xiaoxia commented 2026-07-19 17:15:09 +08:00
Owner
Copy Link
Copy Source

背景

当前存在两套渲染引擎（legacy FFmpeg filter_complex + unified分层渲染），维护成本高，出问题排查困难。

原则

一个软件，一套引擎。越简单越好，维护成本越低越好。

改动范围

后端删除项

  • _render_with_legacy_engine() 函数（generation.py）
  • render_engine_resolver.py 整个feature flag切换层
  • RenderEngineResolver 相关的feature flag代码
  • 所有ENGINE_LEGACY / ENGINE_UNIFIED的判断分支
  • 相关单元测试中的legacy mock

统一为unified

  • generation.py 渲染部分直接走 unified 路径
  • edit_plan_generation.py 同样统一
  • 删除引擎选择逻辑，直接调用RenderAdapter

风险

  • 先确认unified引擎覆盖了所有legacy的功能（直通模式、水印、调速等）
  • 统一前确保unified在staging跑通全量E2E

优先级

P1。架构债务，越早清越好，后续所有渲染相关改动都基于单引擎做。

## 背景 当前存在两套渲染引擎（legacy FFmpeg filter_complex + unified分层渲染），维护成本高，出问题排查困难。 ## 原则 **一个软件，一套引擎。越简单越好，维护成本越低越好。** ## 改动范围 ### 后端删除项 - _render_with_legacy_engine() 函数（generation.py） - render_engine_resolver.py 整个feature flag切换层 - RenderEngineResolver 相关的feature flag代码 - 所有ENGINE_LEGACY / ENGINE_UNIFIED的判断分支 - 相关单元测试中的legacy mock ### 统一为unified - generation.py 渲染部分直接走 unified 路径 - edit_plan_generation.py 同样统一 - 删除引擎选择逻辑，直接调用RenderAdapter ### 风险 - 先确认unified引擎覆盖了所有legacy的功能（直通模式、水印、调速等） - 统一前确保unified在staging跑通全量E2E ## 优先级 P1。架构债务，越早清越好，后续所有渲染相关改动都基于单引擎做。
xiaoxia added the backendP1 labels 2026-07-19 17:15:19 +08:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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 P1 backend
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
auto-approve-bot xiaoxia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#599
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?