Skip to content

Commit 232a8cd

Browse files
authored
Only store session on successful setup (#372)
1 parent 37e3f38 commit 232a8cd

1 file changed

Lines changed: 18 additions & 19 deletions

File tree

src/acp-agent.ts

Lines changed: 18 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1248,24 +1248,6 @@ export class ClaudeAcpAgent implements Agent {
12481248
options,
12491249
});
12501250

1251-
this.sessions[sessionId] = {
1252-
query: q,
1253-
input: input,
1254-
cancelled: false,
1255-
permissionMode,
1256-
settingsManager,
1257-
accumulatedUsage: {
1258-
inputTokens: 0,
1259-
outputTokens: 0,
1260-
cachedReadTokens: 0,
1261-
cachedWriteTokens: 0,
1262-
},
1263-
configOptions: [],
1264-
promptRunning: false,
1265-
pendingMessages: new Map(),
1266-
nextPendingOrder: 0,
1267-
};
1268-
12691251
let initializationResult;
12701252
try {
12711253
initializationResult = await q.initializationResult();
@@ -1319,7 +1301,24 @@ export class ClaudeAcpAgent implements Agent {
13191301
};
13201302

13211303
const configOptions = buildConfigOptions(modes, models);
1322-
this.sessions[sessionId].configOptions = configOptions;
1304+
1305+
this.sessions[sessionId] = {
1306+
query: q,
1307+
input: input,
1308+
cancelled: false,
1309+
permissionMode,
1310+
settingsManager,
1311+
accumulatedUsage: {
1312+
inputTokens: 0,
1313+
outputTokens: 0,
1314+
cachedReadTokens: 0,
1315+
cachedWriteTokens: 0,
1316+
},
1317+
configOptions,
1318+
promptRunning: false,
1319+
pendingMessages: new Map(),
1320+
nextPendingOrder: 0,
1321+
};
13231322

13241323
return {
13251324
sessionId,

0 commit comments

Comments
 (0)