Skip to content

Mark adressed AI review comments as resolved - #58

Open
masih wants to merge 4 commits into
mainfrom
masih/ai-review-comment-mark-as-resolved
Open

Mark adressed AI review comments as resolved#58
masih wants to merge 4 commits into
mainfrom
masih/ai-review-comment-mark-as-resolved

Conversation

@masih

@masih masih commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

On re-review resolve any PR review comments that are resolved.

On re-review resolve any PR review comments that are resolved.
@cursor

cursor Bot commented Aug 11, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes PR review posting and GraphQL thread resolution; wrong model-supplied thread IDs are filtered but eligible-thread logic must stay correct to avoid resolving unrelated threads.

Overview
Extends the seidroid AI review reusable workflow so re-reviews can close the loop on old inline feedback and optionally run again on every push.

On re-review, preflight now loads GitHub review thread IDs and resolved/unresolved state into PREVIOUS_REVIEW_HISTORY.md. The synthesis prompt and Claude JSON schema add resolved_thread_ids (fixed prior findings) and supersedes_thread_ids on inline comments (same issue, new anchor). After the new review posts, the workflow validates those IDs against unresolved threads from earlier seidroid reviews and calls resolveReviewThread; superseded threads are resolved only when replacement inline comments actually posted. Inline bodies are stripped of duplicate severity prefixes before the workflow adds [blocker] / etc.

Callers can opt into re-review-on-push so pull_request.synchronize triggers another automatic review even when seidroid already reviewed once (default remains one automatic review per PR).

Reviewed by Cursor Bugbot for commit 5d25f86. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5951f95. Configure here.

Comment thread .github/workflows/ai-review.yml Outdated
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