Skip to content

Wait for debugger console readiness before returning RunProject - #465

Merged
zunda-pixel merged 1 commit into
mainfrom
fix/debugger-launch-readiness
Oct 30, 2026
Merged

zunda-pixel merged 1 commit into
mainfrom
fix/debugger-launch-readiness

Conversation

@zunda-pixel

Copy link
Copy Markdown
Collaborator

RunProject(attachDebugger: true) could hand a successful launch to the workspace before the native debugger command could find its session. Wait for the matching PID in the execution environment's current launch session, its debugger, and its console adaptor before completing that handoff.

The wait observes the same native state that InvokeDebuggerCommand uses. It remains cancellable, rejects an ended launch, and reports a bounded readiness failure rather than a premature success. Non-debugger launches keep their existing path.

Fixes #462.

Validation:

  • 20 focused debugger-readiness, execution-scope, and application-launch tests passed, including wrong-session, missing-console, cancelled-operation, and ended-launch cases.
  • Xcode 27.2 / physical iPhone 16 on iOS 27.2: ten non-debugger-to-debugger launch transitions, with immediate, 1-second, and 3-second queries. All 30 queries found an active debug session, with zero tool errors.
  • Fixture launches stopped successfully, the owned workspace and host closed without forced termination, and the fixture installed by this run was removed.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-30T15:20:56.197951Z 547cb52 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@zunda-pixel
zunda-pixel merged commit 53c1830 into main Oct 30, 2026
7 checks passed
@zunda-pixel
zunda-pixel deleted the fix/debugger-launch-readiness branch October 30, 2026 16:26
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.

Physical RunProject can return before its debug session is available

1 participant