fix(cli): replace skill installer script with init command

[cameroncooke]

Summary

• replace the standalone scripts/install-skill.sh flow with a built-in xcodebuildmcp init command
• update CLI wiring/help/docs validation and docs references for the new init command
• make init distribution-safe by resolving skills from resource root and including skills/ in portable packaging
• harden uninstall behavior and add init command test coverage

Key changes

• add src/cli/commands/init.ts and tests in src/cli/commands/__tests__/init.test.ts
• register init in src/cli.ts early dispatch and src/cli/yargs-app.ts help/discovery
• add init to scripts/check-docs-cli-commands.js
• remove stale installer rewrite logic from scripts/release.sh
• update README.md and docs/MIGRATION_V2.md to use xcodebuildmcp init
• include and verify skills/ in portable packaging scripts

Validation

• npm run build
• npm run lint (existing warnings in src/daemon.ts, no errors)
• npm run format:check
• npm run typecheck
• npm test
• npm run docs:check
• local distribution smoke tests:
  • npm package install + xcodebuildmcp init --print
  • portable build + verify script (including init --print)
  • local Homebrew tap install + brew test + init --print

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/getsentry/XcodeBuildMCP/xcodebuildmcp@236

commit: c6c83ac

[cameroncooke]

Addressed in 9121d78.\n\n- Replaced redundant with so it now carries user-facing labels.\n- Labels now match intent from the old installer flow:\n - -> \n - -> \n- Updated init command tests to assert the new output strings.

[cameroncooke]

Correction: previous comment formatting was mangled by shell interpolation.

Addressed in 9121d78.

• Replaced redundant skillLabel with skillDisplayName so it now carries user-facing labels.
• Labels now match intent from the old installer flow:
  • mcp -> XcodeBuildMCP (MCP server)
  • cli -> XcodeBuildMCP CLI
• Updated init command tests to assert the new output strings.

[cameroncooke]

Fixed in 99828a1.

What changed:

• resolveTargets is now operation-aware (install vs uninstall).
• For --uninstall with auto-detect and no clients, it now returns an empty target list instead of throwing.
• This allows the handler to reach and print the graceful message: No installed skill directories found to remove.
• Root destination guard text is now neutral: Refusing to use filesystem root as skills destination... (not install-specific).

Also added test coverage for uninstall with no auto-detected clients.

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}

[sentry]

Bug: The init command is dispatched via runInitCommand() before initSentry() is called, so any errors thrown during initialization will not be reported to Sentry.
_{Severity: MEDIUM}

Suggested Fix

Call initSentry({ mode: 'cli' }) at the beginning of the runInitCommand() function. This will ensure that Sentry is initialized before the command logic is executed, allowing it to capture any subsequent errors, similar to the pattern used in startMcpServer().

Prompt for AI Agent

Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: src/cli.ts#L77-L80

Potential issue: The `init` command is dispatched early via `runInitCommand()` at
`src/cli.ts:77-80`, which bypasses the main Sentry initialization on line 81. Unlike
other commands, `runInitCommand()` does not call `initSentry()` itself. As a result, any
errors thrown during the `init` process (e.g., due to file system issues, missing skill
sources, or user cancellation) will propagate up to the main `catch` block without being
captured by Sentry. This creates a monitoring blind spot for a critical user onboarding
command, preventing developers from being alerted to production failures.

_{Did we get this right? 👍 / 👎 to inform future reviews.}