fix(api): editingPlanner 对接后端 /templates API，删除遗留 editPlans.ts #108

2026-06-29T16:06:33+08:00

xiaoxia commented

2026-06-29 16:06:33 +08:00

变更内容

问题1（高优先级）：editingPlanner.ts 仍使用 mock 数据

移除全部 mock 数据和 USE_MOCK 开关（apiClient 导入已启用）
6 个 API 函数对接后端真实端点：
- getEditingTemplates → GET /templates
- getEditingTemplate → GET /templates/{id}
- createEditingTemplate → POST /templates
- updateEditingTemplate → PATCH /templates/{id}（使用 PATCH 与后端一致）
- deleteEditingTemplate → DELETE /templates/{id}
- getTemplateCategories → GET /templates/categories/list
- generateFromTemplate → POST /templates/{id}/validate
类型定义与后端 schema 对齐（is_active、分页 total、ValidateWarning 等）

问题2（低优先级）：清理遗留文件

删除 src/api/editPlans.ts（无任何引用）

变更统计

2 files changed, 194 insertions(+), 412 deletions(-)
editPlans.ts 已删除（-98 行）
editingPlanner.ts 从 314 行精简到 194 行

## 变更内容 ### 问题1（高优先级）：editingPlanner.ts 仍使用 mock 数据 - 移除全部 mock 数据和 `USE_MOCK` 开关（`apiClient` 导入已启用） - 6 个 API 函数对接后端真实端点： - `getEditingTemplates` → `GET /templates` - `getEditingTemplate` → `GET /templates/{id}` - `createEditingTemplate` → `POST /templates` - `updateEditingTemplate` → `PATCH /templates/{id}`（使用 PATCH 与后端一致） - `deleteEditingTemplate` → `DELETE /templates/{id}` - `getTemplateCategories` → `GET /templates/categories/list` - `generateFromTemplate` → `POST /templates/{id}/validate` - 类型定义与后端 schema 对齐（`is_active`、分页 `total`、`ValidateWarning` 等） ### 问题2（低优先级）：清理遗留文件 - 删除 `src/api/editPlans.ts`（无任何引用） ### 变更统计 - 2 files changed, 194 insertions(+), 412 deletions(-) - editPlans.ts 已删除（-98 行） - editingPlanner.ts 从 314 行精简到 194 行

xiaoxia added 1 commit 2026-06-29 16:06:33 +08:00

fix(api): editingPlanner 对接后端 /templates API，删除遗留 editPlans.ts

CI/CD Pipeline / Validate Code Quality And Tests (pull_request) Has been cancelled

Details

CI/CD Pipeline / Frontend Lint (pull_request) Has been cancelled

Details

2d05aa0bcc

- 移除全部 mock 数据和 USE_MOCK 开关
- 6 个函数对接后端真实端点（list/get/create/update/delete/validate）
- getTemplateCategories 对接 /templates/categories/list
- 删除无引用的 editPlans.ts 遗留文件

xiaoxia commented

2026-06-29 16:30:43 +08:00

✅ 代码审计通过

变更概要

删除 editPlans.ts（-98行）：清理无引用的遗留文件 ✅
改造 editingPlanner.ts（+52 -172）：从 mock 数据全面切换到真实后端 API ✅

逐项审查

#	检查项	结论
1	Mock 数据及 `USE_MOCK` 开关完全移除	✅ 清理干净
2	`apiClient` 导入已启用	✅
3	7 个 API 函数均对接正确的后端端点	✅
4	`TemplateMode` 枚举值与后端一致（pip/voice_over/one_take/voice_pip）	✅
5	`TemplateSegment.id` 改为可选 — 新片段由后端生成 ID，合理	✅
6	`ValidateWarning` 新类型 + `GenerateFromTemplateResponse` 改为返回 template+warnings — 与 PR#105 后端 validate 端点对齐	✅
7	`editPlans.ts` 在 API 目录中确认已无残留引用	✅
8	TypeScript 类型定义与后端 schema 匹配	✅

P2 建议（不阻塞合并）

getEditingTemplates 中 category/tag 仍在客户端过滤：函数接收 category/tag 参数并传给后端 query params，但同时也做了客户端 filter。如果后端已支持这些查询参数，客户端过滤是冗余的；如果后端不支持，则不应传给后端。建议对齐。
templates.ts 与 editingPlanner.ts 存在端点重叠：两个文件都调用 GET /templates 和 GET /templates/{id}，但返回不同类型（TemplateItem vs EditingTemplate）。后续可考虑统一或明确区分，避免维护困惑。

结论

✅ 通过，同意合并。 变更简洁清晰，mock 到真实 API 的迁移完整无误。

## ✅ 代码审计通过 ### 变更概要 - **删除** `editPlans.ts`（-98行）：清理无引用的遗留文件 ✅ - **改造** `editingPlanner.ts`（+52 -172）：从 mock 数据全面切换到真实后端 API ✅ ### 逐项审查 | # | 检查项 | 结论 | |---|--------|------| | 1 | Mock 数据及 `USE_MOCK` 开关完全移除 | ✅ 清理干净 | | 2 | `apiClient` 导入已启用 | ✅ | | 3 | 7 个 API 函数均对接正确的后端端点 | ✅ | | 4 | `TemplateMode` 枚举值与后端一致（pip/voice_over/one_take/voice_pip） | ✅ | | 5 | `TemplateSegment.id` 改为可选 — 新片段由后端生成 ID，合理 | ✅ | | 6 | `ValidateWarning` 新类型 + `GenerateFromTemplateResponse` 改为返回 template+warnings — 与 PR#105 后端 validate 端点对齐 | ✅ | | 7 | `editPlans.ts` 在 API 目录中确认已无残留引用 | ✅ | | 8 | TypeScript 类型定义与后端 schema 匹配 | ✅ | ### P2 建议（不阻塞合并） 1. **`getEditingTemplates` 中 category/tag 仍在客户端过滤**：函数接收 `category`/`tag` 参数并传给后端 query params，但同时也做了客户端 `filter`。如果后端已支持这些查询参数，客户端过滤是冗余的；如果后端不支持，则不应传给后端。建议对齐。 2. **`templates.ts` 与 `editingPlanner.ts` 存在端点重叠**：两个文件都调用 `GET /templates` 和 `GET /templates/{id}`，但返回不同类型（`TemplateItem` vs `EditingTemplate`）。后续可考虑统一或明确区分，避免维护困惑。 ### 结论 **✅ 通过，同意合并。** 变更简洁清晰，mock 到真实 API 的迁移完整无误。

xiaoxia merged commit 8c4c428cdc into develop

2026-06-29 16:33:23 +08:00

xiaoxia referenced this issue from a commit

2026-06-29 16:33:24 +08:00

fix(api): editingPlanner 对接后端 /templates API，删除遗留 editPlans.ts (#108)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#108