fix: 标题/换行 + TTS replace模式静音原音 + 配音素材库静音原音 #1504

Merged

auto-approve-bot merged 2 commits from fix/title-slash-newline-and-replace-mute into develop 2026-08-25 23:06:00 +08:00

Conversation 5 Commits 2 Files Changed 5

+82

xiaoxia commented 2026-08-25 22:25:53 +08:00

Owner

Copy Link

Copy Source

修复内容

1. 标题 / 和 ／ 换行

文件: packages/domain/ass_subtitle_builder.py

• 在 escape_ass_text() 中，将半角 / 和全角 ／ 转换为 ASS 硬换行 \N
• 在自动换行 _wrap_title_text() 之前完成替换，用户手动换行优先于自动换行

2. TTS replace 模式静音原视频音轨

文件: apps/worker/video_processing/unified_render_service.py

• _maybe_add_voiceover_layer()：TTS 配音添加成功后，若 overlap_mode == "replace"，遍历 main 和 broll 图层所有 clip，将 config["volume"] 设为 0
• 修复了之前注释 replace 模式在外部处理 但实际逻辑缺失的问题

3. 配音素材库默认静音原音

• _maybe_add_voice_library_layer()：配音素材库音频添加后，同样静音 main 和 broll 图层（配音素材库场景默认替换原音）

测试

• 新增 7 个单元测试覆盖斜杠换行（半角/全角/混合）
• 356 个相关测试全部通过（含 render/tts/voiceover 全套）

## 修复内容 ### 1. 标题 / 和 ／ 换行 **文件**: `packages/domain/ass_subtitle_builder.py` - 在 `escape_ass_text()` 中，将半角 `/` 和全角 `／` 转换为 ASS 硬换行 `\N` - 在自动换行 `_wrap_title_text()` 之前完成替换，用户手动换行优先于自动换行 ### 2. TTS replace 模式静音原视频音轨 **文件**: `apps/worker/video_processing/unified_render_service.py` - `_maybe_add_voiceover_layer()`：TTS 配音添加成功后，若 `overlap_mode == "replace"`，遍历 `main` 和 `broll` 图层所有 clip，将 `config["volume"]` 设为 0 - 修复了之前注释 `replace 模式在外部处理` 但实际逻辑缺失的问题 ### 3. 配音素材库默认静音原音 - `_maybe_add_voice_library_layer()`：配音素材库音频添加后，同样静音 `main` 和 `broll` 图层（配音素材库场景默认替换原音） ### 测试 - 新增 7 个单元测试覆盖斜杠换行（半角/全角/混合） - 356 个相关测试全部通过（含 render/tts/voiceover 全套）

xiaoxia added 1 commit 2026-08-25 22:25:53 +08:00

fix: title slash newline + TTS/voice-library replace mode mutes original audio ... 5b76f500f2

1. escape_ass_text: convert / and ／ to ASS hard newline \\N before
   automatic wrapping, so user manual line breaks take priority.
2. _maybe_add_voiceover_layer: when overlap_mode=='replace', set
   volume=0 on all main/broll layer clips after adding TTS audio.
3. _maybe_add_voice_library_layer: always mute main/broll clips when
   voice library audio is added (voice library replaces original audio).
4. Add 7 new unit tests covering slash conversion and integration.

gitea-actions bot commented 2026-08-25 22:35:53 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-25 22:37:57 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 6aad1c2556

auto-approve-bot commented 2026-08-25 22:41:34 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1504

⚠️ 问题（0个需要修改）

💡 建议（2个可选）

1. apps/worker/video_processing/unified_render_service.py 第829-833行及902-906行：存在重复代码逻辑。
   • 说明：静音 main 和 broll 图层音轨的逻辑在两处完全一致（TTS replace 模式和配音素材库模式）。建议提取为私有方法（如 _mute_original_audio_layers）以提高代码可维护性并避免逻辑不一致。
2. packages/domain/ass_subtitle_builder.py 第150行：斜杠转换逻辑可能影响非标题文本。
   • 说明：将所有 / 和 ／ 替换为 \N 适用于标题换行，但如果此通用函数被用于普通对话字幕（如包含 URL https://... 或分数 1/2），会导致内容显示错误。建议确认函数调用范围，或增加参数控制是否进行此转换，避免破坏普通文本内容。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-25 14:41:34 | 模型:}

## 代码审查结果 - PR #1504 ### ⚠️ 问题（0个需要修改） ### 💡 建议（2个可选） 1. **apps/worker/video_processing/unified_render_service.py 第829-833行及902-906行**：存在重复代码逻辑。 - **说明**：静音 `main` 和 `broll` 图层音轨的逻辑在两处完全一致（TTS replace 模式和配音素材库模式）。建议提取为私有方法（如 `_mute_original_audio_layers`）以提高代码可维护性并避免逻辑不一致。 2. **packages/domain/ass_subtitle_builder.py 第150行**：斜杠转换逻辑可能影响非标题文本。 - **说明**：将所有 `/` 和 `／` 替换为 `\N` 适用于标题换行，但如果此通用函数被用于普通对话字幕（如包含 URL `https://...` 或分数 `1/2`），会导致内容显示错误。建议确认函数调用范围，或增加参数控制是否进行此转换，避免破坏普通文本内容。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-25 14:41:34 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-08-25 23:00:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-25 23:00:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 323bd2da5e into develop 2026-08-25 23:06:00 +08:00

auto-approve-bot deleted branch fix/title-slash-newline-and-replace-mute 2026-08-25 23:06:01 +08:00

gitea-actions bot commented 2026-08-25 23:06:56 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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