chatccc 0.2.231 → 0.2.232

package/src/__tests__/feishu-message-ingress.test.ts

@@ -1,138 +1,138 @@
-import { mkdtemp, rm } from "node:fs/promises";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-
-import { afterEach, describe, expect, it, vi } from "vitest";
-
-import {
-  FeishuMessageLedger,
-  createAckFirstEventHandler,
-} from "../feishu-message-ingress.ts";
-
-const tempDirs: string[] = [];
-
-async function createLedger(maxEntries = 5): Promise<FeishuMessageLedger> {
-  const dir = await mkdtemp(join(tmpdir(), "chatccc-feishu-ingress-"));
-  tempDirs.push(dir);
-  return new FeishuMessageLedger(join(dir, "messages.json"), maxEntries);
-}
-
-afterEach(async () => {
-  await Promise.all(tempDirs.splice(0).map((dir) => rm(dir, { recursive: true, force: true })));
-});
-
-describe("createAckFirstEventHandler", () => {
-  it("returns before starting the asynchronous event worker", async () => {
-    let scheduled: (() => void) | undefined;
-    const schedule = vi.fn((task: () => void) => {
-      scheduled = task;
-    });
-    const worker = vi.fn(async () => {});
-    const onError = vi.fn();
-    const handler = createAckFirstEventHandler(worker, onError, schedule);
-
-    await expect(handler({ message: "test" })).resolves.toBeUndefined();
-    expect(schedule).toHaveBeenCalledOnce();
-    expect(worker).not.toHaveBeenCalled();
-
-    scheduled?.();
-    await vi.waitFor(() => expect(worker).toHaveBeenCalledWith({ message: "test" }));
-    expect(onError).not.toHaveBeenCalled();
-  });
-
-  it("reports detached worker failures without rejecting the SDK callback", async () => {
-    const error = new Error("worker failed");
-    const onError = vi.fn();
-    const handler = createAckFirstEventHandler(
-      async () => {
-        throw error;
-      },
-      onError,
-      (task) => task(),
-    );
-
-    await expect(handler({})).resolves.toBeUndefined();
-    await vi.waitFor(() => expect(onError).toHaveBeenCalledWith(error));
-  });
-});
-
-describe("FeishuMessageLedger", () => {
-  it("rejects an already accepted message after a process restart", async () => {
-    const first = await createLedger();
-    await first.load();
-
-    expect(await first.accept({
-      messageId: "om_001",
-      chatId: "oc_chat",
-      createTime: 100,
-    })).toBe("accepted");
-
-    const restarted = new FeishuMessageLedger(first.filePath, 5);
-    await restarted.load();
-
-    expect(await restarted.accept({
-      messageId: "om_001",
-      chatId: "oc_chat",
-      createTime: 100,
-    })).toBe("duplicate");
-  });
-
-  it("rejects a unique old event after restoring the chat high-water mark", async () => {
-    const first = await createLedger();
-    await first.load();
-
-    expect(await first.accept({
-      messageId: "om_new",
-      chatId: "oc_chat",
-      createTime: 200,
-    })).toBe("accepted");
-
-    const restarted = new FeishuMessageLedger(first.filePath, 5);
-    await restarted.load();
-
-    expect(await restarted.accept({
-      messageId: "om_old",
-      chatId: "oc_chat",
-      createTime: 100,
-    })).toBe("stale");
-  });
-
-  it("accepts distinct messages with the same create timestamp", async () => {
-    const ledger = await createLedger();
-    await ledger.load();
-
-    expect(await ledger.accept({
-      messageId: "om_001",
-      chatId: "oc_chat",
-      createTime: 100,
-    })).toBe("accepted");
-    expect(await ledger.accept({
-      messageId: "om_002",
-      chatId: "oc_chat",
-      createTime: 100,
-    })).toBe("accepted");
-  });
-
-  it("keeps only the configured number of recent message IDs", async () => {
-    const ledger = await createLedger(2);
-    await ledger.load();
-
-    await ledger.accept({ messageId: "om_001", chatId: "oc_chat", createTime: 100 });
-    await ledger.accept({ messageId: "om_002", chatId: "oc_chat", createTime: 200 });
-    await ledger.accept({ messageId: "om_003", chatId: "oc_chat", createTime: 300 });
-
-    const restarted = new FeishuMessageLedger(ledger.filePath, 2);
-    await restarted.load();
-
-    expect(await restarted.accept({
-      messageId: "om_001",
-      chatId: "oc_other",
-      createTime: 100,
-    })).toBe("accepted");
-    expect(await restarted.accept({
-      messageId: "om_003",
-      chatId: "oc_chat",
-      createTime: 300,
-    })).toBe("duplicate");
-  });
-});
+import { mkdtemp, rm } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+
+import { afterEach, describe, expect, it, vi } from "vitest";
+
+import {
+  FeishuMessageLedger,
+  createAckFirstEventHandler,
+} from "../feishu-message-ingress.ts";
+
+const tempDirs: string[] = [];
+
+async function createLedger(maxEntries = 5): Promise<FeishuMessageLedger> {
+  const dir = await mkdtemp(join(tmpdir(), "chatccc-feishu-ingress-"));
+  tempDirs.push(dir);
+  return new FeishuMessageLedger(join(dir, "messages.json"), maxEntries);
+}
+
+afterEach(async () => {
+  await Promise.all(tempDirs.splice(0).map((dir) => rm(dir, { recursive: true, force: true })));
+});
+
+describe("createAckFirstEventHandler", () => {
+  it("returns before starting the asynchronous event worker", async () => {
+    let scheduled: (() => void) | undefined;
+    const schedule = vi.fn((task: () => void) => {
+      scheduled = task;
+    });
+    const worker = vi.fn(async () => {});
+    const onError = vi.fn();
+    const handler = createAckFirstEventHandler(worker, onError, schedule);
+
+    await expect(handler({ message: "test" })).resolves.toBeUndefined();
+    expect(schedule).toHaveBeenCalledOnce();
+    expect(worker).not.toHaveBeenCalled();
+
+    scheduled?.();
+    await vi.waitFor(() => expect(worker).toHaveBeenCalledWith({ message: "test" }));
+    expect(onError).not.toHaveBeenCalled();
+  });
+
+  it("reports detached worker failures without rejecting the SDK callback", async () => {
+    const error = new Error("worker failed");
+    const onError = vi.fn();
+    const handler = createAckFirstEventHandler(
+      async () => {
+        throw error;
+      },
+      onError,
+      (task) => task(),
+    );
+
+    await expect(handler({})).resolves.toBeUndefined();
+    await vi.waitFor(() => expect(onError).toHaveBeenCalledWith(error));
+  });
+});
+
+describe("FeishuMessageLedger", () => {
+  it("rejects an already accepted message after a process restart", async () => {
+    const first = await createLedger();
+    await first.load();
+
+    expect(await first.accept({
+      messageId: "om_001",
+      chatId: "oc_chat",
+      createTime: 100,
+    })).toBe("accepted");
+
+    const restarted = new FeishuMessageLedger(first.filePath, 5);
+    await restarted.load();
+
+    expect(await restarted.accept({
+      messageId: "om_001",
+      chatId: "oc_chat",
+      createTime: 100,
+    })).toBe("duplicate");
+  });
+
+  it("rejects a unique old event after restoring the chat high-water mark", async () => {
+    const first = await createLedger();
+    await first.load();
+
+    expect(await first.accept({
+      messageId: "om_new",
+      chatId: "oc_chat",
+      createTime: 200,
+    })).toBe("accepted");
+
+    const restarted = new FeishuMessageLedger(first.filePath, 5);
+    await restarted.load();
+
+    expect(await restarted.accept({
+      messageId: "om_old",
+      chatId: "oc_chat",
+      createTime: 100,
+    })).toBe("stale");
+  });
+
+  it("accepts distinct messages with the same create timestamp", async () => {
+    const ledger = await createLedger();
+    await ledger.load();
+
+    expect(await ledger.accept({
+      messageId: "om_001",
+      chatId: "oc_chat",
+      createTime: 100,
+    })).toBe("accepted");
+    expect(await ledger.accept({
+      messageId: "om_002",
+      chatId: "oc_chat",
+      createTime: 100,
+    })).toBe("accepted");
+  });
+
+  it("keeps only the configured number of recent message IDs", async () => {
+    const ledger = await createLedger(2);
+    await ledger.load();
+
+    await ledger.accept({ messageId: "om_001", chatId: "oc_chat", createTime: 100 });
+    await ledger.accept({ messageId: "om_002", chatId: "oc_chat", createTime: 200 });
+    await ledger.accept({ messageId: "om_003", chatId: "oc_chat", createTime: 300 });
+
+    const restarted = new FeishuMessageLedger(ledger.filePath, 2);
+    await restarted.load();
+
+    expect(await restarted.accept({
+      messageId: "om_001",
+      chatId: "oc_other",
+      createTime: 100,
+    })).toBe("accepted");
+    expect(await restarted.accept({
+      messageId: "om_003",
+      chatId: "oc_chat",
+      createTime: 300,
+    })).toBe("duplicate");
+  });
+});

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

