feat(from-assets): 按模板segment创建片段，忽略前端required_clips_count #1524

Merged

xiaoxia merged 12 commits from feat/from-assets-by-template-segments into develop 2026-08-28 01:19:58 +08:00

Conversation 5 Commits 12 Files Changed 4

+979 -760

xiaoxia commented 2026-08-27 22:32:35 +08:00

Owner

Copy Link

Copy Source

改动说明

from-assets 端点不再根据前端传的 required_clips_count 创建片段，而是：

1. 通过 template_id 查询 template_segments / clip_configs
2. 片段数量 = segment 数量
3. 每个片段时长在 segment 的 duration_min ~ duration_max 之间随机取值（保留一位小数）
4. 素材按片段顺序轮询分配，素材不够时同一素材切多个片段
5. 创建前先 delete_all_clips 清空旧片段，避免重复调用产生重复数据
6. MediaKit 推荐时间仅用于每个素材的第一个片段，后续片段用随机
7. MediaKit 超时/轮询参数不变（60s / 2s / 15次）
8. 片段创建后立即 mark_clips_ready

测试

• 28 个新单元测试全部通过
• 154 个相关已有测试无回归

## 改动说明 from-assets 端点不再根据前端传的 required_clips_count 创建片段，而是： 1. 通过 template_id 查询 template_segments / clip_configs 2. 片段数量 = segment 数量 3. 每个片段时长在 segment 的 duration_min ~ duration_max 之间随机取值（保留一位小数） 4. 素材按片段顺序轮询分配，素材不够时同一素材切多个片段 5. 创建前先 delete_all_clips 清空旧片段，避免重复调用产生重复数据 6. MediaKit 推荐时间仅用于每个素材的第一个片段，后续片段用随机 7. MediaKit 超时/轮询参数不变（60s / 2s / 15次） 8. 片段创建后立即 mark_clips_ready ## 测试 - 28 个新单元测试全部通过 - 154 个相关已有测试无回归

xiaoxia added 1 commit 2026-08-27 22:32:36 +08:00

feat(from-assets): create clips based on template segments instead of required_clips_count ... a2b37e376d

- Query template_segments/clip_configs by template_id, create exactly N clips matching segment count
- Each clip duration randomly chosen between segment duration_min and duration_max (1 decimal)
- Assets round-robin assigned across segments; same asset can produce multiple clips
- Delete all existing clips for the plan before creating new ones (idempotent)
- MediaKit recommendation only applied to the first clip using each asset; subsequent clips use random start times
- MediaKit timeout/poll settings unchanged (60s/2s/15)
- mark_clips_ready called immediately after creation
- Ignore required_clips_count from frontend
- Clips order starts from 0 since old clips are cleared
- Add _get_template_segments helper supporting both new (clip_configs) and old (template_segments) systems
- 28 unit tests covering segment query, clip creation, round-robin, deletion, duration bounds, MediaKit integration

gitea-actions bot commented 2026-08-27 22:49:39 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-27 22:55:52 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 42c4f22b21

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

fix: rename unused loop var and add type annotation for ruff/mypy 9510f69816

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

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

test: update existing clip tests for template-segment-based creation 030220f24d

auto-approve-bot added 1 commit 2026-08-28 00:11:57 +08:00

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

xiaoxia added 1 commit 2026-08-28 00:20:12 +08:00

fix: atomic clip replacement via replace_all_clips_transactional + asset_total=0 guard ... 5c62dad798

- Use replace_all_clips_transactional for atomic delete+create (fixes data loss risk)
- Guard asset_total <= 0 with HTTP 400 (fixes invalid clip creation)
- Update tests to verify transactional replace and zero-duration guard
- 40 tests passing

auto-approve-bot added 1 commit 2026-08-28 00:29:08 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 753a3206a7

xiaoxia added 1 commit 2026-08-28 00:31:31 +08:00

fix: validate segment durations with _safe_segment_duration + upgrade fallback logs to warning ... d771fee45a

- Add _safe_segment_duration() to handle None/invalid/negative/NaN values
- Swap dur_min/dur_max if dirty data has min > max
- Change fallback exception logs from DEBUG to WARNING for observability
- Add 7 unit tests for _safe_segment_duration
- 47 tests passing

auto-approve-bot added 1 commit 2026-08-28 00:37:28 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 9711b6a545

xiaoxia added 1 commit 2026-08-28 00:47:22 +08:00

fix: guard empty asset_ids + validate clip_duration > 0 + fix old-system default ... ecfa478a24

- Add empty asset_ids check before modulo (defense-in-depth, schema also enforces min_length=1)
- Add clip_duration <= 0 guard after asset_total capping
- Fix old-system fallback default from 0.0 to _DEFAULT_EDITOR_CLIP_DURATION
- Add test for empty asset_ids 400
- 48 tests passing

auto-approve-bot added 1 commit 2026-08-28 00:53:23 +08:00

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

