fix(proxy): persist complete HTTP bridge replay transcripts - #1899
Closed
shaqman wants to merge 8 commits into
Closed
fix(proxy): persist complete HTTP bridge replay transcripts#1899shaqman wants to merge 8 commits into
shaqman wants to merge 8 commits into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
shaqman
force-pushed
the
feat/complete-http-bridge-transcripts
branch
from
August 24, 2026 07:42
c7ea143 to
4984f8c
Compare
Contributor
Author
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
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.
Closes #1898
Summary
This PR makes HTTP Responses bridge recovery replayable when upstream continuity is lost, while keeping the recovery fail-closed when the transcript is incomplete or ambiguous.
The branch is rebased onto
mainatb311aea760aa639fd96f63bd118f775e9b4a89f9(currently v1.24.0-beta.4).Changes
No upstream API or Codex CLI changes are required.
Validation
Local:
pytest -q tests/unit/test_complete_transcript.py: 10 passedpython -m compileallandgit diff --check: passedThe full bridge unit module was also attempted. The local test container currently lacks the
gitexecutable required by an unrelated worktree-path test, so it stops at that environment failure rather than indicating a regression in this change.Live deployment evidence:
response_output_items_complete=trueandresponse_replay_input_complete=true.function_calland onefunction_call_output; no duplicate tool call was emitted.response.completed(27 HTTPstream_incompleteresponses); seven stale sessions had no durable replay snapshot and correctly failed closed.Related work
This is intentionally narrower than the existing recovery work:
previous_response_idrecovery.Those changes do not provide complete transcript materialization plus tool-output deduplication for the replay path addressed here.
Rebase follow-up
The release branch advanced after the initial PR creation. The branch was force-with-lease updated after a clean rebase onto current
mainatb311aea760aa639fd96f63bd118f775e9b4a89f9(currently v1.24.0-beta.4); no conflicts occurred. The focused replay suite and static checks were rerun successfully after the rebase (10/10 tests passed).