fix(memory-core): quiet request-scoped fallback · openclaw/openclaw@c9e6f37
steipete
·
2026-04-27
·
via Recent Commits to openclaw:main
File tree
extensions/memory-core/src
| Original file line number | Diff line number | Diff line change |
|---|
@@ -18,7 +18,7 @@ Docs: https://docs.openclaw.ai
|
18 | 18 | |
19 | 19 | ### Fixes |
20 | 20 | |
21 | | -- Memory-core/dreaming: skip subagent session cleanup after request-scoped narrative fallback and remove duplicate phase-level cleanup, preventing false cleanup warnings when no subagent session was created. Fixes #67152. Thanks @jsompis. |
| 21 | +- Memory-core/dreaming: treat request-scoped narrative fallback as expected, skip session cleanup when no subagent run was created, and remove duplicate phase-level cleanup so fallback no longer emits warning noise. Fixes #67152. Thanks @jsompis. |
22 | 22 | - Config/doctor: stop masking unknown-key validation diagnostics such as `agents.defaults.llm`, and have `openclaw doctor --fix` remove the retired `agents.defaults.llm` timeout block. Thanks @aidiffuser. |
23 | 23 | - CLI/plugins: preserve unversioned ClawHub install specs so `plugins update` can follow newer ClawHub releases instead of pinning to the initially resolved version. Fixes #63010; supersedes #58426. Thanks @kangsen1234 and @robinspt. |
24 | 24 | - Memory-core/subagents: tag plugin-created subagent sessions with their plugin owner so dreaming narrative cleanup can delete its own ephemeral sessions without granting broad admin session deletion. Fixes #72712. Thanks @BSG2000. |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -719,7 +719,8 @@ describe("generateAndAppendDreamNarrative", () => {
|
719 | 719 | |
720 | 720 | const content = await fs.readFile(path.join(workspaceDir, "DREAMS.md"), "utf-8"); |
721 | 721 | expect(content).toContain("API endpoints need authentication"); |
722 | | -expect(logger.warn).toHaveBeenCalledWith(expect.stringContaining("request-scoped")); |
| 722 | +expect(logger.info).toHaveBeenCalledWith(expect.stringContaining("request-scoped")); |
| 723 | +expect(logger.warn).not.toHaveBeenCalledWith(expect.stringContaining("request-scoped")); |
723 | 724 | expect(logger.warn).not.toHaveBeenCalledWith(expect.stringContaining(workspaceDir)); |
724 | 725 | expect(logger.warn).not.toHaveBeenCalledWith( |
725 | 726 | expect.stringContaining("narrative session cleanup failed"), |
@@ -749,6 +750,8 @@ describe("generateAndAppendDreamNarrative", () => {
|
749 | 750 | |
750 | 751 | const content = await fs.readFile(path.join(workspaceDir, "DREAMS.md"), "utf-8"); |
751 | 752 | expect(content).toContain("A durable candidate surfaced."); |
| 753 | +expect(logger.info).toHaveBeenCalledWith(expect.stringContaining("request-scoped")); |
| 754 | +expect(logger.warn).not.toHaveBeenCalledWith(expect.stringContaining("request-scoped")); |
752 | 755 | expect(subagent.deleteSession).not.toHaveBeenCalled(); |
753 | 756 | }); |
754 | 757 | |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -171,7 +171,7 @@ async function startNarrativeRunOrFallback(params: {
|
171 | 171 | nowMs: params.nowMs, |
172 | 172 | timezone: params.timezone, |
173 | 173 | }); |
174 | | -params.logger.warn( |
| 174 | +params.logger.info( |
175 | 175 | `memory-core: narrative generation used fallback for ${params.data.phase} phase because subagent runtime is request-scoped.`, |
176 | 176 | ); |
177 | 177 | } catch (fallbackErr) { |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -319,6 +319,8 @@ describe("memory-core dreaming phases", () => {
|
319 | 319 | const dreams = await fs.readFile(path.join(workspaceDir, "DREAMS.md"), "utf-8"); |
320 | 320 | expect(dreams).toContain("Move backups to S3 Glacier."); |
321 | 321 | expect(logger.error).not.toHaveBeenCalled(); |
| 322 | +expect(logger.info).toHaveBeenCalledWith(expect.stringContaining("request-scoped")); |
| 323 | +expect(logger.warn).not.toHaveBeenCalledWith(expect.stringContaining("request-scoped")); |
322 | 324 | expect(logger.warn).not.toHaveBeenCalledWith( |
323 | 325 | expect.stringContaining("narrative session cleanup failed"), |
324 | 326 | ); |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。