惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

Google DeepMind News
Google DeepMind News
B
Blog RSS Feed
量子位
aimingoo的专栏
aimingoo的专栏
V
Visual Studio Blog
Y
Y Combinator Blog
Vercel News
Vercel News
云风的 BLOG
云风的 BLOG
宝玉的分享
宝玉的分享
Engineering at Meta
Engineering at Meta
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
GbyAI
GbyAI
人人都是产品经理
人人都是产品经理
博客园 - 叶小钗
Stack Overflow Blog
Stack Overflow Blog
大猫的无限游戏
大猫的无限游戏
Microsoft Security Blog
Microsoft Security Blog
B
Blog
Last Week in AI
Last Week in AI
有赞技术团队
有赞技术团队
博客园 - 聂微东
腾讯CDC
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
J
Java Code Geeks

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(channels): preserve implicit default accounts · openc...
steipete · 2026-05-17 · via Recent Commits to openclaw:main
Original file line numberDiff line numberDiff line change

@@ -11,6 +11,7 @@ Docs: https://docs.openclaw.ai

1111

### Fixes

1212
1313

- Agents/skills: apply the full effective tool policy pipeline to inline `command-dispatch: tool` skill dispatch before owner-only filtering, preserving configured allow, deny, sandbox, sender, group, and subagent restrictions. (#78525)

14+

- Channel accounts: keep top-level default channel accounts visible when named accounts are added alongside default credential material, so mixed legacy/new account configs keep resolving `default` instead of silently dropping it.

1415

- Codex/Telegram: synthesize native Codex tool progress from final turn snapshots so Telegram `/verbose` stays visible when command events arrive only at completion.

1516

- Mac app: make Channels settings open faster by deferring config-schema work, avoiding startup channel probes, caching decoded channel status rows, and showing only compact quick settings instead of the full generated channel schema.

1617

- Control UI: include the Control UI and Gateway protocol versions in protocol-mismatch errors so stale app/dashboard pairings identify which side needs rebuilding or restarting.

Original file line numberDiff line numberDiff line change

@@ -1,8 +1,72 @@

11

import { describe, expect, it } from "vitest";

2-

import { resolveClickClackAccount } from "./accounts.js";

2+

import {

3+

listClickClackAccountIds,

4+

resolveClickClackAccount,

5+

resolveDefaultClickClackAccountId,

6+

} from "./accounts.js";

37

import type { CoreConfig } from "./types.js";

48
59

describe("ClickClack account resolution", () => {

10+

it("preserves top-level default account when named accounts are configured", () => {

11+

const cfg = {

12+

channels: {

13+

clickclack: {

14+

baseUrl: "https://app.clickclack.chat",

15+

workspace: "wsp_1",

16+

token: "ccb_default",

17+

accounts: {

18+

work: { enabled: false },

19+

},

20+

},

21+

},

22+

} satisfies CoreConfig;

23+
24+

expect(listClickClackAccountIds(cfg)).toEqual(["default", "work"]);

25+

expect(resolveDefaultClickClackAccountId(cfg)).toBe("default");

26+

expect(resolveClickClackAccount({ cfg }).token).toBe("ccb_default");

27+

});

28+
29+

it("does not synthesize a partial top-level default account from inherited credentials", () => {

30+

const cfg = {

31+

channels: {

32+

clickclack: {

33+

token: "ccb_shared",

34+

accounts: {

35+

work: {

36+

baseUrl: "https://app.clickclack.chat",

37+

workspace: "wsp_1",

38+

},

39+

},

40+

},

41+

},

42+

} satisfies CoreConfig;

43+
44+

expect(listClickClackAccountIds(cfg)).toEqual(["work"]);

45+

expect(resolveDefaultClickClackAccountId(cfg)).toBe("work");

46+

});

47+
48+

it("does not synthesize a default account from blank top-level credentials", () => {

49+

const cfg = {

50+

channels: {

51+

clickclack: {

52+

baseUrl: "https://app.clickclack.chat",

53+

workspace: "wsp_default",

54+

token: " ",

55+

accounts: {

56+

work: {

57+

baseUrl: "https://app.clickclack.chat",

58+

workspace: "wsp_1",

59+

token: "ccb_work",

60+

},

61+

},

62+

},

63+

},

64+

} satisfies CoreConfig;

65+
66+

expect(listClickClackAccountIds(cfg)).toEqual(["work"]);

67+

expect(resolveDefaultClickClackAccountId(cfg)).toBe("work");

68+

});

69+
670

it("resolves env SecretRefs at runtime", () => {

771

const cfg = {

872

channels: {

Original file line numberDiff line numberDiff line change

@@ -1,4 +1,7 @@

1-

import { createAccountListHelpers } from "openclaw/plugin-sdk/account-helpers";

1+

import {

2+

createAccountListHelpers,

3+

hasConfiguredAccountValue,

4+

} from "openclaw/plugin-sdk/account-helpers";

25

import { DEFAULT_ACCOUNT_ID, normalizeAccountId } from "openclaw/plugin-sdk/account-id";

36

import { resolveMergedAccountConfig } from "openclaw/plugin-sdk/account-resolution";

47

import { resolveDefaultSecretProviderAlias } from "openclaw/plugin-sdk/provider-auth";

@@ -15,7 +18,17 @@ const DEFAULT_RECONNECT_MS = 1_500;

1518

const {

1619

listAccountIds: listClickClackAccountIds,

1720

resolveDefaultAccountId: resolveDefaultClickClackAccountId,

18-

} = createAccountListHelpers("clickclack", { normalizeAccountId });

21+

} = createAccountListHelpers("clickclack", {

22+

normalizeAccountId,

23+

hasImplicitDefaultAccount: (cfg) => {

24+

const channel = cfg.channels?.clickclack;

25+

return Boolean(

26+

channel?.baseUrl?.trim() &&

27+

hasConfiguredAccountValue(channel.token) &&

28+

channel.workspace?.trim(),

29+

);

30+

},

31+

});

1932
2033

export { DEFAULT_ACCOUNT_ID, listClickClackAccountIds, resolveDefaultClickClackAccountId };

2134
Original file line numberDiff line numberDiff line change

@@ -7,7 +7,9 @@ import { afterEach, describe, expect, it, vi } from "vitest";

77

import {

88

createDiscordActionGate,

99

isDiscordAccountEnabledForRuntime,

10+

listDiscordAccountIds,

1011

listEnabledDiscordAccounts,

12+

resolveDefaultDiscordAccountId,

1113

resolveDiscordAccount,

1214

resolveDiscordAccountDisabledReason,

1315

resolveDiscordMaxLinesPerMessage,

@@ -72,6 +74,28 @@ describe("Discord defaultAccount omission contract", () => {

7274

assert();

7375

},

7476

);

77+
78+

it("keeps the implicit default account when named accounts are added to top-level credentials", () => {

79+

const cfg = {

80+

channels: {

81+

discord: {

82+

token: "token-default",

83+

accounts: {

84+

work: {

85+

enabled: false,

86+

token: "token-work",

87+

},

88+

},

89+

},

90+

},

91+

} as OpenClawConfig;

92+
93+

expect(listDiscordAccountIds(cfg)).toEqual(["default", "work"]);

94+

expect(resolveDefaultDiscordAccountId(cfg)).toBe("default");

95+

expect(listEnabledDiscordAccounts(cfg).map((account) => account.accountId)).toEqual([

96+

"default",

97+

]);

98+

});

7599

});

76100
77101

describe("resolveDiscordAccount allowFrom precedence", () => {

Original file line numberDiff line numberDiff line change

@@ -27,7 +27,12 @@ export type ResolvedDiscordAccount = {

2727

config: DiscordAccountConfig;

2828

};

2929
30-

const { listAccountIds, resolveDefaultAccountId } = createAccountListHelpers("discord");

30+

const { listAccountIds, resolveDefaultAccountId } = createAccountListHelpers("discord", {

31+

implicitDefaultAccount: {

32+

channelKeys: ["token"],

33+

envVars: ["DISCORD_BOT_TOKEN"],

34+

},

35+

});

3136

export const listDiscordAccountIds = listAccountIds;

3237

export const resolveDefaultDiscordAccountId = resolveDefaultAccountId;

3338
Original file line numberDiff line numberDiff line change

@@ -2,6 +2,7 @@ import { describe, expect, it } from "vitest";

22

import {

33

FeishuSecretRefUnavailableError,

44

inspectFeishuCredentials,

5+

listFeishuAccountIds,

56

resolveDefaultFeishuAccountId,

67

resolveDefaultFeishuAccountSelection,

78

resolveFeishuAccount,

@@ -61,6 +62,23 @@ function expectUnresolvedEnvSecretRefError(key: string) {

6162

}

6263
6364

describe("resolveDefaultFeishuAccountId", () => {

65+

it("preserves top-level default account when named accounts are configured", () => {

66+

const cfg = {

67+

channels: {

68+

feishu: {

69+

appId: "cli_default",

70+

appSecret: "secret_default",

71+

accounts: {

72+

work: { enabled: false },

73+

},

74+

},

75+

},

76+

};

77+
78+

expect(listFeishuAccountIds(cfg as never)).toEqual(["default", "work"]);

79+

expect(resolveDefaultFeishuAccountId(cfg as never)).toBe("default");

80+

});

81+
6482

it("prefers channels.feishu.defaultAccount when configured", () => {

6583

const cfg = {

6684

channels: {

Original file line numberDiff line numberDiff line change

@@ -2,6 +2,7 @@ import {

22

DEFAULT_ACCOUNT_ID,

33

type OpenClawConfig as ClawdbotConfig,

44

createAccountListHelpers,

5+

hasConfiguredAccountValue,

56

normalizeAccountId,

67

normalizeOptionalAccountId,

78

resolveMergedAccountConfig,

@@ -20,6 +21,11 @@ const { listAccountIds: listFeishuAccountIds, resolveDefaultAccountId } = create

2021

"feishu",

2122

{

2223

allowUnlistedDefaultAccount: true,

24+

hasImplicitDefaultAccount: (cfg) =>

25+

Boolean(

26+

cfg.channels?.feishu?.appId?.trim() &&

27+

hasConfiguredAccountValue(cfg.channels.feishu.appSecret),

28+

),

2329

},

2430

);

2531
Original file line numberDiff line numberDiff line change

@@ -36,7 +36,12 @@ const JsonRecordSchema = z.record(z.string(), z.unknown());

3636

const {

3737

listAccountIds: listGoogleChatAccountIds,

3838

resolveDefaultAccountId: resolveDefaultGoogleChatAccountId,

39-

} = createAccountListHelpers("googlechat");

39+

} = createAccountListHelpers("googlechat", {

40+

implicitDefaultAccount: {

41+

channelKeys: ["serviceAccount", "serviceAccountRef", "serviceAccountFile"],

42+

envVars: [ENV_SERVICE_ACCOUNT, ENV_SERVICE_ACCOUNT_FILE],

43+

},

44+

});

4045

export { listGoogleChatAccountIds, resolveDefaultGoogleChatAccountId };

4146
4247

function mergeGoogleChatAccountConfig(

Original file line numberDiff line numberDiff line change

@@ -1,7 +1,28 @@

11

import { describe, expect, it } from "vitest";

2-

import { resolveIMessageAccount } from "./accounts.js";

2+

import {

3+

listIMessageAccountIds,

4+

resolveDefaultIMessageAccountId,

5+

resolveIMessageAccount,

6+

} from "./accounts.js";

37
48

describe("resolveIMessageAccount", () => {

9+

it("preserves top-level default account when named accounts are configured", () => {

10+

const cfg = {

11+

channels: {

12+

imessage: {

13+

cliPath: "/usr/local/bin/imsg",

14+

accounts: {

15+

work: { enabled: false },

16+

},

17+

},

18+

},

19+

} as never;

20+
21+

expect(listIMessageAccountIds(cfg)).toEqual(["default", "work"]);

22+

expect(resolveDefaultIMessageAccountId(cfg)).toBe("default");

23+

expect(resolveIMessageAccount({ cfg }).config.cliPath).toBe("/usr/local/bin/imsg");

24+

});

25+
526

it("uses configured defaultAccount when accountId is omitted", () => {

627

const resolved = resolveIMessageAccount({

728

cfg: {

Original file line numberDiff line numberDiff line change

@@ -15,7 +15,11 @@ export type ResolvedIMessageAccount = {

1515

configured: boolean;

1616

};

1717
18-

const { listAccountIds, resolveDefaultAccountId } = createAccountListHelpers("imessage");

18+

const { listAccountIds, resolveDefaultAccountId } = createAccountListHelpers("imessage", {

19+

implicitDefaultAccount: {

20+

channelKeys: ["cliPath", "dbPath"],

21+

},

22+

});

1923

export const listIMessageAccountIds = listAccountIds;

2024

export const resolveDefaultIMessageAccountId = resolveDefaultAccountId;

2125