Merge branch 'fix/general-improvements' into develop/agent

Author: CodFrm

.gitignore

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

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/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() {

src/pages/confirm/App.tsx

@@ -27,13 +27,17 @@ function PermissionConfirmRequest({ uuid }: { uuid: string }) {
   }, []);
 
   useEffect(() => {
-    window.addEventListener("beforeunload", () => {
+    const handler = () => {
       permissionClient.confirm(uuid, {
         allow: false,
         type: 0,
       });
-    });
+    };
+    window.addEventListener("beforeunload", handler, false);
+    return () => window.removeEventListener("beforeunload", handler, false);
+  }, [uuid]);
 
+  useEffect(() => {
     permissionClient
       .getPermissionInfo(uuid)
       .then((data) => {