chatccc 0.2.183 → 0.2.185

package/src/__tests__/chrome-devtools-guard.test.ts

@@ -0,0 +1,125 @@
+import { describe, expect, it, vi } from "vitest";
+import { spawn } from "node:child_process";
+
+import {
+  ensureChromeCdpRunning,
+  isChromeCdpHealthy,
+  probeChromeCdp,
+  resolveChromeExecutable,
+  resolveChromeUserDataDir,
+} from "../chrome-devtools-guard.ts";
+
+describe("Chrome CDP guard", () => {
+  it("treats /json/version with Browser as healthy", async () => {
+    const fetchImpl = vi.fn(async () =>
+      new Response(JSON.stringify({ Browser: "Chrome/144.0.0.0" }), { status: 200 }),
+    ) as unknown as typeof fetch;
+
+    await expect(isChromeCdpHealthy(15166, { fetchImpl })).resolves.toBe(true);
+  });
+
+  it("does not spawn Chrome when the CDP endpoint is already healthy", async () => {
+    const fetchImpl = vi.fn(async () =>
+      new Response(JSON.stringify({ webSocketDebuggerUrl: "ws://127.0.0.1:15166/devtools/browser/1" }), { status: 200 }),
+    ) as unknown as typeof fetch;
+    const spawnImpl = vi.fn() as unknown as typeof spawn;
+
+    await expect(
+      ensureChromeCdpRunning(
+        { enabled: true, port: 15166, chromePath: "C:/Chrome/chrome.exe" },
+        { fetchImpl, spawnImpl },
+      ),
+    ).resolves.toEqual({ ok: true, started: false, port: 15166 });
+    expect(spawnImpl).not.toHaveBeenCalled();
+  });
+
+  it("spawns Chrome with a dedicated profile when the endpoint is unhealthy", async () => {
+    const fetchImpl = vi.fn()
+      .mockRejectedValueOnce(new Error("ECONNREFUSED"))
+      .mockResolvedValue(new Response(JSON.stringify({ Browser: "Chrome/144.0.0.0" }), { status: 200 })) as unknown as typeof fetch;
+    const unref = vi.fn();
+    const spawnImpl = vi.fn(() => ({ unref })) as unknown as typeof spawn;
+    const mkdirSyncImpl = vi.fn();
+
+    await expect(
+      ensureChromeCdpRunning(
+        { enabled: true, port: 15166, chromePath: "C:/Chrome/chrome.exe" },
+        {
+          fetchImpl,
+          spawnImpl,
+          mkdirSyncImpl: mkdirSyncImpl as never,
+          existsSyncImpl: vi.fn(() => true),
+          env: { LOCALAPPDATA: "C:/Users/me/AppData/Local" },
+        },
+      ),
+    ).resolves.toEqual({ ok: true, started: true, port: 15166 });
+
+    expect(mkdirSyncImpl).toHaveBeenCalledWith("C:\\Users\\me\\AppData\\Local\\chatccc\\chrome-cdp-15166", { recursive: true });
+    expect(spawnImpl).toHaveBeenCalledWith(
+      "C:/Chrome/chrome.exe",
+      expect.arrayContaining([
+        "--remote-debugging-address=127.0.0.1",
+        "--remote-debugging-port=15166",
+        "--user-data-dir=C:\\Users\\me\\AppData\\Local\\chatccc\\chrome-cdp-15166",
+        "--new-window",
+        "about:blank",
+      ]),
+      expect.objectContaining({ detached: true, stdio: "ignore", windowsHide: true }),
+    );
+    expect(unref).toHaveBeenCalled();
+  });
+
+  it("does not spawn Chrome when the port is occupied by a non-CDP service", async () => {
+    const fetchImpl = vi.fn(async () => new Response("not found", { status: 404 })) as unknown as typeof fetch;
+    const spawnImpl = vi.fn() as unknown as typeof spawn;
+
+    await expect(probeChromeCdp(15166, { fetchImpl })).resolves.toBe("occupied");
+    await expect(
+      ensureChromeCdpRunning(
+        { enabled: true, port: 15166, chromePath: "C:/Chrome/chrome.exe" },
+        { fetchImpl, spawnImpl },
+      ),
+    ).resolves.toMatchObject({
+      ok: false,
+      started: false,
+      port: 15166,
+      error: expect.stringContaining("not a healthy Chrome CDP endpoint"),
+    });
+    expect(spawnImpl).not.toHaveBeenCalled();
+  });
+
+  it("reports a clear error when Chrome cannot be found", async () => {
+    const fetchImpl = vi.fn(async () => { throw new Error("ECONNREFUSED"); }) as unknown as typeof fetch;
+
+    await expect(
+      ensureChromeCdpRunning(
+        { enabled: true, port: 15166, chromePath: "" },
+        {
+          fetchImpl,
+          existsSyncImpl: vi.fn(() => false),
+          platform: "win32",
+          env: {},
+        },
+      ),
+    ).resolves.toMatchObject({
+      ok: false,
+      started: false,
+      port: 15166,
+      error: expect.stringContaining("Cannot find chrome executable"),
+    });
+  });
+
+  it("resolves explicit and default Chrome paths", () => {
+    expect(resolveChromeExecutable("C:/Chrome/chrome.exe", { existsSyncImpl: vi.fn(() => true) })).toBe("C:/Chrome/chrome.exe");
+    expect(resolveChromeExecutable("", {
+      existsSyncImpl: vi.fn((p: unknown) => String(p).endsWith("Google\\Chrome\\Application\\chrome.exe")),
+      platform: "win32",
+      env: { ProgramFiles: "C:/Program Files" },
+    })).toBe("C:\\Program Files\\Google\\Chrome\\Application\\chrome.exe");
+  });
+
+  it("uses LocalAppData for the Chrome CDP user data directory", () => {
+    expect(resolveChromeUserDataDir(15166, { LOCALAPPDATA: "C:/Users/me/AppData/Local" }))
+      .toBe("C:\\Users\\me\\AppData\\Local\\chatccc\\chrome-cdp-15166");
+  });
+});

