Repository navigation
feat(codex): restore bounded capacity recovery [L15] - #100
Merged
Mjones13 merged 3 commits intoOct 4, 2026
Merged
Conversation
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
A confirmed Codex serverOverloaded failure schedules at most five promptless retries after a ten-second backoff. The original logical turn stays open, each retry retains its settings, and the prompt and attachments are sent once. Stop, supersession, scope closure, revision drift and unconfirmed starts prevent stale replay.
Restores owner PR #19 on pinned nightly source, using the narrow capacity reducer and cases from reference #91. Managed runtime validation has a separate bounded 30-second check after the ten-second delay. This layer follows merged account-switch PR #97 and preserves its regressions.
Also fixes the ACP replay fixture race exposed by this PR's Test Server 6 job: the fixture now atomically publishes a validated replay checkpoint before exposing its answer. The strict completeness reader and existing assertions remain intact. A captured synthetic child demonstrates that interruption during publication preserves the prior valid checkpoint and withholds the final answer.
Validation: all 336 tests in the capacity affected union passed. The old ACP writer deterministically reproduced the truncated-JSON failure; the repaired ACP/replay union passed all 254 tests with no skips. After codec-only corrections to the new test, all ten publication cases passed again; the other three modules and writer remained byte-identical. Final server typechecking and scoped formatting/lint passed. Current-head CI remains pending. No live account, provider request, runtime replacement or adoption was performed.
Coordinated with GPT-6.1 Sol through the Codex harness.