feat(apple): Add non-interactive Apple Snapshots mode

[cameroncooke]

Stack

This is 3/3 in the Apple Snapshots stack replacing the original unified PR #1293.

• 1/3: feat(apple): Refactor Xcode project primitives for snapshots #1295 — Xcode project primitives
• 2/3: feat(apple): Add interactive Apple Snapshots wizard #1296 — Interactive Apple Snapshots wizard
• 3/3: feat(apple): Add non-interactive Apple Snapshots mode #1297 — Non-interactive Apple Snapshots mode (this PR)

This is the top of the stack. The final tree of this stack was verified to exactly match original PR #1293.

Summary

Adds unattended/agent-friendly execution for the Apple Snapshots wizard introduced in #1296.

The main goal is that non-interactive mode never waits on stdin. It either proceeds from explicit arguments and safe single-target assumptions, or it fails with actionable manual setup instructions and the CLI flag needed to continue.

What changed

• Adds Apple Snapshots CLI options:
  • --non-interactive
  • --app-target
  • --hosted-test-target
• Threads non-interactive behavior through:
  • CLI/run routing
  • project lookup
  • git safety checks
  • preview/missing-preview confirmation
  • sentry-cli preflight
  • hosted XCTest target selection
• Adds app-target and hosted-test-target resolution behavior for unattended runs.
• Adds prompt suppression paths so agentic setup cannot hang on stdin.
• Updates help output for the new flags.
• Adds tests for non-interactive success/failure paths, explicit target handling, and prompt-free behavior.

Behavior contract in this layer

• App target selection uses --app-target when provided.
• Without --app-target, the wizard auto-selects only when there is a single app target.
• If the app target cannot be resolved safely in non-interactive mode, the wizard fails with guidance to pass --app-target.
• Hosted XCTest target selection uses --hosted-test-target when provided, as long as the target exists and has a non-empty TEST_HOST.
• Without --hosted-test-target, the wizard first narrows hosted XCTest targets to those whose TEST_HOST matches the selected app target's bundle/executable names.
• If app-specific hosted-test matching finds one target, the wizard uses it.
• If app-specific hosted-test matching finds multiple targets, non-interactive mode fails with guidance to pass --hosted-test-target.
• If app-specific hosted-test matching is inconclusive but the project has exactly one hosted XCTest target, the wizard assumes the common case that this target belongs to the selected app.
• If there are multiple hosted XCTest targets and no safe match, non-interactive mode fails with guidance.
• Missing Swift previews are not fatal. Non-interactive mode logs the limitation and continues because agents may still want the project wiring and manual next steps.
• Non-interactive mode must never wait on stdin.

What this PR intentionally does not do

• Does not configure Sentry auth, DSNs, runtime SDK initialization, dSYM upload scripts, Fastlane dSYM setup, MCP config, or CI workflow files.
• Does not change the existing ios wizard into a SnapshotPreviews setup path.
• Does not make unsafe guesses when target resolution is ambiguous.

Suggested review focus

• The target-selection contract for explicit CLI choices, single-target assumptions, and non-interactive failures.
• Hosted XCTest detection via TEST_HOST and app bundle/executable name matching.
• Confirming all non-interactive paths avoid stdin prompts.
• Error messages/manual setup guidance for ambiguous target/project structures.
• Help output and CLI option behavior.

Risk

Medium. This layer controls unattended behavior and is the most important one for agentic setup safety. The main risk is choosing the wrong app/test target in unusual projects or leaving a prompt path reachable in non-interactive mode.

Validation

Before submitting the stack:

• yarn test passed: 947 passed, 4 skipped
• yarn build passed
• Final stack tree matched original PR feat(apple): Add Apple Snapshots wizard #1293 exactly
• gt submit --stack --dry-run passed

Co-Authored-By: GPT-5 Codex noreply@openai.com

[cameroncooke]

• feat(apple): Add non-interactive Apple Snapshots mode #1297 👈 (View in Graphite)
• feat(apple): Add interactive Apple Snapshots wizard #1296
• feat(apple): Refactor Xcode project primitives for snapshots #1295
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Autofix Details

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Setup failures exit zero
  • Added abort() calls at lines 79 and 88 to ensure non-zero exit code when target resolution fails in non-interactive mode.