package/src/__tests__/codex-reset-actions.test.ts

@@ -1,146 +1,146 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-import {
-  _resetCodexResetRequestsForTest,
-  handleCodexResetCardAction,
-  type CodexResetActionDeps,
-} from "../codex-reset-actions.ts";
-
-function deps(): CodexResetActionDeps & {
-  getTenantAccessToken: ReturnType<typeof vi.fn>;
-  sendRawCard: ReturnType<typeof vi.fn>;
-  sendTextReply: ReturnType<typeof vi.fn>;
-  sendCardReply: ReturnType<typeof vi.fn>;
-  updateCardMessage: ReturnType<typeof vi.fn>;
-  recallMessage: ReturnType<typeof vi.fn>;
-  consumeCodexRateLimitResetCredit: ReturnType<typeof vi.fn>;
-  createRequestId: ReturnType<typeof vi.fn>;
-} {
-  return {
-    getTenantAccessToken: vi.fn(async () => "tenant-token"),
-    sendRawCard: vi.fn(async () => true),
-    sendTextReply: vi.fn(async () => true),
-    sendCardReply: vi.fn(async () => true),
-    updateCardMessage: vi.fn(async () => true),
-    recallMessage: vi.fn(async () => true),
-    consumeCodexRateLimitResetCredit: vi.fn(async () => ({ code: "reset" as const, windowsReset: 2 })),
-    createRequestId: vi.fn(() => "request-1"),
-  };
-}
-
-function event(value: Record<string, unknown>, messageId: string) {
-  return {
-    event: {
-      open_message_id: messageId,
-      open_chat_id: "chat-1",
-      operator: { open_id: "ou-user" },
-      action: { value },
-    },
-  };
-}
-
-describe("Codex reset card actions", () => {
-  beforeEach(() => {
-    _resetCodexResetRequestsForTest();
-  });
-
-  it("sends a confirmation card tied to the clicked usage card", async () => {
-    const d = deps();
-
-    await expect(handleCodexResetCardAction(
-      event({ action: "codex_reset_request" }, "usage-message"),
-      d,
-    )).resolves.toBe(true);
-
-    expect(d.sendRawCard).toHaveBeenCalledTimes(1);
-    expect(d.sendRawCard.mock.calls[0][1]).toBe("chat-1");
-    const confirmCard = JSON.parse(d.sendRawCard.mock.calls[0][2]);
-    const action = confirmCard.elements.find((element: any) => element.tag === "action");
-    expect(action.actions[0].value).toEqual({
-      action: "codex_reset_confirm",
-      decision: "yes",
-      parentMessageId: "usage-message",
-      requestId: "request-1",
-    });
-  });
-
-  it("consumes a reset credit, sends the result, and collapses the confirmation card when user confirms", async () => {
-    const d = deps();
-    await handleCodexResetCardAction(event({ action: "codex_reset_request" }, "usage-message"), d);
-
-    await expect(handleCodexResetCardAction(
-      event({
-        action: "codex_reset_confirm",
-        decision: "yes",
-        parentMessageId: "usage-message",
-        requestId: "request-1",
-      }, "confirm-message"),
-      d,
-    )).resolves.toBe(true);
-
-    expect(d.consumeCodexRateLimitResetCredit).toHaveBeenCalledWith("request-1");
-    expect(d.updateCardMessage).toHaveBeenCalledTimes(1);
-    expect(d.updateCardMessage.mock.calls[0][1]).toBe("confirm-message");
-    expect(JSON.parse(d.updateCardMessage.mock.calls[0][2]).elements).toEqual([
-      { tag: "markdown", content: " " },
-    ]);
-    expect(d.recallMessage).toHaveBeenCalledWith("tenant-token", "confirm-message");
-    expect(d.sendTextReply).toHaveBeenCalledWith(
-      "tenant-token",
-      "chat-1",
-      expect.stringContaining("重置成功"),
-    );
-    expect(d.sendCardReply).not.toHaveBeenCalled();
-  });
-
-  it("does not consume a reset credit and sends a cancellation message when user declines", async () => {
-    const d = deps();
-    await handleCodexResetCardAction(event({ action: "codex_reset_request" }, "usage-message"), d);
-
-    await expect(handleCodexResetCardAction(
-      event({
-        action: "codex_reset_confirm",
-        decision: "no",
-        parentMessageId: "usage-message",
-        requestId: "request-1",
-      }, "confirm-message"),
-      d,
-    )).resolves.toBe(true);
-
-    expect(d.consumeCodexRateLimitResetCredit).not.toHaveBeenCalled();
-    expect(d.updateCardMessage).toHaveBeenCalledTimes(1);
-    expect(d.recallMessage).toHaveBeenCalledWith("tenant-token", "confirm-message");
-    expect(d.sendTextReply).toHaveBeenCalledWith(
-      "tenant-token",
-      "chat-1",
-      "Codex 主动重置：用户取消了重置。",
-    );
-    expect(d.sendCardReply).not.toHaveBeenCalled();
-  });
-
-  it("can collapse a confirmation card using context.message_id fallback", async () => {
-    const d = deps();
-    await handleCodexResetCardAction(event({ action: "codex_reset_request" }, "usage-message"), d);
-
-    await expect(handleCodexResetCardAction({
-      event: {
-        context: {
-          message_id: "confirm-message-from-context",
-          open_chat_id: "chat-1",
-        },
-        action: {
-          value: {
-            action: "codex_reset_confirm",
-            decision: "no",
-            parentMessageId: "usage-message",
-            requestId: "request-1",
-          },
-        },
-      },
-    }, d)).resolves.toBe(true);
-
-    expect(d.updateCardMessage).toHaveBeenCalledTimes(1);
-    expect(d.updateCardMessage.mock.calls[0][1]).toBe("confirm-message-from-context");
-    expect(d.recallMessage).toHaveBeenCalledWith("tenant-token", "confirm-message-from-context");
-  });
-});
+import { beforeEach, describe, expect, it, vi } from "vitest";
+
+import {
+  _resetCodexResetRequestsForTest,
+  handleCodexResetCardAction,
+  type CodexResetActionDeps,
+} from "../codex-reset-actions.ts";
+
+function deps(): CodexResetActionDeps & {
+  getTenantAccessToken: ReturnType<typeof vi.fn>;
+  sendRawCard: ReturnType<typeof vi.fn>;
+  sendTextReply: ReturnType<typeof vi.fn>;
+  sendCardReply: ReturnType<typeof vi.fn>;
+  updateCardMessage: ReturnType<typeof vi.fn>;
+  recallMessage: ReturnType<typeof vi.fn>;
+  consumeCodexRateLimitResetCredit: ReturnType<typeof vi.fn>;
+  createRequestId: ReturnType<typeof vi.fn>;
+} {
+  return {
+    getTenantAccessToken: vi.fn(async () => "tenant-token"),
+    sendRawCard: vi.fn(async () => true),
+    sendTextReply: vi.fn(async () => true),
+    sendCardReply: vi.fn(async () => true),
+    updateCardMessage: vi.fn(async () => true),
+    recallMessage: vi.fn(async () => true),
+    consumeCodexRateLimitResetCredit: vi.fn(async () => ({ code: "reset" as const, windowsReset: 2 })),
+    createRequestId: vi.fn(() => "request-1"),
+  };
+}
+
+function event(value: Record<string, unknown>, messageId: string) {
+  return {
+    event: {
+      open_message_id: messageId,
+      open_chat_id: "chat-1",
+      operator: { open_id: "ou-user" },
+      action: { value },
+    },
+  };
+}
+
+describe("Codex reset card actions", () => {
+  beforeEach(() => {
+    _resetCodexResetRequestsForTest();
+  });
+
+  it("sends a confirmation card tied to the clicked usage card", async () => {
+    const d = deps();
+
+    await expect(handleCodexResetCardAction(
+      event({ action: "codex_reset_request" }, "usage-message"),
+      d,
+    )).resolves.toBe(true);
+
+    expect(d.sendRawCard).toHaveBeenCalledTimes(1);
+    expect(d.sendRawCard.mock.calls[0][1]).toBe("chat-1");
+    const confirmCard = JSON.parse(d.sendRawCard.mock.calls[0][2]);
+    const action = confirmCard.elements.find((element: any) => element.tag === "action");
+    expect(action.actions[0].value).toEqual({
+      action: "codex_reset_confirm",
+      decision: "yes",
+      parentMessageId: "usage-message",
+      requestId: "request-1",
+    });
+  });
+
+  it("consumes a reset credit, sends the result, and collapses the confirmation card when user confirms", async () => {
+    const d = deps();
+    await handleCodexResetCardAction(event({ action: "codex_reset_request" }, "usage-message"), d);
+
+    await expect(handleCodexResetCardAction(
+      event({
+        action: "codex_reset_confirm",
+        decision: "yes",
+        parentMessageId: "usage-message",
+        requestId: "request-1",
+      }, "confirm-message"),
+      d,
+    )).resolves.toBe(true);
+
+    expect(d.consumeCodexRateLimitResetCredit).toHaveBeenCalledWith("request-1");
+    expect(d.updateCardMessage).toHaveBeenCalledTimes(1);
+    expect(d.updateCardMessage.mock.calls[0][1]).toBe("confirm-message");
+    expect(JSON.parse(d.updateCardMessage.mock.calls[0][2]).elements).toEqual([
+      { tag: "markdown", content: " " },
+    ]);
+    expect(d.recallMessage).toHaveBeenCalledWith("tenant-token", "confirm-message");
+    expect(d.sendTextReply).toHaveBeenCalledWith(
+      "tenant-token",
+      "chat-1",
+      expect.stringContaining("重置成功"),
+    );
+    expect(d.sendCardReply).not.toHaveBeenCalled();
+  });
+
+  it("does not consume a reset credit and sends a cancellation message when user declines", async () => {
+    const d = deps();
+    await handleCodexResetCardAction(event({ action: "codex_reset_request" }, "usage-message"), d);
+
+    await expect(handleCodexResetCardAction(
+      event({
+        action: "codex_reset_confirm",
+        decision: "no",
+        parentMessageId: "usage-message",
+        requestId: "request-1",
+      }, "confirm-message"),
+      d,
+    )).resolves.toBe(true);
+
+    expect(d.consumeCodexRateLimitResetCredit).not.toHaveBeenCalled();
+    expect(d.updateCardMessage).toHaveBeenCalledTimes(1);
+    expect(d.recallMessage).toHaveBeenCalledWith("tenant-token", "confirm-message");
+    expect(d.sendTextReply).toHaveBeenCalledWith(
+      "tenant-token",
+      "chat-1",
+      "Codex 主动重置：用户取消了重置。",
+    );
+    expect(d.sendCardReply).not.toHaveBeenCalled();
+  });
+
+  it("can collapse a confirmation card using context.message_id fallback", async () => {
+    const d = deps();
+    await handleCodexResetCardAction(event({ action: "codex_reset_request" }, "usage-message"), d);
+
+    await expect(handleCodexResetCardAction({
+      event: {
+        context: {
+          message_id: "confirm-message-from-context",
+          open_chat_id: "chat-1",
+        },
+        action: {
+          value: {
+            action: "codex_reset_confirm",
+            decision: "no",
+            parentMessageId: "usage-message",
+            requestId: "request-1",
+          },
+        },
+      },
+    }, d)).resolves.toBe(true);
+
+    expect(d.updateCardMessage).toHaveBeenCalledTimes(1);
+    expect(d.updateCardMessage.mock.calls[0][1]).toBe("confirm-message-from-context");
+    expect(d.recallMessage).toHaveBeenCalledWith("tenant-token", "confirm-message-from-context");
+  });
+});

