Skip to content

Fix delayed Xcode process attachment retry#181

Merged
lynnswap merged 2 commits into
mainfrom
codex/fix-xcode-attach-retry
Jul 15, 2026
Merged

Fix delayed Xcode process attachment retry#181
lynnswap merged 2 commits into
mainfrom
codex/fix-xcode-attach-retry

Conversation

@lynnswap

Copy link
Copy Markdown
Owner

Purpose

Ensure a newly launched Xcode process keeps its bounded mcpbridge activation watchdog when a foreground tools/list request races channel initialization.

Changes

  • Accept channel initialization after the activation attempt has advanced to loadingCatalog.
  • Preserve the in-flight foreground catalog load while still registering the 10-second activation watchdog.
  • Avoid issuing a duplicate activation catalog request when that foreground load already owns the work.
  • Add a regression test for the interleaving that previously left only the 300-second foreground request timeout.

Testing

  • scripts/check.sh
    • 950 standard tests
    • 24 process runtime tests
    • 7 stdio adapter tests
  • codex-review --base main (clean)

lynnswap added 2 commits July 15, 2026 14:39
Keep a process-route activation attempt owned when a foreground tools/list load advances it before channel initialization commits. This ensures the bounded catalog timeout is still registered and can replace and retry an unattached mcpbridge.
Preserve the activation watchdog when a foreground catalog load wins the race, while exposing whether activation still needs to start its own catalog request.
@lynnswap
lynnswap merged commit 82311de into main Jul 15, 2026
9 checks passed
@lynnswap
lynnswap deleted the codex/fix-xcode-attach-retry branch July 15, 2026 06:00
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