chatccc 0.2.201 → 0.2.203

package/src/__tests__/cursor-adapter.test.ts

@@ -1,17 +1,17 @@
-import { afterEach, describe, it, expect, vi } from "vitest";
-import { EventEmitter } from "node:events";
-import { readFileSync } from "node:fs";
-import { join, dirname } from "node:path";
-import { PassThrough } from "node:stream";
-import type { ChildProcess } from "node:child_process";
-import { fileURLToPath } from "node:url";
-import {
-  normalizeCursorMessage,
-  createCursorAdapter,
-  formatCursorAgentEmptyOutputMessage,
-} from "../adapters/cursor-adapter.ts";
-import { BadJsonIdleTimeoutError } from "../adapters/jsonl-stream.ts";
-import type { UnifiedStreamMessage } from "../adapters/adapter-interface.ts";
+import { afterEach, describe, it, expect, vi } from "vitest";
+import { EventEmitter } from "node:events";
+import { readFileSync } from "node:fs";
+import { join, dirname } from "node:path";
+import { PassThrough } from "node:stream";
+import type { ChildProcess } from "node:child_process";
+import { fileURLToPath } from "node:url";
+import {
+  normalizeCursorMessage,
+  createCursorAdapter,
+  formatCursorAgentEmptyOutputMessage,
+} from "../adapters/cursor-adapter.ts";
+import { BadJsonIdleTimeoutError } from "../adapters/jsonl-stream.ts";
+import type { UnifiedStreamMessage } from "../adapters/adapter-interface.ts";
 import type {
   CursorSessionMeta,
   CursorSessionMetaStore,
@@ -45,70 +45,70 @@ function createInMemoryMetaStore(
   };
 }
 
-const __dirname = dirname(fileURLToPath(import.meta.url));
-
-type CursorSpawnForTest = NonNullable<
-  NonNullable<Parameters<typeof createCursorAdapter>[0]>["spawn"]
->;
-
-function readFixture(name: string): unknown[] {
-  const raw = readFileSync(join(__dirname, "fixtures", name), "utf-8");
-  return raw.split(/\r?\n/).filter(Boolean).map((line) => JSON.parse(line));
-}
-
-function createMockCursorProcess(args: {
-  stdout?: string;
-  stderr?: string;
-  exitCode?: number;
-}): ChildProcess {
-  const child = new EventEmitter() as ChildProcess;
-  const stdout = new PassThrough();
-  const stderr = new PassThrough();
-  const stdin = new PassThrough();
-  Object.assign(child, {
-    stdout,
-    stderr,
-    stdin,
-    pid: undefined,
-  });
-
-  setImmediate(() => {
-    if (args.stdout) stdout.write(args.stdout);
-    if (args.stderr) stderr.write(args.stderr);
-    stdout.end();
-    stderr.end();
-    child.emit("close", args.exitCode ?? 0, null);
-  });
-
-  return child;
-}
-
-function createHangingMockCursorProcess(args: {
-  stdout?: string;
-  stderr?: string;
-}): ChildProcess {
-  const child = new EventEmitter() as ChildProcess;
-  const stdout = new PassThrough();
-  const stderr = new PassThrough();
-  const stdin = new PassThrough();
-  Object.assign(child, {
-    stdout,
-    stderr,
-    stdin,
-    pid: undefined,
-  });
-
-  queueMicrotask(() => {
-    if (args.stdout) stdout.write(args.stdout);
-    if (args.stderr) stderr.write(args.stderr);
-  });
-
-  return child;
-}
-
-afterEach(() => {
-  vi.useRealTimers();
-});
+const __dirname = dirname(fileURLToPath(import.meta.url));
+
+type CursorSpawnForTest = NonNullable<
+  NonNullable<Parameters<typeof createCursorAdapter>[0]>["spawn"]
+>;
+
+function readFixture(name: string): unknown[] {
+  const raw = readFileSync(join(__dirname, "fixtures", name), "utf-8");
+  return raw.split(/\r?\n/).filter(Boolean).map((line) => JSON.parse(line));
+}
+
+function createMockCursorProcess(args: {
+  stdout?: string;
+  stderr?: string;
+  exitCode?: number;
+}): ChildProcess {
+  const child = new EventEmitter() as ChildProcess;
+  const stdout = new PassThrough();
+  const stderr = new PassThrough();
+  const stdin = new PassThrough();
+  Object.assign(child, {
+    stdout,
+    stderr,
+    stdin,
+    pid: undefined,
+  });
+
+  setImmediate(() => {
+    if (args.stdout) stdout.write(args.stdout);
+    if (args.stderr) stderr.write(args.stderr);
+    stdout.end();
+    stderr.end();
+    child.emit("close", args.exitCode ?? 0, null);
+  });
+
+  return child;
+}
+
+function createHangingMockCursorProcess(args: {
+  stdout?: string;
+  stderr?: string;
+}): ChildProcess {
+  const child = new EventEmitter() as ChildProcess;
+  const stdout = new PassThrough();
+  const stderr = new PassThrough();
+  const stdin = new PassThrough();
+  Object.assign(child, {
+    stdout,
+    stderr,
+    stdin,
+    pid: undefined,
+  });
+
+  queueMicrotask(() => {
+    if (args.stdout) stdout.write(args.stdout);
+    if (args.stderr) stderr.write(args.stderr);
+  });
+
+  return child;
+}
+
+afterEach(() => {
+  vi.useRealTimers();
+});
 
 // ---------------------------------------------------------------------------
 // normalizeCursorMessage — 核心映射逻辑测试（纯函数）
