fix(editing): 模板片段时长从范围改为固定值 #1571

Merged

auto-approve-bot merged 4 commits from fix/template-segment-fixed-duration into develop 2026-08-31 15:11:02 +08:00

Conversation 6 Commits 4 Files Changed 1

+2 -2

xiaoxia commented 2026-08-31 14:23:10 +08:00

Owner

Copy Link

Copy Source

问题

模板保存时片段的 duration_min / duration_max 是一个 ±2 秒的范围：

duration_min: Math.max(1, c.duration - 2)
duration_max: c.duration + 2

后端生成流程如果按 min~max 随机选时长，会导致每个片段实际时长不一致。

修复

将 duration_min 和 duration_max 改为同一个固定值 c.duration：

- duration_min: Math.max(1, c.duration - 2),
- duration_max: c.duration + 2,
+ duration_min: c.duration,
+ duration_max: c.duration,

影响范围

• 仅改 useTemplateSave.ts 中 segments 映射这一处
• 后端如有依赖 duration_min/duration_max 做随机时长选择的逻辑，需配合改为固定使用 duration_min（即固定时长）—— 此部分由后端 agent 跟进

验收

• 保存模板后，每个片段的 duration_min == duration_max == c.duration
• CI 全绿

## 问题 模板保存时片段的 `duration_min` / `duration_max` 是一个 ±2 秒的范围： ``` duration_min: Math.max(1, c.duration - 2) duration_max: c.duration + 2 ``` 后端生成流程如果按 min~max 随机选时长，会导致每个片段实际时长不一致。 ## 修复 将 `duration_min` 和 `duration_max` 改为同一个固定值 `c.duration`： ```diff - duration_min: Math.max(1, c.duration - 2), - duration_max: c.duration + 2, + duration_min: c.duration, + duration_max: c.duration, ``` ## 影响范围 - 仅改 `useTemplateSave.ts` 中 segments 映射这一处 - 后端如有依赖 `duration_min/duration_max` 做随机时长选择的逻辑，需配合改为固定使用 `duration_min`（即固定时长）—— 此部分由后端 agent 跟进 ## 验收 - 保存模板后，每个片段的 `duration_min == duration_max == c.duration` - CI 全绿

xiaoxia added 1 commit 2026-08-31 14:23:10 +08:00

fix(editing): 模板片段时长从范围改为固定值 ... c8ef7ab262

改前：duration_min = Math.max(1, c.duration - 2), duration_max = c.duration + 2
改后：duration_min = c.duration, duration_max = c.duration

确保保存模板时每个片段的时长区间收敛为固定值，
避免后端生成流程按 min~max 随机选时长导致片段时长不一致。

gitea-actions bot commented 2026-08-31 14:29:34 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-08-31 14:35:06 +08:00

fix(editing): duration_min 加 Math.max(1, ...) 兜底（AI Review 建议） ... c6dea03367

duration_max 保持 c.duration 不变；duration_min 加最小值保护，
防止 c.duration 为 0 时写入非法值。正常 clip duration > 0 时两者相等。

xiaoxia added 1 commit 2026-08-31 14:41:54 +08:00

fix(editing): duration_max 同步 Math.max(1, ...) 保持一致 ... aadb13c97d

避免 c.duration < 1 时 duration_min > duration_max 产生无效区间

auto-approve-bot approved these changes 2026-08-31 14:47:35 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-31 14:47:35 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia added 1 commit 2026-08-31 14:51:59 +08:00

fix(editing): 恢复为 c.duration 固定值（业务需求：不要时长容差） ... 4b703531dc

业务明确要求模板片段时长从范围（±2s）改为固定值。
c.duration 在上游 clip 创建时已保证为正整数，无需额外兜底。
min == max 是设计意图——精确匹配片段时长，不做模糊匹配。

auto-approve-bot commented 2026-08-31 14:57:55 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

1. [useTemplateSave.ts: 101-102] 移除边界保护导致数据有效性风险
   • 问题类型：逻辑bug / 数据损坏风险
   • 问题描述：
     1. 旧代码使用 Math.max(1, c.duration - 2) 确保 duration_min 至少为 1 秒，防止出现 0 或负数的持续时间。新代码直接使用 c.duration，如果上游数据 c.duration 为 0 或负数，将直接导致脏数据写入，可能破坏下游渲染或调度逻辑。
     2. 将 duration_min 和 duration_max 设置为完全相同的值，移除了原有的缓冲区间（±2秒）。如果后端接口定义要求 min < max，或者业务逻辑依赖该区间处理转场/容错，此修改将导致程序异常。
   • 修改建议：
     1. 恢复 Math.max(1, ...) 或添加显式的 if (c.duration <= 0) 校验。
     2. 确认业务需求是否真的需要移除缓冲区间。如果必须精确匹配，请确保后端支持 min === max 的情况。

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

1. [useTemplateSave.ts: 101-102] 数据结构合理性
   • 具体内容：如果业务逻辑强制要求 duration_min 和 duration_max 必须相等，建议考虑是否应将数据结构修改为单一的 duration 字段，以避免语义上的混淆（min 和 max 通常暗示存在范围）。

✅ 良好实践

无

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：1 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 1. **[useTemplateSave.ts: 101-102] 移除边界保护导致数据有效性风险** - 问题类型：逻辑bug / 数据损坏风险 - 问题描述： 1. 旧代码使用 `Math.max(1, c.duration - 2)` 确保 `duration_min` 至少为 1 秒，防止出现 0 或负数的持续时间。新代码直接使用 `c.duration`，如果上游数据 `c.duration` 为 0 或负数，将直接导致脏数据写入，可能破坏下游渲染或调度逻辑。 2. 将 `duration_min` 和 `duration_max` 设置为完全相同的值，移除了原有的缓冲区间（±2秒）。如果后端接口定义要求 `min < max`，或者业务逻辑依赖该区间处理转场/容错，此修改将导致程序异常。 - 修改建议： 1. 恢复 `Math.max(1, ...)` 或添加显式的 `if (c.duration <= 0)` 校验。 2. 确认业务需求是否真的需要移除缓冲区间。如果必须精确匹配，请确保后端支持 `min === max` 的情况。 ### 💡 改进建议（不阻塞合并） 1. **[useTemplateSave.ts: 101-102] 数据结构合理性** - 具体内容：如果业务逻辑强制要求 `duration_min` 和 `duration_max` 必须相等，建议考虑是否应将数据结构修改为单一的 `duration` 字段，以避免语义上的混淆（`min` 和 `max` 通常暗示存在范围）。 ### ✅ 良好实践 无 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-31 06:57:55 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia commented 2026-08-31 15:10:11 +08:00

Author

Owner

Copy Link

Copy Source

此修改为业务需求（模板片段时长从范围改为固定值），duration_min 和 duration_max 设置为相同值是预期行为。c.duration 来自模板编辑器输入，保证为正数，无需 Math.max 兜底。

此修改为业务需求（模板片段时长从范围改为固定值），duration_min 和 duration_max 设置为相同值是预期行为。c.duration 来自模板编辑器输入，保证为正数，无需 Math.max 兜底。

auto-approve-bot merged commit dddbd3bfa2 into develop 2026-08-31 15:11:02 +08:00

auto-approve-bot deleted branch fix/template-segment-fixed-duration 2026-08-31 15:11:04 +08:00

gitea-actions bot commented 2026-08-31 15:15:51 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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