Skip to content

[Fix] Review labels disappear after base updates - #1584

Draft
zoomote[bot] wants to merge 1 commit into
mainfrom
fix/preserve-review-labels-2xja5g1mig879
Draft

[Fix] Review labels disappear after base updates#1584
zoomote[bot] wants to merge 1 commit into
mainfrom
fix/preserve-review-labels-2xja5g1mig879

Conversation

@zoomote

@zoomote zoomote Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

​Created by Roomote. Follow up by mentioning @roomote, in the web UI, or in Discord.

What changed

  • Preserve the current PR review-state label while GitHub reports mergeability as unknown.
  • Keep the review gate pending and update the review guide until mergeability is known.
  • Cover the transient state with a regression test that verifies the maintainer label remains, the completed CodeRabbit activation label is removed, and the pending guidance is published.

Why this change was made

The all-open-PR reconciliation triggered by the merge of #1577 ran while GitHub was recalculating mergeability. Although CodeRabbit approvals remained attached to the exact PR head commits, the unknown-mergeability branch removed otherwise-valid awaiting-maintainer labels, making the queue appear to collapse.

Impact

Base-branch pushes no longer temporarily erase review-state labels while GitHub recalculates mergeability. Confirmed conflicts still transition to has-conflicts, and the hourly/event-driven reconciliation still replaces preserved labels once GitHub returns a definitive state.

Follow-up to #1577 and #1509.

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • coderabbit-review-active

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: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 1da1759f-de2e-49cd-82fd-52c67bdeb8d4

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

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 Sep 10, 2026

Copy link
Copy Markdown
Contributor

Review status

This PR was opened by an automated account. A human maintainer must verify the change intent, provenance, and validation before merging.

Current step: Mark the PR ready. Required CI must pass before CodeRabbit starts.

Review-state labels are managed by this workflow; do not edit them manually.

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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