From d0a5dd3ea5a3504d7aafff9cc56ea47bb0624c6d Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 9 May 2026 06:17:39 +0100 Subject: [PATCH] Revert "test: use sqlite locators in command session fixtures" This reverts commit dbdee9f8f2d71d126655988f6c3f542777bd8b0a. --- .../reply/agent-runner.misc.runreplyagent.test.ts | 2 +- src/auto-reply/reply/commands-core.test.ts | 8 ++++---- src/auto-reply/reply/commands-plugin.test.ts | 6 +++--- src/auto-reply/reply/commands-session-usage.test.ts | 6 +++--- src/auto-reply/reply/queue/state.test.ts | 2 +- 5 files changed, 12 insertions(+), 12 deletions(-) diff --git a/src/auto-reply/reply/agent-runner.misc.runreplyagent.test.ts b/src/auto-reply/reply/agent-runner.misc.runreplyagent.test.ts index 87793c67198..f98b21c2e6c 100644 --- a/src/auto-reply/reply/agent-runner.misc.runreplyagent.test.ts +++ b/src/auto-reply/reply/agent-runner.misc.runreplyagent.test.ts @@ -331,7 +331,7 @@ describe("runReplyAgent auto-compaction token update", () => { sessionId: "session", sessionKey: "main", messageProvider: "whatsapp", - sessionFile: params.sessionFile ?? createTestTranscriptLocator(), + sessionFile: params.sessionFile ?? "/tmp/session.jsonl", workspaceDir: params.workspaceDir ?? "/tmp", config: params.config ?? {}, skillsSnapshot: {}, diff --git a/src/auto-reply/reply/commands-core.test.ts b/src/auto-reply/reply/commands-core.test.ts index d0a41e502b9..725e58f70b0 100644 --- a/src/auto-reply/reply/commands-core.test.ts +++ b/src/auto-reply/reply/commands-core.test.ts @@ -116,7 +116,7 @@ describe("emitResetCommandHooks", () => { sessionKey: "agent:main:telegram:group:-1003826723328:topic:8428", previousSessionEntry: { sessionId: "prev-session", - sessionFile: "sqlite-transcript://main/prev-session.jsonl", + sessionFile: "/tmp/prev-session.jsonl", } as HandleCommandsParams["previousSessionEntry"], workspaceDir: "/tmp/openclaw-workspace", }); @@ -124,7 +124,7 @@ describe("emitResetCommandHooks", () => { await vi.waitFor(() => expect(hookRunnerMocks.runBeforeReset).toHaveBeenCalledTimes(1)); expect(hookRunnerMocks.runBeforeReset).toHaveBeenCalledWith( expect.objectContaining({ - sessionFile: "sqlite-transcript://main/prev-session.jsonl", + sessionFile: "/tmp/prev-session.jsonl", messages: [], reason: "new", }), @@ -164,7 +164,7 @@ describe("emitResetCommandHooks", () => { sessionKey: "agent:target:main", previousSessionEntry: { sessionId: "prev-session", - sessionFile: "sqlite-transcript://main/prev-session.jsonl", + sessionFile: "/tmp/prev-session.jsonl", } as HandleCommandsParams["previousSessionEntry"], workspaceDir: "/tmp/openclaw-workspace", }); @@ -180,7 +180,7 @@ describe("emitResetCommandHooks", () => { }); expect(hookRunnerMocks.runBeforeReset).toHaveBeenCalledWith( expect.objectContaining({ - sessionFile: "sqlite-transcript://main/prev-session.jsonl", + sessionFile: "/tmp/prev-session.jsonl", messages: [{ role: "assistant", content: "Recovered from SQLite" }], reason: "reset", }), diff --git a/src/auto-reply/reply/commands-plugin.test.ts b/src/auto-reply/reply/commands-plugin.test.ts index 06915551063..902733373e5 100644 --- a/src/auto-reply/reply/commands-plugin.test.ts +++ b/src/auto-reply/reply/commands-plugin.test.ts @@ -86,13 +86,13 @@ describe("handlePluginCommand", () => { } as OpenClawConfig); params.sessionEntry = { sessionId: "wrapper-session", - sessionFile: "sqlite-transcript://main/wrapper-session.jsonl", + sessionFile: "/tmp/wrapper-session.jsonl", updatedAt: Date.now(), } as HandleCommandsParams["sessionEntry"]; params.sessionStore = { [params.sessionKey]: { sessionId: "target-session", - sessionFile: "sqlite-transcript://main/target-session.jsonl", + sessionFile: "/tmp/target-session.jsonl", updatedAt: Date.now(), }, }; @@ -102,7 +102,7 @@ describe("handlePluginCommand", () => { expect(executePluginCommandMock).toHaveBeenCalledWith( expect.objectContaining({ sessionId: "target-session", - sessionFile: "sqlite-transcript://main/target-session.jsonl", + sessionFile: "/tmp/target-session.jsonl", }), ); }); diff --git a/src/auto-reply/reply/commands-session-usage.test.ts b/src/auto-reply/reply/commands-session-usage.test.ts index 899da1f774e..1fcfd709c34 100644 --- a/src/auto-reply/reply/commands-session-usage.test.ts +++ b/src/auto-reply/reply/commands-session-usage.test.ts @@ -142,13 +142,13 @@ describe("handleUsageCommand", () => { const params = buildUsageParams(); params.sessionEntry = { sessionId: "wrapper-session", - sessionFile: "sqlite-transcript://target/wrapper-session.jsonl", + sessionFile: "/tmp/wrapper-session.jsonl", updatedAt: Date.now(), }; params.sessionStore = { [params.sessionKey]: { sessionId: "target-session", - sessionFile: "sqlite-transcript://target/target-session.jsonl", + sessionFile: "/tmp/target-session.jsonl", updatedAt: Date.now(), }, }; @@ -158,7 +158,7 @@ describe("handleUsageCommand", () => { expect(loadSessionCostSummaryMock).toHaveBeenCalledWith( expect.objectContaining({ sessionId: "target-session", - sessionFile: "sqlite-transcript://target/target-session.jsonl", + sessionFile: "/tmp/target-session.jsonl", }), ); }); diff --git a/src/auto-reply/reply/queue/state.test.ts b/src/auto-reply/reply/queue/state.test.ts index 5550a594aa2..164923a85f2 100644 --- a/src/auto-reply/reply/queue/state.test.ts +++ b/src/auto-reply/reply/queue/state.test.ts @@ -14,7 +14,7 @@ function makeRun(): FollowupRun["run"] { agentDir: "/tmp/agent", sessionId: "session-1", sessionKey: QUEUE_KEY, - sessionFile: "sqlite-transcript://main/session-1.jsonl", + sessionFile: "/tmp/session-1.jsonl", workspaceDir: "/tmp/workspace", config: {} as FollowupRun["run"]["config"], provider: "anthropic",