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

推荐订阅源

腾讯CDC
aimingoo的专栏
aimingoo的专栏
S
SegmentFault 最新的问题
A
About on SuperTechFans
Engineering at Meta
Engineering at Meta
宝玉的分享
宝玉的分享
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
博客园 - 叶小钗
WordPress大学
WordPress大学
N
Netflix TechBlog - Medium
MyScale Blog
MyScale Blog
Stack Overflow Blog
Stack Overflow Blog
Microsoft Azure Blog
Microsoft Azure Blog
博客园 - 聂微东
M
MIT News - Artificial intelligence
F
Fortinet All Blogs
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
Blog — PlanetScale
Blog — PlanetScale
T
Tailwind CSS Blog
Recent Announcements
Recent Announcements
Jina AI
Jina AI
大猫的无限游戏
大猫的无限游戏
Apple Machine Learning Research
Apple Machine Learning Research
J
Java Code Geeks

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(agents): drop malformed reasoning before orphan close...
steipete · 2026-04-27 · via Recent Commits to openclaw:main
Original file line numberDiff line numberDiff line change

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

2121

### Fixes

2222
2323

- Control UI/Gateway: preserve WebChat client version labels across localhost, 127.0.0.1, and IPv6 loopback aliases on the same port, avoiding misleading `vcontrol-ui` connection logs while investigating duplicate-message reports. Refs #72753 and #72742. Thanks @LumenFromTheFuture and @allesgutefy.

24+

- Agents/reasoning: treat orphan closing reasoning tags with following answer text as a privacy boundary across delivery, history, streaming, and Control UI sanitizers so malformed local-model output cannot leak chain-of-thought text. Fixes #67092. Thanks @AnildoSilva.

2425

- Memory-core: run one-shot memory CLI commands through transient builtin and QMD managers so `memory index`, `memory status --index`, and `memory search` no longer start long-lived file watchers that can hit macOS `EMFILE` limits. Fixes #59101; carries forward #49851. Thanks @mbear469210-coder and @maoyuanxue.

2526

- Memory-core: re-resolve the active runtime config whenever `memory_search` or `memory_get` executes, so provider changes made by `config.patch` stop leaving stale embedding backends behind in existing tool instances. Fixes #61098. Thanks @BradGroux and @Linux2010.

2627

- WebChat: keep bare `/new` and `/reset` startup instructions out of visible chat history while preserving `/reset <note>` as user-visible transcript text. Fixes #72369. Thanks @collynes and @haishmg.

Original file line numberDiff line numberDiff line change

@@ -630,6 +630,18 @@ describe("subscribeEmbeddedPiSession", () => {

630630

expect(payloads[1]?.delta).toBe(" world");

631631

});

632632
633+

it("drops malformed streamed reasoning before orphan close tags when final text follows", () => {

634+

const { emit, onAgentEvent } = createAgentEventHarness();

635+
636+

emit({ type: "message_start", message: { role: "assistant" } });

637+

emitAssistantTextDelta(emit, "private chain of thought </think> Visible answer");

638+
639+

const payloads = extractAgentEventPayloads(onAgentEvent.mock.calls);

640+

expect(payloads).toHaveLength(1);

641+

expect(payloads[0]?.text).toBe("Visible answer");

642+

expect(payloads[0]?.delta).toBe("Visible answer");

643+

});

644+
633645

