Skip to content

Commit f2ee091

Browse files
tclaude
andcommitted
test(core): keep the spill test inside its temp directory
`new SessionManager(sessionsRoot)` passes a string where the constructor wants `{ root }`, so `opts.root` was undefined and the manager silently fell back to the real `~/.deepcode/sessions`. The test still passed — it reads the file back through the locator it was handed — while writing a session and a spill artifact into the user's actual home directory on every run. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent 1b5811e commit f2ee091

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

packages/core/src/agent.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1251,8 +1251,8 @@ describe('runAgent', () => {
12511251
});
12521252

12531253
it('saves the omitted output where the model can read it back', async () => {
1254-
const manager = new SessionManager(sessionsRoot);
1255-
const session = await manager.create({ cwd });
1254+
const manager = new SessionManager({ root: sessionsRoot });
1255+
const session = await manager.create(cwd);
12561256
const result = await runAgent({
12571257
provider: new MockProvider([toolUse('flooding', floodCall()), endTurn('done')]),
12581258
tools: new ToolRegistry([floodTool]),

0 commit comments

Comments
 (0)