@openclaw/feishu 2026.5.2 → 2026.5.3-beta.2

package/src/config-schema.test.ts

@@ -1,309 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { FeishuConfigSchema, FeishuGroupSchema } from "./config-schema.js";
-
-function expectSchemaIssue(
-  result: ReturnType<typeof FeishuConfigSchema.safeParse>,
-  issuePath: string,
-) {
-  expect(result.success).toBe(false);
-  if (!result.success) {
-    expect(result.error.issues.some((issue) => issue.path.join(".") === issuePath)).toBe(true);
-  }
-}
-
-describe("FeishuConfigSchema webhook validation", () => {
-  it("applies top-level defaults", () => {
-    const result = FeishuConfigSchema.parse({});
-    expect(result.domain).toBe("feishu");
-    expect(result.connectionMode).toBe("websocket");
-    expect(result.webhookPath).toBe("/feishu/events");
-    expect(result.dmPolicy).toBe("pairing");
-    expect(result.groupPolicy).toBe("allowlist");
-    // requireMention has no schema-level default now — it is resolved at runtime
-    // through shared channel group-policy resolution, with an open-group override
-    // that defaults to false only when requireMention is otherwise unset.
-    expect(result.requireMention).toBeUndefined();
-  });
-
-  it("does not force top-level policy defaults into account config", () => {
-    const result = FeishuConfigSchema.parse({
-      accounts: {
-        main: {},
-      },
-    });
-
-    expect(result.accounts?.main?.dmPolicy).toBeUndefined();
-    expect(result.accounts?.main?.groupPolicy).toBeUndefined();
-    expect(result.accounts?.main?.requireMention).toBeUndefined();
-  });
-
-  it("normalizes legacy groupPolicy allowall to open", () => {
-    const result = FeishuConfigSchema.parse({
-      groupPolicy: "allowall",
-    });
-
-    expect(result.groupPolicy).toBe("open");
-  });
-
-  it("rejects top-level webhook mode without verificationToken", () => {
-    const result = FeishuConfigSchema.safeParse({
-      connectionMode: "webhook",
-      appId: "cli_top",
-      appSecret: "secret_top", // pragma: allowlist secret
-    });
-
-    expectSchemaIssue(result, "verificationToken");
-  });
-
-  it("rejects top-level webhook mode without encryptKey", () => {
-    const result = FeishuConfigSchema.safeParse({
-      connectionMode: "webhook",
-      verificationToken: "token_top",
-      appId: "cli_top",
-      appSecret: "secret_top", // pragma: allowlist secret
-    });
-
-    expectSchemaIssue(result, "encryptKey");
-  });
-
-  it("accepts top-level webhook mode with verificationToken and encryptKey", () => {
-    const result = FeishuConfigSchema.safeParse({
-      connectionMode: "webhook",
-      verificationToken: "token_top",
-      encryptKey: "encrypt_top",
-      appId: "cli_top",
-      appSecret: "secret_top", // pragma: allowlist secret
-    });
-
-    expect(result.success).toBe(true);
-  });
-
-  it("rejects account webhook mode without verificationToken", () => {
-    const result = FeishuConfigSchema.safeParse({
-      accounts: {
-        main: {
-          connectionMode: "webhook",
-          appId: "cli_main",
-          appSecret: "secret_main", // pragma: allowlist secret
-        },
-      },
-    });
-
-    expectSchemaIssue(result, "accounts.main.verificationToken");
-  });
-
-  it("rejects account webhook mode without encryptKey", () => {
-    const result = FeishuConfigSchema.safeParse({
-      accounts: {
-        main: {
-          connectionMode: "webhook",
-          verificationToken: "token_main",
-          appId: "cli_main",
-          appSecret: "secret_main", // pragma: allowlist secret
-        },
-      },
-    });
-
-    expectSchemaIssue(result, "accounts.main.encryptKey");
-  });
-
-  it("accepts account webhook mode inheriting top-level verificationToken and encryptKey", () => {
-    const result = FeishuConfigSchema.safeParse({
-      verificationToken: "token_top",
-      encryptKey: "encrypt_top",
-      accounts: {
-        main: {
-          connectionMode: "webhook",
-          appId: "cli_main",
-          appSecret: "secret_main", // pragma: allowlist secret
-        },
-      },
-    });
-
-    expect(result.success).toBe(true);
-  });
-
-  it("accepts SecretRef verificationToken in webhook mode", () => {
-    const result = FeishuConfigSchema.safeParse({
-      connectionMode: "webhook",
-      verificationToken: {
-        source: "env",
-        provider: "default",
-        id: "FEISHU_VERIFICATION_TOKEN",
-      },
-      encryptKey: "encrypt_top",
-      appId: "cli_top",
-      appSecret: {
-        source: "env",
-        provider: "default",
-        id: "FEISHU_APP_SECRET",
-      },
-    });
-
-    expect(result.success).toBe(true);
-  });
-
-  it("accepts SecretRef encryptKey in webhook mode", () => {
-    const result = FeishuConfigSchema.safeParse({
-      connectionMode: "webhook",
-      verificationToken: {
-        source: "env",
-        provider: "default",
-        id: "FEISHU_VERIFICATION_TOKEN",
-      },
-      encryptKey: {
-        source: "env",
-        provider: "default",
-        id: "FEISHU_ENCRYPT_KEY",
-      },
-      appId: "cli_top",
-      appSecret: {
-        source: "env",
-        provider: "default",
-        id: "FEISHU_APP_SECRET",
-      },
-    });
-
-    expect(result.success).toBe(true);
-  });
-});
-
-describe("FeishuConfigSchema replyInThread", () => {
-  it("accepts replyInThread at top level", () => {
-    const result = FeishuConfigSchema.parse({ replyInThread: "enabled" });
-    expect(result.replyInThread).toBe("enabled");
-  });
-
-  it("defaults replyInThread to undefined when not set", () => {
-    const result = FeishuConfigSchema.parse({});
-    expect(result.replyInThread).toBeUndefined();
-  });
-
-  it("rejects invalid replyInThread value", () => {
-    const result = FeishuConfigSchema.safeParse({ replyInThread: "always" });
-    expect(result.success).toBe(false);
-  });
-
-  it("accepts replyInThread in group config", () => {
-    const result = FeishuGroupSchema.parse({ replyInThread: "enabled" });
-    expect(result.replyInThread).toBe("enabled");
-  });
-
-  it("accepts replyInThread in account config", () => {
-    const result = FeishuConfigSchema.parse({
-      accounts: {
-        main: { replyInThread: "enabled" },
-      },
-    });
-    expect(result.accounts?.main?.replyInThread).toBe("enabled");
-  });
-});
-
-describe("FeishuConfigSchema optimization flags", () => {
-  it("defaults top-level typingIndicator and resolveSenderNames to true", () => {
-    const result = FeishuConfigSchema.parse({});
-    expect(result.typingIndicator).toBe(true);
-    expect(result.resolveSenderNames).toBe(true);
-  });
-
-  it("accepts account-level optimization flags", () => {
-    const result = FeishuConfigSchema.parse({
-      accounts: {
-        main: {
-          typingIndicator: false,
-          resolveSenderNames: false,
-        },
-      },
-    });
-    expect(result.accounts?.main?.typingIndicator).toBe(false);
-    expect(result.accounts?.main?.resolveSenderNames).toBe(false);
-  });
-});
-
-describe("FeishuConfigSchema TTS overrides", () => {
-  it("accepts top-level and account-level TTS overrides", () => {
-    const result = FeishuConfigSchema.parse({
-      tts: {
-        auto: "always",
-        provider: "openai",
-        providers: {
-          openai: {
-            voice: "alloy",
-          },
-        },
-      },
-      accounts: {
-        english: {
-          tts: {
-            providers: {
-              openai: {
-                voice: "shimmer",
-              },
-            },
-          },
-        },
-      },
-    });
-
-    expect(result.tts).toMatchObject({
-      auto: "always",
-      provider: "openai",
-    });
-    expect(result.accounts?.english?.tts).toMatchObject({
-      providers: {
-        openai: {
-          voice: "shimmer",
-        },
-      },
-    });
-  });
-});
-
-describe("FeishuConfigSchema actions", () => {
-  it("accepts top-level reactions action gate", () => {
-    const result = FeishuConfigSchema.parse({
-      actions: { reactions: false },
-    });
-    expect(result.actions?.reactions).toBe(false);
-  });
-
-  it("accepts account-level reactions action gate", () => {
-    const result = FeishuConfigSchema.parse({
-      accounts: {
-        main: {
-          actions: { reactions: false },
-        },
-      },
-    });
-    expect(result.accounts?.main?.actions?.reactions).toBe(false);
-  });
-});
-
-describe("FeishuConfigSchema defaultAccount", () => {
-  it("accepts defaultAccount when it matches an account key", () => {
-    const result = FeishuConfigSchema.safeParse({
-      defaultAccount: "router-d",
-      accounts: {
-        "router-d": { appId: "cli_router", appSecret: "secret_router" }, // pragma: allowlist secret
-      },
-    });
-
-    expect(result.success).toBe(true);
-  });
-
-  it("rejects defaultAccount when it does not match an account key", () => {
-    const result = FeishuConfigSchema.safeParse({
-      defaultAccount: "router-d",
-      accounts: {
-        backup: { appId: "cli_backup", appSecret: "secret_backup" }, // pragma: allowlist secret
-      },
-    });
-
-    expect(result.success).toBe(false);
-    if (!result.success) {
-      expect(result.error.issues.some((issue) => issue.path.join(".") === "defaultAccount")).toBe(
-        true,
-      );
-    }
-  });
-});

