feat: Step2选素材同步写入edit_plan_clips #1469

Merged

xiaoxia merged 1 commits from feat/sync-edit-plan-clips into develop 2026-08-23 17:42:26 +08:00

Conversation 3 Commits 1 Files Changed 6

+156 -1

xiaoxia commented 2026-08-23 16:16:58 +08:00

Owner

Copy Link

Copy Source

背景

后端正在改造 Worker 渲染逻辑，改为直接读取数据库 edit_plan_clips 表渲染（不再从 asset_ids 重建）。前端需要在选素材时把 clips 写入数据库。

改动

1. 新增 API updateEditPlanClips(templateId, clips, signal?) — PUT /templates/{id}/editor/clips
2. 新增共享工具 buildClipsFromAssets() — clips 构建逻辑与 FrontendPreviewPlayer.buildPlaybackSegments 完全一致（clamp(assetDuration, seg.duration_min, seg.duration_max)，start_time=0）
3. useStep2Materials — 选素材防抖 800ms 后同步写入 clips；手动/智能匹配两种模式都支持；AbortController 取消上一次未完成请求；失败 console.warn，不弹窗不阻塞
4. prop 透传 — Step2MaterialSelect / GenerateStepContent 透传 templateSegments
5. 生成参数 — 确认 source_edit_plan_id、title_config、cover_url、asset_ids 都已正确传递（无需额外改动）

验收

• TypeScript 编译通过
• ESLint 0 warning
• Prettier 格式化
• clips duration 与前端预览一致

## 背景 后端正在改造 Worker 渲染逻辑，改为直接读取数据库 `edit_plan_clips` 表渲染（不再从 `asset_ids` 重建）。前端需要在选素材时把 clips 写入数据库。 ## 改动 1. **新增 API** `updateEditPlanClips(templateId, clips, signal?)` — PUT `/templates/{id}/editor/clips` 2. **新增共享工具** `buildClipsFromAssets()` — clips 构建逻辑与 `FrontendPreviewPlayer.buildPlaybackSegments` 完全一致（`clamp(assetDuration, seg.duration_min, seg.duration_max)`，`start_time=0`） 3. **useStep2Materials** — 选素材防抖 800ms 后同步写入 clips；手动/智能匹配两种模式都支持；AbortController 取消上一次未完成请求；失败 console.warn，不弹窗不阻塞 4. **prop 透传** — Step2MaterialSelect / GenerateStepContent 透传 `templateSegments` 5. **生成参数** — 确认 `source_edit_plan_id`、`title_config`、`cover_url`、`asset_ids` 都已正确传递（无需额外改动） ## 验收 - [x] TypeScript 编译通过 - [x] ESLint 0 warning - [x] Prettier 格式化 - [x] clips duration 与前端预览一致

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

feat: sync selected materials to edit_plan_clips on Step2 ... 9e7f04e312

- New API updateEditPlanClips (PUT /templates/{id}/editor/clips)
- useStep2Materials: debounced 800ms, builds clips with same logic
  as FrontendPreviewPlayer.buildPlaybackSegments (clamp asset
  duration to template segment duration_min/max)
- buildClipsFromAssets shared util ensures duration consistency
  between preview and clips written to DB
- Supports both manual and smart-match modes
- Failures are silent (console.warn), never block the user
- Step2MaterialSelect/GenerateStepContent pass templateSegments

gitea-actions bot commented 2026-08-23 16:20:13 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-08-23 16:21:42 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1469

⚠️ 问题（1个需要修改）

1. apps/web/src/pages/generate/hooks/useStep2Materials.ts 第80行：当用户取消选择所有素材时（ids.length 为 0），代码直接 return 导致未调用 updateEditPlanClips。这会导致后端无法清空 clips 数据，造成前后端数据不一致，生成任务可能残留旧的 clips 数据。
2. apps/web/src/pages/generate/utils/buildClipsFromAssets.ts 第48行：当 templateSegments 为空数组时，lastSeg 为 undefined，且 tplSeg 也会为 undefined。虽然代码有 fallback 逻辑 Math.min(assetDuration, 10)，但在没有模板结构信息的情况下硬编码 10 秒作为默认时长可能不符合业务预期（通常应阻止生成或报错），建议确认此边界场景的业务逻辑。

💡 建议（2个可选）

1. apps/web/src/pages/generate/hooks/useStep2Materials.ts：useEffect 依赖数组中未包含 materials，仅通过 useRef 获取最新值。如果素材列表中的元数据（如 duration）在加载后被更新，已选中的 clips 将不会自动同步更新时长。建议评估是否需要监听 materials 的变化或在关键操作时强制刷新。
2. apps/web/src/pages/generate/utils/buildClipsFromAssets.ts：注释提到逻辑需与 FrontendPreviewPlayer.tsx 保持一致，建议抽取公共逻辑函数或增加单元测试以防止未来修改导致两边逻辑不一致。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能良好

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-23 08:21:42 | 模型:}

## 代码审查结果 - PR #1469 ### ⚠️ 问题（1个需要修改） 1. **apps/web/src/pages/generate/hooks/useStep2Materials.ts 第80行**：当用户取消选择所有素材时（`ids.length` 为 0），代码直接 return 导致未调用 `updateEditPlanClips`。这会导致后端无法清空 clips 数据，造成前后端数据不一致，生成任务可能残留旧的 clips 数据。 2. **apps/web/src/pages/generate/utils/buildClipsFromAssets.ts 第48行**：当 `templateSegments` 为空数组时，`lastSeg` 为 undefined，且 `tplSeg` 也会为 undefined。虽然代码有 fallback 逻辑 `Math.min(assetDuration, 10)`，但在没有模板结构信息的情况下硬编码 10 秒作为默认时长可能不符合业务预期（通常应阻止生成或报错），建议确认此边界场景的业务逻辑。 ### 💡 建议（2个可选） 1. **apps/web/src/pages/generate/hooks/useStep2Materials.ts**：`useEffect` 依赖数组中未包含 `materials`，仅通过 `useRef` 获取最新值。如果素材列表中的元数据（如 duration）在加载后被更新，已选中的 clips 将不会自动同步更新时长。建议评估是否需要监听 `materials` 的变化或在关键操作时强制刷新。 2. **apps/web/src/pages/generate/utils/buildClipsFromAssets.ts**：注释提到逻辑需与 `FrontendPreviewPlayer.tsx` 保持一致，建议抽取公共逻辑函数或增加单元测试以防止未来修改导致两边逻辑不一致。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-23 08:21:42 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia referenced this issue from a commit 2026-08-23 16:25:56 +08:00

feat: sync selected materials to edit_plan_clips on Step2 (PR #1469)

xiaoxia merged commit 392a20002c into develop 2026-08-23 17:42:26 +08:00

gitea-actions bot commented 2026-08-23 17:43:25 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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