mirror of
https://github.com/paperclipai/paperclip.git
synced 2026-10-29 12:59:13 +01:00
The Codex CLI can return "no rollout found for thread id ..." when resuming a heartbeat thread whose rollout has been garbage-collected. Extend isCodexUnknownSessionError() to match this wording so the existing single-retry path in execute.ts activates correctly. Add parse.test.ts covering the new pattern, existing stale-session wordings, parseCodexJsonl, and a negative case. Co-Authored-By: Paperclip <noreply@paperclip.ing>
51 lines
1.7 KiB
TypeScript
51 lines
1.7 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
import { isCodexUnknownSessionError, parseCodexJsonl } from "./parse.js";
|
|
|
|
describe("parseCodexJsonl", () => {
|
|
it("captures session id, assistant summary, usage, and error message", () => {
|
|
const stdout = [
|
|
JSON.stringify({ type: "thread.started", thread_id: "thread_123" }),
|
|
JSON.stringify({
|
|
type: "item.completed",
|
|
item: { type: "agent_message", text: "Recovered response" },
|
|
}),
|
|
JSON.stringify({
|
|
type: "turn.completed",
|
|
usage: { input_tokens: 10, cached_input_tokens: 2, output_tokens: 4 },
|
|
}),
|
|
JSON.stringify({ type: "turn.failed", error: { message: "resume failed" } }),
|
|
].join("\n");
|
|
|
|
expect(parseCodexJsonl(stdout)).toEqual({
|
|
sessionId: "thread_123",
|
|
summary: "Recovered response",
|
|
usage: {
|
|
inputTokens: 10,
|
|
cachedInputTokens: 2,
|
|
outputTokens: 4,
|
|
},
|
|
errorMessage: "resume failed",
|
|
});
|
|
});
|
|
});
|
|
|
|
describe("isCodexUnknownSessionError", () => {
|
|
it("detects the current missing-rollout thread error", () => {
|
|
expect(
|
|
isCodexUnknownSessionError(
|
|
"",
|
|
"Error: thread/resume: thread/resume failed: no rollout found for thread id d448e715-7607-4bcc-91fc-7a3c0c5a9632",
|
|
),
|
|
).toBe(true);
|
|
});
|
|
|
|
it("still detects existing stale-session wordings", () => {
|
|
expect(isCodexUnknownSessionError("unknown thread id", "")).toBe(true);
|
|
expect(isCodexUnknownSessionError("", "state db missing rollout path for thread abc")).toBe(true);
|
|
});
|
|
|
|
it("does not classify unrelated Codex failures as stale sessions", () => {
|
|
expect(isCodexUnknownSessionError("", "model overloaded")).toBe(false);
|
|
});
|
|
});
|