Skip to content

fix(proxy): retry silent bridge response.create upstreams - #1547

Closed
Komzpa wants to merge 3 commits into
Soju06:mainfrom
Komzpa:fix/http-bridge-missing-created-retry
Closed

fix(proxy): retry silent bridge response.create upstreams#1547
Komzpa wants to merge 3 commits into
Soju06:mainfrom
Komzpa:fix/http-bridge-missing-created-retry

Conversation

@Komzpa

@Komzpa Komzpa commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • retry an HTTP bridge request when the upstream Responses websocket accepts response.create but never emits response.created
  • keep the existing fail-closed behavior when a request has already produced visible output or cannot be replayed safely
  • add a regression test for the silent pre-created websocket case

Test plan

  • uv run pytest tests/integration/test_http_responses_bridge.py -k 'missing_created_retry or precreated or response_create_gate or retired_gate'

@Komzpa
Komzpa force-pushed the fix/http-bridge-missing-created-retry branch 2 times, most recently from 9109d72 to b353348 Compare July 30, 2026 12:58
@Komzpa
Komzpa force-pushed the fix/http-bridge-missing-created-retry branch 2 times, most recently from 85ec2bd to 778acb2 Compare July 30, 2026 15:18
@Komzpa
Komzpa force-pushed the fix/http-bridge-missing-created-retry branch from 778acb2 to 109ca66 Compare July 30, 2026 15:25
@Komzpa

Komzpa commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@Komzpa
Komzpa force-pushed the fix/http-bridge-missing-created-retry branch from cbdaebc to a07c6ad Compare July 30, 2026 16:11
@Komzpa

Komzpa commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@Komzpa Komzpa added the needs rebase Needs rebase or conflict repair against current main label Aug 4, 2026
@Komzpa

Komzpa commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded: the core retry-before-claim idea landed via #1394 and #1563, and today's deployed #1625 covers the dead-owner recovery side. The remaining unique piece here (replaying client-owned anchors up to 20 times before response.created) is intentionally fenced off by the merged design, so closing this rather than rebasing it.

@Komzpa Komzpa closed this Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs rebase Needs rebase or conflict repair against current main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant