🔧 通用修复与基础设施改进 (#1328)

* 🔧 通用基础设施改进

- CI: build/test 流水线支持 feature/* 分支触发
- vitest: 修正 exclude 模式，确保子目录 node_modules/.claude 被排除
- message/server: SenderRuntime.getExtMessageSender 添加 null 检查，
  防止 postMessage 通道（Offscreen→SW）因无 RuntimeMessageSender 崩溃
- message/window_message: connect() 使用 WindowPostMessage 包装，
  确保 sandbox(origin:null)→offscreen 的消息不被丢弃

* 🐛 通用修复与代码规范化

- confirm/App.tsx: 定时器从 setTimeout 递归改为 useEffect+setInterval，
  修复内存泄漏和渲染循环；提取 PermissionConfirmRequest 组件
- utils.ts: sourceMapTo 添加 chrome.runtime?.getURL 防御，
  sandbox 环境中 chrome.runtime 不可用时降级为脚本名
- 5 处 eslint-disable react-hooks/exhaustive-deps 注释
- prettier 格式化：CSS/HTML/JSON 文件统一缩进和引号
- .prettierignore: 忽略 *.md 和 example/
- manifest.json: 纯格式化（prettier）
- playwright.config.ts: 本地也启用 retry

* ⏪ 撤回 eslint-disable 注释和 playwright retry 改动

- 移除 6 处 react-hooks/exhaustive-deps 的 eslint-disable（仅 warn 无需抑制）
- 还原 playwright retries 为 CI ? 1 : 0

* ⏪ 还原格式化改动，保持 release/v1.4 原有格式

CSS/HTML/JSON/manifest 等文件还原为 release/v1.4 原始格式

* 🧪 补充 message 包单测：sender 兜底 + connect targetOrigin

根据 PR #1328 Copilot 评审意见，补充两个单测：
- SenderRuntime.getExtMessageSender() 在 sender 为 null/undefined/空对象时不崩溃并返回默认值
- WindowMessage.connect() 返回的连接 sendMessage 带 "*" targetOrigin

* 🐛 修复权限确认页 beforeunload 监听器泄漏

- 拆分 useEffect：beforeunload 注册与 getPermissionInfo 请求独立
- 提取命名 handler，在 cleanup 中 removeEventListener
- beforeunload effect 只依赖 [uuid]，避免语言切换时重复注册
- .gitignore 忽略 .omc 本地开发目录

根据 PR #1328 @cyfung1031 评审意见处理

* 🐛 修复 UnoCSS 暗色模式默认边框色

暗色模式下 UnoCSS 的 --un-default-border-color 默认使用浅色值，
导致 border-* 类的边框在暗色主题下与背景对比不足。
将其绑定到 Arco CSS 变量 --color-border-2，让边框色随主题自适应。

* 🧪 navigation_handle: 用 resetAttachedForTest 替代 vi.resetModules

vi.resetModules() 会清空全局模块缓存，导致后续测试重新加载 LoggerCore、
chrome mock 等模块时出错，在 CI 上表现为偶发失败。

改为导出 resetAttachedForTest() 重置模块级 attached 单例，
测试无需动态 import 即可验证多次调用的幂等性。

Author: CodFrm

.github/workflows/build.yaml

@@ -5,6 +5,7 @@ on:
     branches:
       - main
       - release/*
+      - feature/*
       - dev
     paths-ignore:
       - ".github/**"

.github/workflows/test.yaml

@@ -5,6 +5,7 @@ on:
     branches:
       - main
       - release/*
+      - feature/*
       - dev
       - develop/*
   pull_request:

.gitignore

@@ -41,3 +41,4 @@ test-results
 playwright-report
 
 superpowers
+.omc

.prettierignore

@@ -5,3 +5,7 @@ yarn.lock
 
 # Claude Code
 .claude
+
+# Docs & examples
+*.md
+example/

packages/message/server.test.ts

@@ -540,6 +540,32 @@ describe("Server", () => {
       const extSender = capturedSender!.getExtMessageSender();
       expect(extSender.tabId).toBe(-1);
     });
+
+    it.concurrent("sender 为 null/undefined 时不崩溃并返回默认值", async () => {
+      // postMessage 通道（如 Offscreen→SW）传入空对象作为 sender，
+      // SenderRuntime.getExtMessageSender() 应该返回默认兜底值
+      const senderNull = new SenderRuntime(null as unknown as RuntimeMessageSender);
+      const extNull = senderNull.getExtMessageSender();
+      expect(extNull.windowId).toBe(-1);
+      expect(extNull.tabId).toBe(-1);
+      expect(extNull.frameId).toBeUndefined();
+      expect(extNull.documentId).toBeUndefined();
+
+      const senderUndefined = new SenderRuntime(undefined as unknown as RuntimeMessageSender);
+      const extUndefined = senderUndefined.getExtMessageSender();
+      expect(extUndefined.windowId).toBe(-1);
+      expect(extUndefined.tabId).toBe(-1);
+      expect(extUndefined.frameId).toBeUndefined();
+      expect(extUndefined.documentId).toBeUndefined();
+
+      // 空对象（ServiceWorkerMessageSend 实际传入的值）也应正常处理
+      const senderEmpty = new SenderRuntime({} as RuntimeMessageSender);
+      const extEmpty = senderEmpty.getExtMessageSender();
+      expect(extEmpty.windowId).toBe(-1);
+      expect(extEmpty.tabId).toBe(-1);
+      expect(extEmpty.frameId).toBeUndefined();
+      expect(extEmpty.documentId).toBeUndefined();
+    });
   });
 
   describe("Connect 功能测试", () => {

packages/message/server.ts

@@ -87,6 +87,15 @@ export class SenderRuntime {
 
   getExtMessageSender(): ExtMessageSender {
     const sender = this.sender as RuntimeMessageSender;
+    if (!sender) {
+      // postMessage 通道（如 Offscreen→SW）没有 RuntimeMessageSender
+      return {
+        windowId: -1,
+        tabId: -1,
+        frameId: undefined,
+        documentId: undefined,
+      };
+    }
     return {
       windowId: sender.tab?.windowId || -1, // -1表示后台脚本
       tabId: sender.tab?.id || -1, // -1表示后台脚本

packages/message/window_message.test.ts

@@ -1,5 +1,10 @@
 import { describe, it, expect, beforeEach, vi, afterEach } from "vitest";
-import { ServiceWorkerMessageSend, ServiceWorkerClientMessage, type WindowMessageBody } from "./window_message";
+import {
+  ServiceWorkerMessageSend,
+  ServiceWorkerClientMessage,
+  WindowMessage,
+  type WindowMessageBody,
+} from "./window_message";
 import { Server } from "./server";
 import type { MessageConnect } from "./types";
 
@@ -160,6 +165,38 @@ describe("ServiceWorkerClientMessage", () => {
   });
 });
 
+describe("WindowMessage.connect", () => {
+  it("connect 返回的连接 sendMessage 应带 '*' targetOrigin", async () => {
+    // 模拟 target window，验证 postMessage 被调用时带 "*"
+    const targetPostMessage = vi.fn();
+    const sourceWindow = {
+      addEventListener: vi.fn(),
+    } as unknown as Window;
+    const targetWindow = {
+      postMessage: targetPostMessage,
+    } as unknown as Window;
+
+    const wm = new WindowMessage(sourceWindow, targetWindow);
+
+    const con = await wm.connect({ action: "test/connect", data: "init" });
+
+    // connect() 本身会调用一次 postMessage（发送 connect 消息）
+    expect(targetPostMessage).toHaveBeenCalledTimes(1);
+    expect(targetPostMessage).toHaveBeenCalledWith(expect.objectContaining({ type: "connect" }), "*");
+
+    targetPostMessage.mockClear();
+
+    // 通过返回的连接发送消息，也应该带 "*"
+    con.sendMessage({ action: "test/msg", data: "hello" });
+
+    expect(targetPostMessage).toHaveBeenCalledTimes(1);
+    expect(targetPostMessage).toHaveBeenCalledWith(
+      expect.objectContaining({ type: "connectMessage", data: { action: "test/msg", data: "hello" } }),
+      "*"
+    );
+  });
+});
+
 describe("ServiceWorkerMessageSend ↔ ServiceWorkerClientMessage 双向通信", () => {
   // 辅助函数: 将两端连接起来，模拟 postMessage 通道
   function createWiredPair() {

packages/message/window_message.ts

@@ -100,7 +100,9 @@ export class WindowMessage implements Message {
         data,
       };
       this.target.postMessage(body, "*");
-      resolve(new WindowMessageConnect(body.messageId, this.EE, this.target));
+      // 使用 WindowPostMessage 包装，确保后续 sendMessage 也带 "*" targetOrigin
+      // 否则沙箱（origin: null）→ offscreen（origin: chrome-extension://）的消息会被丢弃
+      resolve(new WindowMessageConnect(body.messageId, this.EE, new WindowPostMessage(this.target)));
     });
   }
 

src/app/service/content/gm_api/navigation_handle.test.ts

@@ -1,16 +1,5 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
-import { UrlChangeEvent } from "./navigation_handle.js";
-
-// attachNavigateHandler 使用模块级 attached 单例，需要在每个测试前重置模块
-const importFresh = async () => {
-  vi.resetModules();
-  // vi.resetModules() 会清空模块缓存，后续 import 得到全新的 LoggerCore 类，
-  // 需要重新初始化以免后续测试文件中 LoggerCore.getInstance() 返回 undefined
-  // @ts-expect-error 动态 import 路径别名在 tsc nodenext 下无法解析
-  const { default: LC, EmptyWriter: EW } = await import("@App/app/logger/core");
-  new LC({ level: "trace", consoleLevel: "trace", writer: new EW(), labels: { env: "test" } });
-  return await import("./navigation_handle.js");
-};
+import { UrlChangeEvent, attachNavigateHandler, resetAttachedForTest } from "./navigation_handle";
 
 describe("UrlChangeEvent", () => {
   it.concurrent("应包含 url 属性", () => {
@@ -64,10 +53,10 @@ describe("attachNavigateHandler", () => {
 
   beforeEach(() => {
     vi.restoreAllMocks();
+    resetAttachedForTest();
   });
 
-  it("不支持 Navigation API 时不应注册监听器", async () => {
-    const { attachNavigateHandler } = await importFresh();
+  it("不支持 Navigation API 时不应注册监听器", () => {
     const win = { location: { href: "https://example.com/" } } as any;
     attachNavigateHandler(win);
     // 没有 navigation 属性，不应报错也不应标记为 attached
@@ -77,16 +66,14 @@ describe("attachNavigateHandler", () => {
     expect(mock.win.navigation.addEventListener).toHaveBeenCalledWith("navigate", expect.any(Function), false);
   });
 
-  it("应在 win.navigation 上注册 navigate 监听器", async () => {
-    const { attachNavigateHandler } = await importFresh();
+  it("应在 win.navigation 上注册 navigate 监听器", () => {
     const mock = createMockWin();
     attachNavigateHandler(mock.win);
     expect(mock.win.navigation.addEventListener).toHaveBeenCalledTimes(1);
     expect(mock.win.navigation.addEventListener).toHaveBeenCalledWith("navigate", expect.any(Function), false);
   });
 
-  it("多次调用只注册一次", async () => {
-    const { attachNavigateHandler } = await importFresh();
+  it("多次调用只注册一次", () => {
     const mock = createMockWin();
     attachNavigateHandler(mock.win);
     attachNavigateHandler(mock.win);
@@ -95,7 +82,6 @@ describe("attachNavigateHandler", () => {
   });
 
   it("URL 变化时应派发 urlchange 事件", async () => {
-    const { attachNavigateHandler } = await importFresh();
     const mock = createMockWin("https://example.com/");
     attachNavigateHandler(mock.win);
     mock.fireNavigate("https://example.com/new");
@@ -109,7 +95,6 @@ describe("attachNavigateHandler", () => {
   });
 
   it("URL 未变化时不应派发事件", async () => {
-    const { attachNavigateHandler } = await importFresh();
     const mock = createMockWin("https://example.com/");
     attachNavigateHandler(mock.win);
     // destination.url 与当前 href 相同

src/app/service/content/gm_api/navigation_handle.ts

@@ -10,6 +10,11 @@ export class UrlChangeEvent extends Event {
 
 let attached = false;
 
+// 仅供测试使用，重置 attached 标记
+export const resetAttachedForTest = () => {
+  attached = false;
+};
+
 const getPropGetter = <T>(obj: T, key: keyof T) => {
   // 避免直接 obj[key] 读取。或会被 hack
   for (let t = obj; t; t = Native.objectGetPrototypeOf(t)) {