chatccc 0.2.189 → 0.2.191

package/src/__tests__/agent-delegate-task-rpc.test.ts

@@ -1,165 +1,165 @@
-import { Readable } from "node:stream";
-import { describe, expect, it, vi, beforeEach } from "vitest";
-
-import type { PlatformAdapter } from "../platform-adapter.ts";
-
-const mockDelegateAgentTask = vi.hoisted(() => vi.fn());
-
-vi.mock("../agent-delegate-task.ts", () => ({
-  delegateAgentTask: mockDelegateAgentTask,
-}));
-
-import {
-  AGENT_DELEGATE_TASK_PATH,
-  buildAgentDelegateTaskCapabilityPrompt,
-  handleAgentDelegateTaskRequest,
-} from "../agent-delegate-task-rpc.ts";
-import { ABD_APPEND_PROMPT } from "../shared-prefix.ts";
-
-function request(body: unknown, path = AGENT_DELEGATE_TASK_PATH, method = "POST"): Readable & {
-  url?: string;
-  method?: string;
-  headers: Record<string, string>;
-} {
-  const req = Readable.from([Buffer.from(JSON.stringify(body), "utf8")]) as Readable & {
-    url?: string;
-    method?: string;
-    headers: Record<string, string>;
-  };
-  req.url = path;
-  req.method = method;
-  req.headers = { "content-type": "application/json; charset=utf-8" };
-  return req;
-}
-
-function response() {
-  const res = {
-    statusCode: 0,
-    headers: {} as Record<string, string>,
-    body: "",
-    writeHead(status: number, headers: Record<string, string>) {
-      this.statusCode = status;
-      this.headers = headers;
-      return this;
-    },
-    end(chunk?: string) {
-      this.body += chunk ?? "";
-      return this;
-    },
-  };
-  return res;
-}
-
-function platform(kind: "feishu" | "wechat" = "feishu"): PlatformAdapter {
-  return {
-    kind,
-    sendText: vi.fn(async () => true),
-    sendCard: vi.fn(async () => true),
-    sendRawCard: vi.fn(async () => true),
-    createGroup: vi.fn(async () => "chat-id"),
-    updateChatInfo: vi.fn(async () => {}),
-    getChatInfo: vi.fn(async () => ({ name: "chat", description: "" })),
-    disbandChat: vi.fn(async () => {}),
-    setChatAvatar: vi.fn(async () => {}),
-    extractSessionInfo: vi.fn(() => null),
-    cardCreate: vi.fn(async () => "card-id"),
-    cardSend: vi.fn(async () => "message-id"),
-    cardUpdate: vi.fn(async () => {}),
-  };
-}
-
-describe("agent delegate task RPC", () => {
-  beforeEach(() => {
-    mockDelegateAgentTask.mockReset();
-    mockDelegateAgentTask.mockResolvedValue({
-      chatId: "chat-new",
-      sessionId: "sid-new",
-      tool: "codex",
-      cwd: "F:\\repo",
-    });
-  });
-
-  it("delegates a task through the local API", async () => {
-    const req = request({
-      tool: "codex",
-      cwd: "F:\\repo",
-      open_id: "ou-user",
-      prompt: "帮我分析日志",
-    });
-    const res = response();
-
-    await handleAgentDelegateTaskRequest(req as never, res as never, platform());
-
-    expect(res.statusCode).toBe(200);
-    expect(JSON.parse(res.body)).toMatchObject({
-      ok: true,
-      chat_id: "chat-new",
-      session_id: "sid-new",
-      tool: "codex",
-    });
-    expect(mockDelegateAgentTask).toHaveBeenCalledWith(expect.objectContaining({
-      tool: "codex",
-      cwd: expect.stringContaining("repo"),
-      promptText: "帮我分析日志",
-      openIds: ["ou-user"],
-      chatNamePrefix: "帮我分析日志",
-    }));
-  });
-
-  it("applies the shared ABD prefix to API prompts", async () => {
-    const req = request({
-      tool: "claude",
-      cwd: "F:\\repo",
-      openIds: ["ou-user"],
-      prompt: "/abd帮我分析",
-    });
-    const res = response();
-
-    await handleAgentDelegateTaskRequest(req as never, res as never, platform());
-
-    expect(res.statusCode).toBe(200);
-    expect(mockDelegateAgentTask).toHaveBeenCalledWith(expect.objectContaining({
-      promptText: `帮我分析\n\n---\n${ABD_APPEND_PROMPT}`,
-      chatNamePrefix: "帮我分析",
-    }));
-  });
-
-  it("rejects non-Feishu platforms", async () => {
-    const req = request({
-      tool: "codex",
-      cwd: "F:\\repo",
-      open_id: "ou-user",
-      prompt: "task",
-    });
-    const res = response();
-
-    await handleAgentDelegateTaskRequest(req as never, res as never, platform("wechat"));
-
-    expect(res.statusCode).toBe(409);
-    expect(mockDelegateAgentTask).not.toHaveBeenCalled();
-  });
-
-  it("rejects missing required parameters", async () => {
-    const req = request({ tool: "codex", cwd: "F:\\repo", prompt: "task" });
-    const res = response();
-
-    await handleAgentDelegateTaskRequest(req as never, res as never, platform());
-
-    expect(res.statusCode).toBe(400);
-    expect(JSON.parse(res.body).error).toContain("open_id");
-    expect(mockDelegateAgentTask).not.toHaveBeenCalled();
-  });
-
-  it("builds prompt instructions for the delegate endpoint", () => {
-    const prompt = buildAgentDelegateTaskCapabilityPrompt({
-      url: `http://127.0.0.1:18080${AGENT_DELEGATE_TASK_PATH}`,
-      cwd: "F:/repo",
-    });
-
-    expect(prompt).toContain("POST http://127.0.0.1:18080/api/agent/delegate-task");
-    expect(prompt).toContain('"tool":"codex|claude|cursor"');
-    expect(prompt).toContain('"cwd":"absolute working directory"');
-    expect(prompt).toContain("project prompt injection and IM skills still apply");
-    expect(prompt).toContain("Current working directory: F:/repo");
-  });
-});
+import { Readable } from "node:stream";
+import { describe, expect, it, vi, beforeEach } from "vitest";
+
+import type { PlatformAdapter } from "../platform-adapter.ts";
+
+const mockDelegateAgentTask = vi.hoisted(() => vi.fn());
+
+vi.mock("../agent-delegate-task.ts", () => ({
+  delegateAgentTask: mockDelegateAgentTask,
+}));
+
+import {
+  AGENT_DELEGATE_TASK_PATH,
+  buildAgentDelegateTaskCapabilityPrompt,
+  handleAgentDelegateTaskRequest,
+} from "../agent-delegate-task-rpc.ts";
+import { ABD_APPEND_PROMPT } from "../shared-prefix.ts";
+
+function request(body: unknown, path = AGENT_DELEGATE_TASK_PATH, method = "POST"): Readable & {
+  url?: string;
+  method?: string;
+  headers: Record<string, string>;
+} {
+  const req = Readable.from([Buffer.from(JSON.stringify(body), "utf8")]) as Readable & {
+    url?: string;
+    method?: string;
+    headers: Record<string, string>;
+  };
+  req.url = path;
+  req.method = method;
+  req.headers = { "content-type": "application/json; charset=utf-8" };
+  return req;
+}
+
+function response() {
+  const res = {
+    statusCode: 0,
+    headers: {} as Record<string, string>,
+    body: "",
+    writeHead(status: number, headers: Record<string, string>) {
+      this.statusCode = status;
+      this.headers = headers;
+      return this;
+    },
+    end(chunk?: string) {
+      this.body += chunk ?? "";
+      return this;
+    },
+  };
+  return res;
+}
+
+function platform(kind: "feishu" | "wechat" = "feishu"): PlatformAdapter {
+  return {
+    kind,
+    sendText: vi.fn(async () => true),
+    sendCard: vi.fn(async () => true),
+    sendRawCard: vi.fn(async () => true),
+    createGroup: vi.fn(async () => "chat-id"),
+    updateChatInfo: vi.fn(async () => {}),
+    getChatInfo: vi.fn(async () => ({ name: "chat", description: "" })),
+    disbandChat: vi.fn(async () => {}),
+    setChatAvatar: vi.fn(async () => {}),
+    extractSessionInfo: vi.fn(() => null),
+    cardCreate: vi.fn(async () => "card-id"),
+    cardSend: vi.fn(async () => "message-id"),
+    cardUpdate: vi.fn(async () => {}),
+  };
+}
+
+describe("agent delegate task RPC", () => {
+  beforeEach(() => {
+    mockDelegateAgentTask.mockReset();
+    mockDelegateAgentTask.mockResolvedValue({
+      chatId: "chat-new",
+      sessionId: "sid-new",
+      tool: "codex",
+      cwd: "F:\\repo",
+    });
+  });
+
+  it("delegates a task through the local API", async () => {
+    const req = request({
+      tool: "codex",
+      cwd: "F:\\repo",
+      open_id: "ou-user",
+      prompt: "帮我分析日志",
+    });
+    const res = response();
+
+    await handleAgentDelegateTaskRequest(req as never, res as never, platform());
+
+    expect(res.statusCode).toBe(200);
+    expect(JSON.parse(res.body)).toMatchObject({
+      ok: true,
+      chat_id: "chat-new",
+      session_id: "sid-new",
+      tool: "codex",
+    });
+    expect(mockDelegateAgentTask).toHaveBeenCalledWith(expect.objectContaining({
+      tool: "codex",
+      cwd: expect.stringContaining("repo"),
+      promptText: "帮我分析日志",
+      openIds: ["ou-user"],
+      chatNamePrefix: "帮我分析日志",
+    }));
+  });
+
+  it("applies the shared ABD prefix to API prompts", async () => {
+    const req = request({
+      tool: "claude",
+      cwd: "F:\\repo",
+      openIds: ["ou-user"],
+      prompt: "/abd帮我分析",
+    });
+    const res = response();
+
+    await handleAgentDelegateTaskRequest(req as never, res as never, platform());
+
+    expect(res.statusCode).toBe(200);
+    expect(mockDelegateAgentTask).toHaveBeenCalledWith(expect.objectContaining({
+      promptText: `帮我分析\n\n---\n${ABD_APPEND_PROMPT}`,
+      chatNamePrefix: "帮我分析",
+    }));
+  });
+
+  it("rejects non-Feishu platforms", async () => {
+    const req = request({
+      tool: "codex",
+      cwd: "F:\\repo",
+      open_id: "ou-user",
+      prompt: "task",
+    });
+    const res = response();
+
+    await handleAgentDelegateTaskRequest(req as never, res as never, platform("wechat"));
+
+    expect(res.statusCode).toBe(409);
+    expect(mockDelegateAgentTask).not.toHaveBeenCalled();
+  });
+
+  it("rejects missing required parameters", async () => {
+    const req = request({ tool: "codex", cwd: "F:\\repo", prompt: "task" });
+    const res = response();
+
+    await handleAgentDelegateTaskRequest(req as never, res as never, platform());
+
+    expect(res.statusCode).toBe(400);
+    expect(JSON.parse(res.body).error).toContain("open_id");
+    expect(mockDelegateAgentTask).not.toHaveBeenCalled();
+  });
+
+  it("builds prompt instructions for the delegate endpoint", () => {
+    const prompt = buildAgentDelegateTaskCapabilityPrompt({
+      url: `http://127.0.0.1:18080${AGENT_DELEGATE_TASK_PATH}`,
+      cwd: "F:/repo",
+    });
+
+    expect(prompt).toContain("POST http://127.0.0.1:18080/api/agent/delegate-task");
+    expect(prompt).toContain('"tool":"codex|claude|cursor"');
+    expect(prompt).toContain('"cwd":"absolute working directory"');
+    expect(prompt).toContain("project prompt injection and IM skills still apply");
+    expect(prompt).toContain("Current working directory: F:/repo");
+  });
+});

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

