Skip to content

fix(proxy): integrate transport reliability fixes - #1

Merged
kevinsslin merged 4 commits into
mainfrom
codex/fork-main-reliability-integration
Aug 14, 2026
Merged

fix(proxy): integrate transport reliability fixes#1
kevinsslin merged 4 commits into
mainfrom
codex/fork-main-reliability-integration

Conversation

@kevinsslin

Copy link
Copy Markdown
Owner

Summary

Integrates the three transport-reliability fixes we are maintaining for the production fork on top of the current upstream main:

This PR is the deployment integration branch for kevinsslin/codex-lb. The upstream PRs remain open for independent maintainer review.

Why

The live v1.23.0 VPS continued to show missing_response_created_timeout, upstream_request_timeout, cleanup budget warnings, repeated 503 cooldown responses, and Codex tasks pausing after reconnect exhaustion. The official fixes are not yet merged into an upstream release.

Safety

  • preserves the existing shared shutdown deadline;
  • keeps eventful, file-pinned, soft-affinity, cross-account, and repeated failures fail-closed;
  • adds no environment variables, migrations, public APIs, or dependencies;
  • retains the PyPI codex-lb==1.23.0 rollback path.

Validation

  • combined targeted suite: 822 passed;
  • uv run ruff check app tests;
  • uv run ruff format --check app tests — 896 files formatted;
  • uv run ty check;
  • uv run python scripts/check_proxy_architecture.py;
  • all three OpenSpec changes validated with --strict;
  • all OpenSpec specs: 57 passed, 0 failed;
  • git diff --check upstream/main...HEAD.

Deployment

After merge, the VPS will install this fork's exact merged main SHA rather than the moving branch name, restart codex-lb, and run health plus bounded transport/log canaries.

Keep the one-second generic task cancellation bound while allowing normal direct Responses WebSocket scope finalization to use a bounded five-second observation window. Preserve the shared shutdown deadline and add a regression plus OpenSpec contract.

Refs Soju06#1711
@kevinsslin
kevinsslin marked this pull request as ready for review August 14, 2026 05:20
@kevinsslin
kevinsslin merged commit 4cf1b95 into main Aug 14, 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