Skip to content

[fix] app server flaky send_messages test#8874

Merged
celia-oai merged 4 commits intomainfrom
dev/cc/fix-test-2
Jan 8, 2026
Merged

[fix] app server flaky send_messages test#8874
celia-oai merged 4 commits intomainfrom
dev/cc/fix-test-2

Conversation

@celia-oai
Copy link
Collaborator

@celia-oai celia-oai commented Jan 8, 2026

Fix flakiness of CI test: https://github.com/openai/codex/actions/runs/20350530276/job/58473691434?pr=8282

This PR does two things:

  1. move the flakiness test to use responses API instead of chat completion API
  2. make mcp_process agnostic to the order of responses/notifications/requests that come in, by buffering messages not read

@celia-oai celia-oai force-pushed the dev/cc/fix-test-2 branch 2 times, most recently from 54359c6 to 660914f Compare January 8, 2026 01:10
@celia-oai celia-oai changed the title changes [fix] fix flaky send_messages test Jan 8, 2026
@celia-oai celia-oai marked this pull request as ready for review January 8, 2026 01:30
@celia-oai celia-oai changed the title [fix] fix flaky send_messages test [fix] app server flaky send_messages test Jan 8, 2026
Copy link
Contributor

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f98f4f2a99

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@celia-oai celia-oai force-pushed the dev/cc/fix-test-2 branch 2 times, most recently from ef094e3 to 2311521 Compare January 8, 2026 02:03
@celia-oai celia-oai marked this pull request as draft January 8, 2026 02:26
Base automatically changed from dev/cc/fix-tests to main January 8, 2026 18:17
@celia-oai celia-oai marked this pull request as ready for review January 8, 2026 19:48
Copy link
Collaborator

@bolinfest bolinfest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, though I feel iffy about about clear_message_buffer().

@celia-oai celia-oai enabled auto-merge (squash) January 8, 2026 20:12
@celia-oai celia-oai merged commit 051bf81 into main Jan 8, 2026
45 of 47 checks passed
@celia-oai celia-oai deleted the dev/cc/fix-test-2 branch January 8, 2026 20:41
@github-actions github-actions bot locked and limited conversation to collaborators Jan 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants