fix(p0): #1922 from-assets 空 templateId 双斜杠 404 兜底 #1926

Merged

auto-approve-bot merged 2 commits from fix/p0-from-assets-default-template-fallback-v2 into develop 2026-09-15 11:54:06 +08:00

Conversation 2 Commits 2 Files Changed 7

+636 -26

xiaoxia commented 2026-09-15 11:37:43 +08:00

Owner

Copy Link

Copy Source

P0 Bug 修复

关联 Issue: #1922

根因

双 Bug 链：

1. PR#1918 清理模板管理 API 时误删 GET /templates 自动创建默认模板的兜底逻辑；
2. 前端 PR#1913 去掉空 tid 拦截后，新用户首次进入生成页 templateId=''，拼出 URL /templates//editor/clips/from-assets（双斜杠）→ FastAPI 404，完全阻断首次生成流程。

修复方案（三层后端兜底 + 独立端点）

文件	改动	作用
app/api/routes/_default_template.py	🆕	共享 get_or_create_default_template_id()：查有效模板 → 无则 CreateTemplateUseCase 创建默认 voice_over 模板 → 创建异常回滚后重查（防并发唯一键冲突）
app/api/routes/templates.py	修改	GET /templates 列表查询前调用兜底，确保生成页首次拉模板列表时一定有数据
app/api/routes/clips_standalone.py	🆕	独立端点 POST /api/v1/clips/from-assets（tags=[Clips]），template_id 从 body 传入可选，为空时后端自动兜底，彻底消除双斜杠路径问题
app/api/routes/templates_editor/dependencies.py	修改	抽出纯函数 resolve_draft_plan_id()（支持 auto_create_default=True 空 tid 兜底），原 get_draft_plan_id Depends 改为薄封装
app/api/routes/templates_editor/__init__.py	修改	导出 resolve_draft_plan_id
app/api/router.py	修改	注册 clips_standalone_router
tests/unit/test_default_template_fallback_p0.py	🆕	16 个单元测试，覆盖兜底函数/resolve/list端点/standalone端点/路由注册

前端配套

前端 useStep2Materials 需改为调用新端点 POST /api/v1/clips/from-assets，templateId 作为 body 字段传入（为空不拼路径，后端自动兜底）。

验证方式

• CI 全绿（含 unit tests + diff coverage ≥ 40%）
• staging E2E：新用户首次访问生成页 → GET /templates 返回默认模板 → 点击「智能选片」→ POST /api/v1/clips/from-assets 正常创建片段，不再 404

备注

旧 PR #1921 因分支基于过期 develop（未合入 #1919/#1920/#1924），本次重建分支基于最新 develop 重提。

## P0 Bug 修复 关联 Issue: #1922 ### 根因 双 Bug 链： 1. **PR#1918 清理模板管理 API 时误删** GET /templates 自动创建默认模板的兜底逻辑； 2. **前端 PR#1913 去掉空 tid 拦截**后，新用户首次进入生成页 `templateId=''`，拼出 URL `/templates//editor/clips/from-assets`（双斜杠）→ FastAPI 404，完全阻断首次生成流程。 ### 修复方案（三层后端兜底 + 独立端点） | 文件 | 改动 | 作用 | |---|---|---| | `app/api/routes/_default_template.py` | 🆕 | 共享 `get_or_create_default_template_id()`：查有效模板 → 无则 CreateTemplateUseCase 创建默认 voice_over 模板 → 创建异常回滚后重查（防并发唯一键冲突）| | `app/api/routes/templates.py` | 修改 | GET /templates 列表查询前调用兜底，确保生成页首次拉模板列表时一定有数据 | | `app/api/routes/clips_standalone.py` | 🆕 | 独立端点 `POST /api/v1/clips/from-assets`（tags=[Clips]），`template_id` 从 body 传入可选，为空时后端自动兜底，**彻底消除双斜杠路径问题** | | `app/api/routes/templates_editor/dependencies.py` | 修改 | 抽出纯函数 `resolve_draft_plan_id()`（支持 auto_create_default=True 空 tid 兜底），原 `get_draft_plan_id` Depends 改为薄封装 | | `app/api/routes/templates_editor/__init__.py` | 修改 | 导出 `resolve_draft_plan_id` | | `app/api/router.py` | 修改 | 注册 `clips_standalone_router` | | `tests/unit/test_default_template_fallback_p0.py` | 🆕 | 16 个单元测试，覆盖兜底函数/resolve/list端点/standalone端点/路由注册 | ### 前端配套 前端 `useStep2Materials` 需改为调用新端点 `POST /api/v1/clips/from-assets`，`templateId` 作为 body 字段传入（为空不拼路径，后端自动兜底）。 ### 验证方式 - CI 全绿（含 unit tests + diff coverage ≥ 40%） - staging E2E：新用户首次访问生成页 → GET /templates 返回默认模板 → 点击「智能选片」→ POST /api/v1/clips/from-assets 正常创建片段，不再 404 ### 备注 旧 PR #1921 因分支基于过期 develop（未合入 #1919/#1920/#1924），本次重建分支基于最新 develop 重提。

xiaoxia added 1 commit 2026-09-15 11:37:43 +08:00

fix(p0): #1922 from-assets 空 templateId 双斜杠 404 兜底 ... f10106bc51

根因：
1. PR#1918 清理模板管理 API 时误删了 GET /templates 自动创建默认模板兜底逻辑；
2. 前端 PR#1913 去掉空 tid 拦截后，新用户首次进入生成页 templateId=''，
   拼出 URL /templates//editor/clips/from-assets（双斜杠）→ FastAPI 404，
   阻断首次生成流程。

修复（三层兜底）：
- 新增 _default_template.py，提供共享函数 get_or_create_default_template_id：
  查有效模板 → 无则 CreateTemplateUseCase 创建默认 voice_over 模板 →
  创建异常回滚后重查（防并发唯一键冲突）。
- GET /templates 列表查询前调用兜底，确保生成页首次拉模板列表时一定有数据。
- 新增独立端点 POST /api/v1/clips/from-assets（tags=[Clips]），template_id 从
  body 传入（可选），为空时后端自动兜底，彻底消除双斜杠路径问题。
- dependencies.py 抽出纯函数 resolve_draft_plan_id（支持 auto_create_default），
  原 get_draft_plan_id Depends 改为薄封装。
- router.py 注册 clips_standalone_router。

配套：16 个单元测试覆盖兜底函数/resolve/list端点/standalone端点/路由注册，
保证 diff coverage ≥ 40% 过 CI 门禁。

xiaoxia referenced this pull request 2026-09-15 11:37:52 +08:00

fix(p0): from-assets空templateId双斜杠404兜底 #1921

gitea-actions bot commented 2026-09-15 11:39:34 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-09-15 11:42:43 +08:00

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

auto-approve-bot merged commit 352221f199 into develop 2026-09-15 11:54:06 +08:00

auto-approve-bot deleted branch fix/p0-from-assets-default-template-fallback-v2 2026-09-15 11:54:06 +08:00

gitea-actions bot commented 2026-09-15 11:54:53 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1926 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

xiaoxia referenced this issue from a commit 2026-09-15 13:38:04 +08:00

fix(e2e): replace removed POST /templates with GET /templates (#1926 auto-default)

xiaoxia referenced this pull request 2026-09-15 13:38:17 +08:00

fix(e2e): 替换已删除的 POST /templates 为 GET /templates 获取默认模板 #1931

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot frontend-dev xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1926