auto-approve-bot commented 2026-08-28 00:59:45 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

1. [apps/api/app/api/routes/templates_editor/clips.py: 640-650] 构造 clips_data 时丢失 clip_type 字段
   • 问题类型：逻辑bug
   • 问题描述：在重构后的代码中，clips_data 字典仅包含 order, asset_id, start_time, duration 四个字段。旧代码中明确使用了 body.clip_type or "main" 来设置片段类型。新代码完全忽略了该字段，导致 replace_all_clips_transactional 创建的片段可能丢失类型信息（如 "overlay", "audio" 等），或者如果服务层没有默认值，可能导致数据错误。
   • 修改建议：在 clips_data 的字典中补充 clip_type 字段，例如 "clip_type": body.clip_type or "main"。

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

1. [apps/api/app/api/routes/templates_editor/clips.py: 398, 418] 异常捕获范围过大

   • 具体内容：在 _get_template_segments 函数中使用了 except Exception 捕获所有异常。虽然意图是作为回退机制（Fallback），但这可能会掩盖数据库连接错误、超时等严重系统问题，导致错误难以排查。建议捕获更具体的异常（如 ServiceUnavailableError 或特定的 Repository 异常），或者在日志中记录更详细的堆栈信息以便排查。

2. [apps/api/app/api/routes/templates_editor/clips.py: 640-650] 确认服务层对缺失字段的默认值处理

   • 具体内容：旧代码调用 create_clip 时传入了多个字段（如 text_content, transition_effect, playback_speed 等），新代码的 clips_data 仅传入了 4 个字段。请确保 replace_all_clips_transactional 方法内部对这些缺失字段有完善的默认值处理逻辑，否则可能导致数据库字段为 NULL 或默认值不符合预期。

✅ 良好实践

• 使用 _safe_segment_duration 统一处理时长数据的边界情况（None、负数、NaN），增强了健壮性。
• 在 replace_all_clips_transactional 调用前进行全量的内存数据校验和计算，避免部分成功导致的数据不一致状态。
• 对素材时长为 0 或不足的情况进行了显式的 400 错误抛出，避免了创建无效片段。

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：1 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 1. **[apps/api/app/api/routes/templates_editor/clips.py: 640-650] 构造 clips_data 时丢失 clip_type 字段** - 问题类型：逻辑bug - 问题描述：在重构后的代码中，`clips_data` 字典仅包含 `order`, `asset_id`, `start_time`, `duration` 四个字段。旧代码中明确使用了 `body.clip_type or "main"` 来设置片段类型。新代码完全忽略了该字段，导致 `replace_all_clips_transactional` 创建的片段可能丢失类型信息（如 "overlay", "audio" 等），或者如果服务层没有默认值，可能导致数据错误。 - 修改建议：在 `clips_data` 的字典中补充 `clip_type` 字段，例如 `"clip_type": body.clip_type or "main"`。 ### 💡 改进建议（不阻塞合并） 1. **[apps/api/app/api/routes/templates_editor/clips.py: 398, 418] 异常捕获范围过大** - 具体内容：在 `_get_template_segments` 函数中使用了 `except Exception` 捕获所有异常。虽然意图是作为回退机制（Fallback），但这可能会掩盖数据库连接错误、超时等严重系统问题，导致错误难以排查。建议捕获更具体的异常（如 `ServiceUnavailableError` 或特定的 Repository 异常），或者在日志中记录更详细的堆栈信息以便排查。 2. **[apps/api/app/api/routes/templates_editor/clips.py: 640-650] 确认服务层对缺失字段的默认值处理** - 具体内容：旧代码调用 `create_clip` 时传入了多个字段（如 `text_content`, `transition_effect`, `playback_speed` 等），新代码的 `clips_data` 仅传入了 4 个字段。请确保 `replace_all_clips_transactional` 方法内部对这些缺失字段有完善的默认值处理逻辑，否则可能导致数据库字段为 NULL 或默认值不符合预期。 ### ✅ 良好实践 - 使用 `_safe_segment_duration` 统一处理时长数据的边界情况（None、负数、NaN），增强了健壮性。 - 在 `replace_all_clips_transactional` 调用前进行全量的内存数据校验和计算，避免部分成功导致的数据不一致状态。 - 对素材时长为 0 或不足的情况进行了显式的 400 错误抛出，避免了创建无效片段。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-27 16:59:45 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia added 1 commit 2026-08-28 01:02:06 +08:00

fix: include clip_type in clips_data for replace_all_clips_transactional ... 506b29991b

- Add clip_type field to clips_data dict (body.clip_type or main)
- Service layer reads clip_type from clips_data instead of hardcoding
- 48 tests passing

xiaoxia merged commit b6af6892cd into develop 2026-08-28 01:19:58 +08:00

gitea-actions bot commented 2026-08-28 01:23:30 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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