refactor(useGenerateVideo): 拆分 useGenerateVideo Hook（370→221行, -40%） #969

Merged

auto-approve-bot merged 15 commits from refactor/generate-video-hook into develop 2026-07-27 22:36:45 +08:00

Conversation 6 Commits 15 Files Changed 15

+347 -226

xiaoxia commented 2026-07-26 18:08:07 +08:00

Owner

Copy Link

Copy Source

变更说明

useGenerateVideo 视频生成 Hook 拆分重构，主文件 370→221 行（-40%）

拆分文件（generate-video/ 子目录）

• types.ts: UseGenerateVideoProps / GenerationPhase 类型定义
• phase.ts: getGenerationPhase 阶段映射
• errorUtils.ts: safeExtractError / translateError / extractBackendError（3个错误处理工具）
• voiceConfig.ts: buildVoiceConfig 配音配置构建
• useGenerationPolling.ts: 生成状态轮询 Hook（独立封装轮询逻辑）

收益

• 主 Hook 只做状态管理 + 流程编排，可读性大幅提升
• 轮询、错误处理、配音配置等独立模块，便于单测和复用
• 新增功能时修改对应的小文件，不会让主 Hook 继续膨胀

## 变更说明 useGenerateVideo 视频生成 Hook 拆分重构，主文件 370→221 行（-40%） ### 拆分文件（generate-video/ 子目录） - **types.ts**: UseGenerateVideoProps / GenerationPhase 类型定义 - **phase.ts**: getGenerationPhase 阶段映射 - **errorUtils.ts**: safeExtractError / translateError / extractBackendError（3个错误处理工具） - **voiceConfig.ts**: buildVoiceConfig 配音配置构建 - **useGenerationPolling.ts**: 生成状态轮询 Hook（独立封装轮询逻辑） ### 收益 - 主 Hook 只做状态管理 + 流程编排，可读性大幅提升 - 轮询、错误处理、配音配置等独立模块，便于单测和复用 - 新增功能时修改对应的小文件，不会让主 Hook 继续膨胀

xiaoxia added 1 commit 2026-07-26 18:08:07 +08:00

refactor(useGenerateVideo): 拆分 useGenerateVideo Hook（370→221行, -40%） ... 8f0a202130

- 抽 generate-video/ 子目录，按职责拆分6个文件
- types.ts: 入参/返回值/阶段类型
- phase.ts: getGenerationPhase 阶段映射
- errorUtils.ts: safeExtractError/translateError/extractBackendError
- voiceConfig.ts: buildVoiceConfig 配音配置构建
- useGenerationPolling.ts: 生成状态轮询 Hook
- 主 Hook 精简为状态管理 + 编排逻辑，可读性大幅提升

xiaoxia added 1 commit 2026-07-27 12:34:44 +08:00

Merge branch 'develop' into refactor/generate-video-hook a759be19f9

xiaoxia added 1 commit 2026-07-27 15:30:54 +08:00

Merge branch 'develop' into refactor/generate-video-hook ee0a2f7222

xiaoxia added 1 commit 2026-07-27 18:08:00 +08:00

Merge branch 'develop' into refactor/generate-video-hook e8c7562a28

xiaoxia added 1 commit 2026-07-27 18:15:39 +08:00

fix: remove unused imports 6556016471

xiaoxia added 1 commit 2026-07-27 18:28:23 +08:00

fix: correct type alias syntax 942d81e3e7

xiaoxia added 1 commit 2026-07-27 19:04:16 +08:00

style: prettier format types.ts 4dc88c1721

xiaoxia added 1 commit 2026-07-27 19:04:24 +08:00

style: prettier format useGenerateVideo.ts e323d6ebeb

xiaoxia added 1 commit 2026-07-27 19:11:00 +08:00

chore: remove debug console.log 1709f1c165

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

Merge branch 'develop' into refactor/generate-video-hook c579b48e05

xiaoxia added 1 commit 2026-07-27 21:26:14 +08:00

fix: 修复TS错误 — 导入路径层级/类型转换 7591f08faa

gitea-actions bot commented 2026-07-27 21:30:58 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-07-27 21:33:15 +08:00

fix: 移除不必要的useCallback依赖 newTitleType f1348e9fac

xiaoxia referenced this issue from a commit 2026-07-27 21:41:42 +08:00

