fix(voice): 修复克隆音色弹窗「选文件无反应」死锁（isSubmittingRef 三处误用） #1541

Merged

auto-approve-bot merged 2 commits from fix/clone-modal-submit-lock into develop 2026-08-29 17:36:01 +08:00

Conversation 5 Commits 2 Files Changed 1

+6 -6

xiaoxia commented 2026-08-29 17:10:56 +08:00

Owner

Copy Link

Copy Source

背景

克隆音色弹窗：打开后上传/选择音频无任何反应、无提示。浏览器实测确认。

根因

isSubmittingRef 是 handleSubmit 的提交防重入锁，但被错误写进了三处不该用的地方：

1. resetState（弹窗 open 时 useEffect 调用）：上锁后永不复位 → 弹窗每次打开即死锁
2. handleFileChange else 分支（文件校验通过后）：检查到锁直接 return → 选文件被静默拦截
3. handleDrop else 分支：同上，拖拽也被拦截

且这三处上锁后都没有解锁路径。

改动（仅 CloneModal/index.tsx，删 6 行）

• 三处删除 if (isSubmittingRef.current) return + isSubmittingRef.current = true
• handleSubmit 内锁逻辑保持原样（入口上锁、成功/catch 复位）
• isMountedRef、OSS 直传链路、useAudioRecorder 均不动

验收

• 打开克隆弹窗 → 点击上传选音频 → 选中后文件正常显示
• 拖拽音频文件正常
• 提交防重入仍生效（提交中重复点击无效）

## 背景 克隆音色弹窗：打开后上传/选择音频无任何反应、无提示。浏览器实测确认。 ## 根因 `isSubmittingRef` 是 `handleSubmit` 的提交防重入锁，但被错误写进了三处不该用的地方： 1. `resetState`（弹窗 open 时 useEffect 调用）：上锁后**永不复位** → 弹窗每次打开即死锁 2. `handleFileChange` else 分支（文件校验通过后）：检查到锁直接 return → 选文件被静默拦截 3. `handleDrop` else 分支：同上，拖拽也被拦截 且这三处上锁后都没有解锁路径。 ## 改动（仅 CloneModal/index.tsx，删 6 行） - 三处删除 `if (isSubmittingRef.current) return` + `isSubmittingRef.current = true` - `handleSubmit` 内锁逻辑保持原样（入口上锁、成功/catch 复位） - isMountedRef、OSS 直传链路、useAudioRecorder 均不动 ## 验收 - 打开克隆弹窗 → 点击上传选音频 → 选中后文件正常显示 - 拖拽音频文件正常 - 提交防重入仍生效（提交中重复点击无效）

xiaoxia added 1 commit 2026-08-29 17:10:56 +08:00

fix(voice): 修复克隆音色弹窗选文件无反应死锁——移除三处误用的提交锁 ... 1c4eeb3e33

isSubmittingRef 是 handleSubmit 的防重入锁，被错误写进 resetState、
handleFileChange else、handleDrop else：
- resetState 在弹窗打开时执行，上锁后永不复位 → 弹窗一开即死锁
- 选文件/拖拽文件校验通过后先检查锁并 return → 文件被静默拦截
且这三处上锁后都没有对应解锁路径。

移除这三对误用语句；handleSubmit 内的上锁/成功·catch 复位逻辑保持原样。
isMountedRef、OSS 直传、useAudioRecorder 均不动。

gitea-actions bot commented 2026-08-29 17:15:04 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot approved these changes 2026-08-29 17:16:03 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-29 17:16:03 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia added 1 commit 2026-08-29 17:25:56 +08:00

docs(voice): CloneModal 补充提交锁边界注释（回应 AI review） ... 81fac3a6e8

文件选择/拖拽/弹窗重置均为同步 state 操作无异步竞态，防重入只属于
handleSubmit（isSubmittingRef + isProcessing 双保险）；
resetState 此前上锁且无复位路径导致弹窗打开即死锁，故不设锁。

auto-approve-bot commented 2026-08-29 17:30:23 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

无

✅ 良好实践

1. 精准修复死锁Bug：准确识别出 isSubmittingRef 在 resetState 中被错误设置且无复位路径导致弹窗死锁的问题，将锁的作用域严格收窄至 handleSubmit，逻辑修复正确。
2. 注释详尽：添加的注释清晰地解释了锁的职责边界以及修改原因，这对于维护复杂的状态逻辑非常有帮助。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **精准修复死锁Bug**：准确识别出 `isSubmittingRef` 在 `resetState` 中被错误设置且无复位路径导致弹窗死锁的问题，将锁的作用域严格收窄至 `handleSubmit`，逻辑修复正确。 2. **注释详尽**：添加的注释清晰地解释了锁的职责边界以及修改原因，这对于维护复杂的状态逻辑非常有帮助。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-29 09:30:23 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit f327171e9b into develop 2026-08-29 17:36:01 +08:00

auto-approve-bot deleted branch fix/clone-modal-submit-lock 2026-08-29 17:36:01 +08:00

gitea-actions bot commented 2026-08-29 17:39:10 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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