fix(render): 画中画模式全面下线 + 标题ASS自动换行 #1239

Merged

xiaoxia merged 3 commits from fix/pip-removal-and-title-wrap into develop 2026-08-04 18:36:19 +08:00

Conversation 3 Commits 3 Files Changed 7

+153 -63

xiaoxia commented 2026-08-04 17:48:04 +08:00

Owner

Copy Link

Copy Source

Q1 — 画中画(PiP)模式全面下线

产品已确认全面下线画中画，pip/voice_pip 统一走 one_take（顺序拼接）。

改动：

• apps/worker/worker_app/tasks/generation.py：_build_plan_and_clips_from_task 移除 pip/voice_pip 分支，入口处强制映射为 one_take
• apps/api/app/api/routes/generation_preview.py：_resolve_strategy_id_from_template 返回前做 pip→one_take 映射
• apps/api/app/api/routes/generation_tasks.py：strategy_id 中的 pip/voice_pip 统一映射

Q2 — 标题 ASS 自动换行

标题超出画面宽度时自动换行，防止溢出。

改动：

• packages/domain/ass_subtitle_builder.py：新增 _wrap_title_text 函数
  • 根据视频宽度(PlayResX)、字号、边距估算可用宽度
  • CJK 字符按全角(font_size)、英文/数字按半角(font_size×0.55)估算
  • 超出可用宽度时插入 \N (ASS 硬换行)

测试

• 75 个 ASS 字幕单测全绿（含 7 个新增换行测试）
• 10 个四模式集成测试全绿（pip/voice_pip 改为验证 one_take 行为）

## Q1 — 画中画(PiP)模式全面下线 产品已确认全面下线画中画，pip/voice_pip 统一走 one_take（顺序拼接）。 **改动：** - `apps/worker/worker_app/tasks/generation.py`：`_build_plan_and_clips_from_task` 移除 pip/voice_pip 分支，入口处强制映射为 one_take - `apps/api/app/api/routes/generation_preview.py`：`_resolve_strategy_id_from_template` 返回前做 pip→one_take 映射 - `apps/api/app/api/routes/generation_tasks.py`：strategy_id 中的 pip/voice_pip 统一映射 ## Q2 — 标题 ASS 自动换行 标题超出画面宽度时自动换行，防止溢出。 **改动：** - `packages/domain/ass_subtitle_builder.py`：新增 `_wrap_title_text` 函数 - 根据视频宽度(PlayResX)、字号、边距估算可用宽度 - CJK 字符按全角(font_size)、英文/数字按半角(font_size×0.55)估算 - 超出可用宽度时插入 `\N` (ASS 硬换行) ## 测试 - 75 个 ASS 字幕单测全绿（含 7 个新增换行测试） - 10 个四模式集成测试全绿（pip/voice_pip 改为验证 one_take 行为）

xiaoxia added 1 commit 2026-08-04 17:48:04 +08:00

fix(render): 画中画模式全面下线 + 标题ASS自动换行 ... c6a28ea7cd

Q1 — 画中画(PiP)下线：
- worker: _build_plan_and_clips_from_task 移除pip/voice_pip分支，
  入口处强制映射为one_take（顺序拼接）
- api preview: _resolve_strategy_id_from_template 返回前做pip→one_take映射
- api generation_tasks: strategy_id中的pip/voice_pip统一映射为one_take
- 更新集成测试：pip/voice_pip测试改为验证one_take行为

Q2 — 标题自动换行：
- packages/domain/ass_subtitle_builder.py: 新增_wrap_title_text函数
  根据视频宽度、字号、边距估算每行最大字符数，CJK全角+英文半角
  超出可用宽度时插入\N硬换行
- build_ass_content中title Dialogue生成前调用自动换行

测试：75个ASS单测全绿 + 10个四模式测试全绿

gitea-actions bot commented 2026-08-04 17:50:19 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-08-04 18:00:03 +08:00

test: update generation_preview tests for pip→one_take mapping ... 4f5a1d8eff

- test_new_template_found_returns_editing_mode: use 'standard' mode
- test_voice_pip_mode: expect 'one_take' (pip removed, mapped to one_take)
- test_strategy_id_from_new_template: use 'standard' mode

xiaoxia added 1 commit 2026-08-04 18:27:48 +08:00

fix: swap escape_ass_text and _wrap_title_text order ... 32bc11e2c1

先 escape 特殊字符再插入 ASS 换行符 \N，消除 AI Code Review
对 \N 被转义为 \\N 的顾虑（虽然当前 escape_ass_text 不转义
反斜杠，但调换顺序后逻辑更安全清晰）。

auto-approve-bot commented 2026-08-04 18:31:56 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：通过
• 建议级问题数量：0 个

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

无

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

无

✅ 良好实践

1. 防御性编程：在 generation_tasks.py 和 generation_preview.py 中，对已下线的 pip/voice_pip 模式进行了统一映射处理，确保了向后兼容性，避免了因旧数据或旧客户端请求导致的系统错误。
2. 逻辑一致性：Worker 端 (generation.py) 和 API 端同时处理了模式映射，确保了整个链路的数据一致性。
3. 安全性：在 ass_subtitle_builder.py 中，正确处理了转义与换行符插入的顺序（先转义特殊字符，再插入 \N），有效防止了 ASS 注入风险，同时保证了换行功能正常。
4. 测试覆盖：不仅更新了单元测试以反映新的逻辑行为，还保留了针对旧模式映射的测试用例（如 test_pip_mode_maps_to_one_take），确保了代码变更的可靠性。
5. 代码整洁：移除了 generation.py 中不再需要的 pip 和 voice_pip 分支逻辑，简化了代码结构，降低了维护成本。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **防御性编程**：在 `generation_tasks.py` 和 `generation_preview.py` 中，对已下线的 `pip/voice_pip` 模式进行了统一映射处理，确保了向后兼容性，避免了因旧数据或旧客户端请求导致的系统错误。 2. **逻辑一致性**：Worker 端 (`generation.py`) 和 API 端同时处理了模式映射，确保了整个链路的数据一致性。 3. **安全性**：在 `ass_subtitle_builder.py` 中，正确处理了转义与换行符插入的顺序（先转义特殊字符，再插入 `\N`），有效防止了 ASS 注入风险，同时保证了换行功能正常。 4. **测试覆盖**：不仅更新了单元测试以反映新的逻辑行为，还保留了针对旧模式映射的测试用例（如 `test_pip_mode_maps_to_one_take`），确保了代码变更的可靠性。 5. **代码整洁**：移除了 `generation.py` 中不再需要的 `pip` 和 `voice_pip` 分支逻辑，简化了代码结构，降低了维护成本。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-04 10:31:56 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 004de97394 into develop 2026-08-04 18:36:19 +08:00

gitea-actions bot commented 2026-08-04 18:36:49 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1239 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

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#1239