Skip to content

fix preserve Dependabot reviewer intent#111

Merged
Pigbibi merged 1 commit into
mainfrom
agent/preserve-dependabot-review-intent
Jul 20, 2026
Merged

fix preserve Dependabot reviewer intent#111
Pigbibi merged 1 commit into
mainfrom
agent/preserve-dependabot-review-intent

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • model both review_requested and review_request_removed timeline events
  • paginate the complete PR timeline
  • re-check current reviewer state immediately before deletion
  • preserve the latest human request/removal intent during concurrent updates
  • restore a human request only when it is still the latest human action and currently absent
  • handle concurrent DELETE races without hiding genuine API failures

Why

This closes the remaining reviewer-state races in Dependabot CODEOWNERS cleanup. A maintainer can request or withdraw review while cleanup is queued without having that intent overwritten.

Validation

  • bash syntax check
  • jq event-state fixture check
  • git diff --check
  • no checkout or execution of PR code

Co-Authored-By: Codex <noreply@openai.com>
@github-actions

Copy link
Copy Markdown

🤖 Codex PR Review

Merge allowed: No issues found


Review by Codex PR Review bot • PR

@Pigbibi
Pigbibi merged commit 5a22005 into main Jul 20, 2026
3 checks passed
@Pigbibi
Pigbibi deleted the agent/preserve-dependabot-review-intent branch July 20, 2026 06:35
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