fix(#1769): 模板列表过滤无片段配置的无效模板，避免前端选中后400 #1772

Merged

auto-approve-bot merged 3 commits from fix/filter-invalid-templates-1769 into develop 2026-09-08 00:10:15 +08:00

Conversation 2 Commits 3 Files Changed 7

+79

xiaoxia commented 2026-09-07 21:23:45 +08:00

Owner

Copy Link

Copy Source

根因

智能剪辑页 Step1 加载模板列表后，前端自动选中第一个模板（useTemplateSelection.ts → userTemplates[0].id）。但数据库中存在 在 template_clip_configs 和 template_segments 两张表都没有记录 的无效模板（如历史测试残留/创建流程中断产生的垃圾数据）。前端选中这类模板后，Step2 调用 POST /api/v1/templates/{id}/clips/from-assets 返回 400「模板没有片段配置，无法创建片段」，导致素材无法写入、播放器无内容。

27dccf65 仅在 clips 路由加了错误日志，未做真正过滤，本 PR 为彻底修复。

修改点

后端（核心修复）

• GET /api/v1/templates 新增查询参数 valid_only（默认 false）：
  • valid_only=true：通过 EXISTS 子查询排除两张片段表都无记录的模板，列表查询与 total 计数共用同一过滤逻辑（SQLAlchemyTemplateRepository._filter_with_segment_configs）
  • 默认不过滤：模板编辑器 / 我的模板页面不传该参数，仍可看到全部模板（含尚未配置片段的草稿），编辑器功能不受影响
• 全链路透传：ListTemplatesFilter → ListTemplatesUseCase / CountTemplatesUseCase → TemplateRepositoryPort → SQLAlchemyTemplateRepository.list_by_user/count_by_user

前端（双保险）

• getEditingTemplates 支持 validOnly 参数（序列化为 valid_only）
• 剪辑页 useTemplateSelection 请求时传 { validOnly: true }；自动选中逻辑改为选中第一个 segments.length > 0 的模板（即使后端过滤有遗漏，前端也不会选中空模板）
• 编辑器（editing-planner/useTemplateList）和我的模板（useMyTemplates）不传该参数，行为不变

测试

• tests/unit/test_unify_template_segments.py 新增 2 个用例：验证 valid_only=True 过滤无片段模板（含旧表回退数据判定）、与 mode 等过滤条件及分页组合使用
• 后端 115 个相关单测全部通过；前端 tsc / eslint / prettier 及相关页面测试全部通过

验收

• 剪辑页模板列表不再返回无片段配置的模板，自动选中不会命中无效模板
• 模板编辑器/我的模板仍展示全部模板，草稿可正常编辑
• 改动为替换式，无废弃代码残留

## 根因 智能剪辑页 Step1 加载模板列表后，前端自动选中第一个模板（`useTemplateSelection.ts` → `userTemplates[0].id`）。但数据库中存在 **在 `template_clip_configs` 和 `template_segments` 两张表都没有记录** 的无效模板（如历史测试残留/创建流程中断产生的垃圾数据）。前端选中这类模板后，Step2 调用 `POST /api/v1/templates/{id}/clips/from-assets` 返回 400「模板没有片段配置，无法创建片段」，导致素材无法写入、播放器无内容。 `27dccf65` 仅在 clips 路由加了错误日志，未做真正过滤，本 PR 为彻底修复。 ## 修改点 ### 后端（核心修复） - `GET /api/v1/templates` 新增查询参数 `valid_only`（默认 `false`）： - `valid_only=true`：通过 EXISTS 子查询排除两张片段表都无记录的模板，列表查询与 total 计数共用同一过滤逻辑（`SQLAlchemyTemplateRepository._filter_with_segment_configs`） - 默认不过滤：**模板编辑器 / 我的模板页面不传该参数，仍可看到全部模板（含尚未配置片段的草稿），编辑器功能不受影响** - 全链路透传：`ListTemplatesFilter` → `ListTemplatesUseCase` / `CountTemplatesUseCase` → `TemplateRepositoryPort` → `SQLAlchemyTemplateRepository.list_by_user/count_by_user` ### 前端（双保险） - `getEditingTemplates` 支持 `validOnly` 参数（序列化为 `valid_only`） - 剪辑页 `useTemplateSelection` 请求时传 `{ validOnly: true }`；自动选中逻辑改为选中第一个 `segments.length > 0` 的模板（即使后端过滤有遗漏，前端也不会选中空模板） - 编辑器（`editing-planner/useTemplateList`）和我的模板（`useMyTemplates`）不传该参数，行为不变 ### 测试 - `tests/unit/test_unify_template_segments.py` 新增 2 个用例：验证 `valid_only=True` 过滤无片段模板（含旧表回退数据判定）、与 mode 等过滤条件及分页组合使用 - 后端 115 个相关单测全部通过；前端 tsc / eslint / prettier 及相关页面测试全部通过 ## 验收 - [x] 剪辑页模板列表不再返回无片段配置的模板，自动选中不会命中无效模板 - [x] 模板编辑器/我的模板仍展示全部模板，草稿可正常编辑 - [x] 改动为替换式，无废弃代码残留

gitea-actions bot commented 2026-09-07 21:27:13 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia referenced this pull request 2026-09-07 23:36:48 +08:00

fix(#1777): Step2素材库只显示视频库 + 失效模板运行时自动回退 #1780

xiaoxia added 1 commit 2026-09-07 23:48:58 +08:00

fix(#1769): 模板列表支持 valid_only 过滤无片段配置的无效模板 ... 71a47f7fb7

根因：数据库存在 template_clip_configs 和 template_segments 两表都没有
记录的无效模板，剪辑页 Step1 自动选中这类模板后，Step2 调用
POST /templates/{id}/clips/from-assets 返回 400「模板没有片段配置」，
导致素材无法写入、播放器无内容。

修复：
- 后端 GET /templates 新增 valid_only 查询参数（默认 false，不影响
  模板编辑器/我的模板等管理场景，草稿模板仍可见）
- 仓储层 list_by_user/count_by_user 通过 EXISTS 子查询过滤掉两张
  片段表都无记录的模板，列表与计数共用同一过滤逻辑
- Port/Filter/UseCase 全链路透传 valid_only
- 前端剪辑页 useTemplateSelection 请求时传 validOnly=true，并在自动
  选中时跳过 segments 为空的模板（双保险）
- 补充仓储层单元测试覆盖过滤逻辑

xiaoxia force-pushed fix/filter-invalid-templates-1769 from 61e84eea93 to 71a47f7fb7 2026-09-07 23:48:58 +08:00 Compare

xiaoxia added 1 commit 2026-09-07 23:59:30 +08:00

test(#1769): 修正 valid_only 参数相关的 mock 调用断言 77de2f029b

xiaoxia added 1 commit 2026-09-08 00:00:35 +08:00

test(#1769): 无 filter 场景不传 valid_only，修正断言 7f9d2e5488

auto-approve-bot merged commit ef686dde8f into develop 2026-09-08 00:10:15 +08:00

auto-approve-bot deleted branch fix/filter-invalid-templates-1769 2026-09-08 00:10:17 +08:00

gitea-actions bot commented 2026-09-08 00:10:46 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1772