feat: 前端对接确认生成 API (POST /tasks/{task_id}/confirm) #1310

Merged

auto-approve-bot merged 5 commits from feat/confirm-generation-api-frontend into develop 2026-08-09 09:11:01 +08:00

Conversation 5 Commits 5 Files Changed 5

+84

xiaoxia commented 2026-08-08 21:07:54 +08:00

Owner

Copy Link

Copy Source

概述

对接后端 PR #1308 新增的确认生成 API。

改动

1. 新增 API 调用 (confirm.ts)

• confirmGeneration(taskId, params) 调用 POST /tasks/{task_id}/confirm

2. 新增类型定义

• ConfirmGenerationRequest: output_width, output_height, cover_url, custom_title
• ConfirmGenerationResponse, ConfirmGenerationTaskItem

3. 修改 useGenerateVideo.ts

• 有 previewTaskId 时使用新 API（确认生成流程）
• 无 previewTaskId 时使用旧 API（向后兼容）
• 传递 cover_url 和 custom_title 参数

请求体格式

interface ConfirmGenerationRequest {
  output_width?: number  // 默认 1080
  output_height?: number // 默认 1920
  cover_url?: string     // 自定义封面 URL
  custom_title?: string  // 自定义标题
}

前置条件

⚠️ 此 PR 依赖后端 PR #1308 同步到 develop 分支

后端 PR #1308 目前合并到了 main，需要先同步到 develop 后此 PR 才能正常构建和合并。

@saas后端程序 请先将 PR #1308 的改动同步到 develop 分支（cherry-pick 或新 PR）。

关联

• 后端 PR #1308: 确认生成 API 改为复用 worker.generate_video 渲染路径

## 概述 对接后端 PR #1308 新增的确认生成 API。 ## 改动 ### 1. 新增 API 调用 (`confirm.ts`) - `confirmGeneration(taskId, params)` 调用 `POST /tasks/{task_id}/confirm` ### 2. 新增类型定义 - `ConfirmGenerationRequest`: output_width, output_height, cover_url, custom_title - `ConfirmGenerationResponse`, `ConfirmGenerationTaskItem` ### 3. 修改 `useGenerateVideo.ts` - 有 `previewTaskId` 时使用新 API（确认生成流程） - 无 `previewTaskId` 时使用旧 API（向后兼容） - 传递 cover_url 和 custom_title 参数 ## 请求体格式 ```typescript interface ConfirmGenerationRequest { output_width?: number // 默认 1080 output_height?: number // 默认 1920 cover_url?: string // 自定义封面 URL custom_title?: string // 自定义标题 } ``` ## 前置条件 ⚠️ **此 PR 依赖后端 PR #1308 同步到 develop 分支** 后端 PR #1308 目前合并到了 main，需要先同步到 develop 后此 PR 才能正常构建和合并。 @saas后端程序 请先将 PR #1308 的改动同步到 develop 分支（cherry-pick 或新 PR）。 ## 关联 - 后端 PR #1308: 确认生成 API 改为复用 worker.generate_video 渲染路径

xiaoxia added 1 commit 2026-08-08 21:07:55 +08:00

feat: 前端对接确认生成 API (POST /tasks/{task_id}/confirm) ... 13ca16b940

对接后端 PR #1308 新增的确认生成 API：

1. 新增 API 调用函数 confirm.ts
   - confirmGeneration(taskId, params) 调用 POST /tasks/{task_id}/confirm

2. 新增类型定义
   - ConfirmGenerationRequest: output_width, output_height, cover_url, custom_title
   - ConfirmGenerationResponse, ConfirmGenerationTaskItem

3. 修改 useGenerateVideo.ts
   - 有 previewTaskId 时使用新 API（确认生成流程）
   - 无 previewTaskId 时使用旧 API（向后兼容）
   - 传递 cover_url 和 custom_title 参数

前置条件：后端 PR #1308 同步到 develop 分支后此 PR 才可合并

关联：PR #1308 (后端确认生成 API)

xiaoxia added 1 commit 2026-08-09 08:54:28 +08:00

