xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 12 Pull Requests 3 Actions 3 Packages Projects Releases 7 Wiki Activity

fix(#1341): 一次性修复封面生成两个问题（confirm 路径 + 超时） #1349

Closed
xiaoxia wants to merge 1 commits from fix/cover-issues-1341 into develop
pull from: fix/cover-issues-1341
merge into: xiaoxia:develop
Conversation 3 Commits 1 Files Changed 2
+2 -2
xiaoxia commented 2026-08-13 00:19:34 +08:00
Owner
Copy Link
Copy Source

修复内容

1. confirm API 路径（修复 404 错误）

  • 文件：apps/web/src/api/generation/confirm.ts
  • 修改：/tasks/${taskId}/confirm → /generation/tasks/${taskId}/confirm
  • 原因：后端路由注册在 prefix="/generation" 下，前端路径缺少前缀

2. 封面生成超时（修复 180s 超时）

  • 文件：apps/web/src/api/template-editor/aiFeatures.ts
  • 修改：timeout: 180000 → timeout: 300000
  • 原因：MediaKit 抽帧处理时间较长，需要更长超时容忍度

测试

  • 等待 CI 全绿后合并到 develop
  • 部署到 staging 后实测验证
## 修复内容 ### 1. confirm API 路径（修复 404 错误） - 文件：`apps/web/src/api/generation/confirm.ts` - 修改：`/tasks/${taskId}/confirm` → `/generation/tasks/${taskId}/confirm` - 原因：后端路由注册在 `prefix="/generation"` 下，前端路径缺少前缀 ### 2. 封面生成超时（修复 180s 超时） - 文件：`apps/web/src/api/template-editor/aiFeatures.ts` - 修改：`timeout: 180000` → `timeout: 300000` - 原因：MediaKit 抽帧处理时间较长，需要更长超时容忍度 ## 测试 - 等待 CI 全绿后合并到 develop - 部署到 staging 后实测验证
xiaoxia added 1 commit 2026-08-13 00:19:34 +08:00
fix(#1341): 一次性修复封面生成两个问题
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 46s
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped
Details
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m26s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m42s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m46s
Details
AI Code Review / AI Code Review (pull_request) Successful in 2m1s
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 1m17s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m6s
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 1m36s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Failing after 2m31s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m25s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 6m3s
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / CI Gate (pull_request) Failing after 20s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Waiting to run
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 47s
Details
82b4c80326 
1. confirm API 路径：补充 /generation 前缀
   - /tasks/{id}/confirm → /generation/tasks/{id}/confirm
   - 修复 404 错误

2. 封面生成超时：180s → 300s
   - MediaKit 抽帧处理时间较长，增加超时容忍度
gitea-actions bot commented 2026-08-13 00:21:02 +08:00
Copy Link
Copy Source

🚀 预览环境已部署

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

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

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

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

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

【阻塞级判定】

  • 是否存在阻塞级问题：否
  • 阻塞级问题数量：0 个

📊 审查概览

  • 整体评价：有建议
  • 建议级问题数量：2 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

  1. [apps/web/src/api/generation/confirm.ts: 10] API 路径变更一致性

    • 具体内容：API 请求路径从 /tasks/${taskId}/confirm 修改为 /generation/tasks/${taskId}/confirm。请确认后端服务已同步更新路由配置，否则会导致前端发起请求后收到 404 错误，影响功能可用性。

  2. [apps/web/src/api/template-editor/aiFeatures.ts: 27] 超时配置与基础设施匹配

    • 具体内容：请求超时时间调整为 300秒（5分钟）。请确认反向代理（如 Nginx）、API 网关或服务端的 KeepAlive/ReadTimeout 配置均大于 300秒，否则请求会在中间层被提前断开，导致客户端超时设置失效。

✅ 良好实践

  • 注释同步更新：aiFeatures.ts 中超时时间的修改已同步更新了注释说明，保持了代码与注释的一致性。

🤖 由 AI 代码审查机器人自动生成 | 2026-08-12 16:21:27 | 模型:

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：有建议 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/web/src/api/generation/confirm.ts: 10] API 路径变更一致性** - 具体内容：API 请求路径从 `/tasks/${taskId}/confirm` 修改为 `/generation/tasks/${taskId}/confirm`。请确认后端服务已同步更新路由配置，否则会导致前端发起请求后收到 404 错误，影响功能可用性。 2. **[apps/web/src/api/template-editor/aiFeatures.ts: 27] 超时配置与基础设施匹配** - 具体内容：请求超时时间调整为 300秒（5分钟）。请确认反向代理（如 Nginx）、API 网关或服务端的 KeepAlive/ReadTimeout 配置均大于 300秒，否则请求会在中间层被提前断开，导致客户端超时设置失效。 ### ✅ 良好实践 - **注释同步更新**：`aiFeatures.ts` 中超时时间的修改已同步更新了注释说明，保持了代码与注释的一致性。 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-08-12 16:21:27 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
xiaoxia closed this pull request 2026-08-13 00:57:23 +08:00
gitea-actions bot commented 2026-08-13 00:58:09 +08:00
Copy Link
Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1349 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。
Some checks are pending
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 46s
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped
Details
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m26s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m42s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m46s
Details
AI Code Review / AI Code Review (pull_request) Successful in 2m1s
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 1m17s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m6s
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 1m36s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Failing after 2m31s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m25s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 6m3s
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / CI Gate (pull_request) Failing after 20s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Waiting to run
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 47s
Details

Pull request closed

Please reopen this pull request to perform a merge.
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
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1349
Delete Branch "fix/cover-issues-1341"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?