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

推荐订阅源

D
DataBreaches.Net
GbyAI
GbyAI
aimingoo的专栏
aimingoo的专栏
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
月光博客
月光博客
大猫的无限游戏
大猫的无限游戏
M
MIT News - Artificial intelligence
腾讯CDC
博客园 - Franky
Engineering at Meta
Engineering at Meta
C
Check Point Blog
T
The Blog of Author Tim Ferriss
有赞技术团队
有赞技术团队
Microsoft Azure Blog
Microsoft Azure Blog
MyScale Blog
MyScale Blog
I
InfoQ
Blog — PlanetScale
Blog — PlanetScale
P
Proofpoint News Feed
The GitHub Blog
The GitHub Blog
N
Netflix TechBlog - Medium
Last Week in AI
Last Week in AI
S
SegmentFault 最新的问题
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
WordPress大学
WordPress大学

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(discord): restore bare numeric channel sends (#86571)...
joshavant · 2026-05-26 · via Recent Commits to openclaw:main
Original file line numberDiff line numberDiff line change

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

6060

- xAI/LM Studio: avoid buffering ordinary bracketed or `final` prose until stream completion while watching for plain-text tool-call fallbacks.

6161

- Doctor: warn and continue when the cron job store exists but cannot be read so later health checks still run. Fixes #86102. (#86384) Thanks @1052326311.

6262

- Discord: suppress a bot's previous reply body and referenced media from prompt context when a user replies to that bot message, while keeping reply metadata for routing. (#86238) Thanks @fuller-stack-dev.

63+

- Discord: restore bare numeric channel IDs for outbound message-tool sends while keeping explicit DM targets unambiguous. (#86571) Thanks @joshavant.

6364

- Docker E2E: avoid rebuilding the Control UI twice while preparing the shared OpenClaw package tarball for package-backed scenario runs.

6465

- Tests: avoid rebuilding the Control UI twice during the installer Docker smoke now that `pnpm build` includes `ui:build`.

6566

- Tests: give QA config mutation RPCs enough native Windows budget to finish gateway config writes and restart settle after hot scenario runs.

Original file line numberDiff line numberDiff line change

@@ -39,4 +39,21 @@ describe("resolveDiscordOutboundSessionRoute", () => {

3939

});

4040

expect(route?.threadId).toBeUndefined();

4141

});

42+
43+

it("treats bare numeric outbound targets as channel routes", () => {

44+

const route = resolveDiscordOutboundSessionRoute({

45+

cfg: {},

46+

agentId: "main",

47+

target: "123",

48+

});

49+
50+

expect(route).toMatchObject({

51+

baseSessionKey: "agent:main:discord:channel:123",

52+

chatType: "channel",

53+

from: "discord:channel:123",

54+

peer: { kind: "channel", id: "123" },

55+

sessionKey: "agent:main:discord:channel:123",

56+

to: "channel:123",

57+

});

58+

});

4259

});

Original file line numberDiff line numberDiff line change

@@ -68,5 +68,5 @@ function resolveDiscordOutboundTargetKindHint(params: {

6868

if (/^(user:|discord:|@|<@!?)/i.test(target)) {

6969

return "user";

7070

}

71-

return undefined;

71+

return "channel";

7272

}

Original file line numberDiff line numberDiff line change

@@ -37,3 +37,13 @@ export async function parseAndResolveRecipient(

3737

);

3838

return { kind: resolved.kind, id: resolved.id };

3939

}

40+
41+

export async function parseAndResolveChannelRecipient(

42+

raw: string,

43+

cfg: OpenClawConfig,

44+

accountId?: string,

45+

): Promise<DiscordRecipient> {

46+

return await parseAndResolveRecipient(raw, cfg, accountId, {

47+

defaultKind: "channel",

48+

});

49+

}

Original file line numberDiff line numberDiff line change

@@ -170,6 +170,34 @@ describe("sendDiscordComponentMessage", () => {

170170

);

171171

});

172172
173+

