refactor(generate): 拆分 useGenerateFormState 为4个子Hook（309→196行, -37%） #1141

Merged

xiaoxia merged 8 commits from refactor/generate-form-state into develop 2026-07-30 00:23:18 +08:00

Conversation 3 Commits 8 Files Changed 7

+412 -309

xiaoxia commented 2026-07-29 17:21:43 +08:00

Owner

Copy Link

Copy Source

拆分说明

按三阶段模式拆分 useGenerateFormState Hook：

拆分内容

• useTemplateSelection - 模板选择与自动选中（22行）
• useTitleCoverSync - 模板切换时标题/封面自动同步（47行）
• useVoiceState - 配音状态管理 + 预置音色API（30行）
• usePlanConfigLoader - URL参数/编辑计划配置加载（110行）

瘦身效果

• 主文件: 309 → 196 行 (-37%)

向后兼容

• 原Hook路径 useGenerateFormState 保持不变（转为目录结构 index.ts）
• 新增 useGenerateFormState/ 子目录存放子Hook
• 无破坏性改动

## 拆分说明 按三阶段模式拆分 useGenerateFormState Hook： ### 拆分内容 - **useTemplateSelection** - 模板选择与自动选中（22行） - **useTitleCoverSync** - 模板切换时标题/封面自动同步（47行） - **useVoiceState** - 配音状态管理 + 预置音色API（30行） - **usePlanConfigLoader** - URL参数/编辑计划配置加载（110行） ### 瘦身效果 - 主文件: 309 → 196 行 **(-37%)** ### 向后兼容 - 原Hook路径 `useGenerateFormState` 保持不变（转为目录结构 index.ts） - 新增 `useGenerateFormState/` 子目录存放子Hook - 无破坏性改动

xiaoxia added 1 commit 2026-07-29 17:21:43 +08:00

refactor(generate): 拆分 useGenerateFormState 为4个子Hook（309→196行, -37%） 7f4402c88f

xiaoxia added 1 commit 2026-07-29 18:31:20 +08:00

fix: 移除未使用的EditingTemplate import bae53fd705

xiaoxia added 1 commit 2026-07-29 18:31:41 +08:00

fix: 修复useTitleCoverSync语法错误 + 清理重复import d5b3598e6f

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

fix: 替换any类型为具体类型 322bc56350

xiaoxia added 1 commit 2026-07-29 19:03:54 +08:00

fix(generate): 修复 useGenerateFormState 拆分后导入路径错误 78c512d8a9

xiaoxia added 1 commit 2026-07-29 19:03:54 +08:00

fix(generate): 修复 useTemplateSelection 缺少 queryFn 和类型定义 172cf357aa

xiaoxia added 1 commit 2026-07-29 19:03:54 +08:00

fix(generate): 修复 useTitleCoverSync 缺少 CoverConfig 导入和隐式any 68843d9d80

xiaoxia added 1 commit 2026-07-29 19:03:55 +08:00

fix(generate): 修复 usePlanConfigLoader 隐式any类型 9010fb6224

gitea-actions bot commented 2026-07-29 19:13:17 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-07-29 19:13:38 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：通过
• 建议级问题数量：1 个

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

无

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

1. [新增文件: 所有] 建议移除源码文件的执行权限
   • 具体内容：Diff 显示新增的 TypeScript 文件（如 index.ts, usePlanConfigLoader.ts 等）以及测试文件 smoke.test.tsx 的 git mode 为 100755（可执行）。通常源代码文件不需要可执行权限，建议在提交前修正为 100644，以保持仓库权限规范整洁。

✅ 良好实践

1. 逻辑拆分清晰：将庞大的 useGenerateFormState 拆分为 useTemplateSelection、useTitleCoverSync、useVoiceState、usePlanConfigLoader 四个子 Hook，职责单一，提升了代码的可读性和可维护性。
2. 依赖数组处理严谨：在拆分后的自定义 Hook 中，将外部传入的 setter 函数（如 setTitleSettings）正确加入 useEffect 的依赖数组，避免了潜在的闭包陷阱，符合 React Hooks 最佳实践。
3. 执行顺序保持一致：重构后的 Hook 调用顺序与原文件中的逻辑执行顺序（模板初始化 -> 同步配置 -> 加载 URL/远程配置）保持一致，确保了状态更新的时序正确性。
4. 类型安全：拆分过程中保留了完整的 TypeScript 类型定义和导入，类型安全性未受损。

---

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

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 11:13:38 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[新增文件: 所有] 建议移除源码文件的执行权限** - 具体内容：Diff 显示新增的 TypeScript 文件（如 `index.ts`, `usePlanConfigLoader.ts` 等）以及测试文件 `smoke.test.tsx 的 git mode 为 100755`（可执行）。通常源代码文件不需要可执行权限，建议在提交前修正为 `100644`，以保持仓库权限规范整洁。 ### ✅ 良好实践 1. **逻辑拆分清晰**：将庞大的 `useGenerateFormState` 拆分为 `useTemplateSelection`、`useTitleCoverSync`、`useVoiceState`、`usePlanConfigLoader` 四个子 Hook，职责单一，提升了代码的可读性和可维护性。 2. **依赖数组处理严谨**：在拆分后的自定义 Hook 中，将外部传入的 setter 函数（如 `setTitleSettings`）正确加入 `useEffect` 的依赖数组，避免了潜在的闭包陷阱，符合 React Hooks 最佳实践。 3. **执行顺序保持一致**：重构后的 Hook 调用顺序与原文件中的逻辑执行顺序（模板初始化 -> 同步配置 -> 加载 URL/远程配置）保持一致，确保了状态更新的时序正确性。 4. **类型安全**：拆分过程中保留了完整的 TypeScript 类型定义和导入，类型安全性未受损。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 11:13:38 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 9732cc51ee into develop 2026-07-30 00:23:18 +08:00

gitea-actions bot commented 2026-07-30 00:25:18 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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