feat(phase8): 任务 2.03 — API 模板管理 CRUD #144

2026-07-01T14:33:25+08:00

xiaoxia commented

2026-07-01 14:33:25 +08:00

任务 2.03: API 模板管理 CRUD

变更内容

新增 5 个 RESTful 端点:
- GET /api/v1/edit-templates — 列表（分页 + 类型/状态筛选）
- GET /api/v1/edit-templates/{id} — 详情
- POST /api/v1/edit-templates — 创建
- PUT /api/v1/edit-templates/{id} — 更新（部分更新）
- DELETE /api/v1/edit-templates/{id} — 软删除（→ inactive）
Pydantic Schema 请求/响应验证
扩展 SQLAlchemyEditTemplateRepository.count() 支持 template_type 筛选
路由注册到 router.py

测试

28 个单元测试全部通过
覆盖：CRUD 正常流程 + 异常场景（404、400、422）+ 分页 + 筛选 + Schema 验证

关联任务

依赖: #2.01 (EditTemplate 数据模型), #2.02 (TemplateClipConfig)
docs/task-breakdown-v2.md §4.3

## 任务 2.03: API 模板管理 CRUD ### 变更内容 - 新增 5 个 RESTful 端点: - `GET /api/v1/edit-templates` — 列表（分页 + 类型/状态筛选） - `GET /api/v1/edit-templates/{id}` — 详情 - `POST /api/v1/edit-templates` — 创建 - `PUT /api/v1/edit-templates/{id}` — 更新（部分更新） - `DELETE /api/v1/edit-templates/{id}` — 软删除（→ inactive） - Pydantic Schema 请求/响应验证 - 扩展 `SQLAlchemyEditTemplateRepository.count()` 支持 `template_type` 筛选 - 路由注册到 `router.py` ### 测试 - 28 个单元测试全部通过 - 覆盖：CRUD 正常流程 + 异常场景（404、400、422）+ 分页 + 筛选 + Schema 验证 ### 关联任务 - 依赖: #2.01 (EditTemplate 数据模型), #2.02 (TemplateClipConfig) - docs/task-breakdown-v2.md §4.3

xiaoxia added 1 commit 2026-07-01 14:33:26 +08:00

feat(phase8): 实现模板管理 CRUD API (#2.03)

Deploy / Build Production Runtime Images (push) Has been skipped

Details

Deploy / Deploy Production (push) Has been skipped

Details

Deploy / Production Browser E2E (push) Has been skipped

Details

CI/CD Pipeline / Frontend Lint (pull_request) Failing after 189h53m28s

Details

CI/CD Pipeline / Validate Code Quality And Tests (pull_request) Failing after 189h53m31s

Details

Deploy / Deploy Staging (push) Failing after 190h12m9s

Details

CI/CD Pipeline / Frontend Lint (push) Failing after 190h12m43s

Details

CI/CD Pipeline / Validate Code Quality And Tests (push) Failing after 190h12m47s

Details

6182aa4682

- 新增 5 个 RESTful 端点: GET/POST/PUT/DELETE /api/v1/edit-templates
- 支持分页、按类型/状态筛选
- Pydantic Schema 请求/响应验证
- 软删除模式 (DELETE → inactive)
- 扩展 Repository.count() 支持 template_type 筛选
- 28 个单元测试全部通过

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

xiaoxia commented

2026-07-01 14:36:19 +08:00

代码审计 — PR #144 审查结果

结论：✅ 通过，可合并

总体评价

Phase 8 任务 2.03 API 模板管理 CRUD 实现质量高。5 个 RESTful 端点设计规范（GET list/detail、POST create、PUT update、DELETE soft-delete），HTTP 状态码使用正确（201/204/400/404/422），Pydantic Schema 分离 Create/Update/Response 职责清晰。软删除使用领域方法 deactivate() 而非直接设字符串，保持领域逻辑内聚。仓储层 count() 扩展 template_type 筛选参数，与 list_all 保持一致。测试采用 Stub Repository + FastAPI TestClient + dependency_overrides，28 个用例覆盖 CRUD 正常+异常、分页、筛选、Schema 验证。

P3 建议（不阻塞合并）

1. 创建模板无管理员权限校验

docstring 标注 创建模板（管理员） 但代码仅检查 get_current_user（认证），未检查角色（授权）
当前任何已认证用户均可创建/修改/删除模板
建议：后续添加 admin 角色检查（如 require_admin 依赖注入），或在领域层记录审计日志

2. update_template 中 description 的 falsy 检查不一致

new_name 用 strip() if new_name else existing.name（falsy 检查）
new_description 用 strip() if new_description is not None else existing.description（None 检查）
效果差异：name 传空字符串 → 保留原值；description 传空字符串 → 清空为空串
建议：统一为 None 检查（is not None），语义更明确

3. _to_response 的 hasattr 检查可简化

t.status.value if hasattr(t.status, "value") else t.status 过于防御性
EditTemplate.status 类型固定为 EditTemplateStatus（StrEnum），直接 .value 即可
建议：简化为 status=t.status.value

审查基于 commit 6182aa46

## 代码审计 — PR #144 审查结果 **结论：✅ 通过，可合并** ### 总体评价 Phase 8 任务 2.03 API 模板管理 CRUD 实现质量高。5 个 RESTful 端点设计规范（GET list/detail、POST create、PUT update、DELETE soft-delete），HTTP 状态码使用正确（201/204/400/404/422），Pydantic Schema 分离 Create/Update/Response 职责清晰。软删除使用领域方法 `deactivate()` 而非直接设字符串，保持领域逻辑内聚。仓储层 `count()` 扩展 `template_type` 筛选参数，与 `list_all` 保持一致。测试采用 Stub Repository + FastAPI TestClient + dependency_overrides，28 个用例覆盖 CRUD 正常+异常、分页、筛选、Schema 验证。 ### P3 建议（不阻塞合并） **1. 创建模板无管理员权限校验** - docstring 标注 `创建模板（管理员）` 但代码仅检查 `get_current_user`（认证），未检查角色（授权） - 当前任何已认证用户均可创建/修改/删除模板 - 建议：后续添加 admin 角色检查（如 `require_admin` 依赖注入），或在领域层记录审计日志 **2. update_template 中 description 的 falsy 检查不一致** - `new_name` 用 `strip() if new_name else existing.name`（falsy 检查） - `new_description` 用 `strip() if new_description is not None else existing.description`（None 检查） - 效果差异：name 传空字符串 → 保留原值；description 传空字符串 → 清空为空串 - 建议：统一为 None 检查（`is not None`），语义更明确 **3. _to_response 的 hasattr 检查可简化** - `t.status.value if hasattr(t.status, "value") else t.status` 过于防御性 - EditTemplate.status 类型固定为 EditTemplateStatus（StrEnum），直接 `.value` 即可 - 建议：简化为 `status=t.status.value` --- *审查基于 commit 6182aa46*

xiaoxia referenced this issue from a commit

2026-07-01 14:41:35 +08:00

feat(phase8-task203): API 模板管理 CRUD - RESTful接口 + Pydantic验证 + 单元测试 (#144)

xiaoxia closed this pull request

2026-07-01 14:41:39 +08:00

xiaoxia referenced this pull request

2026-07-01 15:08:20 +08:00

feat(task-2.14): 剪辑计划编辑器完整版 — 三栏布局 + 拖拽 + 素材面板 #145