fix(viral-video): 6 E2E bug fixes (fusion_level/concat/ingest/duplicated URL/TTS/Seedance ratio) #2113

Merged

auto-approve-bot merged 4 commits from fix/2110-six-viral-video-bugs into develop 2026-10-01 05:12:36 +08:00

Conversation 2 Commits 4 Files Changed 10

+106 -20

xiaoxia commented 2026-10-01 04:00:33 +08:00

Owner

Copy Link

Copy Source

Summary

修复 E2E 冒烟测试发现的 6 个 bug（2 P0 + 4 P1）。

Bug list

#	优先级	问题	修复
1	P0	fusion_level 前后端枚举不一致（前端发 full_ai，后端只认 ai_full）→ 422	schema VALID_FUSION_LEVELS 加 full_ai，field_validator 自动归一化到 ai_full
2	P0	ffmpeg concat exit 234（Seedance 生成段 gen_audio=False 无音频流，concat filter a=1 找不到 [N:a]）	concat_video_files 对每段调 probe_has_audio，无声段 ConcatSegment(has_audio=False) → concat filter 用 aevalsrc 补静音轨（占位片本就这么做，真实段之前漏了）
3	P1	ingest 按 storage_key 找不到占位 asset（历史数据 file_url 列存旧路径 assets/<project>/...）	find_by_storage_key 改为 (storage_key == k) OR (file_url == k) 双列查询，兼容新老数据
4	P1	duplicated 秒传返回 500：domain Asset 无 file_url 属性 → AttributeError	新增 _get_existing_asset_url() helper，按 storage_key → file_url 顺序取值，再 storage_service.get_url(key) 生成公网 URL
5	P1	TTS 音色/format 不一致（传 voice="default" 而非 persona_id，默认 wav 导致混音链路不兼容）	_step_tts 用 job.persona_id or longxiaochun_v3，统一 format="mp3"；保留 TypeError 兜底
6	P1	首帧图模式同时传 ratio="9:16" → Seedance 400 InvalidParameter（首帧模式 ratio 必须跟随首帧图）	call_video_generation 在 image_url 非空时不传 ratio；ai_client.video_generation ratio 改为 Optional[str]，为 None 时 payload 中省略 ratio 字段

Files changed

• apps/api/app/schemas/viral_video.py: fusion_level 别名兼容
• apps/api/app/api/routes/upload.py: duplicated 公网 URL helper
• packages/adapters/sqlalchemy_impl/asset_repository.py: find_by_storage_key 双列查询
• apps/worker/video_processing/concat_engine.py: 每段探测 has_audio
• apps/worker/worker_app/tasks/viral_video.py: TTS voice_id+format=mp3
• packages/shared/ai_service.py + ai_client.py: 首帧模式省略 ratio 参数

Test

• Python 语法 py_compile 全部通过
• Pydantic validator 验证：full_ai→ai_full, ai_full→ai_full, bad→422

Checklist

• 后端 Python 语法校验通过
• 无硬编码 API Key
• 兼容老数据/老调用方

## Summary 修复 E2E 冒烟测试发现的 6 个 bug（2 P0 + 4 P1）。 ## Bug list | # | 优先级 | 问题 | 修复 | |---|--------|------|------| | 1 | P0 | fusion_level 前后端枚举不一致（前端发 `full_ai`，后端只认 `ai_full`）→ 422 | schema VALID_FUSION_LEVELS 加 `full_ai`，field_validator 自动归一化到 `ai_full` | | 2 | P0 | ffmpeg concat exit 234（Seedance 生成段 gen_audio=False 无音频流，concat filter `a=1` 找不到 `[N:a]`） | `concat_video_files` 对每段调 `probe_has_audio`，无声段 `ConcatSegment(has_audio=False)` → concat filter 用 `aevalsrc` 补静音轨（占位片本就这么做，真实段之前漏了） | | 3 | P1 | ingest 按 storage_key 找不到占位 asset（历史数据 file_url 列存旧路径 `assets/<project>/...`） | `find_by_storage_key` 改为 `(storage_key == k) OR (file_url == k)` 双列查询，兼容新老数据 | | 4 | P1 | duplicated 秒传返回 500：domain Asset 无 `file_url` 属性 → AttributeError | 新增 `_get_existing_asset_url()` helper，按 `storage_key → file_url` 顺序取值，再 `storage_service.get_url(key)` 生成公网 URL | | 5 | P1 | TTS 音色/format 不一致（传 voice="default" 而非 persona_id，默认 wav 导致混音链路不兼容） | `_step_tts` 用 `job.persona_id or longxiaochun_v3`，统一 format="mp3"；保留 TypeError 兜底 | | 6 | P1 | 首帧图模式同时传 ratio="9:16" → Seedance 400 InvalidParameter（首帧模式 ratio 必须跟随首帧图） | `call_video_generation` 在 image_url 非空时不传 ratio；`ai_client.video_generation` ratio 改为 Optional[str]，为 None 时 payload 中省略 ratio 字段 | ## Files changed - apps/api/app/schemas/viral_video.py: fusion_level 别名兼容 - apps/api/app/api/routes/upload.py: duplicated 公网 URL helper - packages/adapters/sqlalchemy_impl/asset_repository.py: find_by_storage_key 双列查询 - apps/worker/video_processing/concat_engine.py: 每段探测 has_audio - apps/worker/worker_app/tasks/viral_video.py: TTS voice_id+format=mp3 - packages/shared/ai_service.py + ai_client.py: 首帧模式省略 ratio 参数 ## Test - Python 语法 py_compile 全部通过 - Pydantic validator 验证：full_ai→ai_full, ai_full→ai_full, bad→422 ## Checklist - [x] 后端 Python 语法校验通过 - [x] 无硬编码 API Key - [x] 兼容老数据/老调用方

