Skip to content

docs(ios): document Xcode 27 development setup - #8400

Merged
brow merged 3 commits into
mainfrom
xcode27-readiness
Oct 16, 2026
Merged

brow merged 3 commits into
mainfrom
xcode27-readiness

Conversation

@brow

@brow brow commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator

Document Xcode 27.0 as the recommended iOS development toolchain while retaining Xcode 26 compatibility. Explain per-command selection, the known Flutter native-assets compiler/SDK mismatch on hosts with multiple Xcode versions, and where CI and release toolchains are configured.

The compatibility changes are already merged in #8379 and #8398. Related open work is #8330; this PR changes documentation only. The release version bump remains separate in buzz-releases#110.

Validation

  • Ordinary unsigned Release build passed on main at 73962435b with exact Xcode 27.0 (27A266a), the pinned Flutter SDK, and normal CocoaPods installation. Buzz.app contains NotificationService.appex. No dependency patches or plugin-validation bypasses were used.
  • The merged direct-C change passed native tests on Xcode 26.5 and 27.0, full local CI, and hosted Native and unsigned Release validation. This documentation-only follow-up preserves that code.
  • Documentation diff checks and local pre-push checks passed.

@brow

brow commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-16T17:10:51.362520Z f617441 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cf3224635e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread mobile/README.md Resolved
@brow

brow commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: 5b2b3393ed

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@brow
brow marked this pull request as ready for review October 16, 2026 16:44
@brow
brow requested a review from a team as a code owner October 16, 2026 16:44
@brow
brow requested a review from jedwards27 October 16, 2026 16:44
@github-actions github-actions Bot added the codex-security-review-current The posted Codex security review matches its recorded range. label Oct 16, 2026
@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

🔐 Codex Security Review

Note: This is an automated, security-focused review generated by Codex.
Use it as a supplement to human review; false positives are possible.

Scope

  • Exact PR diff: c8ea4fb4ebdf124a273321287568407258f51a70...f6174413ebe41914c49717efa3e6714e33581630
  • Model: gpt-6.1-sol

💡 Click "edited" above to see earlier reviews for this PR.


Review Summary

Overall Risk: NONE

The authorized PR range only adds iOS toolchain documentation. No concrete security, correctness, or reliability findings were identified.

Findings

No concrete security, correctness, or reliability findings were identified.

Notes

  • No additional limitations were reported.

Generated by Codex Security Review |
Requested by: @brow |
Workflow run

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5b2b3393ed

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread mobile/README.md Resolved

@jedwards27 jedwards27 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.

:bot: Jude’s code review agent

Verdict: REQUEST CHANGES
Reviewed: c8ea4fb4ebdf124a273321287568407258f51a70..5b2b3393edd7b5ef6a73c8eb211d8907e8bec7f4 (exact live head 5b2b3393edd7b5ef6a73c8eb211d8907e8bec7f4)
Risk: low — documentation-only toolchain guidance, but it establishes the default developer setup path.

Blocking finding

[P2] State Xcode 27's hard host requirements beside the recommendation

mobile/README.md:26-28 tells contributors to use Xcode 27.0 by default and identifies Xcode 26 only as a compatibility path, but omits that Xcode 27 requires Apple silicon and macOS Tahoe 26.6 or later. Apple documents both constraints in the Xcode 27 release notes. Intel contributors and Apple-silicon contributors on older macOS are therefore directed toward a toolchain they cannot install or run, despite the supported Xcode 26 path.

Consequence: the primary setup guide can send a supported contributor into an impossible upgrade without telling them that Xcode 26 is the correct branch for their host.

Author action: state the Apple-silicon and macOS Tahoe 26.6+ requirements next to the Xcode 27 recommendation, link the Apple source, and explicitly direct incompatible hosts to supported Xcode 26.

Verification owner: author for the documentation change; reviewer for exact-head link, anchor, and wording validation.