@@ -1,5 +1,5 @@
 import { describe, it, expect, beforeAll, afterAll } from "vitest";
-import { getPlatform, setPlatform, getTenantAccessToken, getChatInfo, createGroupChat, updateChatInfo, sendTextReply, sendCardReply, sendRawCard, sendPostMessage, extractSessionInfo, formatDelayNotice, reportPermissionResults, verifyAllPermissions, addReaction, recallMessage, updateCardMessage, setChatAvatar, getCodexUsageSummary, consumeCodexRateLimitResetCredit, disbandChat, sendRestartCard, getMergeForwardMessages } from "../feishu-platform.ts";
+import { getPlatform, setPlatform, getTenantAccessToken, getChatInfo, createGroupChat, updateChatInfo, sendTextReply, sendCardReply, sendRawCard, sendPostMessage, extractSessionInfo, formatDelayNotice, reportPermissionResults, verifyAllPermissions, addReaction, recallMessage, updateCardMessage, setChatAvatar, getCodexUsageSummary, consumeCodexRateLimitResetCredit, disbandChat, sendRestartCard, getMergeForwardMessages } from "../feishu-platform.ts";
 import type { FeishuPlatform } from "../feishu-platform.ts";
 
 const realPlatform = getPlatform();
@@ -26,16 +26,16 @@ describe("feishu-platform", () => {
       createGroupChat: async () => "mock_chat",
       updateChatInfo: async () => {},
       getChatInfo: async () => ({ name: "x", description: "y" }),
-      disbandChat: async () => {},
-      setChatAvatar: async () => {},
-      getCodexUsageSummary: async () => ({
-        fiveHour: { usedPercent: 10, remainingPercent: 90, resetAtEpochSeconds: 1781528212, resetAfterSeconds: 10349 },
-        weekly: { usedPercent: 20, remainingPercent: 80, resetAtEpochSeconds: 1781842926, resetAfterSeconds: 325063 },
-        rateLimitResetCreditsAvailable: 1,
-        rateLimitResetCredits: [{ grantedAt: null, expiresAt: "2026-07-12T04:01:47.770016Z" }],
-      }),
-      consumeCodexRateLimitResetCredit: async () => ({ code: "reset", windowsReset: 2 }),
-      getOrDownloadImage: async () => "/tmp/img.png",
+      disbandChat: async () => {},
+      setChatAvatar: async () => {},
+      getCodexUsageSummary: async () => ({
+        fiveHour: { usedPercent: 10, remainingPercent: 90, resetAtEpochSeconds: 1781528212, resetAfterSeconds: 10349 },
+        weekly: { usedPercent: 20, remainingPercent: 80, resetAtEpochSeconds: 1781842926, resetAfterSeconds: 325063 },
+        rateLimitResetCreditsAvailable: 1,
+        rateLimitResetCredits: [{ grantedAt: null, expiresAt: "2026-07-12T04:01:47.770016Z" }],
+      }),
+      consumeCodexRateLimitResetCredit: async () => ({ code: "reset", windowsReset: 2 }),
+      getOrDownloadImage: async () => "/tmp/img.png",
       verifyAllPermissions: async () => [],
       reportPermissionResults: realPlatform.reportPermissionResults,
       extractSessionInfo: realPlatform.extractSessionInfo,
@@ -53,16 +53,16 @@ describe("feishu-platform", () => {
       expect(await getChatInfo("t", "mock_chat")).toEqual({ name: "x", description: "y" });
       const perms = await verifyAllPermissions("t");
       expect(perms).toEqual([]);
-      const mergeMsgs = await getMergeForwardMessages("t", "om_test");
-      expect(mergeMsgs).toEqual([]);
-      expect(await getCodexUsageSummary()).toEqual({
-        fiveHour: { usedPercent: 10, remainingPercent: 90, resetAtEpochSeconds: 1781528212, resetAfterSeconds: 10349 },
-        weekly: { usedPercent: 20, remainingPercent: 80, resetAtEpochSeconds: 1781842926, resetAfterSeconds: 325063 },
-        rateLimitResetCreditsAvailable: 1,
-        rateLimitResetCredits: [{ grantedAt: null, expiresAt: "2026-07-12T04:01:47.770016Z" }],
-      });
-      expect(await consumeCodexRateLimitResetCredit("request-1")).toEqual({ code: "reset", windowsReset: 2 });
-    } finally {
+      const mergeMsgs = await getMergeForwardMessages("t", "om_test");
+      expect(mergeMsgs).toEqual([]);
+      expect(await getCodexUsageSummary()).toEqual({
+        fiveHour: { usedPercent: 10, remainingPercent: 90, resetAtEpochSeconds: 1781528212, resetAfterSeconds: 10349 },
+        weekly: { usedPercent: 20, remainingPercent: 80, resetAtEpochSeconds: 1781842926, resetAfterSeconds: 325063 },
+        rateLimitResetCreditsAvailable: 1,
+        rateLimitResetCredits: [{ grantedAt: null, expiresAt: "2026-07-12T04:01:47.770016Z" }],
+      });
+      expect(await consumeCodexRateLimitResetCredit("request-1")).toEqual({ code: "reset", windowsReset: 2 });
+    } finally {
       // 恢复到真实实现，避免影响后续测试
       setPlatform(realPlatform);
     }
@@ -72,4 +72,4 @@ describe("feishu-platform", () => {
     setPlatform(realPlatform);
     expect(getPlatform()).toBe(realPlatform);
   });
-});
+});

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

