Skip to content

fix: retry transient proxy restoration - #101

Merged
kev1n77 merged 1 commit into
mainfrom
codex/proxy-restore-retry-context
Jul 21, 2026
Merged

fix: retry transient proxy restoration#101
kev1n77 merged 1 commit into
mainfrom
codex/proxy-restore-retry-context

Conversation

@kev1n77

@kev1n77 kev1n77 commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • retry the idempotent proxy restore transaction once after a transient failure
  • retain the watchdog handle until disarm succeeds, so Drop can still retry safely
  • attach stage context to restore failures and preserve both errors when the retry also fails
  • split macOS explicit-restore and owner-disconnect integration scenarios into independent tests

Motivation

PR #100 exposed one transient macOS helper failure (EINVAL) in the owner disconnect restoration path, followed by a clean rerun. Review found that the session removed its watchdog handle before disarm completed, preventing a meaningful retry after any I/O error.

Verification

  • cargo fmt --all -- --check
  • cargo test -p codeischeap-proxy-recovery --all-targets
  • cargo clippy -p codeischeap-proxy-recovery --all-targets -- -D warnings
  • python -m unittest discover -s scripts/tests -v (32 passed, 1 skipped because Windows symlink creation is unavailable)
  • git diff --check

Full local workspace Rust verification is deferred to CI because this Windows environment lacks Perl required by the vendored OpenSSL build. The affected crate and its complete target set pass locally.

@kev1n77
kev1n77 merged commit 4505dc2 into main Jul 21, 2026
10 checks passed
@kev1n77
kev1n77 deleted the codex/proxy-restore-retry-context branch July 21, 2026 19:23
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