Skip to content

fix: ignore superseded top-level review feedback#120

Merged
haasonsaas merged 1 commit into
mainfrom
codex/review-guard-current-head-reviews-20260521
May 21, 2026
Merged

fix: ignore superseded top-level review feedback#120
haasonsaas merged 1 commit into
mainfrom
codex/review-guard-current-head-reviews-20260521

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • teach the review-thread guard to compare top-level review feedback against the current PR head
  • keep blocking current-head high/P1 review bodies, but stop failing generated mirror PRs on stale Codex reviews from superseded commits
  • add a regression for old-head top-level review feedback plus latest-head blocking feedback

Validation

  • ruby -Itest -e 'ARGV.each { |path| require "./#{path}" }' test/*_test.rb

@haasonsaas haasonsaas force-pushed the codex/review-guard-current-head-reviews-20260521 branch from 55b8a2b to 692b8bd Compare May 21, 2026 17:10
@haasonsaas haasonsaas merged commit d45c0bc into main May 21, 2026
3 checks passed
@haasonsaas haasonsaas deleted the codex/review-guard-current-head-reviews-20260521 branch May 21, 2026 17:13
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