Or push these changes by commenting:

@cursor push b0e0508f41

Preview (b0e0508f41)

diff --git a/src/apple/snapshots/apple-snapshots-wizard.ts b/src/apple/snapshots/apple-snapshots-wizard.ts
--- a/src/apple/snapshots/apple-snapshots-wizard.ts
+++ b/src/apple/snapshots/apple-snapshots-wizard.ts
@@ -6,6 +6,7 @@
 
 import { withTelemetry } from '../../telemetry';
 import {
+  abort,
   abortIfCancelled,
   askForItemSelection,
   confirmContinueIfNoOrDirtyGitRepo,
@@ -75,7 +76,7 @@
 
   const appTargetName = await resolveAppTargetName(xcProject, options);
   if (!appTargetName) {
-    return;
+    await abort();
   }
 
   const hostedTestTargetName = await resolveHostedTestTargetName(
@@ -84,7 +85,7 @@
     options,
   );
   if (!hostedTestTargetName) {
-    return;
+    await abort();
   }
 
   const selectedTargetHasSwiftPreviews = targetHasSwiftPreviews(

_{You can send follow-ups to the cloud agent here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Wrong non-interactive no-app-target error
  • Separated the zero-targets case from multiple-targets case in non-interactive mode to show 'No application target found.' instead of misleadingly suggesting --app-target when no targets exist.

Or push these changes by commenting:

@cursor push 29a69bdb06

Preview (29a69bdb06)

diff --git a/src/apple/snapshots/apple-snapshots-wizard.ts b/src/apple/snapshots/apple-snapshots-wizard.ts
--- a/src/apple/snapshots/apple-snapshots-wizard.ts
+++ b/src/apple/snapshots/apple-snapshots-wizard.ts
@@ -191,15 +191,22 @@
     return undefined;
   }
 
-  if (options.nonInteractive && appTargetNames.length !== 1) {
-    clack.log.error(
-      [
-        'Could not automatically select an Xcode app target in non-interactive mode.',
-        `Available app targets: ${formatList(appTargetNames)}.`,
-        'Pass --app-target <target-name> to select the app target that SnapshotPreviews should use.',
-      ].join(' '),
-    );
-    return undefined;
+  if (options.nonInteractive) {
+    if (appTargetNames.length === 0) {
+      clack.log.error('No application target found.');
+      return undefined;
+    }
+
+    if (appTargetNames.length !== 1) {
+      clack.log.error(
+        [
+          'Could not automatically select an Xcode app target in non-interactive mode.',
+          `Available app targets: ${formatList(appTargetNames)}.`,
+          'Pass --app-target <target-name> to select the app target that SnapshotPreviews should use.',
+        ].join(' '),
+      );
+      return undefined;
+    }
   }
 
   return await selectXcodeTarget(xcodeProject, {

_{You can send follow-ups to the cloud agent here.}

^{Reviewed by Cursor Bugbot for commit f1f61e5. Configure here.}

[itaybre]

Some comments here and there.
I am mainly concern about using undefined in the tests and not testing for false too

[itaybre]

l: If nonInteractive is either true / false here, why it is optional in other places?

[cameroncooke]

I've simplified this now, this is no longer optional where it's used exclusively by snapshot wizard codepaths, where it was threaded into existing shared functions like for example confirmContinueIfNoOrDirtyGitRepo we keep as an optional as apple-wizard doesn't need to pass argument as it's always interactive this avoid a ton of churn adding in explicit false at all the callsites.

[itaybre]

m: Should we tell the user why it didn't complete

[cameroncooke]

In all abort paths we have clack.log.error(...) which prints the underlying error message.

[itaybre]

LGTM

[cameroncooke]

Merge activity

• Jul 1, 4:39 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jul 1, 4:48 PM UTC: Graphite rebased this pull request as part of a merge.
• Jul 1, 4:53 PM UTC: Graphite couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'All required jobs passed or were skipped', 'cloudflare-worker E2E Tests (macos-15)').