Skip to content

merge-main: run local guards only when asked - #15016

Merged
teamleaderleo merged 1 commit into
mainfrom
ci/merge-main-guards-opt-in
Sep 27, 2026
Merged

teamleaderleo merged 1 commit into
mainfrom
ci/merge-main-guards-opt-in

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

scripts/merge-main.sh ran the ci guard group after every merge, and that group includes swift test on packages and other builds. CLAUDE.md sends every agent through this script to merge main. On 2026-09-27 many agents doing that at once pushed a MacBook Air's load average past 300.

  • Merge only, by default. --guards runs the ci guards locally after the merge; --all-guards and --strict imply it.
  • --no-guards still parses, as a no-op, for older callers.
  • Pushing runs the same guards in CI, which is where a branch's guard verdict comes from anyway.
  • merge_main.Options keeps its default, so the tests that build it directly are unchanged.

Docs: the flag table and step 4 in docs/ci/merge-main.md, and the header of scripts/merge-main.sh.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.


Summary by cubic

Makes local guard runs opt-in in merge-main.sh so merging main no longer runs swift test and package builds on the laptop by default.

  • --guards runs the ci guards locally after the merge; --all-guards and --strict imply it.
  • --no-guards is now the default and remains accepted as a no-op for older callers.
  • Pushing runs the same guards in CI, which is where a branch's guard verdict comes from anyway.
  • merge_main.Options keeps its default so tests that build it directly are unchanged.
  • Updates the flag table and step 4 in docs/ci/merge-main.md and the header of scripts/merge-main.sh.

Written for commit 1af3281. Summary will update on new commits.

Review in cubic

The guards include swift test and package builds. With every agent
merging main through this script, a laptop's load passed 300
(2026-09-27). Merge only by default; --guards (or --all-guards,
--strict) runs them, and pushing runs the same guards in CI.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 27, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 25 seconds.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: e6075576-aecf-4d21-8d8d-b5199d735466

📥 Commits

Reviewing files that changed from the base of the PR and between 5e19a98 and 1af3281.

📒 Files selected for processing (3)
  • docs/ci/merge-main.md
  • scripts/ci/merge_main.py
  • scripts/merge-main.sh

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.

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@teamleaderleo
teamleaderleo merged commit 9ce512a into main Sep 27, 2026
50 of 51 checks passed
@teamleaderleo
teamleaderleo deleted the ci/merge-main-guards-opt-in branch September 27, 2026 14:38
@github-actions

Copy link
Copy Markdown
Contributor

Merge receipt for 1af3281356: every check was green at merge (15 verified; 15 skipped by policy). Full suite runs on main after merge.

rustybret pushed a commit to rustybret/bmux that referenced this pull request Sep 27, 2026
648d5c1 Add a Paste Last Screenshot action with an unbound shortcut (manaflow-ai#14955)
ff61677 ci: avoid partial blobs in catch-up merges (manaflow-ai#15023)
4d0d112 ci: retry transient catch-up GraphQL failures (manaflow-ai#15021)
212e808 ci: attribution scores a lone suspect and reports app-host crashes apart (manaflow-ai#14952)
4cabdf4 test: settle the window before measuring the unread sidebar-row invalidation (manaflow-ai#14568)
12ec99b Add a release-media capture tool for changelog screenshots and clips (manaflow-ai#15010)
ee2cda0 Backfill Unreleased changelog and draft next release cards (manaflow-ai#14999)
be4adf8 Show a brief notice when Cmd+V fails on an oversized image or a timeout (manaflow-ai#14953)
23d22d7 ci: an owned pool the run starts on now beats an earlier one it queues on (manaflow-ai#14993)
05d0190 ci: catch-up posts once per head, says less, and merges inserted declarations (manaflow-ai#15018)
4ee4b21 ci: fail stalled Swift package tests instead of waiting out the job timeout (manaflow-ai#14997)
9ce512a merge-main: run local guards only when asked (manaflow-ai#15016)
d60108a ci: clear test-e2e's fixed DerivedData with clear-dirs.sh (manaflow-ai#14994)
1d7895e ci: run the shell and CLI no-socket lanes in parallel (manaflow-ai#14990)
6e7d25f Honor macOS Differentiate Without Color, Increase Contrast and Reduce Transparency (manaflow-ai#14991)
966b355 Stop interrupting focused work: sidebar jumps, Computer Use focus steal, quit dialog on logout (manaflow-ai#14961)
e1f1cb2 Strip control characters from feedback attachment filenames (manaflow-ai#14783)
0758c9f test: find the onboarding window the test presented, not a leftover (manaflow-ai#15015)
b35c540 fix(spm): resolve GhosttyKit/GhosttyRuntimeTestStubs target name collisions (manaflow-ai#10569)
ef33bed Map .purs artifacts to the Haskell highlight.js grammar (manaflow-ai#14202)
e2a167a Highlight Elixir and Erlang files in the file editor (manaflow-ai#13732)
972c449 fix: wrap Linux browser download card label (manaflow-ai#11157)
f563884 Add Aside to browser data import detection (manaflow-ai#13379)
091d0ea Add cmux send --paste and hint at it for large multi-line sends (manaflow-ai#14937)
3ffcdbb test(ios): keep folder-tap stat tests off the real 2 s deadline (manaflow-ai#15017)
68d3936 test: keep CmuxTerminal pasteboard tests off the cooperative pool (manaflow-ai#15006)
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.

1 participant