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

推荐订阅源

D
DataBreaches.Net
MongoDB | Blog
MongoDB | Blog
GbyAI
GbyAI
L
LangChain Blog
B
Blog
博客园 - 三生石上(FineUI控件)
Martin Fowler
Martin Fowler
博客园 - 【当耐特】
Recent Announcements
Recent Announcements
P
Proofpoint News Feed
U
Unit 42
Last Week in AI
Last Week in AI
WordPress大学
WordPress大学
有赞技术团队
有赞技术团队
雷峰网
雷峰网
Microsoft Security Blog
Microsoft Security Blog
T
The Blog of Author Tim Ferriss
爱范儿
爱范儿
小众软件
小众软件
I
InfoQ
G
Google Developers Blog
大猫的无限游戏
大猫的无限游戏
人人都是产品经理
人人都是产品经理
C
Check Point Blog

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 message tool session-key route drift (#83004) · openc...
joshavant · 2026-05-17 · via Recent Commits to openclaw:main

@@ -72,6 +72,7 @@ const mocks = vi.hoisted(() => ({

72727373

maybeCollectSecretPath(`channels.${scopedChannel}.token`, scopedConfig.token);

7474

maybeCollectSecretPath(`channels.${scopedChannel}.botToken`, scopedConfig.botToken);

75+

maybeCollectSecretPath(`channels.${scopedChannel}.appPassword`, scopedConfig.appPassword);

7576

if (scopedAccountId) {

7677

const accountRecord =

7778

scopedConfig.accounts &&

@@ -106,6 +107,7 @@ const mocks = vi.hoisted(() => ({

106107

type RunMessageActionInput = {

107108

agentId?: string;

108109

cfg?: unknown;

110+

defaultAccountId?: string;

109111

params?: Record<string, unknown>;

110112

requesterSenderId?: string;

111113

sandboxRoot?: string;

@@ -423,6 +425,241 @@ describe("message tool secret scoping", () => {

423425

expect(input?.toolContext?.currentChannelProvider).toBe("webchat");

424426

});

425427428+

it("uses a non-webchat session key when ambient current channel drifted to webchat", async () => {

429+

mockSendResult();

430+431+

const input = await executeSend({

432+

action: { message: "hi" },

433+

toolOptions: {

434+

config: {

435+

channels: {

436+

telegram: {

437+

botToken: { source: "env", provider: "default", id: "TELEGRAM_BOT_TOKEN" },

438+

},

439+

},

440+

} as never,

441+

sourceReplyDeliveryMode: "message_tool_only",

442+

currentChannelProvider: "webchat",

443+

agentSessionKey: "agent:main:telegram:group:-5150615830",

444+

},

445+

});

446+447+

expect(input?.sourceReplyDeliveryMode).toBe("message_tool_only");

448+

expect(input?.toolContext?.currentChannelProvider).toBe("telegram");

449+

expect(input?.toolContext?.currentChannelId).toBe("-5150615830");

450+

expect(input?.params).toEqual({ action: "send", message: "hi" });

451+452+

const secretResolveCall = latestSecretResolveCall();

453+

expect(Array.from(secretResolveCall.targetIds ?? [])).toEqual(["channels.telegram.botToken"]);

454+

});

455+456+

it("preserves direct session keys as explicit user targets when ambient channel drifted to webchat", async () => {

457+

mockSendResult({ channel: "discord", to: "user:123456789" });

458+459+

const input = await executeSend({

460+

action: { message: "hi" },

461+

toolOptions: {

462+

config: {

463+

channels: {

464+

discord: {

465+

token: { source: "env", provider: "default", id: "DISCORD_TOKEN" },

466+

},

467+

},

468+

} as never,

469+

sourceReplyDeliveryMode: "message_tool_only",

470+

currentChannelProvider: "webchat",

471+

agentSessionKey: "agent:main:discord:direct:123456789",

472+

},

473+

});

474+475+

expect(input?.sourceReplyDeliveryMode).toBe("message_tool_only");

476+

expect(input?.toolContext?.currentChannelProvider).toBe("discord");

477+

expect(input?.toolContext?.currentChannelId).toBe("user:123456789");

478+

expect(input?.params).toEqual({ action: "send", message: "hi" });

479+480+

const secretResolveCall = latestSecretResolveCall();

481+

expect(Array.from(secretResolveCall.targetIds ?? [])).toEqual(["channels.discord.token"]);

482+

});

483+484+

it("preserves MS Teams DM session keys as explicit user targets when ambient channel drifted to webchat", async () => {

485+

mockSendResult({ channel: "msteams", to: "user:user-1" });

486+487+

const input = await executeSend({

488+

action: { message: "hi" },

489+

toolOptions: {

490+

config: {

491+

channels: {

492+

msteams: {

493+

appPassword: { source: "env", provider: "default", id: "MSTEAMS_APP_PASSWORD" },

494+

},

495+

},

496+

} as never,

497+

sourceReplyDeliveryMode: "message_tool_only",

498+

currentChannelProvider: "webchat",

499+

agentSessionKey: "agent:main:msteams:dm:user-1",

500+

},

501+

});

502+503+

expect(input?.sourceReplyDeliveryMode).toBe("message_tool_only");

504+

expect(input?.toolContext?.currentChannelProvider).toBe("msteams");

505+

expect(input?.toolContext?.currentChannelId).toBe("user:user-1");

506+

expect(input?.params).toEqual({ action: "send", message: "hi" });

507+508+

const secretResolveCall = latestSecretResolveCall();

509+

expect(Array.from(secretResolveCall.targetIds ?? [])).toEqual(["channels.msteams.appPassword"]);

510+

});

511+512+

it("keeps provider-native direct session targets when ambient channel drifted to webchat", async () => {

513+

mockSendResult({ channel: "telegram", to: "123456789" });

514+515+

const input = await executeSend({

516+

action: { message: "hi" },

517+

toolOptions: {

518+

config: {

519+

channels: {

520+

telegram: {

521+

botToken: { source: "env", provider: "default", id: "TELEGRAM_BOT_TOKEN" },

522+

},

523+

},

524+

} as never,

525+

sourceReplyDeliveryMode: "message_tool_only",

526+

currentChannelProvider: "webchat",

527+

agentSessionKey: "agent:main:telegram:direct:123456789",

528+

},

529+

});

530+531+

expect(input?.sourceReplyDeliveryMode).toBe("message_tool_only");

532+

expect(input?.toolContext?.currentChannelProvider).toBe("telegram");

533+

expect(input?.toolContext?.currentChannelId).toBe("123456789");

534+

expect(input?.params).toEqual({ action: "send", message: "hi" });

535+536+

const secretResolveCall = latestSecretResolveCall();

537+

expect(Array.from(secretResolveCall.targetIds ?? [])).toEqual(["channels.telegram.botToken"]);

538+

});

539+540+

it("uses account-scoped session keys for secret and account fallback when ambient channel drifted to webchat", async () => {

541+

mockSendResult({ channel: "discord", to: "user:123456789" });

542+543+

const input = await executeSend({

544+

action: { message: "hi" },

545+

toolOptions: {

546+

config: {

547+

channels: {

548+

discord: {

549+

token: { source: "env", provider: "default", id: "DISCORD_TOKEN" },

550+

accounts: {

551+

ops: { token: { source: "env", provider: "default", id: "DISCORD_OPS_TOKEN" } },

552+

},

553+

},

554+

},

555+

} as never,

556+

sourceReplyDeliveryMode: "message_tool_only",

557+

currentChannelProvider: "webchat",

558+

agentSessionKey: "agent:main:discord:ops:direct:123456789",

559+

},

560+

});

561+562+

expect(input?.defaultAccountId).toBe("ops");

563+

expect(input?.params?.accountId).toBe("ops");

564+

expect(input?.toolContext?.currentChannelProvider).toBe("discord");

565+

expect(input?.toolContext?.currentChannelId).toBe("user:123456789");

566+567+

const secretResolveCall = latestSecretResolveCall();

568+

expect(Array.from(secretResolveCall.targetIds ?? [])).toEqual([

569+

"channels.discord.token",

570+

"channels.discord.accounts.ops.token",

571+

]);

572+

});

573+574+

it("keeps account-scoped direct keys when account id matches a peer marker", async () => {

575+

mockSendResult({ channel: "discord", to: "user:123456789" });

576+577+

const input = await executeSend({

578+

action: { message: "hi" },

579+

toolOptions: {

580+

config: {

581+

channels: {

582+

discord: {

583+

token: { source: "env", provider: "default", id: "DISCORD_TOKEN" },

584+

accounts: {

585+

direct: {

586+

token: { source: "env", provider: "default", id: "DISCORD_DIRECT_TOKEN" },

587+

},

588+

},

589+

},

590+

},

591+

} as never,

592+

sourceReplyDeliveryMode: "message_tool_only",

593+

currentChannelProvider: "webchat",

594+

agentSessionKey: "agent:main:discord:direct:direct:123456789",

595+

},

596+

});

597+598+

expect(input?.defaultAccountId).toBe("direct");

599+

expect(input?.params?.accountId).toBe("direct");

600+

expect(input?.toolContext?.currentChannelProvider).toBe("discord");

601+

expect(input?.toolContext?.currentChannelId).toBe("user:123456789");

602+603+

const secretResolveCall = latestSecretResolveCall();

604+

expect(Array.from(secretResolveCall.targetIds ?? [])).toEqual([

605+

"channels.discord.token",

606+

"channels.discord.accounts.direct.token",

607+

]);

608+

});

609+610+

it("handles legacy dm markers when ambient channel drifted to webchat", async () => {

611+

mockSendResult({ channel: "slack", to: "user:u123" });

612+613+

const input = await executeSend({

614+

action: { message: "hi" },

615+

toolOptions: {

616+

config: {

617+

channels: {

618+

slack: {

619+

botToken: { source: "env", provider: "default", id: "SLACK_BOT_TOKEN" },

620+

},

621+

},

622+

} as never,

623+

sourceReplyDeliveryMode: "message_tool_only",

624+

currentChannelProvider: "webchat",

625+

agentSessionKey: "agent:main:slack:dm:u123:thread:171.222",

626+

},

627+

});

628+629+

expect(input?.toolContext?.currentChannelProvider).toBe("slack");

630+

expect(input?.toolContext?.currentChannelId).toBe("user:u123");

631+

expect(input?.toolContext?.currentThreadTs).toBe("171.222");

632+

expect(input?.toolContext?.replyToMode).toBe("all");

633+634+

const secretResolveCall = latestSecretResolveCall();

635+

expect(Array.from(secretResolveCall.targetIds ?? [])).toEqual(["channels.slack.botToken"]);

636+

});

637+638+

it("carries session-key thread suffixes into inferred channel context", async () => {

639+

mockSendResult({ channel: "slack", to: "channel:c1" });

640+641+

const input = await executeSend({

642+

action: { message: "hi" },

643+

toolOptions: {

644+

config: {

645+

channels: {

646+

slack: {

647+

botToken: { source: "env", provider: "default", id: "SLACK_BOT_TOKEN" },

648+

},

649+

},

650+

} as never,

651+

sourceReplyDeliveryMode: "message_tool_only",

652+

currentChannelProvider: "webchat",

653+

agentSessionKey: "agent:main:slack:channel:c1:thread:1710000000.9999",

654+

},

655+

});

656+657+

expect(input?.toolContext?.currentChannelProvider).toBe("slack");

658+

expect(input?.toolContext?.currentChannelId).toBe("c1");

659+

expect(input?.toolContext?.currentThreadTs).toBe("1710000000.9999");

660+

expect(input?.toolContext?.replyToMode).toBe("all");

661+

});

662+426663

it("scopes command-time secret resolution to the selected channel/account", async () => {

427664

mockSendResult({ channel: "discord", to: "discord:123" });

428665

mocks.getRuntimeConfig.mockReturnValue({