From d269aa153a4ab350129508d130b5330ecd1f1d07 Mon Sep 17 00:00:00 2001 From: Jari Mustonen Date: Wed, 15 Apr 2026 09:39:02 +0300 Subject: [PATCH] context-engine: remove sessionKey from factory context --- src/agents/pi-embedded-runner/compact.queued.ts | 1 - src/agents/pi-embedded-runner/run.ts | 1 - src/context-engine/context-engine.test.ts | 4 ---- src/context-engine/registry.ts | 5 +---- 4 files changed, 1 insertion(+), 10 deletions(-) diff --git a/src/agents/pi-embedded-runner/compact.queued.ts b/src/agents/pi-embedded-runner/compact.queued.ts index c045818a7b6..ebd17dabe73 100644 --- a/src/agents/pi-embedded-runner/compact.queued.ts +++ b/src/agents/pi-embedded-runner/compact.queued.ts @@ -55,7 +55,6 @@ export async function compactEmbeddedPiSession( const contextEngine = await resolveContextEngine(params.config, { agentDir, workspaceDir: resolveUserPath(params.workspaceDir), - sessionKey: params.sessionKey, }); let contextTokenBudget = params.contextTokenBudget; if (!contextTokenBudget || !Number.isFinite(contextTokenBudget) || contextTokenBudget <= 0) { diff --git a/src/agents/pi-embedded-runner/run.ts b/src/agents/pi-embedded-runner/run.ts index 29ded8cb8db..7bbd3d3573d 100644 --- a/src/agents/pi-embedded-runner/run.ts +++ b/src/agents/pi-embedded-runner/run.ts @@ -743,7 +743,6 @@ export async function runEmbeddedPiAgent( const contextEngine = await resolveContextEngine(params.config, { agentDir: params.agentDir, workspaceDir: params.workspaceDir, - sessionKey: params.sessionKey, }); try { let activeSessionId = params.sessionId; diff --git a/src/context-engine/context-engine.test.ts b/src/context-engine/context-engine.test.ts index 580bc7dc3f3..e9f380c2312 100644 --- a/src/context-engine/context-engine.test.ts +++ b/src/context-engine/context-engine.test.ts @@ -727,14 +727,12 @@ describe("Factory context passing", () => { await resolveContextEngine(cfg, { agentDir: "/tmp/agent", workspaceDir: "/tmp/workspace", - sessionKey: "agent:main:test", }); expect(receivedCtx).toBeDefined(); expect(receivedCtx!.config).toBe(cfg); expect(receivedCtx!.agentDir).toBe("/tmp/agent"); expect(receivedCtx!.workspaceDir).toBe("/tmp/workspace"); - expect(receivedCtx!.sessionKey).toBe("agent:main:test"); }); it("no-arg factories still work when context is passed", async () => { @@ -761,7 +759,6 @@ describe("Factory context passing", () => { const engine = await resolveContextEngine(configWithSlot(engineId), { agentDir: "/tmp/agent", workspaceDir: "/tmp/workspace", - sessionKey: "agent:main:test", }); expect(called).toBe(true); @@ -797,7 +794,6 @@ describe("Factory context passing", () => { expect(receivedCtx!.config).toBeDefined(); expect(receivedCtx!.agentDir).toBeUndefined(); expect(receivedCtx!.workspaceDir).toBeUndefined(); - expect(receivedCtx!.sessionKey).toBeUndefined(); }); }); diff --git a/src/context-engine/registry.ts b/src/context-engine/registry.ts index 192d663ffe9..90b270b79d3 100644 --- a/src/context-engine/registry.ts +++ b/src/context-engine/registry.ts @@ -13,7 +13,6 @@ export type ContextEngineFactoryContext = { config: OpenClawConfig; agentDir?: string; workspaceDir?: string; - sessionKey?: string; }; /** @@ -479,7 +478,6 @@ function describeResolvedContextEngineContractError( export type ResolveContextEngineOptions = { agentDir?: string; workspaceDir?: string; - sessionKey?: string; }; /** @@ -490,7 +488,7 @@ export type ResolveContextEngineOptions = { * 2. Default slot value ("legacy") * * When `config` is provided it is forwarded to the factory as part of a - * {@link ContextEngineFactoryContext}. Additional runtime paths and keys + * {@link ContextEngineFactoryContext}. Additional runtime paths * can be supplied via `options`. No-arg factories still work — the context * parameter is silently ignored by factories that don't declare it. * @@ -515,7 +513,6 @@ export async function resolveContextEngine( config: config ?? ({} as OpenClawConfig), agentDir: options?.agentDir, workspaceDir: options?.workspaceDir, - sessionKey: options?.sessionKey, }; const entry = getContextEngineRegistryState().engines.get(engineId);