refactor(useGenerateVideo): 拆分 useGenerateVideo Hook（370→221行, -40%） (#969)

xiaoxia added 1 commit 2026-07-27 21:41:46 +08:00

style: prettier格式化5个文件 e48c96633d

xiaoxia closed this pull request 2026-07-27 21:41:48 +08:00

xiaoxia reopened this pull request 2026-07-27 21:46:08 +08:00

gitea-actions bot commented 2026-07-27 21:47:10 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia closed this pull request 2026-07-27 22:01:15 +08:00

gitea-actions bot commented 2026-07-27 22:01:26 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia reopened this pull request 2026-07-27 22:03:51 +08:00

xiaoxia closed this pull request 2026-07-27 22:06:23 +08:00

xiaoxia reopened this pull request 2026-07-27 22:08:36 +08:00

gitea-actions bot commented 2026-07-27 22:09:25 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia added 1 commit 2026-07-27 22:27:39 +08:00

test: 补充generate-video和voice-materials smoke测试覆盖 b4f80d41ed

auto-approve-bot commented 2026-07-27 22:35:06 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #969

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

1. apps/web/src/test/pages/generate/useGenerateVideo.test.ts 第11-14行：测试文件引用了不存在的导出变量，会导致构建失败。

   • 说明phase.ts 仅导出了 getGenerationPhase，未导出 PHASE_ORDER 和 getNextPhase；errorUtils.ts 未导出 extractErrorMessage；voiceConfig.ts 未导出 getDefaultVoiceConfig。
   • 后果：测试运行时会抛出 Module has no exported member 错误，阻断 CI/CD 流程。

2. apps/web/src/pages/titles/hooks/useTitleEdit.ts 第62行：useCallback 依赖数组中移除了 newTitleType，存在闭包陷阱风险。

   • 说明：代码从依赖数组中删除了 newTitleType。如果 handleCreate 函数内部使用了 newTitleType 变量（根据变量名推测极大概率使用），函数将始终使用初始值或旧值，导致创建标题时类型错误。
   • 后果：用户选择新的标题类型后点击创建，实际创建的仍是旧类型的标题，属于严重的逻辑错误。

3. apps/web/src/pages/generate/hooks/generate-video/useGenerationPolling.ts 第30-87行：组件卸载时未清理定时器，存在内存泄漏风险。

   • 说明：useGenerationPolling hook 内部使用了 setTimeout 进行轮询，虽然返回了 clearTimer，但并未在 hook 内部使用 useEffect 注册清理函数。如果用户在轮询进行中退出页面，定时器将继续触发并尝试更新已卸载组件的状态（调用 onProgress 等），导致 React 警告和潜在的内存泄漏。
   • 后果：控制台报错 "Can't perform a React state update on an unmounted component"，且持续消耗网络资源。

💡 建议（2个可选）

1. apps/web/src/pages/generate/hooks/generate-video/useGenerationPolling.ts 第78-81行：轮询异常处理建议增加重试次数限制或退避策略。

   • 说明：当前 catch 块中捕获异常后直接无限递归重试（setTimeout(poll, 3000)）。如果后端服务持续故障，客户端将无限发起请求，造成请求风暴和资源浪费。
   • 建议：引入 retryCount 计数器，超过一定次数（如 5 次）后停止轮询并调用 onFailed。

2. apps/web/src/pages/generate/hooks/generate-video/errorUtils.ts 第32行：错误消息长度判断逻辑较为生硬。

   • 说明：translateError 函数中 msg.length < 100 的判断会导致恰好 100 字符长的错误信息被降级处理为通用提示，可能掩盖具体错误。
   • 建议：移除长度限制或调整为更合理的阈值（如 200），优先展示原始错误信息。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-27 14:35:06 | 模型:}

## 代码审查结果 - PR #969 ### ⚠️ 问题（3个需要修改） 1. **apps/web/src/test/pages/generate/useGenerateVideo.test.ts 第11-14行**：测试文件引用了不存在的导出变量，会导致构建失败。 - **说明**`phase.ts` 仅导出了 `getGenerationPhase`，未导出 `PHASE_ORDER` 和 `getNextPhase`；`errorUtils.ts` 未导出 `extractErrorMessage`；`voiceConfig.ts` 未导出 `getDefaultVoiceConfig`。 - **后果**：测试运行时会抛出 `Module has no exported member` 错误，阻断 CI/CD 流程。 2. **apps/web/src/pages/titles/hooks/useTitleEdit.ts 第62行**：`useCallback` 依赖数组中移除了 `newTitleType`，存在闭包陷阱风险。 - **说明**：代码从依赖数组中删除了 `newTitleType`。如果 `handleCreate` 函数内部使用了 `newTitleType` 变量（根据变量名推测极大概率使用），函数将始终使用初始值或旧值，导致创建标题时类型错误。 - **后果**：用户选择新的标题类型后点击创建，实际创建的仍是旧类型的标题，属于严重的逻辑错误。 3. **apps/web/src/pages/generate/hooks/generate-video/useGenerationPolling.ts 第30-87行**：组件卸载时未清理定时器，存在内存泄漏风险。 - **说明**：`useGenerationPolling` hook 内部使用了 `setTimeout` 进行轮询，虽然返回了 `clearTimer`，但并未在 hook 内部使用 `useEffect` 注册清理函数。如果用户在轮询进行中退出页面，定时器将继续触发并尝试更新已卸载组件的状态（调用 `onProgress` 等），导致 React 警告和潜在的内存泄漏。 - **后果**：控制台报错 "Can't perform a React state update on an unmounted component"，且持续消耗网络资源。 ### 💡 建议（2个可选） 1. **apps/web/src/pages/generate/hooks/generate-video/useGenerationPolling.ts 第78-81行**：轮询异常处理建议增加重试次数限制或退避策略。 - **说明**：当前 `catch` 块中捕获异常后直接无限递归重试（`setTimeout(poll, 3000)`）。如果后端服务持续故障，客户端将无限发起请求，造成请求风暴和资源浪费。 - **建议**：引入 `retryCount` 计数器，超过一定次数（如 5 次）后停止轮询并调用 `onFailed`。 2. **apps/web/src/pages/generate/hooks/generate-video/errorUtils.ts 第32行**：错误消息长度判断逻辑较为生硬。 - **说明**：`translateError` 函数中 `msg.length < 100` 的判断会导致恰好 100 字符长的错误信息被降级处理为通用提示，可能掩盖具体错误。 - **建议**：移除长度限制或调整为更合理的阈值（如 200），优先展示原始错误信息。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-27 14:35:06 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit ff827aba01 into develop 2026-07-27 22:36:45 +08:00

auto-approve-bot deleted branch refactor/generate-video-hook 2026-07-27 22:36:45 +08:00

gitea-actions bot commented 2026-07-27 22:37:25 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

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

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#969