Skip to content

ci: keep a warm owned build over a near seed that would recompile the app - #14444

Merged
teamleaderleo merged 2 commits into
mainfrom
ci/owned-prefer-within-n
Sep 25, 2026
Merged

teamleaderleo merged 2 commits into
mainfrom
ci/owned-prefer-within-n

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Follows #14433 (merged).

prefer still took any bucket seed within CI_OWNED_PREFER_SEED commits, even when the commits since that seed change a package source and the kept build does not. That trades a ~250 s download plus a full app recompile (365 to 1,053 s on owned minis, 2026-09-25) for a start that compiled only the diff. It now runs the same compare as #14433's far-seed case. The kept build stays unless the seed is known not to recompile the app.

A package's Tests/ are not built by the app's schemes, so they no longer count as an app rebuild. On main today, #14421 changed only a package test file, which the first version counted as an app rebuild.

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 3 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: c4998635-db38-4116-a214-b607f29fe48f

📥 Commits

Reviewing files that changed from the base of the PR and between 446c2c4 and ada68e8.

📒 Files selected for processing (2)
  • scripts/ci/owned_build_state.py
  • tests/test_ci_owned_build_state.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.

Base automatically changed from ci/owned-prefer-package-rebuild to main September 25, 2026 09:51
… app

prefer took any bucket seed within CI_OWNED_PREFER_SEED commits, even
when the commits since that seed change a package source and the kept
build does not: a 250 s download followed by a full app recompile, to
replace a start that compiled only the diff. It now asks the same
compare as the far-seed case and keeps the kept build unless the seed is
known not to recompile the app. A package's Tests/ are not built by the
app's schemes and no longer count as an app rebuild.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@teamleaderleo
teamleaderleo force-pushed the ci/owned-prefer-within-n branch from 1ed9b49 to c7a0cf0 Compare September 25, 2026 09:52
@teamleaderleo
teamleaderleo marked this pull request as ready for review September 25, 2026 09:52
@github-actions

Copy link
Copy Markdown
Contributor

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

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

cursor Bot commented Sep 25, 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 enabled auto-merge (squash) September 25, 2026 09:54
@teamleaderleo
teamleaderleo merged commit cc1f806 into main Sep 25, 2026
48 checks passed
@teamleaderleo
teamleaderleo deleted the ci/owned-prefer-within-n branch September 25, 2026 10:13
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