Skip to content

fix(reviewer-bot): stop transition spam and repair freshness recovery#540

Merged
PLeVasseur merged 6 commits intorustfoundation:mainfrom
PLeVasseur:fix/reviewer-bot-transition-spam-and-freshness
Mar 26, 2026
Merged

fix(reviewer-bot): stop transition spam and repair freshness recovery#540
PLeVasseur merged 6 commits intorustfoundation:mainfrom
PLeVasseur:fix/reviewer-bot-transition-spam-and-freshness

Conversation

@PLeVasseur
Copy link
Copy Markdown
Collaborator

Summary

  • make transition notices one-shot, remove the false reassignment message, and backfill existing transition notices so already-affected PRs stop spamming
  • simplify the PR comment observer workflow to build deferred payloads inline instead of bootstrapping bot source at runtime
  • repair reviewer freshness recovery by clearing transition markers on reviewer activity, filtering bot-authored comment gaps, and conservatively recovering from visible submitted reviews

Testing

  • uv run pytest .github/reviewer-bot-tests/test_reviewer_bot.py
  • uv run pytest .github/reviewer-bot-tests/test_main.py
  • uv run ruff check --fix

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 26, 2026

Deploy Preview for scrc-coding-guidelines ready!

Name Link
🔨 Latest commit 2d29dbe
🔍 Latest deploy log https://app.netlify.com/projects/scrc-coding-guidelines/deploys/69c4b2fd959fe10008d8a2bd
😎 Deploy Preview https://deploy-preview-540--scrc-coding-guidelines.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@PLeVasseur PLeVasseur merged commit f30bfab into rustfoundation:main Mar 26, 2026
16 checks passed
@PLeVasseur PLeVasseur deleted the fix/reviewer-bot-transition-spam-and-freshness branch March 26, 2026 04:19
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