Skip to content

fix: restart reviewer after stale running supersession#2263

Open
shrirajpawar4 wants to merge 2 commits into
AgentWrapper:mainfrom
shrirajpawar4:fix/review-run-fails-silently
Open

fix: restart reviewer after stale running supersession#2263
shrirajpawar4 wants to merge 2 commits into
AgentWrapper:mainfrom
shrirajpawar4:fix/review-run-fails-silently

Conversation

@shrirajpawar4

Copy link
Copy Markdown

Summary

  • Restart the shared reviewer terminal when stale running review runs are superseded by a newer PR head
  • Add Destroy to the internal review launcher boundary and wire it to the runtime
  • Keep same-head idempotency and completed-prior-run notify behavior unchanged

Closes #2171

Tests

  • env GOCACHE=/private/tmp/ao-go-cache go test ./internal/review ./internal/service/review ./internal/storage/sqlite/store
  • npm run lint

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.

Restart reviewer terminal when a newer PR head supersedes an in-flight review

1 participant