feat: add /design-review-ios with XcodeBuildMCP integration#262
Open
Skyfox-io wants to merge 1 commit intogarrytan:mainfrom
Open
feat: add /design-review-ios with XcodeBuildMCP integration#262Skyfox-io wants to merge 1 commit intogarrytan:mainfrom
Skyfox-io wants to merge 1 commit intogarrytan:mainfrom
Conversation
Author
|
Updated: MCP-first architecture Reworked this from CLI-only to MCP-first with CLI fallback. XcodeBuildMCP is fundamentally an MCP server, and the CLI is the secondary interface. Making MCP the primary integration means:
CLI remains fully supported as a fallback for users without the MCP server configured. Force-pushed to squash into a clean commit since the architectural premise changed. |
…ration iOS design QA via simulator: finds visual inconsistency, spacing issues, HIG violations, accessibility gaps, and AI slop patterns, then fixes them. Auto-detects integration mode at startup: - MCP server (preferred): calls XcodeBuildMCP tools directly (build_run_sim, screenshot, tap, snapshot_ui, etc.) with no Bash, no PATH issues, no DEVELOPER_DIR workaround - CLI fallback: uses xcodebuildmcp CLI via Bash when MCP tools unavailable Covers 10 design categories (~107 checklist items) across typography, HIG compliance, accessibility, light/dark parity, AI slop detection, and multi-device matrix testing. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
58fc965 to
22cc23b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/design-review-iosskill builds, screenshots, and interacts with iOS apps on the simulator — MCP server preferred, CLI fallbackWhat this enables
gstack can now audit iOS apps the same way
/design-reviewaudits web apps, using the iOS Simulator via XcodeBuildMCP. The skill covers 10 design categories (~107 checklist items):Integration architecture
The skill auto-detects the best integration mode at startup:
build_run_sim,screenshot,tap,snapshot_ui, etc.) — no Bash, no PATH issues, noDEVELOPER_DIRworkaroundxcodebuildmcpCLI via BashThis means zero-config for users who already have XcodeBuildMCP set up as an MCP server (
claude mcp add xcodebuildmcp).How it works
discover_projs/list_schemes/list_simsbuild_run_simscreenshotandsnapshot_uitap,gesture,swipe)Requirements
claude mcp add xcodebuildmcp -- npx -y xcodebuildmcp@latest mcp) or CLI (brew install getsentry/xcodebuildmcp/xcodebuildmcp)Test plan
/design-review-ioson any iOS project with XcodeBuildMCP configured (MCP server or CLI)