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

推荐订阅源

大猫的无限游戏
大猫的无限游戏
aimingoo的专栏
aimingoo的专栏
I
InfoQ
B
Blog RSS Feed
D
DataBreaches.Net
S
SegmentFault 最新的问题
P
Proofpoint News Feed
A
About on SuperTechFans
WordPress大学
WordPress大学
Hugging Face - Blog
Hugging Face - Blog
博客园 - 司徒正美
小众软件
小众软件
博客园 - Franky
有赞技术团队
有赞技术团队
D
Docker
T
Tailwind CSS Blog
雷峰网
雷峰网
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
Blog — PlanetScale
Blog — PlanetScale
酷 壳 – CoolShell
酷 壳 – CoolShell
B
Blog
V
Visual Studio 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(messages): use best-effort for implicit tool-only sou...
tianxiaochan · 2026-05-31 · via Recent Commits to openclaw:main

@@ -27,6 +27,38 @@ function firstMockArg(

2727

return arg as Record<string, unknown>;

2828

}

292930+

const slackConfig = {

31+

channels: {

32+

slack: {

33+

enabled: true,

34+

},

35+

},

36+

} as OpenClawConfig;

37+38+

function registerSlackTextPlugin() {

39+

const sendText = vi.fn().mockResolvedValue({

40+

channel: "slack",

41+

messageId: "m1",

42+

chatId: "C123",

43+

});

44+

setActivePluginRegistry(

45+

createTestRegistry([

46+

{

47+

pluginId: "slack",

48+

source: "test",

49+

plugin: createOutboundTestPlugin({

50+

id: "slack",

51+

outbound: {

52+

deliveryMode: "direct",

53+

sendText,

54+

},

55+

}),

56+

},

57+

]),

58+

);

59+

return sendText;

60+

}

61+3062

describe("runMessageAction core send routing", () => {

3163

afterEach(() => {

3264

setActivePluginRegistry(createTestRegistry([]));

@@ -196,6 +228,136 @@ describe("runMessageAction core send routing", () => {

196228

expect(payload.dryRun).toBe(true);

197229

});

198230231+

it("uses best-effort delivery for implicit message-tool-only source replies", async () => {

232+

const sendText = registerSlackTextPlugin();

233+234+

const result = await runMessageAction({

235+

cfg: slackConfig,

236+

action: "send",

237+

params: {

238+

message: "visible source reply",

239+

bestEffort: false,

240+

},

241+

toolContext: {

242+

currentChannelProvider: "slack",

243+

currentChannelId: "channel:C123",

244+

},

245+

sessionKey: "agent:main:slack:channel:C123",

246+

sourceReplyDeliveryMode: "message_tool_only",

247+

dryRun: false,

248+

});

249+250+

expect(result.kind).toBe("send");

251+

expect(sendText).toHaveBeenCalledOnce();

252+

});

253+254+

it("uses best-effort delivery for explicit current-source message-tool-only replies", async () => {

255+

const sendText = registerSlackTextPlugin();

256+257+

const result = await runMessageAction({

258+

cfg: slackConfig,

259+

action: "send",

260+

params: {

261+

target: "channel:C123",

262+

message: "visible current-channel source reply",

263+

bestEffort: false,

264+

},

265+

toolContext: {

266+

currentChannelProvider: "slack",

267+

currentChannelId: "channel:C123",

268+

},

269+

sessionKey: "agent:main:slack:channel:C123",

270+

sourceReplyDeliveryMode: "message_tool_only",

271+

dryRun: false,

272+

});

273+274+

if (result.kind !== "send") {

275+

throw new Error(`expected send result, got ${result.kind}`);

276+

}

277+

expect(sendText).toHaveBeenCalledOnce();

278+

expect(result.to).toBe("channel:C123");

279+

});

280+281+

it("preserves required delivery when message-tool-only sends target another conversation", async () => {

282+

const sendText = registerSlackTextPlugin();

283+284+

await expect(

285+

runMessageAction({

286+

cfg: slackConfig,

287+

action: "send",

288+

params: {

289+

target: "channel:C999",

290+

message: "explicit durable send",

291+

bestEffort: false,

292+

},

293+

toolContext: {

294+

currentChannelProvider: "slack",

295+

currentChannelId: "channel:C123",

296+

},

297+

sessionKey: "agent:main:slack:channel:C123",

298+

sourceReplyDeliveryMode: "message_tool_only",

299+

dryRun: false,

300+

}),

301+

).rejects.toThrow("missing reconcileUnknownSend");

302+

expect(sendText).not.toHaveBeenCalled();

303+

});

304+305+

it("preserves required delivery when message-tool-only sends to another explicit channel", async () => {

306+

const sendText = vi.fn().mockResolvedValue({

307+

channel: "telegram",

308+

messageId: "m1",

309+

chatId: "C999",

310+

});

311+

setActivePluginRegistry(

312+

createTestRegistry([

313+

{

314+

pluginId: "telegram",

315+

source: "test",

316+

plugin: createOutboundTestPlugin({

317+

id: "telegram",

318+

outbound: {

319+

deliveryMode: "direct",

320+

sendText,

321+

},

322+

}),

323+

},

324+

]),

325+

);

326+327+

await expect(

328+

runMessageAction({

329+

cfg: {

330+

channels: {

331+

telegram: {

332+

enabled: true,

333+

},

334+

},

335+

tools: {

336+

message: {

337+

crossContext: {

338+

allowAcrossProviders: true,

339+

},

340+

},

341+

},

342+

} as OpenClawConfig,

343+

action: "send",

344+

params: {

345+

channel: "telegram",

346+

message: "explicit channel-only durable send",

347+

bestEffort: false,

348+

},

349+

toolContext: {

350+

currentChannelProvider: "slack",

351+

currentChannelId: "channel:C123",

352+

},

353+

sessionKey: "agent:main:slack:channel:C123",

354+

sourceReplyDeliveryMode: "message_tool_only",

355+

dryRun: false,

356+

}),

357+

).rejects.toThrow("missing reconcileUnknownSend");

358+

expect(sendText).not.toHaveBeenCalled();

359+

});

360+199361

it("applies TTS to message-tool sends before core outbound delivery", async () => {

200362

const sendMedia = vi.fn().mockResolvedValue({

201363

channel: "testchat",