fix(ci): keep dispatched and scheduled CI runs alive through main churn#979
Conversation
Give workflow_dispatch and schedule events a per-run concurrency group so pushes to the same ref no longer cancel deliberate one-off runs (the 70-minute release-browser-matrix was repeatedly killed mid-flight by main merges). Push and pull_request runs keep the shared ref group with cancel-in-progress, so superseded heads still stop early. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EXsJcLrbZUXwnBeG91cVo9
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EXsJcLrbZUXwnBeG91cVo9
|
Warning Review limit reached
Next review available in: 4 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Summary
concurrencygroup in.github/workflows/ci.yml:workflow_dispatchandscheduleevents now use a per-run group (github.run_id) instead of the sharedCI-<ref>group, so pushes to the same ref can no longer cancel deliberate one-off runs mid-flight. Push andpull_requestruns keep the shared ref group withcancel-in-progress: true, so superseded PR heads still stop early exactly as today.release-browser-matrix(WebKit/Firefox full UI matrix — the only real coverage for those engines) was cancelled twice on 2026-07-19/20 by main merges landing minutes into the run, and the weekly Sunday 18:00 UTC scheduled run sits in the same cancel-in-progress group, so it is subject to the identical livelock. On a repo where main merges every few minutes, the matrix could effectively never complete.docs/branch-review-ledger.md).Verification
npm run check:github-actions— pin check passed (no newuses:, noubuntu-latest; the checker does not constrain theconcurrencyblock)npm run check:ci-scope— self-test passed (classifier change scope unaffected; this PR classifies asworkflow_changed)npm run format:check— repo files cleanVerification not run:
npm run verify:pr-localfull gate — the diff is a workflow-file expression change plus a ledger row; the three checks above are the complete static surface for it (no source, test, or build behavior changes). UI verification not run: no UI change.Risk and rollout
(github.event_name == 'workflow_dispatch' || github.event_name == 'schedule') && github.run_id || github.reffollows the standard and/or ternary idiom;github.run_idis always non-empty so the fallback can never misfire. Accepted side effect: a deliberate dispatch/scheduled run can now execute concurrently with push-triggered runs on the same ref (rare — dispatches are manual, the schedule is weekly; bounded runner cost).release-browser-matrixis not a required branch-protection check, so no merge-gating semantics change.Clinical Governance Preflight
Not applicable — CI concurrency grouping only; no ingestion, answer generation, search/ranking, source rendering, document access, privacy, production env, or clinical output surface is touched.
Notes
🤖 Generated with Claude Code
https://claude.ai/code/session_01EXsJcLrbZUXwnBeG91cVo9
Generated by Claude Code