From d3ddf893c27d817c0fda5d1ba276d2345cfe0a9e Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:39:48 +0000 Subject: [PATCH 01/13] test: remove redundant store-rotation integration prune case --- src/config/sessions/store.pruning.e2e.test.ts | 36 ------------------- 1 file changed, 36 deletions(-) diff --git a/src/config/sessions/store.pruning.e2e.test.ts b/src/config/sessions/store.pruning.e2e.test.ts index 302d9fefd99..92cd0da77fd 100644 --- a/src/config/sessions/store.pruning.e2e.test.ts +++ b/src/config/sessions/store.pruning.e2e.test.ts @@ -86,42 +86,6 @@ describe("Integration: saveSessionStore with pruning", () => { expect(loaded.fresh).toBeDefined(); }); - it("saveSessionStore rotates file when over size limit and creates .bak", async () => { - mockLoadConfig.mockReturnValue({ - session: { - maintenance: { - mode: "enforce", - pruneAfter: "30d", - maxEntries: 500, - rotateBytes: "100b", - }, - }, - }); - - const now = Date.now(); - const largeStore: Record = {}; - for (let i = 0; i < 50; i++) { - largeStore[`agent:main:session-${crypto.randomUUID()}`] = makeEntry(now - i * 1000); - } - await fs.mkdir(path.dirname(storePath), { recursive: true }); - await fs.writeFile(storePath, JSON.stringify(largeStore, null, 2), "utf-8"); - - const statBefore = await fs.stat(storePath); - expect(statBefore.size).toBeGreaterThan(100); - - const smallStore: Record = { - only: makeEntry(now), - }; - await saveSessionStore(storePath, smallStore); - - const files = await fs.readdir(testDir); - const bakFiles = files.filter((f) => f.startsWith("sessions.json.bak.")); - expect(bakFiles.length).toBeGreaterThanOrEqual(1); - - const loaded = loadSessionStore(storePath); - expect(loaded.only).toBeDefined(); - }); - it("saveSessionStore skips enforcement when maintenance mode is warn", async () => { mockLoadConfig.mockReturnValue({ session: { From 9b351fcbd8217af6e6db1c2ae4dc8ccc8a6d950c Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:41:50 +0000 Subject: [PATCH 02/13] test: remove duplicate whatsapp group heartbeat target case --- ...rtbeat-runner.returns-default-unset.test.ts | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/src/infra/heartbeat-runner.returns-default-unset.test.ts b/src/infra/heartbeat-runner.returns-default-unset.test.ts index 3b492f281b3..3471e77f0f4 100644 --- a/src/infra/heartbeat-runner.returns-default-unset.test.ts +++ b/src/infra/heartbeat-runner.returns-default-unset.test.ts @@ -287,24 +287,6 @@ describe("resolveHeartbeatDeliveryTarget", () => { }); }); - it("keeps WhatsApp group targets even with allowFrom set", () => { - const cfg: OpenClawConfig = { - channels: { whatsapp: { allowFrom: ["+1555"] } }, - }; - const entry = { - ...baseEntry, - lastChannel: "whatsapp" as const, - lastTo: "120363401234567890@g.us", - }; - expect(resolveHeartbeatDeliveryTarget({ cfg, entry })).toEqual({ - channel: "whatsapp", - to: "120363401234567890@g.us", - accountId: undefined, - lastChannel: "whatsapp", - lastAccountId: undefined, - }); - }); - it("normalizes prefixed WhatsApp group targets for heartbeat delivery", () => { const cfg: OpenClawConfig = { channels: { whatsapp: { allowFrom: ["+1555"] } }, From 7a6928712bb697427b9c2547d5f454dde75f82e7 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:43:01 +0000 Subject: [PATCH 03/13] test: remove redundant explicit telegram heartbeat target case --- .../heartbeat-runner.returns-default-unset.test.ts | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/src/infra/heartbeat-runner.returns-default-unset.test.ts b/src/infra/heartbeat-runner.returns-default-unset.test.ts index 3471e77f0f4..a7fc891b754 100644 --- a/src/infra/heartbeat-runner.returns-default-unset.test.ts +++ b/src/infra/heartbeat-runner.returns-default-unset.test.ts @@ -305,19 +305,6 @@ describe("resolveHeartbeatDeliveryTarget", () => { }); }); - it("keeps explicit telegram targets", () => { - const cfg: OpenClawConfig = { - agents: { defaults: { heartbeat: { target: "telegram", to: "123" } } }, - }; - expect(resolveHeartbeatDeliveryTarget({ cfg, entry: baseEntry })).toEqual({ - channel: "telegram", - to: "123", - accountId: undefined, - lastChannel: undefined, - lastAccountId: undefined, - }); - }); - it("uses explicit heartbeat accountId when provided", () => { const cfg: OpenClawConfig = { agents: { From 3fb4a7eb531e89779fa5fc99c635e2a9ca3a8793 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:44:16 +0000 Subject: [PATCH 04/13] test: remove duplicate hook-wake heartbeat empty-file case --- ...tbeat-runner.returns-default-unset.test.ts | 67 ------------------- 1 file changed, 67 deletions(-) diff --git a/src/infra/heartbeat-runner.returns-default-unset.test.ts b/src/infra/heartbeat-runner.returns-default-unset.test.ts index a7fc891b754..f8a76d952c0 100644 --- a/src/infra/heartbeat-runner.returns-default-unset.test.ts +++ b/src/infra/heartbeat-runner.returns-default-unset.test.ts @@ -1093,73 +1093,6 @@ describe("runHeartbeatOnce", () => { } }); - it("does not skip hook-triggered heartbeat when HEARTBEAT.md is effectively empty", async () => { - const tmpDir = await createCaseDir("openclaw-hb"); - const storePath = path.join(tmpDir, "sessions.json"); - const workspaceDir = path.join(tmpDir, "workspace"); - const replySpy = vi.spyOn(replyModule, "getReplyFromConfig"); - try { - await fs.mkdir(workspaceDir, { recursive: true }); - await fs.writeFile( - path.join(workspaceDir, "HEARTBEAT.md"), - "# HEARTBEAT.md\n\n## Tasks\n\n", - "utf-8", - ); - - const cfg: OpenClawConfig = { - agents: { - defaults: { - workspace: workspaceDir, - heartbeat: { every: "5m", target: "whatsapp" }, - }, - }, - channels: { whatsapp: { allowFrom: ["*"] } }, - session: { store: storePath }, - }; - const sessionKey = resolveMainSessionKey(cfg); - - await fs.writeFile( - storePath, - JSON.stringify( - { - [sessionKey]: { - sessionId: "sid", - updatedAt: Date.now(), - lastChannel: "whatsapp", - lastTo: "+1555", - }, - }, - null, - 2, - ), - ); - - replySpy.mockResolvedValue({ text: "hook event processed" }); - const sendWhatsApp = vi.fn().mockResolvedValue({ - messageId: "m1", - toJid: "jid", - }); - - const res = await runHeartbeatOnce({ - cfg, - reason: "hook:wake", - deps: { - sendWhatsApp, - getQueueSize: () => 0, - nowMs: () => 0, - webAuthExists: async () => true, - hasActiveWebListener: () => true, - }, - }); - - expect(res.status).toBe("ran"); - expect(replySpy).toHaveBeenCalled(); - expect(sendWhatsApp).toHaveBeenCalledTimes(1); - } finally { - replySpy.mockRestore(); - } - }); - it("runs heartbeat when HEARTBEAT.md has actionable content", async () => { const tmpDir = await createCaseDir("openclaw-hb"); const storePath = path.join(tmpDir, "sessions.json"); From 82fa526bb06792d19b03db4a794e8fa1ee11dc9c Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:45:51 +0000 Subject: [PATCH 05/13] test: remove duplicate undefined template-variable guard case --- src/auto-reply/reply/reply-utils.test.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index 743568b38c1..68b25fc2349 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -811,10 +811,6 @@ describe("extractShortModelName", () => { }); describe("hasTemplateVariables", () => { - it("returns false for undefined", () => { - expect(hasTemplateVariables(undefined)).toBe(false); - }); - it("returns false for empty string", () => { expect(hasTemplateVariables("")).toBe(false); }); From cd04385f9ffe572763ce9e6f122ea4a36688781b Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:46:44 +0000 Subject: [PATCH 06/13] test: remove redundant provider-plus-date model-name case --- src/auto-reply/reply/reply-utils.test.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index 68b25fc2349..7644c994ea4 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -799,10 +799,6 @@ describe("extractShortModelName", () => { expect(extractShortModelName("claude-opus-4-5")).toBe("claude-opus-4-5"); }); - it("handles full path with provider and date suffix", () => { - expect(extractShortModelName("anthropic/claude-opus-4-5-20251101")).toBe("claude-opus-4-5"); - }); - it("preserves version numbers that look like dates but are not", () => { // Date suffix must be exactly 8 digits at the end expect(extractShortModelName("model-v1234567")).toBe("model-v1234567"); From 523193a91fe0ab30ddc9a81b475871667e4e02f6 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:47:45 +0000 Subject: [PATCH 07/13] test: remove duplicate static template-variable false case --- src/auto-reply/reply/reply-utils.test.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index 7644c994ea4..5af3b495639 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -811,10 +811,6 @@ describe("hasTemplateVariables", () => { expect(hasTemplateVariables("")).toBe(false); }); - it("returns false for static prefix", () => { - expect(hasTemplateVariables("[Claude]")).toBe(false); - }); - it("returns true when template variables present", () => { expect(hasTemplateVariables("[{model}]")).toBe(true); expect(hasTemplateVariables("{provider}")).toBe(true); From deef9f91bf663b2be24a24cc5ec28cfdb12d018b Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:48:51 +0000 Subject: [PATCH 08/13] test: remove duplicate multi-variable template check case --- src/auto-reply/reply/reply-utils.test.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index 5af3b495639..8518212bf4d 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -817,10 +817,6 @@ describe("hasTemplateVariables", () => { expect(hasTemplateVariables("prefix {thinkingLevel} suffix")).toBe(true); }); - it("returns true for multiple variables", () => { - expect(hasTemplateVariables("[{model} | {provider}]")).toBe(true); - }); - it("handles consecutive calls correctly (regex lastIndex reset)", () => { // First call expect(hasTemplateVariables("[{model}]")).toBe(true); From c4297a8d604a1e163eb090ffab2c227973021564 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:49:58 +0000 Subject: [PATCH 09/13] test: remove redundant no-provider short-model case --- src/auto-reply/reply/reply-utils.test.ts | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index 8518212bf4d..4c5a2b8b6e1 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -794,11 +794,6 @@ describe("extractShortModelName", () => { expect(extractShortModelName("claude-sonnet-latest")).toBe("claude-sonnet"); }); - it("handles model without provider", () => { - expect(extractShortModelName("gpt-5.2")).toBe("gpt-5.2"); - expect(extractShortModelName("claude-opus-4-5")).toBe("claude-opus-4-5"); - }); - it("preserves version numbers that look like dates but are not", () => { // Date suffix must be exactly 8 digits at the end expect(extractShortModelName("model-v1234567")).toBe("model-v1234567"); From dbcdcc5d1930bff89a8d7200b46a9c63e084b88a Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:51:09 +0000 Subject: [PATCH 10/13] test: remove duplicate positive template-variable detection case --- src/auto-reply/reply/reply-utils.test.ts | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index 4c5a2b8b6e1..3d00a1e36ad 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -806,12 +806,6 @@ describe("hasTemplateVariables", () => { expect(hasTemplateVariables("")).toBe(false); }); - it("returns true when template variables present", () => { - expect(hasTemplateVariables("[{model}]")).toBe(true); - expect(hasTemplateVariables("{provider}")).toBe(true); - expect(hasTemplateVariables("prefix {thinkingLevel} suffix")).toBe(true); - }); - it("handles consecutive calls correctly (regex lastIndex reset)", () => { // First call expect(hasTemplateVariables("[{model}]")).toBe(true); From 2d034730720308841b4c99064eab230e313565f8 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:52:16 +0000 Subject: [PATCH 11/13] test: trim duplicate provider-prefix assertion in short-model tests --- src/auto-reply/reply/reply-utils.test.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index 3d00a1e36ad..b608be55f82 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -780,7 +780,6 @@ describe("resolveResponsePrefixTemplate", () => { describe("extractShortModelName", () => { it("strips provider prefix", () => { expect(extractShortModelName("openai/gpt-5.2")).toBe("gpt-5.2"); - expect(extractShortModelName("anthropic/claude-opus-4-5")).toBe("claude-opus-4-5"); expect(extractShortModelName("openai-codex/gpt-5.2-codex")).toBe("gpt-5.2-codex"); }); From 9f6462bd56b09a3739029e5b5cf459602ef7740c Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:54:05 +0000 Subject: [PATCH 12/13] test: trim duplicate latest-suffix assertion variant --- src/auto-reply/reply/reply-utils.test.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index b608be55f82..583c1383726 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -790,7 +790,6 @@ describe("extractShortModelName", () => { it("strips -latest suffix", () => { expect(extractShortModelName("gpt-5.2-latest")).toBe("gpt-5.2"); - expect(extractShortModelName("claude-sonnet-latest")).toBe("claude-sonnet"); }); it("preserves version numbers that look like dates but are not", () => { From 3eb9c2105c09f984f313adbedb5b0238e2d91248 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Mon, 16 Feb 2026 09:54:56 +0000 Subject: [PATCH 13/13] test: remove duplicate date-suffix assertion variant --- src/auto-reply/reply/reply-utils.test.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/src/auto-reply/reply/reply-utils.test.ts b/src/auto-reply/reply/reply-utils.test.ts index 583c1383726..bff7e300814 100644 --- a/src/auto-reply/reply/reply-utils.test.ts +++ b/src/auto-reply/reply/reply-utils.test.ts @@ -785,7 +785,6 @@ describe("extractShortModelName", () => { it("strips date suffix", () => { expect(extractShortModelName("claude-opus-4-5-20251101")).toBe("claude-opus-4-5"); - expect(extractShortModelName("gpt-5.2-20250115")).toBe("gpt-5.2"); }); it("strips -latest suffix", () => {