Skip to content

fix(openai_codex): omit previous_response_id on tool resume#613

Merged
mikehostetler merged 2 commits intoagentjido:mainfrom
nsi-inco:main
Apr 17, 2026
Merged

fix(openai_codex): omit previous_response_id on tool resume#613
mikehostetler merged 2 commits intoagentjido:mainfrom
nsi-inco:main

Conversation

@nsi-inco
Copy link
Copy Markdown
Contributor

Description

Drop previous_response_id from ChatGPT Codex tool-resume requests while keeping function_call_output items and store=false. This matches the backend's actual resume contract and prevents tool-calling turns from failing after successful tool execution.

Type of Change

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to change)
  • Documentation update

Testing

  • Tests pass (mix test)
  • Quality checks pass (mix quality)

Checklist

  • My code follows the project's style guidelines
  • I have updated the documentation accordingly
  • I have added tests that prove my fix/feature works
  • All new and existing tests pass
  • My commits follow conventional commit format
  • I have NOT edited CHANGELOG.md (it is auto-generated by git_ops)

Drop previous_response_id from ChatGPT Codex tool-resume requests while keeping function_call_output items and store=false. This matches the backend's actual resume contract and prevents tool-calling turns from failing after successful tool execution.
@mikehostetler mikehostetler added the needs_work Changes requested before merge label Apr 17, 2026
@mikehostetler mikehostetler added ready_to_merge and removed needs_work Changes requested before merge labels Apr 17, 2026
@mikehostetler mikehostetler merged commit 3cf5bb1 into agentjido:main Apr 17, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants