Skip to content

fix: openai/codex mark transport sse parse errors as retryable#2566

Merged
tusharmath merged 6 commits intotailcallhq:mainfrom
bahirul:main
Mar 16, 2026
Merged

fix: openai/codex mark transport sse parse errors as retryable#2566
tusharmath merged 6 commits intotailcallhq:mainfrom
bahirul:main

Conversation

@bahirul
Copy link
Copy Markdown
Contributor

@bahirul bahirul commented Mar 14, 2026

Summary

Im using Codex and often hit SSE error when do task, we need treat SSE error as retryable like in bedrock provider.

Changes

  • Added function to handle SSE error
  • Added test function

Why

We need handle SSE error as retryable.

Testing

Describe how you tested it.

  • Unit tests
  • Manual testing
  • Not tested

Checklist

  • Code follows project style
  • Self-review completed
  • Tests added/updated if needed
  • Documentation updated if needed
  • No breaking changes

Related

Previous PR Closes #2565 (Cannot sign CLA) 😕
Related to #2553

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 14, 2026 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Mar 14, 2026
graphite-app[bot]
graphite-app Bot reviewed Mar 14, 2026
View reviewed changes
Comment thread crates/forge_repo/src/provider/openai_responses/repository.rs Outdated
@bahirul bahirul mentioned this pull request Mar 15, 2026
Closed
8 tasks
@tusharmath tusharmath merged commit 7d63501 into tailcallhq:main Mar 16, 2026
9 checks passed
@tusharmath
Copy link
Copy Markdown
Collaborator

tusharmath commented Mar 16, 2026

thanks @bahirul!

tusharmath pushed a commit that referenced this pull request Mar 16, 2026
@bahirul @autofix-ci
@amitksingh1490 @tusharmath
feat: openai/codex mark transport sse parse errors as retryable (#2566)
55bc00d 
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Amit Singh <amitksingh1490@gmail.com>
@tusharmath tusharmath added type: fix Iterations on existing features or infrastructure. and removed type: feature Brand new functionality, features, pages, workflows, endpoints, etc. labels Mar 16, 2026
@tusharmath tusharmath changed the title feat: openai/codex mark transport sse parse errors as retryable fix: openai/codex mark transport sse parse errors as retryable Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@graphite-app graphite-app[bot] graphite-app[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

type: fix Iterations on existing features or infrastructure.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bahirul @CLAassistant @tusharmath @amitksingh1490