Skip to content

Harden Codex retry and bridge recovery#1041

Closed
zvladru wants to merge 1 commit into
Soju06:mainfrom
zvladru:fix/codex-tool-output-delta-retry
Closed

Harden Codex retry and bridge recovery#1041
zvladru wants to merge 1 commit into
Soju06:mainfrom
zvladru:fix/codex-tool-output-delta-retry

Conversation

@zvladru

@zvladru zvladru commented Jun 17, 2026

Copy link
Copy Markdown

Summary

  • Prevent fresh WebSocket retries for previous_response_id payloads that contain tool outputs without matching tool calls.
  • Add stale HTTP bridge inflight-session cleanup diagnostics for drain/status paths.
  • Normalize quota planner timestamps to naive UTC before database writes and comparisons.

Tests

  • uv run pytest -q tests/unit/test_proxy_http_bridge.py::test_http_bridge_activity_snapshot_counts_pending_and_inflight_sessions tests/unit/test_proxy_http_bridge.py::test_http_bridge_activity_snapshot_cleans_stale_inflight_session tests/unit/test_proxy_http_bridge.py::test_get_or_create_http_bridge_session_inflight_wait_times_out tests/unit/test_proxy_http_bridge.py::test_get_or_create_http_bridge_session_capacity_wait_times_out tests/unit/test_proxy_utils.py::test_prepare_websocket_response_create_request_does_not_fresh_retry_tool_output_delta tests/unit/test_proxy_utils.py::test_prepare_websocket_response_create_request_captures_client_full_resend_anchor_replay tests/unit/test_proxy_utils.py::test_websocket_client_previous_response_full_resend_retry_rejects_tool_output_delta tests/unit/test_proxy_utils.py::test_websocket_client_previous_response_full_resend_retry_allows_self_contained_tool_history tests/unit/test_quota_planner.py

@zvladru

zvladru commented Jun 17, 2026

Copy link
Copy Markdown
Author

Superseded by smaller focused PRs:

Closing this combined PR to keep review scope small.

@zvladru zvladru closed this Jun 17, 2026
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