From 0558305cd3886aeef984616abba38689aef3168d Mon Sep 17 00:00:00 2001 From: Gagancreates Date: Fri, 5 Jun 2026 14:40:07 +0530 Subject: [PATCH] refactor(code-mode): move ACP session files out of WorkDir/config Per-run ACP session state is runtime state that accumulates one file per chat run, not user/app config. Relocate it from WorkDir/config to a dedicated WorkDir/code-mode/sessions/ so it can be listed, cleaned up, and managed on its own without crowding config. Drop the now-redundant codesession- filename prefix (the directory conveys it). --- apps/x/packages/core/src/code-mode/acp/session-store.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/apps/x/packages/core/src/code-mode/acp/session-store.ts b/apps/x/packages/core/src/code-mode/acp/session-store.ts index 82e3eb68..e5e45666 100644 --- a/apps/x/packages/core/src/code-mode/acp/session-store.ts +++ b/apps/x/packages/core/src/code-mode/acp/session-store.ts @@ -13,8 +13,13 @@ export interface StoredSession { sessionId: string; } +// Per-run ACP session state lives in its own directory (not WorkDir/config): it's +// runtime state that accumulates one file per chat run, so it's kept separate from +// user/app config to be listed and cleaned up on its own. +const SESSIONS_DIR = path.join(WorkDir, 'code-mode', 'sessions'); + function sessionFile(runId: string): string { - return path.join(WorkDir, 'config', `codesession-${runId}.json`); + return path.join(SESSIONS_DIR, `${runId}.json`); } export async function readStoredSession(runId: string): Promise {