ton/pi_harness
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(coding-agent): shorten invalid session error

Browse Source 
Follow-up to #6002
This commit is contained in:
Mario Zechner
2026-06-25 15:20:08 +02:00
parent 774288587f
commit 0d145e895c
3 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/coding-agent/src/core/session-manager.ts
+1 -1
View File
@@ -800,7 +800,7 @@ export class SessionManager {
if (this.fileEntries.length === 0) { if (this.fileEntries.length === 0) {
const explicitPath = this.sessionFile; const explicitPath = this.sessionFile;
if (statSync(explicitPath).size > 0) { if (statSync(explicitPath).size > 0) {
throw new Error(`Session file is not a valid pi session and was not modified: ${explicitPath}`); throw new Error(`Session file is not a valid pi session: ${explicitPath}`);
} }
this.newSession(); this.newSession();
this.sessionFile = explicitPath; this.sessionFile = explicitPath;
packages/coding-agent/test/session-file-invalid.test.ts
+1 -3
View File
@@ -57,9 +57,7 @@ describe("--session invalid file handling", () => {
const result = await runCli(["--session", sessionFile, "-p", "hi"], projectDir, agentDir); const result = await runCli(["--session", sessionFile, "-p", "hi"], projectDir, agentDir);
expect(result.code).toBe(1); expect(result.code).toBe(1);
expect(result.stderr).toContain( expect(result.stderr).toContain(`Error: Session file is not a valid pi session: ${sessionFile}`);
`Error: Session file is not a valid pi session and was not modified: ${sessionFile}`,
);
expect(result.stderr).not.toContain("SessionManager.open"); expect(result.stderr).not.toContain("SessionManager.open");
expect(result.stderr).not.toContain("at "); expect(result.stderr).not.toContain("at ");
expect(readFileSync(sessionFile, "utf8")).toBe(originalContent); expect(readFileSync(sessionFile, "utf8")).toBe(originalContent);
packages/coding-agent/test/session-manager/file-operations.test.ts
+2 -2
View File
@@ -275,7 +275,7 @@ describe("SessionManager.setSessionFile with corrupted files", () => {
writeFileSync(noHeaderFile, originalContent); writeFileSync(noHeaderFile, originalContent);
expect(() => SessionManager.open(noHeaderFile, tempDir)).toThrow( expect(() => SessionManager.open(noHeaderFile, tempDir)).toThrow(
`Session file is not a valid pi session and was not modified: ${noHeaderFile}`, `Session file is not a valid pi session: ${noHeaderFile}`,
); );
expect(readFileSync(noHeaderFile, "utf-8")).toBe(originalContent); expect(readFileSync(noHeaderFile, "utf-8")).toBe(originalContent);
}); });
@@ -286,7 +286,7 @@ describe("SessionManager.setSessionFile with corrupted files", () => {
writeFileSync(nonSessionFile, originalContent); writeFileSync(nonSessionFile, originalContent);
expect(() => SessionManager.open(nonSessionFile, tempDir)).toThrow( expect(() => SessionManager.open(nonSessionFile, tempDir)).toThrow(
`Session file is not a valid pi session and was not modified: ${nonSessionFile}`, `Session file is not a valid pi session: ${nonSessionFile}`,
); );
expect(readFileSync(nonSessionFile, "utf-8")).toBe(originalContent); expect(readFileSync(nonSessionFile, "utf-8")).toBe(originalContent);
}); });