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

fix(matrix): skip reasoning-only messages in reply delivery

Browse Source 
When `includeReasoning` is active (or `reasoningLevel` falls back to the
model default), the agent emits reasoning blocks as separate reply
payloads prefixed with "Reasoning:\n".  Matrix has no dedicated reasoning
lane, so these internal thinking traces leak into the chat as regular
user-visible messages.

Filter out pure-reasoning payloads (those starting with "Reasoning:\n" or
a `<thinking>` tag) before delivery so internal reasoning never reaches
the Matrix room.

Fixes #24411

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
justinhuangcode 2026-02-23 17:41:12 +00:00 committed by Peter Steinberger
parent 5ac70b36a4
commit 1298bd4e1b
1 changed files with 24 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
extensions/matrix/src/matrix/monitor/replies.ts
View File

@ -41,6 +41,11 @@ export async function deliverMatrixReplies(params: {
params.runtime.error?.("matrix reply missing text/media"); params.runtime.error?.("matrix reply missing text/media");
continue; continue;
} }
// Skip pure reasoning messages so internal thinking traces are never delivered.
if (reply.text && isReasoningOnlyMessage(reply.text)) {
logVerbose("matrix reply is reasoning-only; skipping");
continue;
}
const replyToIdRaw = reply.replyToId?.trim(); const replyToIdRaw = reply.replyToId?.trim();
const replyToId = params.threadId || params.replyToMode === "off" ? undefined : replyToIdRaw; const replyToId = params.threadId || params.replyToMode === "off" ? undefined : replyToIdRaw;
const rawText = reply.text ?? ""; const rawText = reply.text ?? "";
@ -98,3 +103,22 @@ export async function deliverMatrixReplies(params: {
} }
} }
} }
const REASONING_PREFIX = "Reasoning:\n";
const THINKING_TAG_RE = /^\s*<\s*(?:think(?:ing)?|thought|antthinking)\b/i;
/**
* Detect messages that contain only reasoning/thinking content and no user-facing answer.
* These are emitted by the agent when `includeReasoning` is active but should not
* be forwarded to channels that do not support a dedicated reasoning lane.
*/
function isReasoningOnlyMessage(text: string): boolean {
const trimmed = text.trim();
if (trimmed.startsWith(REASONING_PREFIX)) {
return true;
}
if (THINKING_TAG_RE.test(trimmed)) {
return true;
}
return false;
}