Skip to content

Run slow source-lint guards in parallel - #13497

Merged
teamleaderleo merged 5 commits into
mainfrom
codex/ci-parallel-source-lints
Sep 21, 2026
Merged

teamleaderleo merged 5 commits into
mainfrom
codex/ci-parallel-source-lints

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Split the two independent slow scans in workflow-guard-source-lints into a two-way matrix:

  • sidebar-layout: sidebar lazy-layout policy;
  • dispatch-ownership: stored DispatchWorkItem ownership, including its Bonsplit fixture.

Bonsplit initialization is scoped only to the group that needs it. The reusable workflow keeps the same job ID, so guard-status still receives one fail-closed aggregate.

Measured motivation from #13431's CI run:

  • source-lints job: ~109s;
  • sidebar lazy-layout: ~63s;
  • stored DispatchWorkItem ownership: ~21s;
  • setup/checkout made up the rest.

Related: #13095, #13325.


Summary by cubic

Runs the two slow source-lint guards in parallel to cut CI wall time.

  • Splits workflow-guard-source-lints into a two-way matrix: sidebar-layout and dispatch-ownership run concurrently.
  • Scopes Bonsplit submodule initialization to the dispatch-ownership group only.
  • Keeps the same job ID so the aggregate guard-status still gets one fail-closed result.
  • Adds a test that asserts the matrix and step guards are configured.

Written for commit 7cc98bf. Summary will update on new commits.

Review in cubic

@coderabbitai

coderabbitai Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 16 minutes.

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: 511a0941-22a3-4df5-9709-b33b35d929d0

📥 Commits

Reviewing files that changed from the base of the PR and between 7739f10 and 7cc98bf.

📒 Files selected for processing (2)
  • .github/workflows/ci-guards.yml
  • tests/test_ci_source_lint_guard_structure.py

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.

@cursor

cursor Bot commented Sep 21, 2026

Copy link
Copy Markdown

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

@teamleaderleo
teamleaderleo merged commit b478397 into main Sep 21, 2026
37 of 39 checks passed
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