fix: 预览视频3问题修复 — URL签名/时长约束/分辨率 (#1209) #1229

Merged

xiaoxia merged 11 commits from fix/preview-generation-3-issues into develop 2026-08-04 00:21:41 +08:00

Conversation 3 Commits 11 Files Changed 4

+411 -28

xiaoxia commented 2026-08-03 22:46:12 +08:00

Owner

Copy Link

Copy Source

修复内容

P0 - 预览视频黑屏

• 根因: generation_preview.py 的 _to_preview_response 返回裸 OSS URL，私有 bucket 403 → 黑屏
• 修复: 新增 _sign_video_url() 函数，返回前调用 get_storage_service().get_download_url() 生成预签名 URL（有效期 2 小时），签名失败降级返回原始 URL

P1 - 预览视频时长不符合模板

• 根因: _build_plan_and_clips_from_task() 中 duration=path_duration[p] 用了素材完整时长，模板 segment 的 duration_max 约束没生效
• 修复: 新增 _load_template_segment_durations() 读取模板 segment 的 duration_max 列表，在 clip 创建后对每个 clip 取 min(素材时长, segment.duration_max)

P1 - 预览分辨率横屏应竖屏

• 根因: _render_video() 中 is_preview=True 时硬编码 "854x480" 横屏，忽略了前端传入的 video_ratio 参数
• 修复: 预览模式优先使用传入的 resolution 参数（如 "480x854" 竖屏），未传入时才降级为默认 "854x480"

测试

• 新增 tests/unit/test_preview_generation_fixes.py，覆盖 URL 签名(5)、时长约束(6)、分辨率(7) = 18 个测试

验收标准

• staging 部署后，模板"718测试"预览：视频可播放、时长≈9s、竖屏 9:16

## 修复内容 ### P0 - 预览视频黑屏 - **根因**: `generation_preview.py` 的 `_to_preview_response` 返回裸 OSS URL，私有 bucket 403 → 黑屏 - **修复**: 新增 `_sign_video_url()` 函数，返回前调用 `get_storage_service().get_download_url()` 生成预签名 URL（有效期 2 小时），签名失败降级返回原始 URL ### P1 - 预览视频时长不符合模板 - **根因**: `_build_plan_and_clips_from_task()` 中 `duration=path_duration[p]` 用了素材完整时长，模板 segment 的 `duration_max` 约束没生效 - **修复**: 新增 `_load_template_segment_durations()` 读取模板 segment 的 `duration_max` 列表，在 clip 创建后对每个 clip 取 `min(素材时长, segment.duration_max)` ### P1 - 预览分辨率横屏应竖屏 - **根因**: `_render_video()` 中 `is_preview=True` 时硬编码 `"854x480"` 横屏，忽略了前端传入的 `video_ratio` 参数 - **修复**: 预览模式优先使用传入的 `resolution` 参数（如 `"480x854"` 竖屏），未传入时才降级为默认 `"854x480"` ## 测试 - 新增 `tests/unit/test_preview_generation_fixes.py`，覆盖 URL 签名(5)、时长约束(6)、分辨率(7) = 18 个测试 ## 验收标准 - staging 部署后，模板"718测试"预览：视频可播放、时长≈9s、竖屏 9:16

xiaoxia added 4 commits 2026-08-03 22:46:12 +08:00

fix: 预览视频URL签名 — 私有bucket返回预签名URL避免403黑屏 a9d55dc2cf

fix: 预览视频URL签名 — 私有bucket返回预签名URL避免403黑屏 2788d00593

fix: 预览视频时长约束(模板segment duration_max) + 分辨率尊重video_ratio 4861d298cc

test: 预览生成3个修复的单元测试(URL签名/时长约束/分辨率) 3b90da1050

gitea-actions bot commented 2026-08-03 22:47:38 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-03 22:48:01 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 11fa1f31da

xiaoxia added 1 commit 2026-08-03 23:08:46 +08:00

test: 更新预览测试 — URL签名mock + 分辨率逻辑匹配修复后行为 6800ee31a3

xiaoxia added 1 commit 2026-08-03 23:09:01 +08:00

chore: 移除独立测试文件（已合并到 test_generation_preview.py） b8e466eb75

auto-approve-bot added 1 commit 2026-08-03 23:10:40 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 91364f15d5

xiaoxia added 1 commit 2026-08-03 23:25:58 +08:00

fix: URL签名降级逻辑 — bucket未配置时返回原始URL而非构造URL 9808a6eef9

xiaoxia added 1 commit 2026-08-03 23:27:03 +08:00

fix: 修复TestToPreviewResponse URL签名mock + 匹配降级逻辑 b9566c7a93

xiaoxia added 1 commit 2026-08-04 00:09:55 +08:00

test: generation.py worker侧修复的单测(时长约束+segment加载) d494f368d0

auto-approve-bot commented 2026-08-04 00:13:53 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：需修改
• 建议级问题数量：0 个

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

1. [tests/unit/test_generation_preview.py: 1064] 测试引用了不存在的函数
   • 问题类型：语法错误 / 逻辑bug
   • 问题描述：新增的测试类 TestCalcPreviewResolution（第1064行开始）中，测试方法尝试从 app.api.routes.generation_preview 导入并调用 _calc_preview_resolution 函数。然而，在 apps/api/app/api/routes/generation_preview.py 的代码变更中，并未定义或引入该函数（仅新增了 _sign_video_url）。这将导致测试运行时出现 ImportError 或 AttributeError，测试无法通过。
   • 修改建议：请确认 _calc_preview_resolution 函数是否遗漏在本次代码提交中，或者该测试类属于错误的文件/PR。如果该逻辑应在 API 层实现，请补充函数定义；如果该逻辑仅在 Worker 层（generation.py）实现，请删除或调整该测试类。

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

无

✅ 良好实践

1. 降级处理：_sign_video_url 和 _load_template_segment_durations 均实现了完善的异常捕获和降级逻辑，确保非核心流程错误（如签名失败、DB查询失败）不阻断主任务流程，提高了系统的鲁棒性。
2. 测试覆盖：新增的 test_generation_worker_fixes.py 对 Worker 端的时长截断逻辑进行了细致的单元测试（包括边界条件、部分匹配等），测试覆盖面较全。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-03 16:13:53 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：1 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 1. **[tests/unit/test_generation_preview.py: 1064] 测试引用了不存在的函数** - 问题类型：语法错误 / 逻辑bug - 问题描述：新增的测试类 `TestCalcPreviewResolution`（第1064行开始）中，测试方法尝试从 `app.api.routes.generation_preview` 导入并调用 `_calc_preview_resolution` 函数。然而，在 `apps/api/app/api/routes/generation_preview.py` 的代码变更中，并未定义或引入该函数（仅新增了 `_sign_video_url`）。这将导致测试运行时出现 `ImportError` 或 `AttributeError`，测试无法通过。 - 修改建议：请确认 `_calc_preview_resolution` 函数是否遗漏在本次代码提交中，或者该测试类属于错误的文件/PR。如果该逻辑应在 API 层实现，请补充函数定义；如果该逻辑仅在 Worker 层（`generation.py`）实现，请删除或调整该测试类。 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **降级处理**：`_sign_video_url` 和 `_load_template_segment_durations` 均实现了完善的异常捕获和降级逻辑，确保非核心流程错误（如签名失败、DB查询失败）不阻断主任务流程，提高了系统的鲁棒性。 2. **测试覆盖**：新增的 `test_generation_worker_fixes.py` 对 Worker 端的时长截断逻辑进行了细致的单元测试（包括边界条件、部分匹配等），测试覆盖面较全。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-03 16:13:53 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 0e5f2eb036 into develop 2026-08-04 00:21:41 +08:00

gitea-actions bot commented 2026-08-04 00:22:47 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia referenced this pull request 2026-08-04 00:45:48 +08:00

fix: 预览分辨率自动从模板mode推断（Fix3补充） #1230

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