@@ -1,33 +1,33 @@
-import { afterEach, describe, expect, it } from "vitest";
-
-import { ChatSession } from "../builtin/index.ts";
-import { config } from "../config.ts";
-
-const originalDeepSeekApiKey = process.env.DEEPSEEK_API_KEY;
-const originalCccConfig = structuredClone(config.ccc);
-
-afterEach(() => {
-  if (originalDeepSeekApiKey === undefined) {
-    delete process.env.DEEPSEEK_API_KEY;
-  } else {
-    process.env.DEEPSEEK_API_KEY = originalDeepSeekApiKey;
-  }
-  config.ccc = structuredClone(originalCccConfig);
-});
-
-describe("builtin ChatSession config", () => {
-  it("does not fall back to DEEPSEEK_API_KEY environment variable", () => {
-    config.ccc = {
-      DEEPSEEK_API_KEY: "",
-      DEEPSEEK_BASE_URL: "https://api.deepseek.com/v1",
-      model: "deepseek-v4-pro[1m]",
-    };
-    process.env.DEEPSEEK_API_KEY = "sk-env-should-not-be-used";
-
-    expect(() => new ChatSession()).toThrow("ccc.DEEPSEEK_API_KEY 未设置");
-  });
-
-  it("allows constructor parameters to override config defaults", () => {
-    expect(() => new ChatSession({ apiKey: "sk-test" })).not.toThrow();
-  });
-});
+import { afterEach, describe, expect, it } from "vitest";
+
+import { ChatSession } from "../builtin/index.ts";
+import { config } from "../config.ts";
+
+const originalDeepSeekApiKey = process.env.DEEPSEEK_API_KEY;
+const originalCccConfig = structuredClone(config.ccc);
+
+afterEach(() => {
+  if (originalDeepSeekApiKey === undefined) {
+    delete process.env.DEEPSEEK_API_KEY;
+  } else {
+    process.env.DEEPSEEK_API_KEY = originalDeepSeekApiKey;
+  }
+  config.ccc = structuredClone(originalCccConfig);
+});
+
+describe("builtin ChatSession config", () => {
+  it("does not fall back to DEEPSEEK_API_KEY environment variable", () => {
+    config.ccc = {
+      DEEPSEEK_API_KEY: "",
+      DEEPSEEK_BASE_URL: "https://api.deepseek.com/v1",
+      model: "deepseek-v4-pro",
+    };
+    process.env.DEEPSEEK_API_KEY = "sk-env-should-not-be-used";
+
+    expect(() => new ChatSession()).toThrow("ccc.DEEPSEEK_API_KEY 未设置");
+  });
+
+  it("allows constructor parameters to override config defaults", () => {
+    expect(() => new ChatSession({ apiKey: "sk-test" })).not.toThrow();
+  });
+});

