Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(whatsapp): suppress reasoning/thinking content from WhatsApp delivery

Browse Source 
The deliver callback in process-message.ts was forwarding all payload
kinds (tool, block, final) to WhatsApp. Block payloads contain the
model's reasoning/thinking content, which should only be visible in
the internal web UI. This caused chain-of-thought to leak to end users
as separate WhatsApp messages.

Add an early return for non-final payloads so only the actual response
is delivered to the WhatsApp channel, matching how Telegram already
filters by info.kind === "final".

Fixes #24954
Fixes #24605

Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
SidQin-cyber 2026-02-24 10:13:35 +08:00 committed by Peter Steinberger
parent 3a653082d8
commit 3d22af692c
1 changed files with 15 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
src/web/auto-reply/monitor/process-message.ts
View File

@ -368,6 +368,12 @@ export async function processMessage(params: {
}
},
deliver: async (payload: ReplyPayload, info) => {
if (info.kind !== "final") {
// Only deliver final replies to external messaging channels (WhatsApp).
// Block (reasoning/thinking) and tool updates are meant for the internal
// web UI only; sending them here leaks chain-of-thought to end users.
return;
}
await deliverWebReply({
replyResult: payload,
msg: params.msg,
@ -377,30 +383,23 @@ export async function processMessage(params: {
chunkMode,
replyLogger: params.replyLogger,
connectionId: params.connectionId,
// Tool + block updates are noisy; skip their log lines.
skipLog: info.kind !== "final",
skipLog: false,
tableMode,
});
didSendReply = true;
if (info.kind === "tool") {
params.rememberSentText(payload.text, {});
return;
}
const shouldLog = info.kind === "final" && payload.text ? true : undefined;
const shouldLog = payload.text ? true : undefined;
params.rememberSentText(payload.text, {
combinedBody,
combinedBodySessionKey: params.route.sessionKey,
logVerboseMessage: shouldLog,
});
if (info.kind === "final") {
const fromDisplay =
params.msg.chatType === "group" ? conversationId : (params.msg.from ?? "unknown");
const hasMedia = Boolean(payload.mediaUrl || payload.mediaUrls?.length);
whatsappOutboundLog.info(`Auto-replied to ${fromDisplay}${hasMedia ? " (media)" : ""}`);
if (shouldLogVerbose()) {
const preview = payload.text != null ? elide(payload.text, 400) : "<media>";
whatsappOutboundLog.debug(`Reply body: ${preview}${hasMedia ? " (media)" : ""}`);
}
const fromDisplay =
params.msg.chatType === "group" ? conversationId : (params.msg.from ?? "unknown");
const hasMedia = Boolean(payload.mediaUrl || payload.mediaUrls?.length);
whatsappOutboundLog.info(`Auto-replied to ${fromDisplay}${hasMedia ? " (media)" : ""}`);
if (shouldLogVerbose()) {
const preview = payload.text != null ? elide(payload.text, 400) : "<media>";
whatsappOutboundLog.debug(`Reply body: ${preview}${hasMedia ? " (media)" : ""}`);
}
},
onError: (err, info) => {