Skip to content

fix: Harden scaffold and utility edge cases#476

Merged
cameroncooke merged 4 commits into
mainfrom
codex/fix/issue-459-warden-findings
Jul 16, 2026
Merged

fix: Harden scaffold and utility edge cases#476
cameroncooke merged 4 commits into
mainfrom
codex/fix/issue-459-warden-findings

Conversation

@cameroncooke

Copy link
Copy Markdown
Collaborator

Fixes the verified correctness, concurrency, and integrity findings reported in #459. iOS scaffolding now emits valid orientation and device-family settings; LLDB drains continue output before later commands and escapes interpolated values; physical-device name lookup no longer blocks the event loop; run-destination parsing works without an active scheme; and template/xcodemake helpers no longer mutate the process-wide working directory.

The xcodemake installer now downloads an exact upstream commit, verifies its SHA-256 digest before writing executable content, and atomically replaces the installed script. Device ID formatting remains synchronous, returning the raw ID until the asynchronously populated name cache is available.

Fixes #459

Correct scaffold settings, isolate LLDB command output, and make device lookup asynchronous.

Also harden xcuserstate parsing, template extraction, and xcodemake installation.

Fixes #459
@pkg-pr-new

pkg-pr-new Bot commented Jul 16, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/xcodebuildmcp@476

commit: 0f2e39a

@cameroncooke
cameroncooke marked this pull request as ready for review July 16, 2026 16:19
Comment thread src/utils/debugger/backends/lldb-cli-backend.ts
Comment thread src/utils/device-name-resolver.ts

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3f77e1b. Configure here.

Comment thread src/utils/debugger/backends/lldb-cli-backend.ts
@cameroncooke
cameroncooke merged commit 60cfdc3 into main Jul 16, 2026
45 checks passed
@cameroncooke
cameroncooke deleted the codex/fix/issue-459-warden-findings branch July 16, 2026 18:55
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.

Warden: code-review

1 participant