ci: skip smart e2e ai selection for cherry-pick PRs targeting release branch#29658
Merged
Conversation
… branch Re-introduces the release-branch skip logic from #28331 on top of #29305's refactor. Cherry-pick PRs into release/* branches now bypass AI selection and run the full E2E suite — release cherry-picks are low-volume and high-impact, and we cannot afford a partial CI signal there. Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
9d2049e to
94855dc
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #29658 +/- ##
==========================================
- Coverage 82.15% 81.84% -0.32%
==========================================
Files 5178 5244 +66
Lines 137450 138672 +1222
Branches 31079 31460 +381
==========================================
+ Hits 112924 113498 +574
- Misses 16875 17449 +574
- Partials 7651 7725 +74 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
chrisleewilcox
approved these changes
May 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Re-introduces the release-branch skip in
.github/actions/smart-e2e-selection/action.yml.The original logic was added in #28331 and accidentally removed by #29305 ("ci: add automated E2E readiness label and CI checks - Part1"), which refactored the action and didn't carry the release-branch handling forward. Since the merge of #29305, every cherry-pick PR targeting
release/*runs the AI-based reduced E2E selection — same as amainPR — instead of running the full suite.Why release branches deserve different handling (same reasoning as #28331):
main, iterating quickly with a risk-scored subset is a reasonable trade-off — we accept some residual risk for speed and catch the rest in follow-up PRs.release/*is effectively "this build is a candidate to ship." Release cherry-picks are low-volume and high-impact, and we cannot afford to merge a cherry-pick that breaks CI on the release branch under release urgency.Changelog
CHANGELOG entry: null
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/INFRA-3464
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Made with Cursor
Note
Low Risk
Low risk CI-only change that just alters when the composite action runs AI selection vs defaulting to the full E2E suite; main risk is unintentionally skipping AI selection if
base-refis mis-set.Overview
Smart E2E selection now skips AI for PRs targeting
release/*. Thesmart-e2e-selectioncomposite action adds acheck-release-targetgate usinginputs.base-ref; when it matchesrelease/*, the action bypasses checkout/deps/analysis and keeps the defaultai_e2e_test_tags=["ALL"]withai_confidence=100.The action’s
base-refinput description is updated to document this behavior, and the AI analysis step now reports a specific skip reason for release-branch targets (in addition to the existing skip-label behavior).Reviewed by Cursor Bugbot for commit 94855dc. Bugbot is set up for automated code reviews on this repo. Configure here.