Codecov Report

❌ Patch coverage is 65.11628% with 30 lines in your changes missing coverage. Please review.

📢 Thoughts on this report? Let us know!

Code Review：feat(weixin): add WebUI QR-code login via SSE (#1744)

变更概述

本 PR 为 WebUI 模式（浏览器无 electronAPI）添加了微信扫码登录支持：在主进程新增 SSE 路由 /api/channel/weixin/login，驱动现有 WeixinLogin.ts 流程并向前端推送 qr/scanned/done/error 事件；前端检测 WebUI 环境并使用 EventSource + QRCodeSVG 渲染二维码。同时补充了断开连接功能，并修复了 hasToken=true, enabled=false 时误显示已连接的 bug。

方案评估

结论：✅ 方案合理

整体方案与项目架构一致：SSE 路由放在 src/process/webserver/routes/，前端通过运行时检测 electronAPI 分支，不影响 Electron 路径。将 qrcode_img_content（页面 URL）直接作为 QR 编码内容符合微信 liteapp 的实际行为（在 PR 描述中有验证）。enableWeixinPlugin 抽离复用 Electron 和 WebUI 两条路径，逻辑清晰，无不必要的抽象。

问题清单

🟡 MEDIUM — EventSource 未在组件卸载时关闭，存在连接泄漏

文件：src/renderer/components/settings/SettingsModal/contents/channels/WeixinConfigForm.tsx，第 277 行

问题代码：

const handleLoginWebUI = () => {
  ...
  const es = new EventSource('/api/channel/weixin/login', { withCredentials: true });
  // es 仅保存在局部变量中，组件卸载时无法关闭
  ...
};

问题说明：如果用户在登录流程中关闭 Settings 弹窗，EventSource 连接不会被关闭，服务端的 startLogin 流程继续运行，SSE 连接保持存活直至超时。同时 React 的 state setter 仍会被回调，可能触发无副作用但不必要的状态更新。

修复建议：将 EventSource 存入 useRef，并在 useEffect 清理函数中关闭：

const eventSourceRef = useRef<EventSource | null>(null);

// handleLoginWebUI 中：
const es = new EventSource('/api/channel/weixin/login', { withCredentials: true });
eventSourceRef.current = es;

// 组件卸载清理：
useEffect(() => {
  return () => {
    eventSourceRef.current?.close();
    eventSourceRef.current = null;
  };
}, []);

🟡 MEDIUM — 补丁覆盖率 60.29%，低于项目 80% 下限

文件：src/renderer/components/settings/SettingsModal/contents/channels/WeixinConfigForm.tsx

问题说明：Codecov 报告补丁覆盖率 60.29%（19 行未覆盖 + 8 行部分覆盖），未达到项目 ≥80% 的覆盖率要求。主要缺口包括：

• enableWeixinPlugin 失败路径（success=false）
• handleLoginWebUI 中 error 事件的 expired/too many 分支
• onerror 回调
• handleDisconnect 错误路径

修复建议：补充以下测试场景：

// enableWeixinPlugin 失败路径
mockEnablePlugin.mockResolvedValueOnce({ success: false, msg: 'Error msg' });
// → 应显示错误 Message，loginState 回到 'idle'

// SSE error 事件中 'expired' 分支
es.emit('error', { message: 'QR code expired' });
// → 应显示 warning，loginState 回到 'idle'

// handleDisconnect 失败路径
mockDisablePlugin.mockResolvedValueOnce({ success: false, msg: 'Disable failed' });
// → 应显示错误 Message

🔵 LOW — es.onerror 与 addEventListener('error') 重复处理连接级错误（oxlint 警告）

文件：src/renderer/components/settings/SettingsModal/contents/channels/WeixinConfigForm.tsx，第 300–316 行

问题代码：

es.addEventListener('error', (e: MessageEvent) => {
  es.close();
  // ...处理服务端发送的 error 事件或连接错误
});

es.onerror = () => {           // oxlint: prefer-add-event-listener
  es.close();
  setLoginState('idle');
  setQrcodeDataUrl(null);
};

问题说明：连接级错误会同时触发 addEventListener('error') 和 onerror，导致 es.close() 和状态重置被调用两次。oxlint 报告 prefer-add-event-listener 违规。

修复建议：删除 onerror 赋值，addEventListener('error') 已通过 e.data ? ... : '' 检查兼容处理两种情形（服务端 error 事件和连接级错误）。

🔵 LOW — 变量注释描述不准确

文件：src/renderer/components/settings/SettingsModal/contents/channels/WeixinConfigForm.tsx，第 64 行

问题代码：

// In Electron mode this holds a base64 data URL; in WebUI mode it holds the raw QR ticket string.

问题说明：WebUI 模式下存储的是 qrcode_img_content 页面 URL（非 raw ticket）。

修复建议：

// In Electron mode this holds a base64 data URL; in WebUI mode it holds the qrcode_img_content page URL.

汇总

结论

⚠️ 有条件批准 — 无阻塞性问题，EventSource 未清理和覆盖率不足需补充修复后可合并。

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

PR Fix 验证报告

原始 PR: #1744
修复方式: 直接推送到 feat/weixin-webui-login

总结： ✅ 已修复 2 个 | ❌ 未能修复 0 个

🔵 LOW 级别问题已跳过（不阻塞合并，修复优先级低）。

Code Review：feat(weixin): add WebUI QR-code login via SSE (#1744)

变更概述

本 PR 为 WebUI 模式下的微信登录增加了 SSE（Server-Sent Events）通道。核心改动包括：新增 /api/channel/weixin/login SSE 端点（weixinLoginRoutes.ts），扩展 LoginCallbacks.onQR 签名以暴露原始二维码 ticket，在 WeixinConfigForm 中检测 WebUI 模式并通过 EventSource + QRCodeSVG 展示登录二维码，同时补充了对应的单元测试。

方案评估

结论：✅ 方案合理

方案正确区分了 Electron（IPC + 隐藏窗口截图）和 WebUI（SSE + QRCodeSVG）两条路径，对已有 Electron 流程没有破坏性改动。SSE 端点做了认证中间件（validateApiAccess）保护，req.on('close', handle.abort) 确保客户端断开后服务端登录流程也会终止。将 qrcode_img_content 页面 URL 而非原始 ticket 作为二维码内容，与 PR 描述中对微信 liteapp JS bundle 的验证结论一致，方案合理。

问题清单

🔵 LOW — es.onerror 与 es.addEventListener('error', ...) 双重注册导致重复处理

文件：src/renderer/components/settings/SettingsModal/contents/channels/WeixinConfigForm.tsx，第 310–326 行

问题说明：EventSource 规范中 onerror 与 addEventListener('error', ...) 均监听同一 error 事件类型（网络断开和服务端发送的 event: error 消息都会触发两者）。当前实现在任何错误场景下都会执行两次 state 重置，属于冗余处理。addEventListener('error', ...) 里已通过 e.data ? ... : '' 分支处理了无数据的网络错误，onerror 可以安全移除。此问题也正是 oxlint (prefer-add-event-listener) 警告的根源。

修复建议：移除 es.onerror，保留 addEventListener('error', ...) 单一处理即可。

🔵 LOW — weixinLoginRoutes.test.ts 未覆盖 onError 路径

文件：tests/unit/weixinLoginRoutes.test.ts

问题说明：当前测试只验证了 qr → scanned → done 的正常链路，未覆盖 onError 回调（SSE 发送 event: error 后 res.end()）。Codecov 报告显示 weixinLoginRoutes.ts 缺少 2 行覆盖（87.5%，仍高于 80% 阈值，CI 通过）。

汇总

结论

✅ 批准合并 — 无阻塞性问题。核心功能正确，Electron 路径不受影响，测试覆盖率满足项目要求（CI 全部通过）。上述两个 LOW 级别问题可在后续 PR 中跟进修复。

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

✅ 已自动 review，无阻塞性问题，正在触发自动合并。