@@ -708,7 +708,7 @@ describe("Cursor stream fixture - 端到端不重复", () => {
     });
   });
 
-  it("mapToolCallKey: maps known keys to readable names", () => {
+  it("mapToolCallKey: maps known keys to readable names", () => {
     const cases: [string, string][] = [
       ["globToolCall", "Glob"],
       ["shellToolCall", "Bash"],
@@ -724,150 +724,150 @@ describe("Cursor stream fixture - 端到端不重复", () => {
         tool_call: { [raw]: { args: {} } },
       } as Parameters<typeof normalizeCursorMessage>[0]);
       expect(r!.blocks[0]).toMatchObject({ type: "tool_use", name: expected });
-    }
-  });
-});
-
-describe("Cursor adapter process failures", () => {
-  it("formats empty stdout auth stderr as a cautious visible message", () => {
-    const message = formatCursorAgentEmptyOutputMessage({
-      exitCode: 1,
-      stdoutLength: 0,
-      stderr: "Error: Authentication required. Please run 'agent login' first, or set CURSOR_API_KEY environment variable.\n",
-    });
-
-    expect(message).toContain("检测到认证相关错误");
-    expect(message).toContain("可能需要重新登录 Cursor Agent");
-    expect(message).toContain("CURSOR_API_KEY");
-    expect(message).toContain("agent status");
-    expect(message).toContain("agent login");
-    expect(message).not.toContain("登录态已失效");
-  });
-
-  it("formats empty stdout action-required stderr with the actionable Cursor error", () => {
-    const message = formatCursorAgentEmptyOutputMessage({
-      exitCode: 1,
-      stdoutLength: 0,
-      stderr: "ActionRequiredError: You have an unpaid invoice Your team has an unpaid invoice. Please contact your team administrator to pay your invoice and continue using Cursor.\n",
-    });
-
-    expect(message).toContain("底层命令异常退出");
-    expect(message).toContain("[Cursor stderr] exit=1");
-    expect(message).toContain("ActionRequiredError");
-    expect(message).toContain("unpaid invoice");
-    expect(message).toContain("team administrator");
-  });
-
-  it("redacts obvious secrets from visible Cursor stderr", () => {
-    const message = formatCursorAgentEmptyOutputMessage({
-      exitCode: 1,
-      stdoutLength: 0,
-      stderr: "Error: failed with token=secret-token-value and Bearer abcdefghijklmnop\n",
-    });
-
-    expect(message).toContain("token=<redacted>");
-    expect(message).toContain("Bearer <redacted>");
-    expect(message).not.toContain("secret-token-value");
-    expect(message).not.toContain("abcdefghijklmnop");
-  });
-
-  it("does not format a message when stdout is non-empty", () => {
-    expect(
-      formatCursorAgentEmptyOutputMessage({
-        exitCode: 1,
-        stdoutLength: 10,
-        stderr: "Error: Authentication required",
-      }),
-    ).toBeNull();
-  });
-
-  it("prompt surfaces auth-related empty output before failing the stream", async () => {
-    const store = createInMemoryMetaStore({ sid: { cwd: "F:/repo" } });
-    const spawnImpl = (() =>
-      createMockCursorProcess({
-        exitCode: 1,
-        stderr: "Error: Authentication required. Please run 'agent login' first, or set CURSOR_API_KEY environment variable.\n",
-      })) as CursorSpawnForTest;
-    const adapter = createCursorAdapter({ metaStore: store, spawn: spawnImpl });
-    const iterator = adapter.prompt(
-      "sid",
-      "[User message]\nhello\n[/User message]",
-      "F:/repo",
-    )[Symbol.asyncIterator]();
-
-    const first = await iterator.next();
-    expect(first.done).toBe(false);
-    expect(first.value.blocks).toHaveLength(1);
-    expect(first.value.blocks[0]).toMatchObject({ type: "text_final" });
-    expect(first.value.blocks[0]).toHaveProperty(
-      "text",
-      expect.stringContaining("可能需要重新登录 Cursor Agent"),
-    );
-    expect(first.value.blocks[0]).toHaveProperty(
-      "text",
-      expect.not.stringContaining("登录态已失效"),
-    );
-
-    await expect(iterator.next()).rejects.toThrow(
-      /Cursor Agent exited without stream-json output/,
-    );
-  });
-
-  it("prompt surfaces action-required stderr before failing the stream", async () => {
-    const store = createInMemoryMetaStore({ sid: { cwd: "F:/repo" } });
-    const spawnImpl = (() =>
-      createMockCursorProcess({
-        exitCode: 1,
-        stderr: "ActionRequiredError: You have an unpaid invoice Your team has an unpaid invoice. Please contact your team administrator to pay your invoice and continue using Cursor.\n",
-      })) as CursorSpawnForTest;
-    const adapter = createCursorAdapter({ metaStore: store, spawn: spawnImpl });
-    const iterator = adapter.prompt(
-      "sid",
-      "[User message]\nhello\n[/User message]",
-      "F:/repo",
-    )[Symbol.asyncIterator]();
-
-    const first = await iterator.next();
-    expect(first.done).toBe(false);
-    expect(first.value.blocks).toHaveLength(1);
-    expect(first.value.blocks[0]).toMatchObject({ type: "text_final" });
-    expect(first.value.blocks[0]).toHaveProperty(
-      "text",
-      expect.stringContaining("ActionRequiredError"),
-    );
-    expect(first.value.blocks[0]).toHaveProperty(
-      "text",
-      expect.stringContaining("unpaid invoice"),
-    );
-
-    await expect(iterator.next()).rejects.toThrow(
-      /Cursor Agent exited without stream-json output/,
-    );
-  });
-
-  it("prompt fails when Cursor emits bad JSON and then leaves stdout idle", async () => {
-    const store = createInMemoryMetaStore({ sid: { cwd: "F:/repo" } });
-    const spawnImpl = (() =>
-      createHangingMockCursorProcess({
-        stdout: "{\"type\":\"tool_call\",\"subtype\":\"started\"\n",
-      })) as CursorSpawnForTest;
-    const adapter = createCursorAdapter({
-      metaStore: store,
-      spawn: spawnImpl,
-      badJsonIdleTimeoutMs: 10,
-    });
-
-    const pending = (async () => {
-      for await (const _event of adapter.prompt(
-        "sid",
-        "[User message]\nhello\n[/User message]",
-        "F:/repo",
-      )) {
-        // No normalized events are expected before the watchdog fires.
-      }
-    })().catch((error: unknown) => error);
-
-    const error = await pending;
-    expect(error).toBeInstanceOf(BadJsonIdleTimeoutError);
-  });
-});
+    }
+  });
+});
+
+describe("Cursor adapter process failures", () => {
+  it("formats empty stdout auth stderr as a cautious visible message", () => {
+    const message = formatCursorAgentEmptyOutputMessage({
+      exitCode: 1,
+      stdoutLength: 0,
+      stderr: "Error: Authentication required. Please run 'agent login' first, or set CURSOR_API_KEY environment variable.\n",
+    });
+
+    expect(message).toContain("检测到认证相关错误");
+    expect(message).toContain("可能需要重新登录 Cursor Agent");
+    expect(message).toContain("CURSOR_API_KEY");
+    expect(message).toContain("agent status");
+    expect(message).toContain("agent login");
+    expect(message).not.toContain("登录态已失效");
+  });
+
+  it("formats empty stdout action-required stderr with the actionable Cursor error", () => {
+    const message = formatCursorAgentEmptyOutputMessage({
+      exitCode: 1,
+      stdoutLength: 0,
+      stderr: "ActionRequiredError: You have an unpaid invoice Your team has an unpaid invoice. Please contact your team administrator to pay your invoice and continue using Cursor.\n",
+    });
+
+    expect(message).toContain("底层命令异常退出");
+    expect(message).toContain("[Cursor stderr] exit=1");
+    expect(message).toContain("ActionRequiredError");
+    expect(message).toContain("unpaid invoice");
+    expect(message).toContain("team administrator");
+  });
+
+  it("redacts obvious secrets from visible Cursor stderr", () => {
+    const message = formatCursorAgentEmptyOutputMessage({
+      exitCode: 1,
+      stdoutLength: 0,
+      stderr: "Error: failed with token=secret-token-value and Bearer abcdefghijklmnop\n",
+    });
+
+    expect(message).toContain("token=<redacted>");
+    expect(message).toContain("Bearer <redacted>");
+    expect(message).not.toContain("secret-token-value");
+    expect(message).not.toContain("abcdefghijklmnop");
+  });
+
+  it("does not format a message when stdout is non-empty", () => {
+    expect(
+      formatCursorAgentEmptyOutputMessage({
+        exitCode: 1,
+        stdoutLength: 10,
+        stderr: "Error: Authentication required",
+      }),
+    ).toBeNull();
+  });
+
+  it("prompt surfaces auth-related empty output before failing the stream", async () => {
+    const store = createInMemoryMetaStore({ sid: { cwd: "F:/repo" } });
+    const spawnImpl = (() =>
+      createMockCursorProcess({
+        exitCode: 1,
+        stderr: "Error: Authentication required. Please run 'agent login' first, or set CURSOR_API_KEY environment variable.\n",
+      })) as CursorSpawnForTest;
+    const adapter = createCursorAdapter({ metaStore: store, spawn: spawnImpl });
+    const iterator = adapter.prompt(
+      "sid",
+      "[User message]\nhello\n[/User message]",
+      "F:/repo",
+    )[Symbol.asyncIterator]();
+
+    const first = await iterator.next();
+    expect(first.done).toBe(false);
+    expect(first.value.blocks).toHaveLength(1);
+    expect(first.value.blocks[0]).toMatchObject({ type: "text_final" });
+    expect(first.value.blocks[0]).toHaveProperty(
+      "text",
+      expect.stringContaining("可能需要重新登录 Cursor Agent"),
+    );
+    expect(first.value.blocks[0]).toHaveProperty(
+      "text",
+      expect.not.stringContaining("登录态已失效"),
+    );
+
+    await expect(iterator.next()).rejects.toThrow(
+      /Cursor Agent exited without stream-json output/,
+    );
+  });
+
+  it("prompt surfaces action-required stderr before failing the stream", async () => {
+    const store = createInMemoryMetaStore({ sid: { cwd: "F:/repo" } });
+    const spawnImpl = (() =>
+      createMockCursorProcess({
+        exitCode: 1,
+        stderr: "ActionRequiredError: You have an unpaid invoice Your team has an unpaid invoice. Please contact your team administrator to pay your invoice and continue using Cursor.\n",
+      })) as CursorSpawnForTest;
+    const adapter = createCursorAdapter({ metaStore: store, spawn: spawnImpl });
+    const iterator = adapter.prompt(
+      "sid",
+      "[User message]\nhello\n[/User message]",
+      "F:/repo",
+    )[Symbol.asyncIterator]();
+
+    const first = await iterator.next();
+    expect(first.done).toBe(false);
+    expect(first.value.blocks).toHaveLength(1);
+    expect(first.value.blocks[0]).toMatchObject({ type: "text_final" });
+    expect(first.value.blocks[0]).toHaveProperty(
+      "text",
+      expect.stringContaining("ActionRequiredError"),
+    );
+    expect(first.value.blocks[0]).toHaveProperty(
+      "text",
+      expect.stringContaining("unpaid invoice"),
+    );
+
+    await expect(iterator.next()).rejects.toThrow(
+      /Cursor Agent exited without stream-json output/,
+    );
+  });
+
+  it("prompt fails when Cursor emits bad JSON and then leaves stdout idle", async () => {
+    const store = createInMemoryMetaStore({ sid: { cwd: "F:/repo" } });
+    const spawnImpl = (() =>
+      createHangingMockCursorProcess({
+        stdout: "{\"type\":\"tool_call\",\"subtype\":\"started\"\n",
+      })) as CursorSpawnForTest;
+    const adapter = createCursorAdapter({
+      metaStore: store,
+      spawn: spawnImpl,
+      badJsonIdleTimeoutMs: 10,
+    });
+
+    const pending = (async () => {
+      for await (const _event of adapter.prompt(
+        "sid",
+        "[User message]\nhello\n[/User message]",
+        "F:/repo",
+      )) {
+        // No normalized events are expected before the watchdog fires.
+      }
+    })().catch((error: unknown) => error);
+
+    const error = await pending;
+    expect(error).toBeInstanceOf(BadJsonIdleTimeoutError);
+  });
+});

