Skip to content

feat(mcp): add get_reviews tool for PR review submissions#1021

Merged
acreeger merged 1 commit into
mainfrom
feat/get-reviews-mcp-tool
May 26, 2026
Merged

feat(mcp): add get_reviews tool for PR review submissions#1021
acreeger merged 1 commit into
mainfrom
feat/get-reviews-mcp-tool

Conversation

@acreeger

Copy link
Copy Markdown
Collaborator

Summary

  • Adds get_reviews MCP tool that fetches PR review submissions (verdicts + body text from the "Submit review" flow), distinct from inline review comments and PR comments
  • Updates pr-prompt.txt to cross-reference review submissions against inline comments before reporting feedback as "unresolved" — prevents false reports when findings have already been addressed
  • Normalizes pullRequestReviewId from number to string for provider-agnostic ID handling across the VCS abstraction
  • Adds NaN guard on PR number parsing in the get_reviews handler
  • Instructs agents to reply inline to specific comment threads (in_reply_to) rather than posting standalone reviews
  • Adds no-op getReviewSubmissions to BitBucket VCS provider (BitBucket has no equivalent "Submit review" flow)

Test plan

  • 7 new tests for getReviewSubmissions in GitHubIssueManagementProvider.test.ts (normal response, empty body, empty list, repo override, default repo, invalid PR number, null user)
  • Existing pullRequestReviewId test assertions updated for string type
  • All 5236 tests pass
  • Build succeeds
  • Lint and TypeScript compile clean

Adds a new `get_reviews` MCP tool that fetches PR review submissions (the
body text and verdict submitted via the "Submit review" flow). This is
distinct from inline review comments (get_review_comments) and PR comments
(get_pr).

The pr-prompt now cross-references review submissions against inline comments
to identify truly unresolved feedback, preventing false reports when findings
have already been addressed via a review submission.

Also normalizes `pullRequestReviewId` from number to string for
provider-agnostic ID handling, adds NaN guard on PR number parsing, and
instructs agents to reply inline to specific comment threads rather than
posting standalone reviews.
@acreeger acreeger merged commit c7db123 into main May 26, 2026
4 checks passed
@acreeger acreeger deleted the feat/get-reviews-mcp-tool branch May 26, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant