refactor(voice-materials): 拆分 useVoiceMaterialActions Hook（216→80行, -63%） #1059

Merged

auto-approve-bot merged 3 commits from refactor/use-voice-material-actions into develop 2026-07-28 12:49:52 +08:00

Conversation 2 Commits 3 Files Changed 5

+268 -174

xiaoxia commented 2026-07-28 10:14:59 +08:00

Owner

Copy Link

Copy Source

No description provided.

xiaoxia added 1 commit 2026-07-28 10:14:59 +08:00

refactor(voice-materials): 拆分 useVoiceMaterialActions Hook ... 18f99ba32b

三阶段重构：
- Phase 1: 拆分为 3 个领域子 Hook
  - useVoiceUpload: 上传流程（库→上传→时长→创建→标签）
  - useVoiceEdit: 编辑流程（更新信息+标签差异同步）
  - useVoiceDelete: 删除操作
- Phase 2: 主 Hook 组合子 Hook + UI 弹窗状态
- Phase 3: 主文件 216→80 行（-63%）

保持导出签名向后兼容

xiaoxia added 1 commit 2026-07-28 10:19:45 +08:00

fix(useVoiceMaterialActions): fix prettier formatting 3eef40bbf8

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

fix(useVoiceMaterialActions): 修复子 Hook 相对路径层级错误 ... ab9b7409b2

子 Hook 位于 actions/ 子目录，引用 types/utils 需要上三级（../../../）
而非上两级（../../），导致 TS 编译失败。

auto-approve-bot commented 2026-07-28 12:27:44 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1059

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

1. useVoiceMaterialActions.ts 第45行：编辑弹窗在请求提交后立即关闭，导致用户无法感知加载状态，且若请求失败弹窗已关闭，体验不佳。
   • 原因：代码在 editMutation.mutate 后立即执行了 setEditingMaterial(null)，这与 handleUpload 中将关闭逻辑放在 onSuccess 回调中的做法不一致。
   • 后果：编辑失败时用户界面已重置，可能造成困惑；且无法在弹窗内展示加载中的 Spinner。
   • 建议：将 setEditingMaterial(null) 移至 mutate 方法的 onSuccess 回调选项中，确保仅在成功后关闭。

💡 建议（4个可选）

1. useVoiceEdit.ts 第45行：untagAsset 在 for...of 循环中使用 await 串行执行，若删除标签数量较多会导致接口响应时间累加。建议改为 Promise.all 并行执行以提高性能。
2. useVoiceEdit.ts 第62行：该 Hook 导出的 handleEdit 方法在父组件 useVoiceMaterialActions 中被重新定义覆盖，属于死代码，建议删除以减小包体积。
3. useVoiceDelete.ts 第32行：该 Hook 导出 handleDelete 方法，但在父组件 useVoiceMaterialActions 中并未使用（父组件重新定义了同名函数），属于死代码，建议删除。
4. useVoiceEdit.ts 第17行：mutationFn 依赖闭包中的 materials 变量。若父组件传入的 materials 引用频繁变动，会导致 useMutation 配置频繁重建。建议确保 materials 引用稳定或使用 useCallback 稳定化 mutationFn。

---

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

---

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

## 代码审查结果 - PR #1059 ### ⚠️ 问题（1个需要修改） 1. **useVoiceMaterialActions.ts 第45行**：编辑弹窗在请求提交后立即关闭，导致用户无法感知加载状态，且若请求失败弹窗已关闭，体验不佳。 - **原因**：代码在 `editMutation.mutate` 后立即执行了 `setEditingMaterial(null)`，这与 `handleUpload` 中将关闭逻辑放在 `onSuccess` 回调中的做法不一致。 - **后果**：编辑失败时用户界面已重置，可能造成困惑；且无法在弹窗内展示加载中的 Spinner。 - **建议**：将 `setEditingMaterial(null)` 移至 `mutate` 方法的 `onSuccess` 回调选项中，确保仅在成功后关闭。 ### 💡 建议（4个可选） 1. **useVoiceEdit.ts 第45行**：`untagAsset` 在 `for...of` 循环中使用 `await` 串行执行，若删除标签数量较多会导致接口响应时间累加。建议改为 `Promise.all` 并行执行以提高性能。 2. **useVoiceEdit.ts 第62行**：该 Hook 导出的 `handleEdit` 方法在父组件 `useVoiceMaterialActions` 中被重新定义覆盖，属于死代码，建议删除以减小包体积。 3. **useVoiceDelete.ts 第32行**：该 Hook 导出 `handleDelete` 方法，但在父组件 `useVoiceMaterialActions` 中并未使用（父组件重新定义了同名函数），属于死代码，建议删除。 4. **useVoiceEdit.ts 第17行**：`mutationFn` 依赖闭包中的 `materials` 变量。若父组件传入的 `materials` 引用频繁变动，会导致 `useMutation` 配置频繁重建。建议确保 `materials` 引用稳定或使用 `useCallback` 稳定化 `mutationFn`。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-28 04:27:44 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit 29448aaf9b into develop 2026-07-28 12:49:52 +08:00

auto-approve-bot deleted branch refactor/use-voice-material-actions 2026-07-28 12:49:53 +08:00

gitea-actions bot commented 2026-07-28 12:59:07 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia referenced this pull request 2026-07-28 13:40:56 +08:00

fix(voice-materials): 修复 useVoiceMaterialActions 的 types 相对路径层级 #1083

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#1059