Skip to content

Conversation

@ebarronh
Copy link
Contributor

@ebarronh ebarronh commented Oct 5, 2025

It looks like the -H is being ignored, specially for commands with workflows. -e is working perfectly. Took me a bit to figure it out.

claude mcp add XcodeBuildMCP npx xcodebuildmcp@latest -e XCODEBUILDMCP_ENABLED_WORKFLOWS=simulator,project-discovery,logging

Summary by CodeRabbit

  • Documentation
    • Updated CLI example to use short-form environment flags (-e) instead of legacy host flags (-H).
    • Aligned command syntax in the docs with current CLI conventions for clarity.
    • No functional changes to the product; this is a documentation-only update.

Note

Update README to use -e environment flags instead of -H in the Claude Code CLI example.

Written by Cursor Bugbot for commit 4ae0b61. This will update automatically on new commits. Configure here.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 5, 2025

Walkthrough

Updated README command example: replaced legacy host flags (-H) with short environment flags (-e) for INCREMENTAL_BUILDS_ENABLED and XCODEBUILDMCP_SENTRY_DISABLED. No code or API changes.

Changes

Cohort / File(s) Summary
Docs
README.md
Revised CLI example to use -e env flags instead of -H host flags for two variables; no functional change.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I twitch my nose at flags that change,
From -H to -e—how neat, how strange!
The builds still hop, the docs now gleam,
A tidier path for the same old stream.
Thump-thump—approved! Now onward, team. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title succinctly conveys that the pull request changes flags from -H to -e in the README, directly reflecting the change described without extra noise. It is concise, specific, and clearly communicates the main update.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

@cameroncooke
Copy link
Collaborator

Thanks for the contribution @ebarronh

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 5, 2025

Open in StackBlitz

npm i https://pkg.pr.new/cameroncooke/XcodeBuildMCP/xcodebuildmcp@119

commit: 4ae0b61

@cameroncooke cameroncooke merged commit 5c210b7 into getsentry:main Oct 5, 2025
4 of 6 checks passed
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.

2 participants