Skip to content

fix(server): treat a missing Codex rollout as a recoverable resume error - #6671

Merged
juliusmarminge merged 1 commit into
pingdotgg:mainfrom
Rishet11:fix/5680-codex-missing-rollout
Aug 15, 2026
Merged

juliusmarminge merged 1 commit into
pingdotgg:mainfrom
Rishet11:fix/5680-codex-missing-rollout

fix(server): treat a missing Codex rollout as a recoverable resume error

507caa5
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Effect Service Conventions succeeded Aug 15, 2026 in 24s

All clear

All clear

Details

Note

Your check run agent prompt is: .macroscope/check-run-agents/effect-service-conventions.md
More information about how Check Run Agents work can be found in our Help Center.


🔍 Agent activity — reasoning, no tool calls

💭 I'm checking the diff to verify it follows conventions—just adding a snippet string to the recoverable error list and its corresponding test. I need to dig into the isRecoverableThreadResumeError implementation to understand how "no rollout found" is handled — it seems like a behavior question rather than a convention issue. The test shows "Config file not found" returns false, which suggests there's additional logic checking for thread mentions. I'm wondering if "no rollout found" without a thread reference behaves differently, so let me look at the actual predicate to see what's being checked. The pre-existing catchIf with a message-based predicate is legacy code not introduced by this PR, and the diff only adds a single line plus a test without violating any conventions. I need to return the response as JSON schema even though the instruction specifies "All clear" exactly—the schema structure takes precedence, so I'll set the state to success with "All clear" as the title and summary.


Agent Credits: 25 credits