Skip to content

fix: report issue work timeout cause - #178

Merged
jeremymcs merged 1 commit into
mainfrom
jeremymcs/fix-codex-cache-issues
Aug 18, 2026
Merged

fix: report issue work timeout cause#178
jeremymcs merged 1 commit into
mainfrom
jeremymcs/fix-codex-cache-issues

Conversation

@jeremymcs

Copy link
Copy Markdown
Owner

Summary

  • use the agent-aware failure summarizer for failed issue work
  • surface the terminal timeout instead of early Codex startup/cache warnings
  • add regression coverage for the reported base_instructions cache warning followed by exit 124

Root cause

Codex continued running after the models-cache parse warning. Patchdeck later timed it out, but issue work displayed the beginning of raw stderr and hid the appended timeout reason at the end.

Verification

  • node --import tsx --test --test-name-pattern='reports a Codex timeout instead of a models cache warning' server/issueWorkAgent.test.ts
  • node --import tsx --test server/agentRunner.test.ts server/issueWorkAgent.test.ts
  • npm run test (701 passed)
  • npm run check
  • node --experimental-test-coverage --import tsx --test server/agentRunner.test.ts server/issueWorkAgent.test.ts (issueWorkAgent.ts: 92.42% lines, 74.42% branches)

@github-actions

Copy link
Copy Markdown
Contributor

📄 Documentation Review Reminder

This PR changes source files but does not update any public documentation.

Changed source files:

server/issueWorkAgent.test.ts
server/issueWorkAgent.ts

Please check if documentation in docs/public/ needs to be updated:

  • No documentation update needed
  • Documentation updated in this PR
  • Follow-up documentation issue created

Public docs are Markdown files in docs/public/. They are automatically
rendered to HTML and deployed to GitHub Pages when merged to main.

Available docs: getting-started.md, pr-babysitter.md, agent-dispatch.md, configuration.md, pr-questions.md

@jeremymcs
jeremymcs merged commit f61be7f into main Aug 18, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant