Skip to content

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

Open
nsi-inco wants to merge 1 commit intoagentjido:mainfrom
nsi-inco:main
Open

fix(openai_codex): omit previous_response_id on tool resume#613
nsi-inco wants to merge 1 commit intoagentjido:mainfrom
nsi-inco:main

Conversation

@nsi-inco
Copy link
Copy Markdown

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.
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