it("emits agent events on message_end for non-streaming assistant text", () => {

634646

const { session, emit } = createStubSessionHarness();

635647
Original file line numberDiff line numberDiff line change

@@ -9,6 +9,7 @@ import { createSubsystemLogger } from "../logging/subsystem.js";

99

import type { InlineCodeState } from "../markdown/code-spans.js";

1010

import { buildCodeSpanIndex, createInlineCodeState } from "../markdown/code-spans.js";

1111

import { normalizeOptionalString } from "../shared/string-coerce.js";

12+

import { hasOrphanReasoningCloseBoundary } from "../shared/text/reasoning-tags.js";

1213

import { EmbeddedBlockChunker } from "./pi-embedded-block-chunker.js";

1314

import {

1415

isMessagingToolDuplicateNormalized,

@@ -531,10 +532,22 @@ export function subscribeEmbeddedPiSession(params: SubscribeEmbeddedPiSessionPar

531532

if (codeSpans.isInside(idx)) {

532533

continue;

533534

}

535+

const isClose = match[1] === "/";

534536

if (!inThinking) {

537+

if (isClose) {

538+

const afterIndex = idx + match[0].length;

539+

const before = text.slice(lastIndex, idx);

540+

const after = text.slice(afterIndex);

541+

if (hasOrphanReasoningCloseBoundary({ before, after })) {

542+

processed = "";

543+

} else {

544+

processed += before;

545+

}

546+

lastIndex = afterIndex;

547+

continue;

548+

}

535549

processed += text.slice(lastIndex, idx);

536550

}

537-

const isClose = match[1] === "/";

538551

inThinking = !isClose;

539552

lastIndex = idx + match[0].length;

540553

}

Original file line numberDiff line numberDiff line change

@@ -528,6 +528,12 @@ describe("sanitizeAssistantVisibleText", () => {

528528
529529

expect(sanitizeAssistantVisibleText(input)).toBe("Visible answer");

530530

});

531+
532+

it("drops malformed reasoning before orphan close tags when final text follows", () => {

533+

expect(sanitizeAssistantVisibleText("private chain of thought </think> Visible answer")).toBe(

534+

"Visible answer",

535+

);

536+

});

531537

});

532538
533539

describe("sanitizeAssistantVisibleTextWithProfile", () => {

@@ -537,6 +543,15 @@ describe("sanitizeAssistantVisibleTextWithProfile", () => {

537543

expect(sanitizeAssistantVisibleTextWithProfile(input, "history")).toBe("Hi there");

538544

});

539545
546+

it("uses the history profile to drop malformed reasoning before orphan close tags", () => {

547+

expect(

548+

sanitizeAssistantVisibleTextWithProfile(

549+

"private chain of thought </think> Visible answer",

550+

"history",

551+

),

552+

).toBe(" Visible answer");

553+

});

554+
540555

it("uses the internal-scaffolding profile to preserve downgraded tool text behavior", () => {

541556

const input = [

542557

"[Tool Call: read (ID: toolu_1)]",

Original file line numberDiff line numberDiff line change

@@ -120,6 +120,14 @@ describe("stripReasoningTagsFromText", () => {

120120

input: "You can start with <think and then close with </think>",

121121

expected: "You can start with <think and then close with",

122122

},

123+

{

124+

input: "Internal reasoning </think> final answer",

125+

expected: "final answer",

126+

},

127+

{

128+

input: "Use `<think>` to open and `</think>` to close. Final sentence.",

129+

expected: "Use `<think>` to open and `</think>` to close. Final sentence.",

130+

},

123131

{

124132

input: "A < think >content< /think > B",

125133

expected: "A B",

@@ -168,7 +176,7 @@ describe("stripReasoningTagsFromText", () => {

168176

it.each([

169177

{

170178

input: "<think>outer <think>inner</think> still outer</think>visible",

171-

expected: "still outervisible",

179+

expected: "visible",

172180

},

173181

{

174182

input: "A<final>1</final>B<final>2</final>C",

Original file line numberDiff line numberDiff line change

@@ -17,6 +17,13 @@ function applyTrim(value: string, mode: ReasoningTagTrim): string {

1717

return value.trim();

1818

}

1919
20+

export function hasOrphanReasoningCloseBoundary(params: {

21+

before: string;

22+

after: string;

23+

}): boolean {

24+

return params.before.trim().length > 0 && params.after.trim().length > 0;

25+

}

26+
2027

export function stripReasoningTagsFromText(

2128

text: string,

2229

options?: {

@@ -63,7 +70,7 @@ export function stripReasoningTagsFromText(

6370

THINKING_TAG_RE.lastIndex = 0;

6471

let result = "";

6572

let lastIndex = 0;

66-

let inThinking = false;

73+

let thinkingDepth = 0;

6774
6875

for (const match of cleaned.matchAll(THINKING_TAG_RE)) {

6976

const idx = match.index ?? 0;

@@ -73,19 +80,31 @@ export function stripReasoningTagsFromText(

7380

continue;

7481

}

7582
76-

if (!inThinking) {

77-

result += cleaned.slice(lastIndex, idx);

78-

if (!isClose) {

79-

inThinking = true;

83+

if (thinkingDepth === 0) {

84+

if (isClose) {

85+

const afterIndex = idx + match[0].length;

86+

const before = cleaned.slice(lastIndex, idx);

87+

const after = cleaned.slice(afterIndex);

88+

if (hasOrphanReasoningCloseBoundary({ before, after })) {

89+

result = "";

90+

} else {

91+

result += before;

92+

}

93+

lastIndex = afterIndex;

94+

continue;

8095

}

96+

result += cleaned.slice(lastIndex, idx);

97+

thinkingDepth = 1;

8198

} else if (isClose) {

82-

inThinking = false;

99+

thinkingDepth -= 1;

100+

} else {

101+

thinkingDepth += 1;

83102

}

84103
85104

lastIndex = idx + match[0].length;

86105

}

87106
88-

if (!inThinking || mode === "preserve") {

107+

if (thinkingDepth === 0 || mode === "preserve") {

89108

result += cleaned.slice(lastIndex);

90109

}

91110
Original file line numberDiff line numberDiff line change

@@ -53,6 +53,12 @@ describe("stripThinkingTags", () => {

5353

expect(stripThinkingTags("Hello\n</think>")).toBe("Hello\n");

5454

});

5555
56+

it("drops malformed reasoning before orphan close tags when final text follows", () => {

57+

expect(stripThinkingTags("private chain of thought </think> Visible answer")).toBe(

58+

"Visible answer",

59+

);

60+

});

61+
5662

it("returns original text when no tags exist", () => {

5763

expect(stripThinkingTags("Hello")).toBe("Hello");

5864

});