Skip to content

Add gerrit_get_file_diff MCP tool and diff-anchored draft comments#11

Draft
adityabhat-1 wants to merge 1 commit into
mainfrom
comment-navigation-loading-fix
Draft

Add gerrit_get_file_diff MCP tool and diff-anchored draft comments#11
adityabhat-1 wants to merge 1 commit into
mainfrom
comment-navigation-loading-fix

Conversation

@adityabhat-1

@adityabhat-1 adityabhat-1 commented Apr 21, 2026

Copy link
Copy Markdown
Collaborator

adds mcp tool to ensure that comments are posted at the right locations

Adds a new gerrit_get_file_diff MCP tool that returns simplified hunk ranges (aStart/aLines on PARENT, bStart/bLines on REVISION) so the agent reviewer can pick exact line numbers and the correct side before posting.

gerrit_post_draft_comment now accepts side (PARENT/REVISION) and range for multi-line regions.

The AI review prompt is updated to require fetching the diff in a new step 3b and to only anchor comments on lines that actually appear in a hunk.

Made-with: Cursor

@adityabhat-1
adityabhat-1 marked this pull request as draft April 21, 2026 10:14
@adityabhat-1
adityabhat-1 marked this pull request as ready for review April 29, 2026 18:49
@adityabhat-1
adityabhat-1 force-pushed the comment-navigation-loading-fix branch from 99637bf to 4486c34 Compare April 29, 2026 18:55
@adityabhat-1
adityabhat-1 marked this pull request as draft April 30, 2026 08:54
Adds a new gerrit_get_file_diff MCP tool that returns simplified hunk ranges (aStart/aLines on PARENT, bStart/bLines on REVISION) so the agent reviewer can pick exact line numbers and the correct side before posting.

gerrit_post_draft_comment now accepts side (PARENT/REVISION) and range for multi-line regions.

The AI review prompt is updated to require fetching the diff in a new step 3b and to only anchor comments on lines that actually appear in a hunk.

Made-with: Cursor
@adityabhat-1
adityabhat-1 force-pushed the comment-navigation-loading-fix branch from 4486c34 to 8468164 Compare April 30, 2026 09:09
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