it("treats bare numeric component edit targets as channels", async () => {

174+

const { rest, patchMock, getMock } = makeDiscordRest();

175+

getMock.mockResolvedValueOnce({

176+

type: ChannelType.GuildText,

177+

id: "273512430271856640",

178+

});

179+

patchMock.mockResolvedValueOnce({ id: "msg1", channel_id: "273512430271856640" });

180+
181+

await editDiscordComponentMessage(

182+

"273512430271856640",

183+

"msg1",

184+

{

185+

text: "Updated picker",

186+

blocks: [{ type: "actions", buttons: [{ label: "Tap" }] }],

187+

},

188+

{

189+

cfg: DISCORD_TEST_CFG,

190+

rest,

191+

token: "t",

192+

sessionKey: "agent:main:discord:channel:273512430271856640",

193+

agentId: "main",

194+

},

195+

);

196+
197+

expect(patchMock).toHaveBeenCalledTimes(1);

198+

expect(readMockCall(patchMock, 0)[0]).toContain("/channels/273512430271856640/messages/msg1");

199+

});

200+
173201

it("registers a prebuilt component message against an edited message id", () => {

174202

registerBuiltDiscordComponentMessage({

175203

messageId: "msg1",

@@ -312,6 +340,36 @@ describe("sendDiscordComponentMessage classic message downgrade", () => {

312340

expect(modals[0]?.fields?.[0]?.label).toBe("Notes");

313341

});

314342
343+

it("treats bare numeric component send targets as channels", async () => {

344+

const { rest, postMock, getMock } = makeDiscordRest();

345+

getMock.mockResolvedValueOnce({

346+

type: ChannelType.GuildText,

347+

id: "273512430271856640",

348+

});

349+

postMock.mockResolvedValueOnce({ id: "msg1", channel_id: "273512430271856640" });

350+
351+

await sendDiscordComponentMessage(

352+

"273512430271856640",

353+

{

354+

text: "report",

355+

modal: {

356+

title: "Feedback",

357+

fields: [{ type: "text", label: "Notes" }],

358+

},

359+

},

360+

{

361+

cfg: DISCORD_TEST_CFG,

362+

rest,

363+

token: "t",

364+

mediaUrl: "https://example.com/report.pdf",

365+

},

366+

);

367+
368+

expect(sendMessageDiscordMock).not.toHaveBeenCalled();

369+

expect(postMock).toHaveBeenCalledTimes(1);

370+

expect(readMockCall(postMock, 0)[0]).toContain("/channels/273512430271856640/messages");

371+

});

372+
315373

it("keeps spoiler file blocks on the component path", async () => {

316374

const { rest, postMock, getMock } = makeDiscordRest();

317375

getMock.mockResolvedValueOnce({

Original file line numberDiff line numberDiff line change

@@ -21,7 +21,7 @@ import {

2121

type MessagePayloadObject,

2222

type RequestClient,

2323

} from "./internal/discord.js";

24-

import { parseAndResolveRecipient } from "./recipient-resolution.js";

24+

import { parseAndResolveChannelRecipient } from "./recipient-resolution.js";

2525

import { loadOutboundMediaFromUrl } from "./runtime-api.js";

2626

import { sendMessageDiscord } from "./send.outbound.js";

2727

import { createDiscordSendResult } from "./send.receipt.js";

@@ -290,7 +290,7 @@ export async function sendDiscordComponentMessage(

290290

const cfg = requireRuntimeConfig(opts.cfg, "Discord component send");

291291

const accountInfo = resolveDiscordAccount({ cfg, accountId: opts.accountId });

292292

const { token, rest, request } = createDiscordClient({ ...opts, cfg });

293-

const recipient = await parseAndResolveRecipient(to, cfg, opts.accountId);

293+

const recipient = await parseAndResolveChannelRecipient(to, cfg, opts.accountId);

294294

const { channelId } = await resolveChannelId(rest, recipient, request);

295295
296296

const channelType = await resolveDiscordChannelType(rest, channelId);

@@ -353,7 +353,7 @@ export async function editDiscordComponentMessage(

353353

const cfg = requireRuntimeConfig(opts.cfg, "Discord component edit");

354354

const accountInfo = resolveDiscordAccount({ cfg, accountId: opts.accountId });

355355

const { token, rest, request } = createDiscordClient({ ...opts, cfg });

356-

const recipient = await parseAndResolveRecipient(to, cfg, opts.accountId);

356+

const recipient = await parseAndResolveChannelRecipient(to, cfg, opts.accountId);

357357

const { channelId } = await resolveChannelId(rest, recipient, request);

358358

const { body, buildResult } = await buildDiscordComponentPayload({

359359

spec,

Original file line numberDiff line numberDiff line change

@@ -11,7 +11,7 @@ import { convertMarkdownTables } from "openclaw/plugin-sdk/text-chunking";

1111

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

1212

import { createChannelMessage, createThread, type RequestClient } from "./internal/discord.js";

1313

import { rewriteDiscordKnownMentions } from "./mentions.js";

14-

import { parseAndResolveRecipient } from "./recipient-resolution.js";

14+

import { parseAndResolveChannelRecipient } from "./recipient-resolution.js";

1515

import { createDiscordSendResult, type DiscordReceiptResultSource } from "./send.receipt.js";

1616

import {

1717

buildDiscordMessageRequest,

@@ -140,7 +140,7 @@ async function resolveDiscordSendTarget(

140140

): Promise<{ rest: RequestClient; request: DiscordClientRequest; channelId: string }> {

141141

const cfg = requireRuntimeConfig(opts.cfg, "Discord send target resolution");

142142

const { rest, request } = createDiscordClient({ ...opts, cfg });

143-

const recipient = await parseAndResolveRecipient(to, cfg, opts.accountId);

143+

const recipient = await parseAndResolveChannelRecipient(to, cfg, opts.accountId);

144144

const { channelId } = await resolveChannelId(rest, recipient, request);

145145

return { rest, request, channelId };

146146

}

@@ -181,7 +181,7 @@ export async function sendMessageDiscord(

181181

mentionAliases: accountInfo.config.mentionAliases,

182182

});

183183

const { token, rest, request } = createDiscordClient({ ...opts, cfg });

184-

const recipient = await parseAndResolveRecipient(to, cfg, opts.accountId);

184+

const recipient = await parseAndResolveChannelRecipient(to, cfg, opts.accountId);

185185

const { channelId } = await resolveChannelId(rest, recipient, request);

186186
187187

// Forum/Media channels reject POST /messages; auto-create a thread post instead.

Original file line numberDiff line numberDiff line change

@@ -469,29 +469,23 @@ describe("sendMessageDiscord", () => {

469469

expect(res.channelId).toBe("chan1");

470470

});

471471
472-

it("rejects bare numeric IDs as ambiguous", async () => {

473-

const { rest } = makeDiscordRest();

474-

await expect(

475-

sendMessageDiscord("273512430271856640", "hello", {

476-

rest,

477-

token: "t",

478-

cfg: DISCORD_TEST_CFG,

479-

}),

480-

).rejects.toThrow(/Ambiguous Discord recipient/);

481-

await expect(

482-

sendMessageDiscord("273512430271856640", "hello", {

483-

rest,

484-

token: "t",

485-

cfg: DISCORD_TEST_CFG,

486-

}),

487-

).rejects.toThrow(/user:273512430271856640/);

488-

await expect(

489-

sendMessageDiscord("273512430271856640", "hello", {

490-

rest,

491-

token: "t",

492-

cfg: DISCORD_TEST_CFG,

493-

}),

494-

).rejects.toThrow(/channel:273512430271856640/);

472+

it("treats bare numeric outbound IDs as channels", async () => {

473+

const { rest, postMock, getMock } = makeDiscordRest();

474+

getMock.mockResolvedValueOnce({ type: ChannelType.GuildText });

475+

postMock.mockResolvedValueOnce({

476+

id: "msg1",

477+

channel_id: "273512430271856640",

478+

});

479+
480+

const result = await sendMessageDiscord("273512430271856640", "hello", {

481+

rest,

482+

token: "t",

483+

cfg: DISCORD_TEST_CFG,

484+

});

485+
486+

expect(result.channelId).toBe("273512430271856640");

487+

expectRestRoute(postMock, 0, Routes.channelMessages("273512430271856640"));

488+

expect(requireRestBody(postMock).content).toBe("hello");

495489

});

496490
497491

it("adds missing permission hints on 50013", async () => {

Original file line numberDiff line numberDiff line change

@@ -0,0 +1,63 @@

1+

import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";

2+

import { makeDiscordRest } from "./send.test-harness.js";

3+
4+

const loadWebMediaRawMock = vi.hoisted(() => vi.fn());

5+

vi.mock("openclaw/plugin-sdk/web-media", () => ({

6+

loadWebMediaRaw: loadWebMediaRawMock,

7+

}));

8+
9+

const voiceMocks = vi.hoisted(() => ({

10+

ensureOggOpus: vi.fn(),

11+

getVoiceMessageMetadata: vi.fn(),

12+

sendDiscordVoiceMessage: vi.fn(),

13+

}));

14+

vi.mock("./voice-message.js", () => voiceMocks);

15+
16+

const DISCORD_TEST_CFG = {

17+

channels: { discord: { token: "t" } },

18+

};

19+
20+

let sendVoiceMessageDiscord: typeof import("./send.voice.js").sendVoiceMessageDiscord;

21+
22+

describe("sendVoiceMessageDiscord", () => {

23+

beforeAll(async () => {

24+

({ sendVoiceMessageDiscord } = await import("./send.voice.js"));

25+

});

26+
27+

beforeEach(() => {

28+

vi.clearAllMocks();

29+

loadWebMediaRawMock.mockResolvedValue({

30+

buffer: Buffer.from("voice"),

31+

fileName: "voice.ogg",

32+

contentType: "audio/ogg",

33+

kind: "audio",

34+

});

35+

voiceMocks.ensureOggOpus.mockImplementation(async (inputPath: string) => ({

36+

path: inputPath,

37+

cleanup: false,

38+

}));

39+

voiceMocks.getVoiceMessageMetadata.mockResolvedValue({ duration_secs: 1, waveform: "" });

40+

voiceMocks.sendDiscordVoiceMessage.mockResolvedValue({

41+

id: "msg1",

42+

channel_id: "273512430271856640",

43+

});

44+

});

45+
46+

it("treats bare numeric voice targets as channels", async () => {

47+

const { rest } = makeDiscordRest();

48+
49+

const result = await sendVoiceMessageDiscord(

50+

"273512430271856640",

51+

"https://example.com/voice.ogg",

52+

{

53+

cfg: DISCORD_TEST_CFG,

54+

rest,

55+

token: "t",

56+

},

57+

);

58+
59+

expect(result.channelId).toBe("273512430271856640");

60+

expect(voiceMocks.sendDiscordVoiceMessage).toHaveBeenCalledTimes(1);

61+

expect(voiceMocks.sendDiscordVoiceMessage.mock.calls[0]?.[1]).toBe("273512430271856640");

62+

});

63+

});

Original file line numberDiff line numberDiff line change

@@ -13,7 +13,7 @@ import { tempWorkspace, resolvePreferredOpenClawTmpDir } from "openclaw/plugin-s

1313

import { loadWebMediaRaw } from "openclaw/plugin-sdk/web-media";

1414

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

1515

import type { RequestClient } from "./internal/discord.js";

16-

import { parseAndResolveRecipient } from "./recipient-resolution.js";

16+

import { parseAndResolveChannelRecipient } from "./recipient-resolution.js";

1717

import { createDiscordSendResult } from "./send.receipt.js";

1818

import { buildDiscordSendError, createDiscordClient, resolveChannelId } from "./send.shared.js";

1919

import type { DiscordSendResult } from "./send.types.js";

@@ -95,7 +95,7 @@ export async function sendVoiceMessageDiscord(

9595

token = client.token;

9696

rest = client.rest;

9797

const request = client.request;

98-

const recipient = await parseAndResolveRecipient(to, cfg, opts.accountId);

98+

const recipient = await parseAndResolveChannelRecipient(to, cfg, opts.accountId);

9999

channelId = (await resolveChannelId(rest, recipient, request)).channelId;

100100
101101

const ogg = await ensureOggOpus(localInputPath);