Code Review：fix(settings): catch form validation rejection in EditModeModal (#1779)

变更概述

本 PR 为 EditModeModal 的 onOk 回调新增了 try-catch 包裹，修复了 Arco Form validate() 在校验失败时抛出 rejection 导致的未处理 Promise 异常（Sentry ELECTRON-1E）。同时新增了一个 DOM 测试文件，验证校验失败不产生 unhandled rejection、校验成功正确调用 onChange。

方案评估

结论：✅ 方案合理

修复方式最小且正确——Arco Form 在校验失败时会自动 reject，同时在 UI 上标红字段，因此空 catch 加注释是标准处理方式。测试覆盖了核心场景（失败不回调、成功正确回调）。无过度工程化，与项目架构完全一致。

问题清单

🔵 LOW — capturedOnOk 声明但未使用，触发 lint 警告

文件：tests/unit/EditModeModal.dom.test.tsx，第 136 行

问题代码：

let capturedOnOk: (() => Promise<void>) | undefined;
vi.mock('@/renderer/components/base/AionModal', () => ({
  default: ({ children, onOk, ...props }: any) => {
    capturedOnOk = onOk; // 赋值但从未读取

问题说明：该变量的初衷是"暴露 onOk 供直接调用"，但实际测试改为通过 fireEvent.click 触发，capturedOnOk 从未被读取。oxlint 报 no-unused-vars 警告。

修复建议：将变量名前缀改为 _，或直接移除赋值语句（mock 仍可工作，onOk 已通过 onClick={onOk} 绑定）。

🔵 LOW — 第二个测试未移除 unhandledrejection 监听器

文件：tests/unit/EditModeModal.dom.test.tsx，第 193–217 行

问题说明：beforeEach 添加了监听器，第一个测试在结束时手动移除，但第二个测试未移除，导致监听器在测试完成后仍挂载。建议统一在 afterEach 中清理。

修复建议：

afterEach(() => {
  window.removeEventListener('unhandledrejection', rejectionHandler);
});
// 并删除第一个测试中手动的 removeEventListener 调用

汇总

结论

⚠️ 有条件批准 — 生产代码修复正确，两处均为测试文件的小问题，处理后可合并。

本报告由本地 pr-review skill 生成，包含完整项目上下文，无截断限制。

CONCLUSION: CONDITIONAL
IS_CRITICAL_PATH: false
PR_NUMBER: 1779

PR Fix 验证报告

原始 PR: #1779
修复方式: 所有 review 问题均为 🔵 LOW 级别，已跳过（不阻塞合并）

总结： ✅ 已修复 0 个 | ⏭️ 跳过 2 个（均为 LOW）

🔵 LOW 级别问题已跳过（不阻塞合并，修复优先级低）。生产代码修复正确，可安全合并。