Skip to content

fix(cli): guide Xcode 27 simulator recovery - #3995

Merged
riderx merged 2 commits into
mainfrom
t3/6ac1ee1d
Oct 29, 2026
Merged

riderx merged 2 commits into
mainfrom
t3/6ac1ee1d

Conversation

@riderx

@riderx riderx commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Summary (AI generated)

When Xcode 27 provides Device Hub but Capacitor's installed native-run still expects Simulator.app, the CLI, doctor, and OTA onboarding now report the incompatibility and provide a manual Xcode launch path. MCP onboarding waits until the user confirms the baseline app is running before proceeding to the OTA test.

The check follows the selected Xcode and resolves native-run from the app's installed @capacitor/cli, including hoisted and symlinked installs. It recognizes the upstream package-manager patch and leaves physical-device selection available. CLI help, published usage skills, README, and webdocs include recovery and upgrade guidance.

Motivation (AI generated)

Xcode 27 replaced Simulator.app with Device Hub. The upstream fix is merged, but as of October 29, 2026, npm still publishes native-run 2.0.3 and Capacitor CLI 8.6.0 still depends on it. There is no released minimum Capacitor version to recommend for this fix yet. Recommending a Capacitor upgrade alone would leave users stuck.

Sources: ionic-team/capacitor#8620 and ionic-team/native-run#408.

Business Impact (AI generated)

Users setting up Capgo with a recent Xcode can continue through Xcode or the documented upstream patch instead of repeatedly failing simulator launches. Confirming the baseline app opens prevents onboarding from advancing to an OTA test with no running native app.

Test Plan (AI generated)

  • bun run --cwd cli lint
  • bun run --cwd cli build
  • bun run --cwd cli test:mcp
  • bun run --cwd cli test:bundle
  • bun run --cwd cli test:onboarding-run-targets, including 16 compatibility tests and compiled CLI/doctor checks
  • bun run --cwd cli test:mcp-live-update-onboarding, including the manual-launch confirmation gate
  • bun run --cwd cli test:run-device-command
  • bun run --cwd cli test:doctor-analytics
  • bun run --cwd cli test:cli-help
  • git diff --check

Compatibility coverage uses filesystem fixtures for Xcode applications and installed dependencies. A live Xcode 27 simulator session was not run.

Generated with AI


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

View guided diff Turn on auto-fix

@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 8 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: ed3489bc-2156-4e3e-8d08-18d2971ab58f
Commits

Reviewing files that changed from the base of the PR and between b630cd7 and d2a0ada.

Files selected for processing: 15
  • cli/README.md
  • cli/package.json
  • cli/skills/usage/SKILL.md
  • cli/src/app/info.ts
  • cli/src/index.ts
  • cli/src/init/command.ts
  • cli/src/init/mcp/engine.ts
  • cli/src/init/mcp/live-update-tools.ts
  • cli/src/run/device.ts
  • cli/src/run/ios-compatibility.ts
  • cli/test/test-ios-simulator-compatibility.mjs
  • cli/test/test-mcp-live-update-onboarding.mjs
  • cli/webdocs/doctor.mdx
  • cli/webdocs/init.mdx
  • cli/webdocs/run.mdx
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@riderx
riderx marked this pull request as ready for review October 29, 2026 13:43
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 29, 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-29T13:45:32.001685Z d2a0ada Draft marked ready
ℹ️ 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.

@codspeed

codspeed Bot commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 43 untouched benchmarks
⏩ 2 skipped benchmarks1


Comparing t3/6ac1ee1d (d2a0ada) with main (3f2461b)

Open in CodSpeed

Footnotes

  1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

@riderx
riderx merged commit 401c270 into main Oct 29, 2026
61 of 62 checks passed
@riderx
riderx deleted the t3/6ac1ee1d branch October 29, 2026 13:45
@sonarqubecloud

Copy link
Copy Markdown

This branch was previously deployed

1 inactive deployment
deepsec-pr — d2a0adae Deployed Oct 29, 2026 by riderx via Scan PR changes #10388
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