package/src/__tests__/config-reload.test.ts

@@ -40,6 +40,7 @@ import {
 const baseAppConfig: AppConfig = {
   feishu: { appId: "INITIAL_APP", appSecret: "INITIAL_SECRET" },
   platforms: { feishu: { enabled: true }, ilink: { enabled: true } },
+  chromeDevtools: { enabled: false, port: 15166, chromePath: "" },
   port: 18080,
   gitTimeoutSeconds: 180,
   allowInterrupt: false,
@@ -195,6 +196,18 @@ describe("applyLoadedConfig — config 对象引用契约", () => {
     expect(config.codex.path).toBe("/refresh/codex");
   });
 
+  it("刷新 chromeDevtools 配置但保持 config 引用", () => {
+    const refBefore = config;
+
+    applyLoadedConfig({
+      ...structuredClone(baseAppConfig),
+      chromeDevtools: { enabled: true, port: 15166, chromePath: "C:/Chrome/chrome.exe" },
+    });
+
+    expect(config).toBe(refBefore);
+    expect(config.chromeDevtools).toEqual({ enabled: true, port: 15166, chromePath: "C:/Chrome/chrome.exe" });
+  });
+
   it("空 feishu 凭证也能正确刷入（向导回滚到空的反向场景）", () => {
     applyLoadedConfig({
       ...structuredClone(baseAppConfig),

package/src/__tests__/config-sample.test.ts

@@ -27,4 +27,15 @@ describe("config.sample.json", () => {
     expect(sample.claude?.model).toBe("");
     expect(sample.claude?.subagentModel).toBe("");
   });
+
+  it("keeps Chrome CDP guard disabled by default with port 15166", () => {
+    const configSamplePath = join(process.cwd(), "config.sample.json");
+    const sample = JSON.parse(readFileSync(configSamplePath, "utf8")) as {
+      chromeDevtools?: { enabled?: unknown; port?: unknown; chromePath?: unknown };
+    };
+
+    expect(sample.chromeDevtools?.enabled).toBe(false);
+    expect(sample.chromeDevtools?.port).toBe(15166);
+    expect(sample.chromeDevtools?.chromePath).toBe("");
+  });
 });

package/src/__tests__/format-message.test.ts

@@ -11,52 +11,52 @@ vi.mock("../feishu-platform.ts", () => ({
   getMergeForwardMessages: (...args: unknown[]) => mockGetMergeForwardMessages(...args),
 }));
 
-import { formatMessageContent, formatPostContent, formatMergeForward } from "../format-message.ts";
-
-describe("formatMessageContent mixed post images", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockGetTenantAccessToken.mockResolvedValue("mock_token");
-  });
-
-  it("includes downloaded images from mixed post messages", async () => {
-    mockGetOrDownloadImage.mockResolvedValue("C:\\tmp\\img_001.png");
-
-    const result = await formatMessageContent({
-      message_id: "om_post",
-      message_type: "post",
-      content: JSON.stringify({
-        content: [[
-          { tag: "text", text: "use this image" },
-          { tag: "img", image_key: "img_001" },
-        ]],
-      }),
-    });
-
-    expect(result).toBe("use this image\n[图片] C:\\tmp\\img_001.png");
-    expect(mockGetTenantAccessToken).toHaveBeenCalled();
-    expect(mockGetOrDownloadImage).toHaveBeenCalledWith("mock_token", "om_post", "img_001");
-  });
-
-  it("keeps post image key when mixed post image download fails", async () => {
-    mockGetOrDownloadImage.mockRejectedValue(new Error("download failed"));
-
-    const result = await formatMessageContent({
-      message_id: "om_post",
-      message_type: "post",
-      content: JSON.stringify({
-        content: [[
-          { tag: "text", text: "caption" },
-          { tag: "img", image_key: "img_002" },
-        ]],
-      }),
-    });
-
-    expect(result).toBe("caption\n[图片: img_002]");
-  });
-});
-
-describe("formatMessageContent", () => {
+import { formatMessageContent, formatPostContent, formatMergeForward } from "../format-message.ts";
+
+describe("formatMessageContent mixed post images", () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    mockGetTenantAccessToken.mockResolvedValue("mock_token");
+  });
+
+  it("includes downloaded images from mixed post messages", async () => {
+    mockGetOrDownloadImage.mockResolvedValue("C:\\tmp\\img_001.png");
+
+    const result = await formatMessageContent({
+      message_id: "om_post",
+      message_type: "post",
+      content: JSON.stringify({
+        content: [[
+          { tag: "text", text: "use this image" },
+          { tag: "img", image_key: "img_001" },
+        ]],
+      }),
+    });
+
+    expect(result).toBe("use this image\n[图片] C:\\tmp\\img_001.png");
+    expect(mockGetTenantAccessToken).toHaveBeenCalled();
+    expect(mockGetOrDownloadImage).toHaveBeenCalledWith("mock_token", "om_post", "img_001");
+  });
+
+  it("keeps post image key when mixed post image download fails", async () => {
+    mockGetOrDownloadImage.mockRejectedValue(new Error("download failed"));
+
+    const result = await formatMessageContent({
+      message_id: "om_post",
+      message_type: "post",
+      content: JSON.stringify({
+        content: [[
+          { tag: "text", text: "caption" },
+          { tag: "img", image_key: "img_002" },
+        ]],
+      }),
+    });
+
+    expect(result).toBe("caption\n[图片: img_002]");
+  });
+});
+
+describe("formatMessageContent", () => {
   beforeEach(() => {
     vi.clearAllMocks();
     mockGetTenantAccessToken.mockResolvedValue("mock_token");
@@ -313,4 +313,4 @@ describe("formatPostContent", () => {
     const result = formatPostContent({ content: [null, undefined, "string"] });
     expect(result).toBe("");
   });
-});
+});