fix(assets): 智能选素材传 limit 参数，避免「有几个选几个」 #1570

Merged

auto-approve-bot merged 2 commits from fix/smart-match-limit into develop 2026-08-31 14:21:00 +08:00

Conversation 5 Commits 2 Files Changed 3

+37 -9

xiaoxia commented 2026-08-31 13:58:19 +08:00

Owner

Copy Link

Copy Source

问题

线上实测：AI 自动选素材时，视频库里有几个素材就返回几个，没有数量上限控制。

根因：smartMatchAssets 调用 POST /assets/smart-match 时未传 limit 参数，后端 SmartMatchRequest.limit 默认为 None（返回全部）。

修复

1. apps/web/src/api/assets/assets.ts

• smartMatchAssets 增加可选 limit?: number 参数
• 仅在 limit > 0 时序列化进请求体

2. apps/web/src/pages/generate/hooks/step2-materials/useSmartMatch.ts

• 接受 templateSegments?: TemplateSegment[] 选项
• 新增 computeLimitFromSegments：
  • 取各 segment duration_min 之和作为目标视频总时长
  • 按 15 秒/素材估算所需素材数：Math.ceil(totalSeconds / 15)
  • 结果钳制到 [1, 200]（后端 SmartMatchRequest.limit 上限 200）
  • 拿不到时长信息时默认 limit = 10

3. apps/web/src/pages/generate/hooks/useStep2Materials.ts

• 透传 templateSegments 到 useSmartMatch

影响范围

• 仅影响自动模式（materialMode === "auto"）
• 手动选素材模式不受影响
• 后端已支持 limit 字段（SmartMatchRequest.limit: int | None = Field(default=None, ge=1, le=200)）

验收

• 自动模式选素材数量 ≈ ceil(目标视频时长 / 15)，不再等于库里素材总数
• 手动模式不受影响
• CI 全绿

## 问题 线上实测：AI 自动选素材时，视频库里有几个素材就返回几个，没有数量上限控制。 根因：`smartMatchAssets` 调用 `POST /assets/smart-match` 时未传 `limit` 参数，后端 `SmartMatchRequest.limit` 默认为 `None`（返回全部）。 ## 修复 ### 1. `apps/web/src/api/assets/assets.ts` - `smartMatchAssets` 增加可选 `limit?: number` 参数 - 仅在 `limit > 0` 时序列化进请求体 ### 2. `apps/web/src/pages/generate/hooks/step2-materials/useSmartMatch.ts` - 接受 `templateSegments?: TemplateSegment[]` 选项 - 新增 `computeLimitFromSegments`： - 取各 segment `duration_min` 之和作为目标视频总时长 - 按 15 秒/素材估算所需素材数：`Math.ceil(totalSeconds / 15)` - 结果钳制到 `[1, 200]`（后端 `SmartMatchRequest.limit` 上限 200） - 拿不到时长信息时默认 `limit = 10` ### 3. `apps/web/src/pages/generate/hooks/useStep2Materials.ts` - 透传 `templateSegments` 到 `useSmartMatch` ## 影响范围 - 仅影响自动模式（`materialMode === "auto"`） - 手动选素材模式不受影响 - 后端已支持 `limit` 字段（`SmartMatchRequest.limit: int | None = Field(default=None, ge=1, le=200)`） ## 验收 - 自动模式选素材数量 ≈ `ceil(目标视频时长 / 15)`，不再等于库里素材总数 - 手动模式不受影响 - CI 全绿

xiaoxia added 1 commit 2026-08-31 13:58:20 +08:00

fix(assets): 智能选素材传 limit 参数，避免「有几个选几个」 ... 80247c969a

- smartMatchAssets 增加可选 limit 参数，传给后端 /assets/smart-match
- useSmartMatch 根据模板 segments 总时长按 15s/素材估算 limit，钳制 [1,200]
- 拿不到时长信息时默认 limit=10（后端上限 200）
- useStep2Materials 透传 templateSegments 到 useSmartMatch

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

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-08-31 14:09:33 +08:00

fix(assets): useSmartMatch 兜底过滤空 id 素材（AI Review 建议） 140d3c7e67

auto-approve-bot commented 2026-08-31 14:13:38 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：通过
• 建议级问题数量：1 个

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

无

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

1. [apps/web/src/pages/generate/hooks/step2-materials/useSmartMatch.ts: 97] 依赖数组引用稳定性
   • 具体内容：useCallback 的依赖数组中包含了 templateSegments。如果父组件每次渲染都传递一个新的数组引用（这在 React 中很常见），会导致 handleSmartMatch 函数在每次渲染时都被重新创建。建议确认父组件传入的 templateSegments 引用是否稳定，或者在父组件中使用 useMemo 包装该数据，以避免不必要的子组件重渲染或函数重创建。

✅ 良好实践

1. 边界条件处理完善：在 computeLimitFromSegments 函数中，对空数组、总时长小于等于0的情况均做了兜底处理，并对最终结果进行了 [1, 200] 的区间钳制，逻辑严密。
2. 空指针安全：在 handleSmartMatch 中使用 !!a?.id 进行过滤，有效防止了 undefined 或 null 导致的运行时错误。
   2/3. 类型安全与防御性编程：smartMatchAssets 函数中对 limit 参数进行了 limit > 0 的校验，确保了只有合法的正整数才会被发送到后端。

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/web/src/pages/generate/hooks/step2-materials/useSmartMatch.ts: 97] 依赖数组引用稳定性** - 具体内容：`useCallback` 的依赖数组中包含了 `templateSegments`。如果父组件每次渲染都传递一个新的数组引用（这在 React 中很常见），会导致 `handleSmartMatch` 函数在每次渲染时都被重新创建。建议确认父组件传入的 `templateSegments` 引用是否稳定，或者在父组件中使用 `useMemo` 包装该数据，以避免不必要的子组件重渲染或函数重创建。 ### ✅ 良好实践 1. **边界条件处理完善**：在 `computeLimitFromSegments` 函数中，对空数组、总时长小于等于0的情况均做了兜底处理，并对最终结果进行了 `[1, 200]` 的区间钳制，逻辑严密。 2. **空指针安全**：在 `handleSmartMatch` 中使用 `!!a?.id` 进行过滤，有效防止了 `undefined` 或 `null` 导致的运行时错误。 2/3. **类型安全与防御性编程**：`smartMatchAssets` 函数中对 `limit` 参数进行了 `limit > 0` 的校验，确保了只有合法的正整数才会被发送到后端。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-31 06:13:38 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-08-31 14:20:30 +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:20:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit ab23ca7e5a into develop 2026-08-31 14:21:00 +08:00

auto-approve-bot deleted branch fix/smart-match-limit 2026-08-31 14:21:01 +08:00

gitea-actions bot commented 2026-08-31 14:24:53 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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