package/src/config-schema.ts

@@ -1,333 +0,0 @@
-import { normalizeAccountId } from "openclaw/plugin-sdk/account-id";
-import { z } from "openclaw/plugin-sdk/zod";
-export { z };
-import { buildSecretInputSchema, hasConfiguredSecretInput } from "./secret-input.js";
-
-const ChannelActionsSchema = z
-  .object({
-    reactions: z.boolean().optional(),
-  })
-  .strict()
-  .optional();
-
-const DmPolicySchema = z.enum(["open", "pairing", "allowlist"]);
-const GroupPolicySchema = z.union([
-  z.enum(["open", "allowlist", "disabled"]),
-  z.literal("allowall").transform(() => "open" as const),
-]);
-const FeishuDomainSchema = z.union([
-  z.enum(["feishu", "lark"]),
-  z.string().url().startsWith("https://"),
-]);
-const FeishuConnectionModeSchema = z.enum(["websocket", "webhook"]);
-const TtsOverrideSchema = z
-  .object({
-    auto: z.enum(["off", "always", "inbound", "tagged"]).optional(),
-    enabled: z.boolean().optional(),
-    mode: z.enum(["final", "all"]).optional(),
-    provider: z.string().optional(),
-    persona: z.string().optional(),
-    personas: z.record(z.string(), z.record(z.string(), z.unknown())).optional(),
-    summaryModel: z.string().optional(),
-    modelOverrides: z.record(z.string(), z.unknown()).optional(),
-    providers: z.record(z.string(), z.record(z.string(), z.unknown())).optional(),
-    prefsPath: z.string().optional(),
-    maxTextLength: z.number().int().min(1).optional(),
-    timeoutMs: z.number().int().min(1000).max(120000).optional(),
-  })
-  .strict()
-  .optional();
-
-const ToolPolicySchema = z
-  .object({
-    allow: z.array(z.string()).optional(),
-    deny: z.array(z.string()).optional(),
-  })
-  .strict()
-  .optional();
-
-const DmConfigSchema = z
-  .object({
-    enabled: z.boolean().optional(),
-    systemPrompt: z.string().optional(),
-  })
-  .strict()
-  .optional();
-
-const MarkdownConfigSchema = z
-  .object({
-    mode: z.enum(["native", "escape", "strip"]).optional(),
-    tableMode: z.enum(["native", "ascii", "simple"]).optional(),
-  })
-  .strict()
-  .optional();
-
-// Message render mode: auto (default) = detect markdown, raw = plain text, card = always card
-const RenderModeSchema = z.enum(["auto", "raw", "card"]).optional();
-
-// Streaming card mode: when enabled, card replies use Feishu's Card Kit streaming API
-// for incremental text display with a "Thinking..." placeholder
-const StreamingModeSchema = z.boolean().optional();
-
-const BlockStreamingCoalesceSchema = z
-  .object({
-    enabled: z.boolean().optional(),
-    minDelayMs: z.number().int().positive().optional(),
-    maxDelayMs: z.number().int().positive().optional(),
-  })
-  .strict()
-  .optional();
-
-const ChannelHeartbeatVisibilitySchema = z
-  .object({
-    visibility: z.enum(["visible", "hidden"]).optional(),
-    intervalMs: z.number().int().positive().optional(),
-  })
-  .strict()
-  .optional();
-
-/**
- * Dynamic agent creation configuration.
- * When enabled, a new agent is created for each unique DM user.
- */
-const DynamicAgentCreationSchema = z
-  .object({
-    enabled: z.boolean().optional(),
-    workspaceTemplate: z.string().optional(),
-    agentDirTemplate: z.string().optional(),
-    maxAgents: z.number().int().positive().optional(),
-  })
-  .strict()
-  .optional();
-
-/**
- * Feishu tools configuration.
- * Controls which tool categories are enabled.
- *
- * Dependencies:
- * - wiki requires doc (wiki content is edited via doc tools)
- * - perm can work independently but is typically used with drive
- */
-const FeishuToolsConfigSchema = z
-  .object({
-    doc: z.boolean().optional(), // Document operations (default: true)
-    chat: z.boolean().optional(), // Chat info + member query operations (default: true)
-    wiki: z.boolean().optional(), // Knowledge base operations (default: true, requires doc)
-    drive: z.boolean().optional(), // Cloud storage operations (default: true)
-    perm: z.boolean().optional(), // Permission management (default: false, sensitive)
-    scopes: z.boolean().optional(), // App scopes diagnostic (default: true)
-  })
-  .strict()
-  .optional();
-
-/**
- * Group session scope for routing Feishu group messages.
- * - "group" (default): one session per group chat
- * - "group_sender": one session per (group + sender)
- * - "group_topic": one session per group topic thread (falls back to group if no topic)
- * - "group_topic_sender": one session per (group + topic thread + sender),
- *   falls back to (group + sender) if no topic
- */
-const GroupSessionScopeSchema = z
-  .enum(["group", "group_sender", "group_topic", "group_topic_sender"])
-  .optional();
-
-/**
- * @deprecated Use groupSessionScope instead.
- *
- * Topic session isolation mode for group chats.
- * - "disabled" (default): All messages in a group share one session
- * - "enabled": Messages in different topics get separate sessions
- *
- * Topic routing uses Feishu topic-group `thread_id` when the event identifies a
- * native topic group, and keeps `root_id` precedence for normal groups so
- * reply-created threads stay on the initiating message session.
- */
-const TopicSessionModeSchema = z.enum(["disabled", "enabled"]).optional();
-const ReactionNotificationModeSchema = z.enum(["off", "own", "all"]).optional();
-
-/**
- * Reply-in-thread mode for group chats.
- * - "disabled" (default): Bot replies are normal inline replies
- * - "enabled": Bot replies create or continue a Feishu topic thread
- *
- * When enabled, the Feishu reply API is called with `reply_in_thread: true`,
- * causing the reply to appear as a topic (话题) under the original message.
- */
-const ReplyInThreadSchema = z.enum(["disabled", "enabled"]).optional();
-
-export const FeishuGroupSchema = z
-  .object({
-    requireMention: z.boolean().optional(),
-    tools: ToolPolicySchema,
-    skills: z.array(z.string()).optional(),
-    enabled: z.boolean().optional(),
-    allowFrom: z.array(z.union([z.string(), z.number()])).optional(),
-    systemPrompt: z.string().optional(),
-    groupSessionScope: GroupSessionScopeSchema,
-    topicSessionMode: TopicSessionModeSchema,
-    replyInThread: ReplyInThreadSchema,
-  })
-  .strict();
-
-const FeishuSharedConfigShape = {
-  webhookHost: z.string().optional(),
-  webhookPort: z.number().int().positive().optional(),
-  capabilities: z.array(z.string()).optional(),
-  markdown: MarkdownConfigSchema,
-  configWrites: z.boolean().optional(),
-  dmPolicy: DmPolicySchema.optional(),
-  allowFrom: z.array(z.union([z.string(), z.number()])).optional(),
-  groupPolicy: GroupPolicySchema.optional(),
-  groupAllowFrom: z.array(z.union([z.string(), z.number()])).optional(),
-  groupSenderAllowFrom: z.array(z.union([z.string(), z.number()])).optional(),
-  requireMention: z.boolean().optional(),
-  groups: z.record(z.string(), FeishuGroupSchema.optional()).optional(),
-  historyLimit: z.number().int().min(0).optional(),
-  dmHistoryLimit: z.number().int().min(0).optional(),
-  dms: z.record(z.string(), DmConfigSchema).optional(),
-  textChunkLimit: z.number().int().positive().optional(),
-  chunkMode: z.enum(["length", "newline"]).optional(),
-  blockStreamingCoalesce: BlockStreamingCoalesceSchema,
-  mediaMaxMb: z.number().positive().optional(),
-  httpTimeoutMs: z.number().int().positive().max(300_000).optional(),
-  heartbeat: ChannelHeartbeatVisibilitySchema,
-  renderMode: RenderModeSchema,
-  streaming: StreamingModeSchema,
-  tools: FeishuToolsConfigSchema,
-  actions: ChannelActionsSchema,
-  replyInThread: ReplyInThreadSchema,
-  reactionNotifications: ReactionNotificationModeSchema,
-  typingIndicator: z.boolean().optional(),
-  resolveSenderNames: z.boolean().optional(),
-  tts: TtsOverrideSchema,
-};
-
-/**
- * Per-account configuration.
- * All fields are optional - missing fields inherit from top-level config.
- */
-export const FeishuAccountConfigSchema = z
-  .object({
-    enabled: z.boolean().optional(),
-    name: z.string().optional(), // Display name for this account
-    appId: z.string().optional(),
-    appSecret: buildSecretInputSchema().optional(),
-    encryptKey: buildSecretInputSchema().optional(),
-    verificationToken: buildSecretInputSchema().optional(),
-    domain: FeishuDomainSchema.optional(),
-    connectionMode: FeishuConnectionModeSchema.optional(),
-    webhookPath: z.string().optional(),
-    ...FeishuSharedConfigShape,
-    groupSessionScope: GroupSessionScopeSchema,
-    topicSessionMode: TopicSessionModeSchema,
-  })
-  .strict();
-
-export const FeishuConfigSchema = z
-  .object({
-    enabled: z.boolean().optional(),
-    defaultAccount: z.string().optional(),
-    // Top-level credentials (backward compatible for single-account mode)
-    appId: z.string().optional(),
-    appSecret: buildSecretInputSchema().optional(),
-    encryptKey: buildSecretInputSchema().optional(),
-    verificationToken: buildSecretInputSchema().optional(),
-    domain: FeishuDomainSchema.optional().default("feishu"),
-    connectionMode: FeishuConnectionModeSchema.optional().default("websocket"),
-    webhookPath: z.string().optional().default("/feishu/events"),
-    ...FeishuSharedConfigShape,
-    dmPolicy: DmPolicySchema.optional().default("pairing"),
-    reactionNotifications: ReactionNotificationModeSchema.optional().default("own"),
-    groupPolicy: GroupPolicySchema.optional().default("allowlist"),
-    requireMention: z.boolean().optional(),
-    groupSessionScope: GroupSessionScopeSchema,
-    topicSessionMode: TopicSessionModeSchema,
-    // Dynamic agent creation for DM users
-    dynamicAgentCreation: DynamicAgentCreationSchema,
-    // Optimization flags
-    typingIndicator: z.boolean().optional().default(true),
-    resolveSenderNames: z.boolean().optional().default(true),
-    // Multi-account configuration
-    accounts: z.record(z.string(), FeishuAccountConfigSchema.optional()).optional(),
-  })
-  .strict()
-  .superRefine((value, ctx) => {
-    const defaultAccount = value.defaultAccount?.trim();
-    if (defaultAccount && value.accounts && Object.keys(value.accounts).length > 0) {
-      const normalizedDefaultAccount = normalizeAccountId(defaultAccount);
-      if (!Object.prototype.hasOwnProperty.call(value.accounts, normalizedDefaultAccount)) {
-        ctx.addIssue({
-          code: z.ZodIssueCode.custom,
-          path: ["defaultAccount"],
-          message: `channels.feishu.defaultAccount="${defaultAccount}" does not match a configured account key`,
-        });
-      }
-    }
-
-    const defaultConnectionMode = value.connectionMode ?? "websocket";
-    const defaultVerificationTokenConfigured = hasConfiguredSecretInput(value.verificationToken);
-    const defaultEncryptKeyConfigured = hasConfiguredSecretInput(value.encryptKey);
-    if (defaultConnectionMode === "webhook") {
-      if (!defaultVerificationTokenConfigured) {
-        ctx.addIssue({
-          code: z.ZodIssueCode.custom,
-          path: ["verificationToken"],
-          message:
-            'channels.feishu.connectionMode="webhook" requires channels.feishu.verificationToken',
-        });
-      }
-      if (!defaultEncryptKeyConfigured) {
-        ctx.addIssue({
-          code: z.ZodIssueCode.custom,
-          path: ["encryptKey"],
-          message: 'channels.feishu.connectionMode="webhook" requires channels.feishu.encryptKey',
-        });
-      }
-    }
-
-    for (const [accountId, account] of Object.entries(value.accounts ?? {})) {
-      if (!account) {
-        continue;
-      }
-      const accountConnectionMode = account.connectionMode ?? defaultConnectionMode;
-      if (accountConnectionMode !== "webhook") {
-        continue;
-      }
-      const accountVerificationTokenConfigured =
-        hasConfiguredSecretInput(account.verificationToken) || defaultVerificationTokenConfigured;
-      const accountEncryptKeyConfigured =
-        hasConfiguredSecretInput(account.encryptKey) || defaultEncryptKeyConfigured;
-      if (!accountVerificationTokenConfigured) {
-        ctx.addIssue({
-          code: z.ZodIssueCode.custom,
-          path: ["accounts", accountId, "verificationToken"],
-          message:
-            `channels.feishu.accounts.${accountId}.connectionMode="webhook" requires ` +
-            "a verificationToken (account-level or top-level)",
-        });
-      }
-      if (!accountEncryptKeyConfigured) {
-        ctx.addIssue({
-          code: z.ZodIssueCode.custom,
-          path: ["accounts", accountId, "encryptKey"],
-          message:
-            `channels.feishu.accounts.${accountId}.connectionMode="webhook" requires ` +
-            "an encryptKey (account-level or top-level)",
-        });
-      }
-    }
-
-    if (value.dmPolicy === "open") {
-      const allowFrom = value.allowFrom ?? [];
-      const hasWildcard = allowFrom.some((entry) => String(entry).trim() === "*");
-      if (!hasWildcard) {
-        ctx.addIssue({
-          code: z.ZodIssueCode.custom,
-          path: ["allowFrom"],
-          message:
-            'channels.feishu.dmPolicy="open" requires channels.feishu.allowFrom to include "*"',
-        });
-      }
-    }
-  });

package/src/conversation-id.test.ts

@@ -1,18 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { buildFeishuModelOverrideParentCandidates } from "./conversation-id.js";
-
-describe("buildFeishuModelOverrideParentCandidates", () => {
-  it("returns topic and chat fallback ids for sender-scoped topics", () => {
-    expect(
-      buildFeishuModelOverrideParentCandidates(
-        "oc_group_chat:Topic:om_topic_root:Sender:ou_topic_user",
-      ),
-    ).toEqual(["oc_group_chat:topic:om_topic_root", "oc_group_chat"]);
-  });
-
-  it("returns chat fallback ids for sender-scoped chats", () => {
-    expect(buildFeishuModelOverrideParentCandidates("oc_group_chat:sender:ou_topic_user")).toEqual([
-      "oc_group_chat",
-    ]);
-  });
-});