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

推荐订阅源

月光博客
月光博客
J
Java Code Geeks
F
Fortinet All Blogs
Blog — PlanetScale
Blog — PlanetScale
P
Proofpoint News Feed
U
Unit 42
B
Blog
宝玉的分享
宝玉的分享
腾讯CDC
Microsoft Azure Blog
Microsoft Azure Blog
酷 壳 – CoolShell
酷 壳 – CoolShell
Last Week in AI
Last Week in AI
博客园 - Franky
博客园 - 三生石上(FineUI控件)
人人都是产品经理
人人都是产品经理
Martin Fowler
Martin Fowler
博客园_首页
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
云风的 BLOG
云风的 BLOG
L
LangChain Blog
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
Y
Y Combinator Blog
The GitHub Blog
The GitHub 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(cli): avoid local preload for gateway-owned message a...
steipete · 2026-05-04 · via Recent Commits to openclaw:main

@@ -5,6 +5,11 @@ vi.mock("../../../commands/message.js", () => ({

55

messageCommand: messageCommandMock,

66

}));

778+

const getChannelPluginMock = vi.fn();

9+

vi.mock("../../../channels/plugins/index.js", () => ({

10+

getChannelPlugin: getChannelPluginMock,

11+

}));

12+813

vi.mock("../../../globals.js", () => ({

914

danger: (s: string) => s,

1015

setVerbose: vi.fn(),

@@ -70,6 +75,14 @@ async function runSendAction(opts: Record<string, unknown> = {}) {

7075

await expect(runMessageAction("send", { ...baseSendOptions, ...opts })).rejects.toThrow("exit");

7176

}

727778+

function mockChannelExecutionModes(modes: Record<string, "gateway" | "local"> = {}) {

79+

getChannelPluginMock.mockImplementation((id: string) => ({

80+

actions: {

81+

resolveExecutionMode: () => modes[id] ?? "local",

82+

},

83+

}));

84+

}

85+7386

function expectNoAccountFieldInPassedOptions() {

7487

const passedOpts = (

7588

messageCommandMock.mock.calls as unknown as Array<[Record<string, unknown>]>

@@ -84,6 +97,8 @@ function expectNoAccountFieldInPassedOptions() {

8497

describe("runMessageAction", () => {

8598

beforeEach(() => {

8699

vi.clearAllMocks();

100+

getChannelPluginMock.mockReset();

101+

mockChannelExecutionModes({ telegram: "gateway" });

87102

messageCommandMock.mockClear().mockResolvedValue(undefined);

88103

hasHooksMock.mockClear().mockReturnValue(false);

89104

runGatewayStopMock.mockClear().mockResolvedValue(undefined);

@@ -113,12 +128,152 @@ describe("runMessageAction", () => {

113128

});

114129115130

it("narrows plugin loading from a channel-prefixed target", async () => {

131+

await runSendAction({ channel: undefined, target: "discord:channel:12345" });

132+133+

expect(ensurePluginRegistryLoaded).toHaveBeenCalledWith({

134+

scope: "configured-channels",

135+

onlyChannelIds: ["discord"],

136+

});

137+

});

138+139+

it("skips local plugin preload for any gateway-owned scoped channel action", async () => {

140+

mockChannelExecutionModes({ discord: "gateway" });

141+142+

await runSendAction({ target: "channel:12345" });

143+144+

expect(ensurePluginRegistryLoaded).not.toHaveBeenCalled();

145+

expect(messageCommandMock).toHaveBeenCalledWith(

146+

expect.objectContaining({

147+

action: "send",

148+

channel: "discord",

149+

target: "channel:12345",

150+

message: "hi",

151+

}),

152+

expect.anything(),

153+

expect.anything(),

154+

);

155+

expect(exitMock).toHaveBeenCalledWith(0);

156+

});

157+158+

it("keeps broadcast on the local preload path for same-channel prefixed targets", async () => {

159+

const runMessageAction = createRunMessageAction();

160+161+

await expect(

162+

runMessageAction("broadcast", {

163+

targets: ["telegram:1", "telegram:2"],

164+

message: "hi",

165+

}),

166+

).rejects.toThrow("exit");

167+168+

expect(ensurePluginRegistryLoaded).toHaveBeenCalledWith({

169+

scope: "configured-channels",

170+

onlyChannelIds: ["telegram"],

171+

});

172+

expect(messageCommandMock).toHaveBeenCalledWith(

173+

expect.objectContaining({

174+

action: "broadcast",

175+

targets: ["telegram:1", "telegram:2"],

176+

message: "hi",

177+

}),

178+

expect.anything(),

179+

expect.anything(),

180+

);

181+

});

182+183+

it("keeps unknown actions on the local preload path", async () => {

184+

mockChannelExecutionModes({ discord: "gateway" });

185+

const runMessageAction = createRunMessageAction();

186+187+

await expect(

188+

runMessageAction("custom-action", {

189+

...baseSendOptions,

190+

target: "channel:12345",

191+

}),

192+

).rejects.toThrow("exit");

193+194+

expect(ensurePluginRegistryLoaded).toHaveBeenCalledWith({

195+

scope: "configured-channels",

196+

onlyChannelIds: ["discord"],

197+

});

198+

expect(messageCommandMock).toHaveBeenCalledWith(

199+

expect.objectContaining({

200+

action: "custom-action",

201+

}),

202+

expect.anything(),

203+

expect.anything(),

204+

);

205+

});

206+207+

it("preloads when the scoped channel plugin is not cheaply available", async () => {

208+

getChannelPluginMock.mockReturnValue(undefined);

209+210+

await runSendAction({ target: "channel:12345" });

211+212+

expect(ensurePluginRegistryLoaded).toHaveBeenCalledWith({

213+

scope: "configured-channels",

214+

onlyChannelIds: ["discord"],

215+

});

216+

});

217+218+

it("keeps target-prefixed Telegram sends from local plugin preload", async () => {

116219

await runSendAction({ channel: undefined, target: "telegram:12345" });

117220221+

expect(ensurePluginRegistryLoaded).not.toHaveBeenCalled();

222+

expect(messageCommandMock).toHaveBeenCalledWith(

223+

expect.objectContaining({

224+

action: "send",

225+

target: "telegram:12345",

226+

message: "hi",

227+

}),

228+

expect.anything(),

229+

expect.anything(),

230+

);

231+

expect(exitMock).toHaveBeenCalledWith(0);

232+

});

233+234+

it("keeps explicit Telegram sends on the normal command path without local plugin preload", async () => {

235+

await runSendAction({

236+

channel: "telegram",

237+

account: "default",

238+

target: "@ops",

239+

media: "./diagram.png",

240+

presentation: '{"blocks":[{"type":"buttons","buttons":[{"label":"OK","value":"ok"}]}]}',

241+

delivery: '{"pin":true}',

242+

forceDocument: true,

243+

});

244+245+

expect(ensurePluginRegistryLoaded).not.toHaveBeenCalled();

246+

expect(messageCommandMock).toHaveBeenCalledWith(

247+

expect.objectContaining({

248+

action: "send",

249+

channel: "telegram",

250+

accountId: "default",

251+

target: "@ops",

252+

message: "hi",

253+

media: "./diagram.png",

254+

presentation: '{"blocks":[{"type":"buttons","buttons":[{"label":"OK","value":"ok"}]}]}',

255+

delivery: '{"pin":true}',

256+

forceDocument: true,

257+

}),

258+

expect.anything(),

259+

expect.anything(),

260+

);

261+

expectNoAccountFieldInPassedOptions();

262+

expect(exitMock).toHaveBeenCalledWith(0);

263+

});

264+265+

it("keeps Telegram dry-runs on the local preload path for local validation", async () => {

266+

await runSendAction({

267+

channel: "telegram",

268+

target: "@ops",

269+

dryRun: true,

270+

});

271+118272

expect(ensurePluginRegistryLoaded).toHaveBeenCalledWith({

119273

scope: "configured-channels",

120274

onlyChannelIds: ["telegram"],

121275

});

276+

expect(messageCommandMock).toHaveBeenCalled();

122277

});

123278124279

it("loads configured channel plugins for mixed broadcast target prefixes", async () => {