Skip to content

fix(reviewer-bot): prefer current-head reviews and track thread replies#544

Merged
PLeVasseur merged 1 commit intorustfoundation:mainfrom
PLeVasseur:fix/reviewer-bot-freshness-holistic
Mar 26, 2026
Merged

fix(reviewer-bot): prefer current-head reviews and track thread replies#544
PLeVasseur merged 1 commit intorustfoundation:mainfrom
PLeVasseur:fix/reviewer-bot-freshness-holistic

Conversation

@PLeVasseur
Copy link
Copy Markdown
Collaborator

Summary

  • prefer current-head reviewer reviews over newer stale-head reviews when projecting reviewer-response state, and refresh stored reviewer-review state during reconcile and sweeper repair
  • add deferred observer, reconcile, and sweeper coverage for pull_request_review_comment so review-thread replies update freshness state and diagnostics
  • add regression coverage for the #368 stale-review case, the #256 preservation case, and review-comment workflow routing

Testing

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

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 26, 2026

Deploy Preview for scrc-coding-guidelines ready!

Name Link
🔨 Latest commit 3780a5d
🔍 Latest deploy log https://app.netlify.com/projects/scrc-coding-guidelines/deploys/69c5b7cbb482360008cc2fd1
😎 Deploy Preview https://deploy-preview-544--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 20dfff8 into rustfoundation:main Mar 26, 2026
16 checks passed
@PLeVasseur PLeVasseur deleted the fix/reviewer-bot-freshness-holistic branch March 26, 2026 22:54
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