feat: Autofix CI failures and merge conflicts#137
Conversation
OpenDiff SummaryWhat This PR ChangesThis PR extends Review JudgementOpenDiff completed the review and found 2 warnings to consider before merging. Open Issues Across Reviews
New Issues
Addressed Since Earlier Reviews
Rating: 80/100 Reviewed by opendiff |
There was a problem hiding this comment.
## Status Update
This PR expands the review agent to react to failing CI signals and merge conflicts, adding new webhook handling in apps/review-agent/src/index.ts, GitHub client helpers for associated PRs and check annotations, and new triage flows in apps/review-agent/src/webhook/triage-handler.ts. It also adds prompt plumbing for CI-failure and merge-conflict autofix paths, records those runs in review accounting, and surfaces queue stats and summary comments for the new workflows. The main behavior change is that failed check_run/status events can now enqueue autofix jobs, while pull-request review jobs can try a synthetic base-branch merge before running the normal review flow.
### Overview
-
### Highlighted In This Review
- apps/review-agent/src/index.ts:383 check_run only considers the first associated PR
- apps/review-agent/src/index.ts:720 merge-conflict detection is disabled with autofix
Resolved in a later revision. See the living OpenDiff Summary comment for the current state of this PR.
Remediation Summary✅ 2 issues fixed automatically ✅ Fixed
|
There was a problem hiding this comment.
Status Update
This PR extends apps/review-agent/src/index.ts and apps/review-agent/src/webhook/triage-handler.ts so the review agent can enqueue autofix jobs from failing check_run and status webhooks, gather CI context via new GitHub client helpers, and attempt a synthetic base-branch merge before normal PR review. It also adds prompt plumbing and accounting for CI-failure and merge-conflict autofix flows, plus summary comments and tests around the new queueing and triage behavior. The main effect is that CI failures and merge conflicts can now trigger dedicated remediation paths before or alongside the existing review flow.
Overview
⚠️ 2 warnings
Highlighted In This Review
apps/review-agent/src/index.ts:755review still runs after unresolved merge conflictapps/review-agent/src/index.ts:388one bad associated PR aborts CI autofix resolution
Preview Deployment
Updated: 2026-06-24 21:22:09 UTC |
Summary
Validation