@coze-arch/cli 0.0.17 → 0.0.19-beta.1

package/lib/__templates__/pi-agent/tests/dashboard-models-api.test.ts

@@ -0,0 +1,171 @@
+import assert from "node:assert/strict";
+import { mkdtempSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import test from "node:test";
+import { ValidationError, readModelsResponse, saveModelsRequest } from "../src/dashboard/api/models.js";
+
+function createConfigFile(config: unknown): { configPath: string; cleanup: () => void } {
+  const tempDir = mkdtempSync(join(tmpdir(), "pi-bot-dashboard-models-"));
+  const workspaceDir = join(tempDir, "workspace");
+  const configPath = join(workspaceDir, "config.json");
+
+  mkdirSync(workspaceDir, { recursive: true });
+  writeFileSync(configPath, JSON.stringify(config, null, 2));
+
+  return {
+    configPath,
+    cleanup: () => rmSync(tempDir, { recursive: true, force: true }),
+  };
+}
+
+test("readModelsResponse exposes default model and selectable options", () => {
+  const { configPath, cleanup } = createConfigFile({
+    agents: {
+      defaults: {
+        model: {
+          primary: "coze/auto",
+        },
+      },
+    },
+    models: {
+      providers: {
+        coze: {
+          api: "openai-completions",
+          apiKey: "${COZE_API_KEY}",
+          baseUrl: "https://coze.example.com/v1",
+          authHeader: true,
+          models: [
+            {
+              id: "auto",
+              name: "Auto",
+              input: ["text", "image"],
+              reasoning: true,
+              contextWindow: 200000,
+              maxTokens: 8192,
+            },
+          ],
+        },
+      },
+    },
+  });
+
+  const result = readModelsResponse({ configPath });
+  cleanup();
+
+  assert.equal(result.defaultModel, "coze/auto");
+  assert.deepEqual(result.options, [{ value: "coze/auto", label: "coze / Auto" }]);
+});
+
+test("saveModelsRequest updates default model while preserving unrelated config", () => {
+  const { configPath, cleanup } = createConfigFile({
+    agents: {
+      defaults: {
+        model: {
+          primary: "coze/auto",
+        },
+      },
+    },
+    models: {
+      providers: {
+        coze: {
+          api: "openai-completions",
+          apiKey: "${COZE_API_KEY}",
+          baseUrl: "https://coze.example.com/v1",
+          headers: {
+            "X-Provider": "keep-me",
+          },
+          models: [
+            {
+              id: "auto",
+              name: "Auto",
+              input: ["text"],
+              reasoning: false,
+              contextWindow: 200000,
+              maxTokens: 8192,
+              cost: {
+                input: 0,
+                output: 0,
+                cacheRead: 0,
+                cacheWrite: 0,
+              },
+            },
+            {
+              id: "glm-4.7",
+              name: "GLM-4.7",
+              input: ["text"],
+              reasoning: false,
+              contextWindow: 256000,
+              maxTokens: 8192,
+              cost: {
+                input: 0,
+                output: 0,
+                cacheRead: 0,
+                cacheWrite: 0,
+              },
+            },
+          ],
+        },
+      },
+    },
+    channels: {
+      feishu: {
+        enabled: true,
+      },
+    },
+  });
+
+  saveModelsRequest({
+    configPath,
+    body: {
+      models: {
+        defaultModel: "coze/glm-4.7",
+      },
+    },
+  });
+
+  const saved = JSON.parse(readFileSync(configPath, "utf-8")) as {
+    agents: { defaults: { model: { primary: string } } };
+    models: {
+      providers: {
+        coze: {
+          apiKey: string;
+          baseUrl: string;
+          authHeader: boolean;
+          headers: Record<string, string>;
+          models: Array<Record<string, unknown>>;
+        };
+      };
+    };
+    channels: { feishu: { enabled: boolean } };
+  };
+
+  cleanup();
+
+  assert.equal(saved.agents.defaults.model.primary, "coze/glm-4.7");
+  assert.deepEqual(saved.models.providers.coze.headers, {
+    "X-Provider": "keep-me",
+  });
+  assert.equal(saved.channels.feishu.enabled, true);
+});
+
+test("saveModelsRequest rejects invalid defaultModel", () => {
+  const { configPath, cleanup } = createConfigFile({
+    agents: { defaults: { model: { primary: "coze/auto" } } },
+    models: { providers: { coze: { models: [{ id: "auto", name: "Auto" }] } } },
+  });
+  try {
+    assert.throws(() => {
+      saveModelsRequest({
+        configPath,
+        body: {
+          models: {
+            defaultModel: "coze/does-not-exist",
+          },
+        },
+      });
+    }, ValidationError);
+  } finally {
+    cleanup();
+  }
+});

package/lib/__templates__/pi-agent/tests/feishu-channel.test.ts

@@ -0,0 +1,149 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import { createFeishuChannel } from "../src/channels/feishu/index.js";
+
+test("feishu channel adds and removes thinking reaction around replies", async () => {
+  const operations: string[] = [];
+  let nextReactionId = 0;
+
+  const channel = createFeishuChannel(
+    {
+      thinkingReaction: {
+        enabled: true,
+        emojiType: "OneSecond"
+      },
+      transport: {
+        addReaction({ messageId, emojiType }) {
+          nextReactionId += 1;
+          operations.push(`add:${messageId}:${emojiType}`);
+          return {
+            reactionId: `reaction-${nextReactionId}`,
+            emojiType
+          };
+        },
+        removeReaction({ messageId, reactionId, emojiType }) {
+          operations.push(`remove:${messageId}:${reactionId}:${emojiType}`);
+        },
+        send(message) {
+          operations.push(`send:${message.replyToMessageId}:${message.text}`);
+        }
+      }
+    },
+    {
+      async onMessage() {
+        operations.push("handler:start");
+        await Promise.resolve();
+        operations.push("handler:end");
+        return {
+          text: "reply text"
+        };
+      }
+    }
+  );
+
+  const result = await channel.simulateIncomingText({
+    text: "hello",
+    senderId: "user-1",
+    conversationId: "dm-1",
+    isDirectMessage: true
+  });
+
+  assert.equal(result.handled, true);
+  assert.equal(result.reply?.text, "reply text");
+  assert.deepEqual(operations, [
+    `add:${result.message?.messageId}:OneSecond`,
+    "handler:start",
+    "handler:end",
+    `remove:${result.message?.messageId}:reaction-1:OneSecond`,
+    `send:${result.message?.messageId}:reply text`
+  ]);
+});
+
+test("feishu channel defaults thinking reaction emoji to OneSecond", async () => {
+  const operations: string[] = [];
+
+  const channel = createFeishuChannel(
+    {
+      transport: {
+        addReaction({ messageId, emojiType }) {
+          operations.push(`add:${messageId}:${emojiType}`);
+          return {
+            reactionId: "reaction-1",
+            emojiType
+          };
+        },
+        removeReaction({ messageId, reactionId, emojiType }) {
+          operations.push(`remove:${messageId}:${reactionId}:${emojiType}`);
+        },
+        send(message) {
+          operations.push(`send:${message.replyToMessageId}:${message.text}`);
+        }
+      }
+    },
+    {
+      onMessage() {
+        return {
+          text: "reply text"
+        };
+      }
+    }
+  );
+
+  const result = await channel.simulateIncomingText({
+    text: "hello",
+    senderId: "user-default",
+    conversationId: "dm-default",
+    isDirectMessage: true
+  });
+
+  assert.equal(result.handled, true);
+  assert.deepEqual(operations, [
+    `add:${result.message?.messageId}:OneSecond`,
+    `remove:${result.message?.messageId}:reaction-1:OneSecond`,
+    `send:${result.message?.messageId}:reply text`
+  ]);
+});
+
+test("feishu channel still replies when thinking reaction fails", async () => {
+  const operations: string[] = [];
+
+  const channel = createFeishuChannel(
+    {
+      thinkingReaction: {
+        enabled: true,
+        emojiType: "OneSecond"
+      },
+      transport: {
+        addReaction() {
+          operations.push("add-failed");
+          throw new Error("reaction unavailable");
+        },
+        send(message) {
+          operations.push(`send:${message.replyToMessageId}:${message.text}`);
+        }
+      }
+    },
+    {
+      onMessage() {
+        operations.push("handler");
+        return {
+          text: "reply after failure"
+        };
+      }
+    }
+  );
+
+  const result = await channel.simulateIncomingText({
+    text: "hello",
+    senderId: "user-2",
+    conversationId: "dm-2",
+    isDirectMessage: true
+  });
+
+  assert.equal(result.handled, true);
+  assert.deepEqual(operations, [
+    "add-failed",
+    "handler",
+    `send:${result.message?.messageId}:reply after failure`
+  ]);
+});

package/lib/__templates__/pi-agent/tests/feishu-streaming-card.test.ts

@@ -0,0 +1,15 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import { mergeStreamingText } from "../src/channels/feishu/streaming-card.js";
+
+test("mergeStreamingText prefers progressive full-text updates", () => {
+  assert.equal(mergeStreamingText("", "hel"), "hel");
+  assert.equal(mergeStreamingText("hel", "hello"), "hello");
+  assert.equal(mergeStreamingText("hello", "hello world"), "hello world");
+});
+
+test("mergeStreamingText merges overlapping chunks without duplicating suffixes", () => {
+  assert.equal(mergeStreamingText("hello", "lo world"), "hello world");
+  assert.equal(mergeStreamingText("abc", "bcdef"), "abcdef");
+});
+

package/lib/__templates__/pi-agent/tests/pi-resources.test.ts

@@ -0,0 +1,73 @@
+import assert from "node:assert/strict";
+import { mkdtempSync, mkdirSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import path from "node:path";
+import test from "node:test";
+import { SettingsManager } from "@mariozechner/pi-coding-agent";
+import {
+  buildPiResourceLoaderOptions,
+  createPiResourceLoader,
+  getPiResourcesDir
+} from "../src/pi-resources.js";
+
+test("buildPiResourceLoaderOptions includes repo-managed resource directories", () => {
+  const cwd = mkdtempSync(path.join(tmpdir(), "pi-bot-pi-resources-"));
+  const options = buildPiResourceLoaderOptions({
+    cwd,
+    agentDir: cwd,
+    settingsManager: SettingsManager.inMemory()
+  });
+
+  const resourcesDir = getPiResourcesDir();
+  assert.ok((options.additionalExtensionPaths ?? []).includes(path.join(resourcesDir, "extensions", "test-ping.ts")));
+  assert.ok(
+    (options.additionalExtensionPaths ?? []).includes(
+      path.join(resourcesDir, "extensions", "preference-memory", "index.ts")
+    )
+  );
+  assert.deepEqual(options.additionalSkillPaths, [path.join(resourcesDir, "skills")]);
+  assert.deepEqual(options.additionalPromptTemplatePaths, [path.join(resourcesDir, "prompts")]);
+  assert.deepEqual(options.agentsFilesOverride?.({ agentsFiles: [{ path: "/tmp/AGENTS.md", content: "x" }] }), {
+    agentsFiles: []
+  });
+});
+
+test("buildPiResourceLoaderOptions prefers local .pi SYSTEM.md over repo-managed SYSTEM.md", () => {
+  const cwd = mkdtempSync(path.join(tmpdir(), "pi-bot-pi-resources-local-system-"));
+  mkdirSync(path.join(cwd, ".pi"), { recursive: true });
+  writeFileSync(path.join(cwd, ".pi", "SYSTEM.md"), "local-system");
+
+  const options = buildPiResourceLoaderOptions({
+    cwd,
+    agentDir: cwd,
+    settingsManager: SettingsManager.inMemory()
+  });
+
+  assert.equal(options.systemPrompt, path.join(cwd, ".pi", "SYSTEM.md"));
+});
+
+test("buildPiResourceLoaderOptions falls back to repo-managed SYSTEM.md when no local override exists", () => {
+  const cwd = mkdtempSync(path.join(tmpdir(), "pi-bot-pi-resources-source-system-"));
+  const options = buildPiResourceLoaderOptions({
+    cwd,
+    agentDir: cwd,
+    settingsManager: SettingsManager.inMemory()
+  });
+
+  assert.equal(options.systemPrompt, path.join(getPiResourcesDir(), "SYSTEM.md"));
+});
+
+test("createPiResourceLoader loads repo-managed extensions without errors", async () => {
+  const cwd = mkdtempSync(path.join(tmpdir(), "pi-bot-pi-resources-loader-"));
+  const loader = createPiResourceLoader({
+    cwd,
+    agentDir: cwd,
+    settingsManager: SettingsManager.inMemory()
+  });
+
+  await loader.reload();
+
+  const result = loader.getExtensions();
+  assert.equal(result.errors.length, 0);
+  assert.ok(result.extensions.length >= 2);
+});

package/lib/__templates__/pi-agent/tests/preference-memory.test.ts

@@ -0,0 +1,43 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import {
+  extractPreferencesFromText,
+  getPreferencesFilePath,
+  renderPreferencesDocument,
+  upsertPreferencesDocument
+} from "../pi-resources/extensions/preference-memory/index.js";
+
+test("getPreferencesFilePath points to workspace root preference.md", () => {
+  const filePath = getPreferencesFilePath("/tmp/pi-workspace");
+  assert.equal(filePath, "/tmp/pi-workspace/preference.md");
+});
+
+test("extractPreferencesFromText captures stable reply and project preferences", () => {
+  const preferences = extractPreferencesFromText(
+    "以后都用中文回复，回答尽量简洁。这个项目统一使用 pnpm。后续资源都写到 workspace/.pi。"
+  );
+
+  assert.deepEqual(
+    preferences.map((preference) => preference.key).sort(),
+    ["package-manager", "reply-language", "reply-style", "runtime-resource-dir"]
+  );
+});
+
+test("extractPreferencesFromText ignores temporary one-off instructions", () => {
+  const preferences = extractPreferencesFromText("这次请用中文回复，先不要删文件。");
+  assert.deepEqual(preferences, []);
+});
+
+test("upsertPreferencesDocument replaces existing values instead of appending duplicates", () => {
+  const initial = renderPreferencesDocument([
+    { key: "reply-language", section: "回复偏好", label: "回复语言", value: "中文" }
+  ]);
+  const updated = upsertPreferencesDocument(initial, [
+    { key: "reply-language", section: "回复偏好", label: "回复语言", value: "英文" },
+    { key: "package-manager", section: "项目约定", label: "包管理器", value: "pnpm" }
+  ]);
+
+  assert.match(updated, /回复语言：英文/);
+  assert.doesNotMatch(updated, /回复语言：中文/);
+  assert.match(updated, /包管理器：pnpm/);
+});

package/lib/__templates__/pi-agent/tests/session-store.test.ts

@@ -0,0 +1,61 @@
+import assert from "node:assert/strict";
+import { mkdtempSync, readFileSync, rmSync, existsSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import test from "node:test";
+import { SessionStore } from "../src/session-store.js";
+
+test("SessionStore persists index + transcript header and can reload", () => {
+  const root = mkdtempSync(join(tmpdir(), "pi-bot-session-store-"));
+  try {
+    const store1 = new SessionStore(root);
+    store1.load();
+
+    const key = "dashboard:dm:dashboard-user";
+    const record1 = store1.ensureSession(key);
+
+    assert.equal(record1.sessionKey, key);
+    assert.ok(record1.sessionId);
+    assert.ok(record1.sessionFile.endsWith(".jsonl"));
+    assert.ok(existsSync(record1.sessionFile));
+
+    const firstLine = readFileSync(record1.sessionFile, "utf-8").split(/\r?\n/)[0] ?? "";
+    const header = JSON.parse(firstLine) as { type?: string; id?: string };
+    assert.equal(header.type, "session");
+    assert.equal(header.id, record1.sessionId);
+
+    const store2 = new SessionStore(root);
+    store2.load();
+    assert.deepEqual(store2.listSessionKeys(), [key]);
+    const record2 = store2.ensureSession(key);
+    assert.equal(record2.sessionId, record1.sessionId);
+    assert.equal(record2.sessionFile, record1.sessionFile);
+  } finally {
+    rmSync(root, { recursive: true, force: true });
+  }
+});
+
+test("SessionStore reset archives prior transcript and rotates sessionId", () => {
+  const root = mkdtempSync(join(tmpdir(), "pi-bot-session-store-reset-"));
+  try {
+    const store = new SessionStore(root);
+    store.load();
+
+    const key = "feishu:dm:user-1";
+    const first = store.ensureSession(key);
+    assert.ok(existsSync(first.sessionFile));
+
+    const second = store.resetSession(key);
+    assert.notEqual(second.sessionId, first.sessionId);
+    assert.notEqual(second.sessionFile, first.sessionFile);
+    assert.ok(existsSync(second.sessionFile));
+
+    const sessionsDir = join(root, ".pi-bot", "sessions");
+    const archiveDir = join(sessionsDir, "archive");
+    // Old file should have been moved into archive (best-effort rename).
+    assert.ok(existsSync(archiveDir));
+  } finally {
+    rmSync(root, { recursive: true, force: true });
+  }
+});
+