@nextclaw/channel-plugin-feishu 0.2.15 → 0.2.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nextclaw/channel-plugin-feishu",
-  "version": "0.2.15",
+  "version": "0.2.16",
   "private": false,
   "description": "NextClaw Feishu/Lark channel plugin with doc/wiki/drive tools.",
   "type": "module",

package/src/accounts.test.ts

@@ -1,5 +1,6 @@
 import { describe, expect, it } from "vitest";
 import {
+  listFeishuAccountIds,
   resolveDefaultFeishuAccountId,
   resolveDefaultFeishuAccountSelection,
   resolveFeishuAccount,
@@ -56,6 +57,15 @@ function expectUnresolvedEnvSecretRefError(key: string) {
 }
 
 describe("resolveDefaultFeishuAccountId", () => {
+  it("falls back safely when channel config is missing", () => {
+    expect(listFeishuAccountIds(undefined)).toEqual(["default"]);
+    expect(resolveDefaultFeishuAccountId(undefined)).toBe("default");
+    expect(resolveDefaultFeishuAccountSelection(undefined)).toEqual({
+      accountId: "default",
+      source: "mapped-default",
+    });
+  });
+
   it("prefers channels.feishu.defaultAccount when configured", () => {
     const cfg = {
       channels: {

package/src/accounts.ts

@@ -12,8 +12,8 @@ import type {
 /**
  * List all configured account IDs from the accounts field.
  */
-function listConfiguredAccountIds(cfg: ClawdbotConfig): string[] {
-  const accounts = (cfg.channels?.feishu as FeishuConfig)?.accounts;
+function listConfiguredAccountIds(cfg?: ClawdbotConfig): string[] {
+  const accounts = (cfg?.channels?.feishu as FeishuConfig | undefined)?.accounts;
   if (!accounts || typeof accounts !== "object") {
     return [];
   }
@@ -24,7 +24,7 @@ function listConfiguredAccountIds(cfg: ClawdbotConfig): string[] {
  * List all Feishu account IDs.
  * If no accounts are configured, returns [DEFAULT_ACCOUNT_ID] for backward compatibility.
  */
-export function listFeishuAccountIds(cfg: ClawdbotConfig): string[] {
+export function listFeishuAccountIds(cfg?: ClawdbotConfig): string[] {
   const ids = listConfiguredAccountIds(cfg);
   if (ids.length === 0) {
     // Backward compatibility: no accounts configured, use default
@@ -36,11 +36,11 @@ export function listFeishuAccountIds(cfg: ClawdbotConfig): string[] {
 /**
  * Resolve the default account selection and its source.
  */
-export function resolveDefaultFeishuAccountSelection(cfg: ClawdbotConfig): {
+export function resolveDefaultFeishuAccountSelection(cfg?: ClawdbotConfig): {
   accountId: string;
   source: FeishuDefaultAccountSelectionSource;
 } {
-  const preferredRaw = (cfg.channels?.feishu as FeishuConfig | undefined)?.defaultAccount?.trim();
+  const preferredRaw = (cfg?.channels?.feishu as FeishuConfig | undefined)?.defaultAccount?.trim();
   const preferred = preferredRaw ? normalizeAccountId(preferredRaw) : undefined;
   if (preferred) {
     return {
@@ -64,7 +64,7 @@ export function resolveDefaultFeishuAccountSelection(cfg: ClawdbotConfig): {
 /**
  * Resolve the default account ID.
  */
-export function resolveDefaultFeishuAccountId(cfg: ClawdbotConfig): string {
+export function resolveDefaultFeishuAccountId(cfg?: ClawdbotConfig): string {
   return resolveDefaultFeishuAccountSelection(cfg).accountId;
 }
 
@@ -72,10 +72,10 @@ export function resolveDefaultFeishuAccountId(cfg: ClawdbotConfig): string {
  * Get the raw account-specific config.
  */
 function resolveAccountConfig(
-  cfg: ClawdbotConfig,
+  cfg: ClawdbotConfig | undefined,
   accountId: string,
 ): FeishuAccountConfig | undefined {
-  const accounts = (cfg.channels?.feishu as FeishuConfig)?.accounts;
+  const accounts = (cfg?.channels?.feishu as FeishuConfig | undefined)?.accounts;
   if (!accounts || typeof accounts !== "object") {
     return undefined;
   }
@@ -86,8 +86,8 @@ function resolveAccountConfig(
  * Merge top-level config with account-specific config.
  * Account-specific fields override top-level fields.
  */
-function mergeFeishuAccountConfig(cfg: ClawdbotConfig, accountId: string): FeishuConfig {
-  const feishuCfg = cfg.channels?.feishu as FeishuConfig | undefined;
+function mergeFeishuAccountConfig(cfg: ClawdbotConfig | undefined, accountId: string): FeishuConfig {
+  const feishuCfg = cfg?.channels?.feishu as FeishuConfig | undefined;
 
   // Extract base config (exclude accounts field to avoid recursion)
   const { accounts: _ignored, defaultAccount: _ignoredDefaultAccount, ...base } = feishuCfg ?? {};