gitea-actions bot commented 2026-10-01 04:02:35 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-10-01 04:14:56 +08:00

fix(viral-video): fix 6 E2E bugs — fusion_level alias, concat silent segments, ingest lookup, duplicated URL, TTS voice/format, Seedance first-frame ratio ... cdce1b2e10

Bug1 (P0): schema accepts 'full_ai' as alias for 'ai_full' (Pydantic field_validator normalizes)
Bug2 (P0): concat_video_files probes each segment audio stream; Seedance gen_audio=False
       segments now marked has_audio=False so concat filter uses aevalsrc silence instead
       of failing with ffmpeg exit 234
Bug3 (P1): find_by_storage_key now queries (storage_key OR file_url) to cover historical
       data where the legacy file_url column held assets/<project>/<date>/... paths
Bug4 (P1): duplicated-hit response no longer accesses non-existent domain Asset.file_url;
       new helper _get_existing_asset_url uses storage_key (fallback file_url) through
       storage_service.get_url()
Bug5 (P1): _step_tts passes job.persona_id as voice_id (default longxiaochun_v3) and
       forces format='mp3' so downstream ffmpeg -map 1:a:0 works regardless of provider
Bug6 (P1): call_video_generation omits ratio param in first-frame (image_url) mode;
       ai_client.video_generation ratio becomes Optional[str] and is omitted from payload
       when None, fixing 400 InvalidParameter from Seedance

xiaoxia force-pushed fix/2110-six-viral-video-bugs from 72cffe664b to cdce1b2e10 2026-10-01 04:14:56 +08:00 Compare

xiaoxia added 1 commit 2026-10-01 04:39:06 +08:00

test: fix unit tests to match TTS import path and storage_service.get_url mock ... 09b8b2990f

- test_viral_video_p0: patch new apps.worker.services.tts_service_factory path,
  assert format='mp3' is passed
- test_prepare_dedup: storage_service.get_url.return_value = '' so
  DirectUploadPrepareResponse.url: str doesn't receive a MagicMock

xiaoxia force-pushed fix/2110-six-viral-video-bugs from daf1b3b380 to 09b8b2990f 2026-10-01 04:39:06 +08:00 Compare

auto-approve-bot added 1 commit 2026-10-01 04:43:49 +08:00

style: auto-format with black + isort + ruff + prettier [skip ci-format-check] a74be7c717

xiaoxia added 1 commit 2026-10-01 04:59:45 +08:00

fix(tests+robustness): mock-safe status_code int-cast in ai_client; seedance test responses default 200 ... 3d8882c479

- test_ai_client_video: fake task/poll responses explicitly set status_code=200/text=''
  to avoid MagicMock()>=int TypeError leaking into retry logic
- ai_client.video_generation: wrap resp.status_code comparisons in try/except
  (int() cast) so any non-int status (MagicMock in tests) degrades to 200 instead
  of bombing out with TypeError during create/poll

auto-approve-bot merged commit fc6ebbecb6 into develop 2026-10-01 05:12:36 +08:00

auto-approve-bot deleted branch fix/2110-six-viral-video-bugs 2026-10-01 05:12:39 +08:00

gitea-actions bot commented 2026-10-01 05:13:35 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

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

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot frontend-dev xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#2113