Skip to content

Stop review/implementation cycle when PR is merged or closed#19

Merged
vkehfdl1 merged 1 commit intomainfrom
Feature/stop-on-pr-close
Apr 6, 2026
Merged

Stop review/implementation cycle when PR is merged or closed#19
vkehfdl1 merged 1 commit intomainfrom
Feature/stop-on-pr-close

Conversation

@vkehfdl1
Copy link
Copy Markdown
Contributor

@vkehfdl1 vkehfdl1 commented Apr 6, 2026

Summary

  • Fixes The review round - implementation still progress after PR is merged or closed #17: review round / implementation / final verdict agent handlers now check PR state before enqueueing the next job
  • Added _is_pr_open() guard in all 4 agent event handlers — returns {"status": "ignored", "reason": "pr_not_open"} when PR is no longer open
  • 3 new tests covering review round, implementation, and final verdict stopping on closed PR

Test plan

  • uv run pytest — 79 tests pass
  • Manually close/merge a PR mid-cycle and verify no further jobs are enqueued

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vkehfdl1 vkehfdl1 merged commit 5c3ea45 into main Apr 6, 2026
6 checks passed
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.

The review round - implementation still progress after PR is merged or closed

1 participant