Repository navigation
fix(manager): mark a new pull request waiting while its first CI run is in progress - #640
Merged
TheGreatAxios merged 1 commit intoOct 26, 2026
Conversation
…is in progress A pass whose newest mail for its head is the hub holding it for CI marks the head waiting even before any pull request mail was recorded.
TheGreatAxios
marked this pull request as ready for review
October 26, 2026 19:14
Contributor
|
Triage: Unsure Waiting on:
Some rules could not run this time. Triage will re-check this pull request. |
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Closes CL-11533
Why
When a pull request is opened while a check is already running, the hub's first mail about it says only "wait for CI". The manager triaged a head only once it had a pull request mail, so it skipped that pass and the pull request showed no "Triage: Waiting for CI" label until CI finished.
What changed
No docs changed: ARCHITECTURE.md already describes this behaviour ("a head held for CI is marked waiting when that mail is the newest for it").
Verified
bun --conditions=intx-src test packages/triage-manager: 47 pass. The two new tests, one per host, fail on main.bun run check,bun run lintand the policy check against origin/main are clean.