package/src/__tests__/jsonl-stream.test.ts

@@ -1,79 +1,79 @@
-import { PassThrough } from "node:stream";
-
-import { afterEach, describe, expect, it, vi } from "vitest";
-
-import {
-  BadJsonIdleTimeoutError,
-  readJsonLinesWithBadJsonIdleWatchdog,
-} from "../adapters/jsonl-stream.ts";
-
-function createReader(input: PassThrough, idleTimeoutMs = 100): AsyncIterator<unknown> {
-  return readJsonLinesWithBadJsonIdleWatchdog({
-    input,
-    tool: "test-agent",
-    tag: "sid-test",
-    idleTimeoutMs,
-    parse: (line) => JSON.parse(line) as unknown,
-  })[Symbol.asyncIterator]();
-}
-
-afterEach(() => {
-  vi.useRealTimers();
-});
-
-describe("readJsonLinesWithBadJsonIdleWatchdog", () => {
-  it("throws when a JSON-like bad line remains the last stdout past the idle timeout", async () => {
-    vi.useFakeTimers();
-    const input = new PassThrough();
-    const iterator = createReader(input);
-
-    const pending = iterator.next().catch((error: unknown) => error);
-    input.write("{\"type\":\"tool_call\",\"subtype\":\"started\"\n");
-
-    await vi.advanceTimersByTimeAsync(100);
-
-    const error = await pending;
-    expect(error).toBeInstanceOf(BadJsonIdleTimeoutError);
-    expect(error).toMatchObject({
-      code: "BAD_JSON_IDLE_TIMEOUT",
-      tool: "test-agent",
-      tag: "sid-test",
-    });
-  });
-
-  it("does not throw when a valid JSON line arrives after the bad line before timeout", async () => {
-    vi.useFakeTimers();
-    const input = new PassThrough();
-    const iterator = createReader(input);
-
-    const pending = iterator.next();
-    input.write("{\"type\":\"tool_call\"\n");
-    await vi.advanceTimersByTimeAsync(50);
-    input.write("{\"type\":\"ok\"}\n");
-
-    await expect(pending).resolves.toEqual({
-      done: false,
-      value: { type: "ok" },
-    });
-
-    const done = iterator.next();
-    input.end();
-    await expect(done).resolves.toEqual({ done: true, value: undefined });
-  });
-
-  it("ignores non-JSON banner lines for the bad JSON idle watchdog", async () => {
-    vi.useFakeTimers();
-    const input = new PassThrough();
-    const iterator = createReader(input);
-
-    const pending = iterator.next();
-    input.write("Reading prompt from stdin...\n");
-    await vi.advanceTimersByTimeAsync(200);
-    input.write("{\"type\":\"ok\"}\n");
-
-    await expect(pending).resolves.toEqual({
-      done: false,
-      value: { type: "ok" },
-    });
-  });
-});
+import { PassThrough } from "node:stream";
+
+import { afterEach, describe, expect, it, vi } from "vitest";
+
+import {
+  BadJsonIdleTimeoutError,
+  readJsonLinesWithBadJsonIdleWatchdog,
+} from "../adapters/jsonl-stream.ts";
+
+function createReader(input: PassThrough, idleTimeoutMs = 100): AsyncIterator<unknown> {
+  return readJsonLinesWithBadJsonIdleWatchdog({
+    input,
+    tool: "test-agent",
+    tag: "sid-test",
+    idleTimeoutMs,
+    parse: (line) => JSON.parse(line) as unknown,
+  })[Symbol.asyncIterator]();
+}
+
+afterEach(() => {
+  vi.useRealTimers();
+});
+
+describe("readJsonLinesWithBadJsonIdleWatchdog", () => {
+  it("throws when a JSON-like bad line remains the last stdout past the idle timeout", async () => {
+    vi.useFakeTimers();
+    const input = new PassThrough();
+    const iterator = createReader(input);
+
+    const pending = iterator.next().catch((error: unknown) => error);
+    input.write("{\"type\":\"tool_call\",\"subtype\":\"started\"\n");
+
+    await vi.advanceTimersByTimeAsync(100);
+
+    const error = await pending;
+    expect(error).toBeInstanceOf(BadJsonIdleTimeoutError);
+    expect(error).toMatchObject({
+      code: "BAD_JSON_IDLE_TIMEOUT",
+      tool: "test-agent",
+      tag: "sid-test",
+    });
+  });
+
+  it("does not throw when a valid JSON line arrives after the bad line before timeout", async () => {
+    vi.useFakeTimers();
+    const input = new PassThrough();
+    const iterator = createReader(input);
+
+    const pending = iterator.next();
+    input.write("{\"type\":\"tool_call\"\n");
+    await vi.advanceTimersByTimeAsync(50);
+    input.write("{\"type\":\"ok\"}\n");
+
+    await expect(pending).resolves.toEqual({
+      done: false,
+      value: { type: "ok" },
+    });
+
+    const done = iterator.next();
+    input.end();
+    await expect(done).resolves.toEqual({ done: true, value: undefined });
+  });
+
+  it("ignores non-JSON banner lines for the bad JSON idle watchdog", async () => {
+    vi.useFakeTimers();
+    const input = new PassThrough();
+    const iterator = createReader(input);
+
+    const pending = iterator.next();
+    input.write("Reading prompt from stdin...\n");
+    await vi.advanceTimersByTimeAsync(200);
+    input.write("{\"type\":\"ok\"}\n");
+
+    await expect(pending).resolves.toEqual({
+      done: false,
+      value: { type: "ok" },
+    });
+  });
+});

