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

推荐订阅源

V
Visual Studio Blog
量子位
大猫的无限游戏
大猫的无限游戏
Hugging Face - Blog
Hugging Face - Blog
S
SegmentFault 最新的问题
Blog — PlanetScale
Blog — PlanetScale
月光博客
月光博客
Google DeepMind News
Google DeepMind News
小众软件
小众软件
WordPress大学
WordPress大学
宝玉的分享
宝玉的分享
MongoDB | Blog
MongoDB | Blog
B
Blog RSS Feed
博客园 - Franky
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
B
Blog
博客园 - 聂微东
The GitHub Blog
The GitHub Blog
Recent Announcements
Recent Announcements
Y
Y Combinator Blog
Microsoft Security Blog
Microsoft Security Blog
雷峰网
雷峰网
Jina AI
Jina AI
酷 壳 – CoolShell
酷 壳 – CoolShell

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
Preserve delivered assistant replies in session repair (#...
obviyus · 2026-05-03 · via Recent Commits to openclaw:main

@@ -117,7 +117,7 @@ describe("repairSessionFileIfNeeded", () => {

117117

errorMessage: "transient stream failure",

118118

},

119119

};

120-

// Follow-up so the session doesn't end on assistant (trailing-trim is tested separately).

120+

// Follow-up keeps this case focused on empty error-turn repair.

121121

const followUp = {

122122

type: "message",

123123

id: "msg-3",

@@ -293,7 +293,7 @@ describe("repairSessionFileIfNeeded", () => {

293293

stopReason: "stop",

294294

},

295295

};

296-

// Follow-up so the session doesn't end on assistant (trailing-trim is tested separately).

296+

// Follow-up keeps this case focused on silent-reply preservation.

297297

const followUp = {

298298

type: "message",

299299

id: "msg-3",

@@ -312,7 +312,7 @@ describe("repairSessionFileIfNeeded", () => {

312312

expect(after).toBe(original);

313313

});

314314315-

it("trims trailing assistant messages from the session file", async () => {

315+

it("preserves delivered trailing assistant messages in the session file", async () => {

316316

const { file } = await createTempSessionPath();

317317

const { header, message } = buildSessionHeaderAndMessage();

318318

const assistantEntry = {

@@ -329,19 +329,15 @@ describe("repairSessionFileIfNeeded", () => {

329329

const original = `${JSON.stringify(header)}\n${JSON.stringify(message)}\n${JSON.stringify(assistantEntry)}\n`;

330330

await fs.writeFile(file, original, "utf-8");

331331332-

const debug = vi.fn();

333-

const result = await repairSessionFileIfNeeded({ sessionFile: file, debug });

332+

const result = await repairSessionFileIfNeeded({ sessionFile: file });

334333335-

expect(result.repaired).toBe(true);

336-

expect(result.trimmedTrailingAssistantMessages).toBe(1);

337-

expect(debug.mock.calls[0]?.[0]).toContain("trimmed 1 trailing assistant message(s)");

334+

expect(result.repaired).toBe(false);

338335339-

const repaired = await fs.readFile(file, "utf-8");

340-

const repairedLines = repaired.trim().split("\n");

341-

expect(repairedLines).toHaveLength(2);

336+

const after = await fs.readFile(file, "utf-8");

337+

expect(after).toBe(original);

342338

});

343339344-

it("trims multiple consecutive trailing assistant messages", async () => {

340+

it("preserves multiple consecutive delivered trailing assistant messages", async () => {

345341

const { file } = await createTempSessionPath();

346342

const { header, message } = buildSessionHeaderAndMessage();

347343

const assistantEntry1 = {

@@ -371,12 +367,10 @@ describe("repairSessionFileIfNeeded", () => {

371367372368

const result = await repairSessionFileIfNeeded({ sessionFile: file });

373369374-

expect(result.repaired).toBe(true);

375-

expect(result.trimmedTrailingAssistantMessages).toBe(2);

370+

expect(result.repaired).toBe(false);

376371377-

const repaired = await fs.readFile(file, "utf-8");

378-

const repairedLines = repaired.trim().split("\n");

379-

expect(repairedLines).toHaveLength(2);

372+

const after = await fs.readFile(file, "utf-8");

373+

expect(after).toBe(original);

380374

});

381375382376

it("does not trim non-trailing assistant messages", async () => {

@@ -406,7 +400,6 @@ describe("repairSessionFileIfNeeded", () => {

406400

const result = await repairSessionFileIfNeeded({ sessionFile: file });

407401408402

expect(result.repaired).toBe(false);

409-

expect(result.trimmedTrailingAssistantMessages ?? 0).toBe(0);

410403

});

411404412405

it("preserves trailing assistant messages that contain tool calls", async () => {

@@ -432,12 +425,11 @@ describe("repairSessionFileIfNeeded", () => {

432425

const result = await repairSessionFileIfNeeded({ sessionFile: file });

433426434427

expect(result.repaired).toBe(false);

435-

expect(result.trimmedTrailingAssistantMessages ?? 0).toBe(0);

436428

const after = await fs.readFile(file, "utf-8");

437429

expect(after).toBe(original);

438430

});

439431440-

it("trims non-tool-call assistant but stops at tool-call assistant", async () => {

432+

it("preserves adjacent trailing tool-call and text assistant messages", async () => {

441433

const { file } = await createTempSessionPath();

442434

const { header, message } = buildSessionHeaderAndMessage();

443435

const toolCallAssistant = {

@@ -467,16 +459,13 @@ describe("repairSessionFileIfNeeded", () => {

467459468460

const result = await repairSessionFileIfNeeded({ sessionFile: file });

469461470-

expect(result.repaired).toBe(true);

471-

expect(result.trimmedTrailingAssistantMessages).toBe(1);

462+

expect(result.repaired).toBe(false);

472463473-

const repaired = await fs.readFile(file, "utf-8");

474-

const repairedLines = repaired.trim().split("\n");

475-

expect(repairedLines).toHaveLength(3);

476-

expect(JSON.parse(repairedLines[2]).id).toBe("msg-asst-tc");

464+

const after = await fs.readFile(file, "utf-8");

465+

expect(after).toBe(original);

477466

});

478467479-

it("never trims below the session header", async () => {

468+

it("preserves assistant-only session history after the header", async () => {

480469

const { file } = await createTempSessionPath();

481470

const { header } = buildSessionHeaderAndMessage();

482471

const assistantEntry = {

@@ -495,13 +484,10 @@ describe("repairSessionFileIfNeeded", () => {

495484496485

const result = await repairSessionFileIfNeeded({ sessionFile: file });

497486498-

expect(result.repaired).toBe(true);

499-

expect(result.trimmedTrailingAssistantMessages).toBe(1);

487+

expect(result.repaired).toBe(false);

500488501-

const repaired = await fs.readFile(file, "utf-8");

502-

const repairedLines = repaired.trim().split("\n");

503-

expect(repairedLines).toHaveLength(1);

504-

expect(JSON.parse(repairedLines[0]).type).toBe("session");

489+

const after = await fs.readFile(file, "utf-8");

490+

expect(after).toBe(original);

505491

});

506492507493

it("is a no-op on a session that was already repaired", async () => {

@@ -522,7 +508,7 @@ describe("repairSessionFileIfNeeded", () => {

522508

stopReason: "error",

523509

},

524510

};

525-

// Follow-up so the session doesn't end on assistant (trailing-trim is tested separately).

511+

// Follow-up keeps this case focused on idempotent empty error-turn repair.

526512

const followUp = {

527513

type: "message",

528514

id: "msg-3",