Merge branch 'main' into custom-workflows

cameroncooke · web-flow · commit a96ef1783c81 · 2026-03-02T13:43:56.000Z
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,10 +2,15 @@
 
 ## [Unreleased]
 
+
 ### Added
 
 - Added support for `customWorkflows` in `.xcodebuildmcp/config.yaml`, so user-defined workflow names can be referenced from `enabledWorkflows` and mapped to explicit tool lists.
 
+### Fixed
+
+- Fixed `swift_package_build`, `swift_package_test`, and `swift_package_clean` swallowing compiler diagnostics on failure by treating empty stderr as falsy, so stdout diagnostics are included in the error response ([#243](https://github.com/getsentry/XcodeBuildMCP/issues/243)).
+
 ## [2.1.0]
 
 ### Added
diff --git a/docs/TOOLS-CLI.md b/docs/TOOLS-CLI.md
@@ -189,4 +189,4 @@ XcodeBuildMCP provides 73 canonical tools organized into 13 workflow groups.
 
 ---
 
-*This documentation is automatically generated by `scripts/update-tools-docs.ts` from the tools manifest. Last updated: 2026-02-28T20:48:25.650Z UTC*
+*This documentation is automatically generated by `scripts/update-tools-docs.ts` from the tools manifest. Last updated: 2026-03-02T13:10:22.681Z UTC*
diff --git a/docs/TOOLS.md b/docs/TOOLS.md
@@ -205,4 +205,4 @@ This document lists MCP tool names as exposed to MCP clients. XcodeBuildMCP prov
 
 ---
 
-*This documentation is automatically generated by `scripts/update-tools-docs.ts` from the tools manifest. Last updated: 2026-02-28T20:48:25.650Z UTC*
+*This documentation is automatically generated by `scripts/update-tools-docs.ts` from the tools manifest. Last updated: 2026-03-02T13:10:22.681Z UTC*
diff --git a/manifests/tools/boot_sim.yaml b/manifests/tools/boot_sim.yaml
@@ -6,7 +6,7 @@ names:
 description: Boot iOS simulator for manual/non-build flows. Not required before simulator build-and-run (build_run_sim).
 annotations:
   title: Boot Simulator
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Open the Simulator app (makes it visible)
     toolId: open_sim
diff --git a/manifests/tools/build_device.yaml b/manifests/tools/build_device.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Build Device
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/build_macos.yaml b/manifests/tools/build_macos.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Build macOS
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/build_run_macos.yaml b/manifests/tools/build_run_macos.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Build Run macOS
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/build_run_sim.yaml b/manifests/tools/build_run_sim.yaml
@@ -8,7 +8,7 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Build Run Simulator
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Capture structured logs (app continues running)
     toolId: start_sim_log_cap
diff --git a/manifests/tools/build_sim.yaml b/manifests/tools/build_sim.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Build Simulator
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/button.yaml b/manifests/tools/button.yaml
@@ -6,4 +6,4 @@ names:
 description: Press simulator hardware button.
 annotations:
   title: Hardware Button
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/gesture.yaml b/manifests/tools/gesture.yaml
@@ -6,4 +6,4 @@ names:
 description: Simulator gesture preset.
 annotations:
   title: Gesture
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/install_app_device.yaml b/manifests/tools/install_app_device.yaml
@@ -6,4 +6,4 @@ names:
 description: Install app on device.
 annotations:
   title: Install App Device
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/install_app_sim.yaml b/manifests/tools/install_app_sim.yaml
@@ -6,7 +6,7 @@ names:
 description: Install app on sim.
 annotations:
   title: Install App Simulator
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Open the Simulator app
     toolId: open_sim
diff --git a/manifests/tools/key_press.yaml b/manifests/tools/key_press.yaml
@@ -6,4 +6,4 @@ names:
 description: Press key by keycode.
 annotations:
   title: Key Press
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/key_sequence.yaml b/manifests/tools/key_sequence.yaml
@@ -6,4 +6,4 @@ names:
 description: Press a sequence of keys by their keycodes.
 annotations:
   title: Key Sequence
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/launch_app_device.yaml b/manifests/tools/launch_app_device.yaml
@@ -6,7 +6,7 @@ names:
 description: Launch app on device.
 annotations:
   title: Launch App Device
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Stop the app
     toolId: stop_app_device
diff --git a/manifests/tools/launch_app_logs_sim.yaml b/manifests/tools/launch_app_logs_sim.yaml
@@ -8,7 +8,7 @@ routing:
   stateful: true
 annotations:
   title: Launch App Logs Simulator
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Stop capture and retrieve logs
     toolId: stop_sim_log_cap
diff --git a/manifests/tools/launch_app_sim.yaml b/manifests/tools/launch_app_sim.yaml
@@ -6,7 +6,7 @@ names:
 description: Launch app on simulator.
 annotations:
   title: Launch App Simulator
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Open Simulator app to see it
     toolId: open_sim
diff --git a/manifests/tools/launch_mac_app.yaml b/manifests/tools/launch_mac_app.yaml
@@ -6,4 +6,4 @@ names:
 description: Launch macOS app.
 annotations:
   title: Launch macOS App
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/long_press.yaml b/manifests/tools/long_press.yaml
@@ -6,4 +6,4 @@ names:
 description: Long press at coords.
 annotations:
   title: Long Press
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/open_sim.yaml b/manifests/tools/open_sim.yaml
@@ -6,7 +6,7 @@ names:
 description: Open Simulator.app for visibility/manual workflows. Not required before simulator build-and-run (build_run_sim).
 annotations:
   title: Open Simulator
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Boot a simulator for manual workflows
     toolId: boot_sim
diff --git a/manifests/tools/record_sim_video.yaml b/manifests/tools/record_sim_video.yaml
@@ -8,7 +8,7 @@ routing:
   stateful: true
 annotations:
   title: Record Simulator Video
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Stop and save the recording
     toolId: record_sim_video
diff --git a/manifests/tools/reset_sim_location.yaml b/manifests/tools/reset_sim_location.yaml
@@ -6,4 +6,4 @@ names:
 description: Reset sim location.
 annotations:
   title: Reset Simulator Location
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/scaffold_ios_project.yaml b/manifests/tools/scaffold_ios_project.yaml
@@ -8,7 +8,7 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Scaffold iOS Project
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: "Important: Before working on the project make sure to read the README.md file in the workspace root directory."
   - label: Build for simulator
diff --git a/manifests/tools/scaffold_macos_project.yaml b/manifests/tools/scaffold_macos_project.yaml
@@ -8,7 +8,7 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Scaffold macOS Project
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: "Important: Before working on the project make sure to read the README.md file in the workspace root directory."
   - label: Build for macOS
diff --git a/manifests/tools/session_clear_defaults.yaml b/manifests/tools/session_clear_defaults.yaml
@@ -6,4 +6,4 @@ names:
 description: Clear session defaults for the active profile or a specified profile.
 annotations:
   title: Clear Session Defaults
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/session_set_defaults.yaml b/manifests/tools/session_set_defaults.yaml
@@ -6,4 +6,4 @@ names:
 description: Set session defaults for the active profile, or for a specified profile and make it active.
 annotations:
   title: Set Session Defaults
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/session_use_defaults_profile.yaml b/manifests/tools/session_use_defaults_profile.yaml
@@ -7,3 +7,4 @@ description: Switch the active session defaults profile.
 annotations:
   title: Use Session Defaults Profile
   readOnlyHint: false
+  destructiveHint: false
diff --git a/manifests/tools/set_sim_appearance.yaml b/manifests/tools/set_sim_appearance.yaml
@@ -6,4 +6,4 @@ names:
 description: Set sim appearance.
 annotations:
   title: Set Simulator Appearance
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/set_sim_location.yaml b/manifests/tools/set_sim_location.yaml
@@ -6,4 +6,4 @@ names:
 description: Set sim location.
 annotations:
   title: Set Simulator Location
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/sim_statusbar.yaml b/manifests/tools/sim_statusbar.yaml
@@ -6,4 +6,4 @@ names:
 description: Set sim status bar network.
 annotations:
   title: Simulator Statusbar
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/start_device_log_cap.yaml b/manifests/tools/start_device_log_cap.yaml
@@ -8,7 +8,7 @@ routing:
   stateful: true
 annotations:
   title: Start Device Log Capture
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Stop capture and retrieve logs
     toolId: stop_device_log_cap
diff --git a/manifests/tools/start_sim_log_cap.yaml b/manifests/tools/start_sim_log_cap.yaml
@@ -8,7 +8,7 @@ routing:
   stateful: true
 annotations:
   title: Start Simulator Log Capture
-  destructiveHint: true
+  destructiveHint: false
 nextSteps:
   - label: Stop capture and retrieve logs
     toolId: stop_sim_log_cap
diff --git a/manifests/tools/stop_app_device.yaml b/manifests/tools/stop_app_device.yaml
@@ -6,4 +6,4 @@ names:
 description: Stop device app.
 annotations:
   title: Stop App Device
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/stop_app_sim.yaml b/manifests/tools/stop_app_sim.yaml
@@ -6,4 +6,4 @@ names:
 description: Stop sim app.
 annotations:
   title: Stop App Simulator
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/stop_device_log_cap.yaml b/manifests/tools/stop_device_log_cap.yaml
@@ -8,4 +8,4 @@ routing:
   stateful: true
 annotations:
   title: Stop Device and Return Logs
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/stop_mac_app.yaml b/manifests/tools/stop_mac_app.yaml
@@ -6,4 +6,4 @@ names:
 description: Stop macOS app.
 annotations:
   title: Stop macOS App
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/stop_sim_log_cap.yaml b/manifests/tools/stop_sim_log_cap.yaml
@@ -8,4 +8,4 @@ routing:
   stateful: true
 annotations:
   title: Stop Simulator and Return Logs
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/swift_package_build.yaml b/manifests/tools/swift_package_build.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Swift Package Build
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/swift_package_run.yaml b/manifests/tools/swift_package_run.yaml
@@ -8,4 +8,4 @@ routing:
   stateful: true
 annotations:
   title: Swift Package Run
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/swift_package_stop.yaml b/manifests/tools/swift_package_stop.yaml
@@ -8,4 +8,4 @@ routing:
   stateful: true
 annotations:
   title: Swift Package Stop
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/swift_package_test.yaml b/manifests/tools/swift_package_test.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Swift Package Test
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/swipe.yaml b/manifests/tools/swipe.yaml
@@ -6,4 +6,4 @@ names:
 description: Swipe between points.
 annotations:
   title: Swipe
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/tap.yaml b/manifests/tools/tap.yaml
@@ -6,4 +6,4 @@ names:
 description: Tap UI element by accessibility id/label (recommended) or coordinates as fallback.
 annotations:
   title: Tap
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/test_device.yaml b/manifests/tools/test_device.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Test Device
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/test_macos.yaml b/manifests/tools/test_macos.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Test macOS
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/test_sim.yaml b/manifests/tools/test_sim.yaml
@@ -8,4 +8,4 @@ predicates:
   - hideWhenXcodeAgentMode
 annotations:
   title: Test Simulator
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/touch.yaml b/manifests/tools/touch.yaml
@@ -6,4 +6,4 @@ names:
 description: Touch down/up at coords.
 annotations:
   title: Touch
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/type_text.yaml b/manifests/tools/type_text.yaml
@@ -6,4 +6,4 @@ names:
 description: Type text.
 annotations:
   title: Type Text
-  destructiveHint: true
+  destructiveHint: false
diff --git a/manifests/tools/xcode_ide_call_tool.yaml b/manifests/tools/xcode_ide_call_tool.yaml
@@ -9,3 +9,4 @@ predicates:
 annotations:
   title: Call Xcode IDE Tool
   readOnlyHint: false
+  destructiveHint: false
diff --git a/manifests/tools/xcode_tools_bridge_disconnect.yaml b/manifests/tools/xcode_tools_bridge_disconnect.yaml
@@ -9,3 +9,4 @@ predicates:
 annotations:
   title: Disconnect Xcode Tools Bridge
   readOnlyHint: false
+  destructiveHint: false
diff --git a/manifests/tools/xcode_tools_bridge_sync.yaml b/manifests/tools/xcode_tools_bridge_sync.yaml
@@ -9,3 +9,4 @@ predicates:
 annotations:
   title: Sync Xcode Tools Bridge
   readOnlyHint: false
+  destructiveHint: false
diff --git a/src/cli/commands/__tests__/init.test.ts b/src/cli/commands/__tests__/init.test.ts
@@ -159,6 +159,30 @@ describe('init command', () => {
       stdoutSpy.mockRestore();
     });
 
+    it('errors with skip reason when only Claude is detected for MCP auto-install', async () => {
+      const fakeHome = join(tempDir, 'home-only-claude');
+      mkdirSync(join(fakeHome, '.claude'), { recursive: true });
+      mockedHomedir.mockReturnValue(fakeHome);
+
+      const yargs = (await import('yargs')).default;
+      const mod = await loadInitModule();
+
+      const app = yargs(['init', '--skill', 'mcp']).scriptName('').fail(false);
+      mod.registerInitCommand(app);
+
+      const stdoutSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+      await expect(app.parseAsync()).rejects.toThrow(
+        'No eligible install targets after applying skill policy. Skipped: Claude Code: MCP skill is unnecessary because Claude Code already uses server instructions.',
+      );
+
+      const output = stdoutSpy.mock.calls.map((c) => String(c[0])).join('');
+      expect(output).toContain(
+        'Skipped Claude Code: MCP skill is unnecessary because Claude Code already uses server instructions.',
+      );
+
+      stdoutSpy.mockRestore();
+    });
+
     it('allows explicit Claude MCP install with --client claude', async () => {
       const fakeHome = join(tempDir, 'home-explicit-claude');
       mkdirSync(join(fakeHome, '.claude'), { recursive: true });
diff --git a/src/cli/commands/init.ts b/src/cli/commands/init.ts
diff --git a/src/mcp/tools/swift-package/__tests__/swift_package_build.test.ts b/src/mcp/tools/swift-package/__tests__/swift_package_build.test.ts
diff --git a/src/mcp/tools/swift-package/__tests__/swift_package_test.test.ts b/src/mcp/tools/swift-package/__tests__/swift_package_test.test.ts
diff --git a/src/mcp/tools/swift-package/swift_package_build.ts b/src/mcp/tools/swift-package/swift_package_build.ts
diff --git a/src/mcp/tools/swift-package/swift_package_clean.ts b/src/mcp/tools/swift-package/swift_package_clean.ts
diff --git a/src/mcp/tools/swift-package/swift_package_test.ts b/src/mcp/tools/swift-package/swift_package_test.ts