package/src/__tests__/package-files.test.ts

@@ -1,24 +1,24 @@
-import { existsSync, readFileSync } from "node:fs";
-import { join } from "node:path";
-
-import { describe, expect, it } from "vitest";
-
-const CREATE_APP_SKILL_DIRS = [
-  ".agents/skills/create-chatccc-feishu-app/",
-  ".claude/skills/create-chatccc-feishu-app/",
-  ".cursor/skills/create-chatccc-feishu-app/",
-];
-
-describe("npm package files", () => {
-  it("ships the Feishu app creation skill for every supported agent", () => {
-    const root = process.cwd();
-    const packageJson = JSON.parse(readFileSync(join(root, "package.json"), "utf8")) as {
-      files?: string[];
-    };
-
-    for (const skillDir of CREATE_APP_SKILL_DIRS) {
-      expect(packageJson.files).toContain(skillDir);
-      expect(existsSync(join(root, skillDir, "SKILL.md"))).toBe(true);
-    }
-  });
-});
+import { existsSync, readFileSync } from "node:fs";
+import { join } from "node:path";
+
+import { describe, expect, it } from "vitest";
+
+const CREATE_APP_SKILL_DIRS = [
+  ".agents/skills/create-chatccc-feishu-app/",
+  ".claude/skills/create-chatccc-feishu-app/",
+  ".cursor/skills/create-chatccc-feishu-app/",
+];
+
+describe("npm package files", () => {
+  it("ships the Feishu app creation skill for every supported agent", () => {
+    const root = process.cwd();
+    const packageJson = JSON.parse(readFileSync(join(root, "package.json"), "utf8")) as {
+      files?: string[];
+    };
+
+    for (const skillDir of CREATE_APP_SKILL_DIRS) {
+      expect(packageJson.files).toContain(skillDir);
+      expect(existsSync(join(root, skillDir, "SKILL.md"))).toBe(true);
+    }
+  });
+});