package/src/__tests__/feishu-api.test.ts

@@ -1,60 +1,60 @@
-import { afterEach, describe, expect, it, vi } from "vitest";
-
-import { sendTextReply } from "../feishu-api.ts";
-
-function abortError(): Error {
-  const err = new Error("aborted");
-  err.name = "AbortError";
-  return err;
-}
-
-describe("sendTextReply timeout", () => {
-  afterEach(() => {
-    vi.useRealTimers();
-    vi.restoreAllMocks();
-    vi.unstubAllGlobals();
-  });
-
-  it("aborts a hung text send request", async () => {
-    vi.useFakeTimers();
-    vi.spyOn(console, "error").mockImplementation(() => {});
-
-    const fetchMock = vi.fn((_url: Parameters<typeof fetch>[0], init?: Parameters<typeof fetch>[1]) => (
-      new Promise<Response>((_resolve, reject) => {
-        init?.signal?.addEventListener("abort", () => reject(abortError()));
-      })
-    ));
-    vi.stubGlobal("fetch", fetchMock);
-
-    const send = expect(sendTextReply("token", "chat-1", "hello"))
-      .resolves.toBe(false);
-    await vi.advanceTimersByTimeAsync(15_000);
-
-    await send;
-    expect(fetchMock).toHaveBeenCalledWith(
-      expect.stringContaining("/im/v1/messages?receive_id_type=chat_id"),
-      expect.objectContaining({ signal: expect.any(AbortSignal) }),
-    );
-  });
-
-  it("also aborts when response body reading hangs", async () => {
-    vi.useFakeTimers();
-    vi.spyOn(console, "error").mockImplementation(() => {});
-
-    const fetchMock = vi.fn((_url: Parameters<typeof fetch>[0], init?: Parameters<typeof fetch>[1]) => (
-      Promise.resolve({
-        status: 200,
-        text: () => new Promise<string>((_resolve, reject) => {
-          init?.signal?.addEventListener("abort", () => reject(abortError()));
-        }),
-      } as Response)
-    ));
-    vi.stubGlobal("fetch", fetchMock);
-
-    const send = expect(sendTextReply("token", "chat-2", "hello"))
-      .resolves.toBe(false);
-    await vi.advanceTimersByTimeAsync(15_000);
-
-    await send;
-  });
-});
+import { afterEach, describe, expect, it, vi } from "vitest";
+
+import { sendTextReply } from "../feishu-api.ts";
+
+function abortError(): Error {
+  const err = new Error("aborted");
+  err.name = "AbortError";
+  return err;
+}
+
+describe("sendTextReply timeout", () => {
+  afterEach(() => {
+    vi.useRealTimers();
+    vi.restoreAllMocks();
+    vi.unstubAllGlobals();
+  });
+
+  it("aborts a hung text send request", async () => {
+    vi.useFakeTimers();
+    vi.spyOn(console, "error").mockImplementation(() => {});
+
+    const fetchMock = vi.fn((_url: Parameters<typeof fetch>[0], init?: Parameters<typeof fetch>[1]) => (
+      new Promise<Response>((_resolve, reject) => {
+        init?.signal?.addEventListener("abort", () => reject(abortError()));
+      })
+    ));
+    vi.stubGlobal("fetch", fetchMock);
+
+    const send = expect(sendTextReply("token", "chat-1", "hello"))
+      .resolves.toBe(false);
+    await vi.advanceTimersByTimeAsync(15_000);
+
+    await send;
+    expect(fetchMock).toHaveBeenCalledWith(
+      expect.stringContaining("/im/v1/messages?receive_id_type=chat_id"),
+      expect.objectContaining({ signal: expect.any(AbortSignal) }),
+    );
+  });
+
+  it("also aborts when response body reading hangs", async () => {
+    vi.useFakeTimers();
+    vi.spyOn(console, "error").mockImplementation(() => {});
+
+    const fetchMock = vi.fn((_url: Parameters<typeof fetch>[0], init?: Parameters<typeof fetch>[1]) => (
+      Promise.resolve({
+        status: 200,
+        text: () => new Promise<string>((_resolve, reject) => {
+          init?.signal?.addEventListener("abort", () => reject(abortError()));
+        }),
+      } as Response)
+    ));
+    vi.stubGlobal("fetch", fetchMock);
+
+    const send = expect(sendTextReply("token", "chat-2", "hello"))
+      .resolves.toBe(false);
+    await vi.advanceTimersByTimeAsync(15_000);
+
+    await send;
+  });
+});