chore: trigger CI re-run after backend sync (#1312) e62da06a27

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

fix: 修复 CoverConfig 属性名 imageUrl → upload_url (TS2339) f149c9d9b3

gitea-actions bot commented 2026-08-09 08:59:47 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-08-09 09:02:12 +08:00

style: prettier format confirm.ts (trailing newline) 33e650e917

xiaoxia added 1 commit 2026-08-09 09:06:17 +08:00

style: fix trailing newline in confirm.ts 6262f6c5c3

auto-approve-bot approved these changes 2026-08-09 09:08:07 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-09 09:08:07 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot commented 2026-08-09 09:08:42 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1310

⚠️ 问题（1个需要修改）

1. apps/web/src/pages/generate/hooks/useGenerateVideo.ts 第66-70行：逻辑错误，忽略了新创建的任务ID。
   • 问题描述：confirmGeneration API 返回了 ConfirmGenerationResponse，其中包含 items 数组，数组内的对象应当包含新创建的正式生成任务的 id。代码当前直接丢弃了返回值并调用 startPolling()。如果 startPolling 默认继续轮询 previewTaskId（预览任务），由于预览任务已完成且状态已固定，轮询将无法获取到新任务的进度，导致前端状态停滞或显示错误。
   • 后果：用户无法看到正式生成任务的进度，功能失效。
   • 修改建议：从 response.data.items[0].id 获取新任务 ID，并更新用于轮询的任务 ID 状态（或将其传给 startPolling）。

💡 建议（2个可选）

1. apps/web/src/pages/generate/hooks/useGenerateVideo.ts 第62行：建议优化分辨率解析逻辑。
   • 具体内容：使用 split("x") 解析分辨率较为脆弱，如果 props.videoRatio 格式异常（如包含空格、非 'x' 分隔符）可能导致解析不符合预期。虽然当前有默认值兜底，但建议使用正则表达式提取数字，或增加对分隔符的校验。
2. apps/web/src/pages/generate/hooks/useGenerateVideo.ts 第67-69行：建议区分“未设置”与“空字符串”。
   • 具体内容：代码使用 || "" 将 undefined 转换为空字符串。如果后端 API 语义中，不传字段表示“使用默认值”，而传空字符串表示“强制清空”，则此逻辑可能导致用户未设置封面/标题时，后端错误地将其清空。建议仅在值存在时才传递字段，或明确后端对空字符串的处理逻辑。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-09 01:08:42 | 模型:}

## 代码审查结果 - PR #1310 ### ⚠️ 问题（1个需要修改） 1. **apps/web/src/pages/generate/hooks/useGenerateVideo.ts 第66-70行**：逻辑错误，忽略了新创建的任务ID。 - **问题描述**：`confirmGeneration` API 返回了 `ConfirmGenerationResponse`，其中包含 `items` 数组，数组内的对象应当包含新创建的正式生成任务的 `id`。代码当前直接丢弃了返回值并调用 `startPolling()`。如果 `startPolling` 默认继续轮询 `previewTaskId`（预览任务），由于预览任务已完成且状态已固定，轮询将无法获取到新任务的进度，导致前端状态停滞或显示错误。 - **后果**：用户无法看到正式生成任务的进度，功能失效。 - **修改建议**：从 `response.data.items[0].id` 获取新任务 ID，并更新用于轮询的任务 ID 状态（或将其传给 `startPolling`）。 ### 💡 建议（2个可选） 1. **apps/web/src/pages/generate/hooks/useGenerateVideo.ts 第62行**：建议优化分辨率解析逻辑。 - **具体内容**：使用 `split("x")` 解析分辨率较为脆弱，如果 `props.videoRatio` 格式异常（如包含空格、非 'x' 分隔符）可能导致解析不符合预期。虽然当前有默认值兜底，但建议使用正则表达式提取数字，或增加对分隔符的校验。 2. **apps/web/src/pages/generate/hooks/useGenerateVideo.ts 第67-69行**：建议区分“未设置”与“空字符串”。 - **具体内容**：代码使用 `|| ""` 将 `undefined` 转换为空字符串。如果后端 API 语义中，不传字段表示“使用默认值”，而传空字符串表示“强制清空”，则此逻辑可能导致用户未设置封面/标题时，后端错误地将其清空。建议仅在值存在时才传递字段，或明确后端对空字符串的处理逻辑。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-09 01:08:42 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit 2c4fabb585 into develop 2026-08-09 09:11:01 +08:00

auto-approve-bot deleted branch feat/confirm-generation-api-frontend 2026-08-09 09:11:01 +08:00

gitea-actions bot commented 2026-08-09 09:12:40 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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