xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 16 Pull Requests 8 Actions 34.2k Packages Projects Releases 7 Wiki Activity

[P2] Worker _pure.py 文件清理与迁移 #1833

New Issue
Closed
opened 2026-09-10 12:07:09 +08:00 by xiaoxia · 1 comment
xiaoxia commented 2026-09-10 12:07:09 +08:00
Owner
Copy Link
Copy Source

背景

apps/worker/video_processing/ 下 5 个 _pure 后缀文件只被同名测试引用，生产代码全部使用非 _pure 版本。

文件清单

_pure 文件 测试引用 生产代码引用
bgm_mixer_pure.py test_bgm_mixer_pure.py 无
concat_engine_pure.py test_concat_engine_pure.py 无
multi_track_mixer_pure.py test_multi_track_mixer_pure.py 无
pip_engine_pure.py test_pip_engine_pure.py 无
sticker_engine_pure.py test_sticker_engine_pure.py 无

处理方案

  1. 确认 _pure 版本的用途（是否为纯函数提取的重构实验）
  2. 如已确认非 _pure 版本为最终实现，删除 _pure 文件及对应测试
  3. 如 _pure 版本包含有价值的纯函数实现，将其集成到生产代码中

验收标准

  • _pure 文件有明确去留决策并执行
  • 对应测试文件同步处理
  • CI 全绿
## 背景 apps/worker/video_processing/ 下 5 个 _pure 后缀文件只被同名测试引用，生产代码全部使用非 _pure 版本。 ## 文件清单 | _pure 文件 | 测试引用 | 生产代码引用 | |-----------|---------|------------| | bgm_mixer_pure.py | test_bgm_mixer_pure.py | 无 | | concat_engine_pure.py | test_concat_engine_pure.py | 无 | | multi_track_mixer_pure.py | test_multi_track_mixer_pure.py | 无 | | pip_engine_pure.py | test_pip_engine_pure.py | 无 | | sticker_engine_pure.py | test_sticker_engine_pure.py | 无 | ## 处理方案 1. 确认 _pure 版本的用途（是否为纯函数提取的重构实验） 2. 如已确认非 _pure 版本为最终实现，删除 _pure 文件及对应测试 3. 如 _pure 版本包含有价值的纯函数实现，将其集成到生产代码中 ## 验收标准 - _pure 文件有明确去留决策并执行 - 对应测试文件同步处理 - CI 全绿
xiaoxia added the backendrefactorcleanup labels 2026-09-10 12:07:09 +08:00
xiaoxia commented 2026-09-15 18:00:53 +08:00
Author
Owner
Copy Link
Copy Source

本轮未推进，关闭。Worker _pure.py 清理后续重构时新开 Issue。

本轮未推进，关闭。Worker _pure.py 清理后续重构时新开 Issue。
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
architecture
架构设计
 backend
后端相关
 blocked
阻塞中
 bug
Bug修复
 ci-bug
CI本身的bug
 ci-optimization
CI优化类任务
 ci-pipeline
CI流水线/workflow相关
 cleanup
代码清理
 devops
运维/部署/CI
 documentation
文档相关
 feature
新功能
 frontend
前端相关
 P0
阻塞性，必须立即处理
 P1
高优先级
 P2
中优先级
 P3
低优先级/后考虑
 question
需要讨论确认
 ready
PR CI通过，可合并
 refactor
代码重构
No Label backend cleanup refactor
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
auto-approve-bot frontend-dev xiaoxia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1833
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?