public/sw.js 里 networkFirst 和 staleWhileRevalidate 两个策略在 cache.put 之前没有检查 response.ok，4xx/5xx 的响应也会被写进缓存。

比如服务器短暂返回 500 之后网络中断，离线时会把缓存的 500 页面返回给用户；staleWhileRevalidate 的后台更新也可能用错误响应覆盖掉正常缓存。

建议加个 response.ok 判断再写缓存：

// networkFirst
const response = await fetch(request);
if (response.ok) {
  cache.put(request, response.clone());
}
return response;

// staleWhileRevalidate
.then((response) => {
  if (response.ok) {
    cache.put(request, response.clone());
  }
  return response;
})

public/sw.js 里 networkFirst 和 staleWhileRevalidate 两个策略在 cache.put 之前没有检查 response.ok，4xx/5xx 的响应也会被写进缓存。

比如服务器短暂返回 500 之后网络中断，离线时会把缓存的 500 页面返回给用户；staleWhileRevalidate 的后台更新也可能用错误响应覆盖掉正常缓存。

建议加个 response.ok 判断再写缓存：

// networkFirst
const response = await fetch(request);
if (response.ok) {
  cache.put(request, response.clone());
}
return response;

// staleWhileRevalidate
.then((response) => {
  if (response.ok) {
    cache.put(request, response.clone());
  }
  return response;
})

已处理，感谢指出。

这边修了两处：

• 在 networkFirst / staleWhileRevalidate 中增加 response.ok 判断，避免把 4xx/5xx 响应写入缓存，防止错误页污染已有缓存。
• 额外补充了一处缓存硬化：将 /qr-login 排除出 service worker 的处理范围，避免带一次性 token 的二维码登录页进入离线缓存。

同时补充了对应测试用例，覆盖上述两个场景。

Codecov Report

❌ Patch coverage is 83.33333% with 3 lines in your changes missing coverage. Please review.

📢 Thoughts on this report? Let us know!

Code Review：feat(webui): add pwa support (#1688)

变更概述

本 PR 为 WebUI 添加 PWA 支持，包含：manifest、PWA 图标、service worker（public/sw.js）、service worker 注册逻辑（registerPwa.ts）及 HTML meta 标签。同时更新了 vite.renderer.config.ts 和 electron.vite.config.ts，将 public/ 目录正确配置为 publicDir。PR 讨论中提到的 response.ok 校验和 /qr-login 排除问题已在本版本中修复。

方案评估

结论：✅ 方案合理

方案采用原生 Service Worker API 而非重型框架（如 Workbox），对于本场景是合适的轻量选择。networkFirst 用于导航请求、staleWhileRevalidate 用于静态资源，策略分层清晰。registerPwa.ts 通过 isElectronDesktop() 正确跳过 Electron 运行时，publicDir 的两处配置（vite.renderer.config.ts 和 electron.vite.config.ts）确保了开发和生产构建均能正确服务 PWA 文件。

问题清单

🟡 MEDIUM — registerPwa.ts 测试覆盖率不足（约 58%，低于项目 80% 目标）

文件：src/renderer/services/registerPwa.ts

问题说明：Codecov 报告 registerPwa.ts 覆盖率约 58%（4 行 missing，3 处 partial branch）。isPwaRegistrationSupported 中的若干防御分支未被测试：

• !('serviceWorker' in navigator) 为 false 的路径
• protocol !== 'http:' 且 protocol !== 'https:' 的路径（非 HTTP 协议，如 file:）
• !window.isSecureContext && !LOCALHOST_HOSTS.has(hostname) 的路径（非 localhost 且非安全上下文 → 返回 false）
• registerPwa() 内 catch 块（serviceWorker.register 抛出异常时的路径）

修复建议：在 registerPwa.dom.test.ts 中补充以下用例：

it('skips registration when serviceWorker is not in navigator', async () => {
  Object.defineProperty(window, 'electronAPI', { configurable: true, value: undefined });
  // navigator.serviceWorker not defined
  await expect(registerPwa()).resolves.toBeUndefined();
});

it('skips registration on insecure non-localhost origin', async () => {
  Object.defineProperty(window, 'electronAPI', { configurable: true, value: undefined });
  Object.defineProperty(navigator, 'serviceWorker', {
    configurable: true,
    value: { register: vi.fn() },
  });
  Object.defineProperty(window, 'location', {
    configurable: true,
    value: { protocol: 'http:', hostname: '192.168.1.1', isSecureContext: false },
  });
  await expect(registerPwa()).resolves.toBeUndefined();
});

it('returns undefined when service worker registration throws', async () => {
  const register = vi.fn().mockRejectedValue(new Error('Security error'));
  Object.defineProperty(window, 'electronAPI', { configurable: true, value: undefined });
  Object.defineProperty(navigator, 'serviceWorker', { configurable: true, value: { register } });
  await expect(registerPwa()).resolves.toBeUndefined();
});

汇总

结论

⚠️ 有条件批准 — 实现本身正确、PR 讨论反馈已处理，仅需补充 registerPwa.ts 的缺失测试分支（约需 3–4 个用例）后可合并。

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

PR Fix 验证报告

原始 PR: #1688
修复方式: 直接推送到 feat/webui-pwa

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

覆盖率由约 58% 提升，新增测试覆盖所有缺失的防御分支。全部 141 个测试文件通过（6 个测试用例→全部通过）。