chatccc 0.2.183 → 0.2.184

package/im-skills/feishu-skill/skill.md

@@ -8,4 +8,5 @@ Current working directory: {{cwd}}
 Use local endpoints instead of calling Feishu Open Platform directly.
 
 - **Send images**: POST `{{send_image_url}}` with `{"session_id":"{{session_id}}","path":"<absolute path>","caption":"<optional>"}` — read `{{im_skills_cache_dir}}/feishu-skill/receive-send-image.md`
-- **Send files/videos**: POST `{{send_file_url}}` with `{"session_id":"{{session_id}}","path":"<absolute path>","caption":"<optional>"}` — read `{{im_skills_cache_dir}}/feishu-skill/receive-send-file.md`
+- **Send files/videos**: POST `{{send_file_url}}` with `{"session_id":"{{session_id}}","path":"<absolute path>","caption":"<optional>"}` — read `{{im_skills_cache_dir}}/feishu-skill/receive-send-file.md`
+- **Delegate a task to a new agent conversation**: POST `{{delegate_task_url}}` with `{"tool":"codex|claude|cursor","cwd":"<absolute working directory>","open_id":"<Feishu user open_id>","prompt":"<first task>"}`. Use `open_ids` for multiple users. This uses the normal ChatCCC prompt flow, so project prompt injection and IM skills still apply.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chatccc",
-  "version": "0.2.183",
+  "version": "0.2.184",
   "description": "Feishu bot bridge for Claude Code",
   "license": "Apache-2.0",
   "type": "module",

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

@@ -0,0 +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");
+  });
+});

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__/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("");
   });
-});
+});