package/src/__tests__/card-plain-text.test.ts

@@ -16,11 +16,11 @@ describe("cardJsonToPlainText", () => {
     expect(text).toContain("/new");
     expect(text).toContain("/new cursor");
     expect(text).toContain("/new codex");
-    expect(text).toContain("/restart");
-    expect(text).toContain("/update");
-    expect(text).toContain("/cd");
-    expect(text).toContain("/abd");
-  });
+    expect(text).toContain("/restart");
+    expect(text).toContain("/update");
+    expect(text).toContain("/cd");
+    expect(text).toContain("/abd");
+  });
 
   it("converts status cards from v1 card format", () => {
     const text = cardJsonToPlainText(buildStatusCard("status body", "green"));

package/src/__tests__/cardkit.test.ts

@@ -1,60 +1,60 @@
-import { afterEach, describe, expect, it, vi } from "vitest";
-
-import { updateCardKitCard } from "../cardkit.ts";
-
-function abortError(): Error {
-  const err = new Error("aborted");
-  err.name = "AbortError";
-  return err;
-}
-
-describe("CardKit request timeout", () => {
-  afterEach(() => {
-    vi.useRealTimers();
-    vi.restoreAllMocks();
-    vi.unstubAllGlobals();
-  });
-
-  it("aborts a hung card update so display.cardBusy can be released", 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 update = expect(updateCardKitCard("token", "card-1", "{}", 7))
-      .rejects.toThrow("updateCard cardId=card-1 seq=7 timeout after 15000ms");
-    await vi.advanceTimersByTimeAsync(15_000);
-
-    await update;
-    expect(fetchMock).toHaveBeenCalledWith(
-      expect.stringContaining("/cardkit/v1/cards/card-1"),
-      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 update = expect(updateCardKitCard("token", "card-2", "{}", 8))
-      .rejects.toThrow("updateCard cardId=card-2 seq=8 timeout after 15000ms");
-    await vi.advanceTimersByTimeAsync(15_000);
-
-    await update;
-  });
-});
+import { afterEach, describe, expect, it, vi } from "vitest";
+
+import { updateCardKitCard } from "../cardkit.ts";
+
+function abortError(): Error {
+  const err = new Error("aborted");
+  err.name = "AbortError";
+  return err;
+}
+
+describe("CardKit request timeout", () => {
+  afterEach(() => {
+    vi.useRealTimers();
+    vi.restoreAllMocks();
+    vi.unstubAllGlobals();
+  });
+
+  it("aborts a hung card update so display.cardBusy can be released", 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 update = expect(updateCardKitCard("token", "card-1", "{}", 7))
+      .rejects.toThrow("updateCard cardId=card-1 seq=7 timeout after 15000ms");
+    await vi.advanceTimersByTimeAsync(15_000);
+
+    await update;
+    expect(fetchMock).toHaveBeenCalledWith(
+      expect.stringContaining("/cardkit/v1/cards/card-1"),
+      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 update = expect(updateCardKitCard("token", "card-2", "{}", 8))
+      .rejects.toThrow("updateCard cardId=card-2 seq=8 timeout after 15000ms");
+    await vi.advanceTimersByTimeAsync(15_000);
+
+    await update;
+  });
+});