Skip to content

fix(sdk): cancel Codex responses resolved during publish - #164

Merged
ross-rl merged 1 commit into
mainfrom
fix/codex-server-request-resolution-race
Aug 13, 2026
Merged

fix(sdk): cancel Codex responses resolved during publish#164
ross-rl merged 1 commit into
mainfrom
fix/codex-server-request-resolution-race

Conversation

@ross-rl

@ross-rl ross-rl commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • establish one request-lifetime controller for every recognized Codex server request
  • retain serverRequest/resolved tracking through handler execution and final result or error publication
  • abort in-flight default, custom-handler, and fallback-error responses when Codex clears the request first
  • suppress stale fallback responses after external resolution
  • cover custom result, handler rejection, and default-response publication races

Tests

  • bun run check
  • bun run typecheck
  • bun run build
  • bun run test (25 files, 590 tests)

@reflex-loop

reflex-loop Bot commented Aug 13, 2026

Copy link
Copy Markdown

Reflex agent status: Completed

The agent completed its work.

This PR was created by Reflex.

View the agent run →

This comment updates in place as the agent works.

@ross-rl ross-rl changed the title fix(codex): cancel responses resolved during publish fix(sdk): cancel Codex responses resolved during publish Aug 13, 2026
@ross-rl
ross-rl force-pushed the fix/codex-server-request-resolution-race branch from 3434054 to 337bdc7 Compare August 13, 2026 20:06
@ross-rl
ross-rl merged commit 50901a8 into main Aug 13, 2026
6 checks passed
@ross-rl
ross-rl deleted the fix/codex-server-request-resolution-race branch August 13, 2026 20:10
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