fix(ci): prevent Homebrew background work during local checks - #297
Conversation
Disable Homebrew updates and analytics for the mobile prerequisite check so the owned CI process group can finish cleanly. Preserve missing-tool failures. Carry-Group: build-ci
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. 👍 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Use the same optional-field handling as the CI runner so the Homebrew fixture satisfies exactOptionalPropertyTypes. Carry-Group: build-ci
|
@codex review |
|
Codex Review: Didn't find any major issues. 🎉 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1 participant
Local package builds could fail after Homebrew successfully verified the mobile tools because an analytics process remained in the owned CI process group.
Disable automatic updates and analytics only for this prerequisite check. Keep the existing process ownership guard and propagate missing-tool failures.
Validation: 37 focused tests passed on Node 24; the corrected Homebrew regression cases, scripts-only typecheck, and targeted formatting and lint passed, and the real Homebrew check reproduced the original failure and passed with this change. Local source review found no issues. GitHub CI will provide the full validation; local Quick CI was not run.
Implemented by GPT-6.1-Sol using the Codex harness in T3 Code.