Validation and residual risk

  • PASS — exact base/head pinning and full two-file diff review; git diff --check passed in a clean detached exact-head lane.
  • PASS — CONTRIBUTING.md points to the unique mobile/README.md#ios-toolchain anchor.
  • PASS — the revised matching-default warning covers both Xcode 26 and Xcode 27; DEVELOPER_DIR selection was exercised on Xcode 27.0 (27A266a).
  • PASS — repository source supports the native C/no-plugin claim and the separation between GitHub CI and internal release ownership.
  • Exact-head mobile checks, all three Dart shards, Android, iOS release, security review, Semgrep, zizmor, and DCO were green when refreshed. Mobile Swift Native was still running; that external gate is a confidence gap, not the reason for this verdict.
  • Xcode 26 and an intentionally mismatched global/default Xcode were not independently exercised in this review. Author action for this gap: none. Verification owner: reviewer/tooling or CI/release gate.

The rest of the documentation change is internally consistent. Re-review can approve after the host gate is made explicit at a fresh head.

@github-actions github-actions Bot removed the codex-security-review-current The posted Codex security review matches its recorded range. label Oct 16, 2026
@brow

brow commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator Author

🤖 Addressed the requested host prerequisites in f6174413e. The recommendation now explicitly requires Apple silicon and macOS Tahoe 26.6 or later, links Apple’s release notes, and directs incompatible hosts to a compatible Xcode 26 version. Both requirements were verified against Apple’s release notes; documentation diff and local checks passed.

@brow
brow dismissed jedwards27’s stale review October 16, 2026 17:08

blockers resolved

@brow
brow requested a review from jedwards27 October 16, 2026 17:08
@brow

brow commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@github-actions github-actions Bot added the codex-security-review-current The posted Codex security review matches its recorded range. label Oct 16, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: f6174413eb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@jedwards27 jedwards27 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.

:bot: Jude’s code review agent

Verdict: APPROVE
Reviewed: c8ea4fb4ebdf124a273321287568407258f51a70..f6174413ebe41914c49717efa3e6714e33581630 (exact live head f6174413ebe41914c49717efa3e6714e33581630)
Risk: low — documentation-only developer toolchain guidance.

The prior P2 is resolved. mobile/README.md:26-32 now states that Xcode 27 requires Apple silicon and macOS Tahoe 26.6+, links Apple’s Xcode 27 release notes, and explicitly routes incompatible hosts to the supported Xcode 26 path. That exactly repairs the previously impossible default journey without changing the supported-toolchain contract.

Findings: no unresolved author-actionable defect.
Author action: none.
Verification owner: reviewer/tooling and CI/release gate for residual platform execution only.

Validation

  • PASS — exact one-commit delta 5b2b3393…f6174413 independently compared with the prior requested remedy.
  • PASS — Apple’s current Xcode 27 release-note source contains both claims: macOS Tahoe 26.6+ and Apple-silicon-only.
  • PASS — Xcode 26 is presented as supported for incompatible hosts; the existing DEVELOPER_DIR and matching-default-Xcode warning remains internally consistent.
  • PASS — full PR scope remains two documentation files; Markdown target mobile/README.md#ios-toolchain, native C/no-plugin claim, and CI/release ownership were source-traced in the prior exact-head review and are unchanged by this repair.
  • PASS — exact source files contain no trailing whitespace; current Mobile checks, all three Dart shards, Android, aggregate Mobile, security review, Semgrep, zizmor, and DCO are green.

Confidence gaps / residual risk

  • Mobile Swift Native and Release were still running when refreshed. These external gates do not establish an author defect in this docs-only delta.
  • This reviewer did not independently execute Xcode 26 on Intel/older macOS or an intentionally mismatched default Xcode. The guidance now states the relevant branch honestly; author action: none, verification owner: reviewer/tooling or CI/release gate.

No material finding remains at this exact head.

@brow
brow merged commit 2de484b into main Oct 16, 2026
47 checks passed
@brow
brow deleted the xcode27-readiness branch October 16, 2026 17:35

This branch was previously deployed

1 inactive deployment
codex-review — f6174413 Deployed Oct 16, 2026 by brow via Run Codex Security Review #8558
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex-security-review-current The posted Codex security review matches its recorded range.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants