Repository navigation
Keep a Codex turn going while the relay catches up on the app-server's output - #431
Merged
Merged
Conversation
…s output The subscription relay authorizes and checks each line the host Codex app-server writes, one at a time, and the queue in front of that work failed closed once 256 lines waited. Codex writes bursts of hundreds of deltas faster than that, so a slow stretch closed the relay (1008, "serial task queue limit exceeded") and failed a healthy turn as "The provider event stream closed unexpectedly". Pausing the child's output does not help: Bun keeps delivering a child's stdout after pause(). The app-server's frames now wait without a limit; the sandbox's frames keep the fail-closed limit of 256. The new relay test holds one frame's authorization while 600 lines arrive; the old code closes the relay.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.