Skip to content

ci: give react-apps-check 40 minutes (cx-v6fbz) - #20854

Merged
teamleaderleo merged 2 commits into
feat-cmux-nextfrom
web-check-cap
Oct 15, 2026
Merged

teamleaderleo merged 2 commits into
feat-cmux-nextfrom
web-check-cap

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Split from #20733 so that PR does not widen its web scope to the web/ unit shards.

Summary

  • ci-web.yml: react-apps-check job cap 30 -> 40 min. On 2026-10-14 the test step alone took 22 min (277 files, about 90 of them dev-server files two at a time), and the --with-deps engine install took 2.5 min on one attempt and 11.5 on the next, which left the tests 17 min, so a rerun was cancelled at 30 (Transcript: a followed reply moves only by what is drawn #20697).

Verification

  • YAML-only change; no local run. Editing ci-web.yml runs every web job here, so this PR also shows whether the web/ unit shards are green on current base.

Changelog

none

🤖 Generated with Claude Code


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


Summary by cubic

Bumps the react-apps-check job timeout in ci-web.yml from 30 to 40 minutes so it no longer gets cancelled mid-run. On 2026-10-14 the test step alone took 22 minutes and a slow engine install left only 17 minutes for the tests, causing a cancelled rerun at the 30-minute cap (cx-v6fbz).

Written for commit e6421ca. Summary will update on new commits.

View guided diff Turn on auto-fix

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

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 1138a905-f21d-4372-b938-8b70a2a55147

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

all web/size checks green

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Passes: CI passes on e6421ca3dd.

CI passes on e6421ca3dd (run 38687456373 attempt 1).

Written by scripts/ci/classify_failures.py (ci-failure-attribution.yml); signatures are its SIGNATURES table. A machine verdict is the runner's fault, not this PR's; yours means the failing file is one this PR changes, also red on main that main's latest full suite fails the same way, seen on other PRs that it failed on another pull request's run lately.

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

CI fast guards passes on e6421ca3dd (https://github.com/manaflow-ai/cmux/actions/runs/38687456373). This covers only the fast guards, not CI: CI's result is the ci-status check, and the CI failure attribution comment names any failing test.

…k-cap

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

Copy link
Copy Markdown
Collaborator Author

Merge note: merged with one check run, compile, still queued (it never started and is not a required check or a feat-cmux-next gate). Every gate check passed on this exact head: react-apps-check, Web tests 1-4/4, Web status, CmuxNextApp size cap and crash ratchet.

@teamleaderleo
teamleaderleo merged commit 1985668 into feat-cmux-next Oct 15, 2026
83 checks passed
@teamleaderleo
teamleaderleo deleted the web-check-cap branch October 15, 2026 07:35
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