Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat:(github) get_pull_request_comments and get_pull_request_reviews #558

Conversation

monkeydaichan
Copy link
Contributor

** Should be merged #557 **

Description

Add new tools to GitHub MCP server for retrieving pull request review comments and reviews.

Server Details

  • Server: github
  • Changes to: tools
    • Added get_pull_request_comments tool
    • Added get_pull_request_reviews tool

Motivation and Context

These new tools will enable LLM clients to fetch and analyze pull request review comments and reviews, enabling better code review assistance and collaboration features.

How Has This Been Tested?

  • Tested with sample pull requests to verify comment and review retrieval
  • Verified correct parsing of review comments and review data
  • Confirmed proper error handling for invalid requests

Breaking Changes

No breaking changes. These are new additions that don't affect existing functionality.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Protocol Documentation
  • My changes follows MCP security best practices
  • I have updated the server's README accordingly
  • I have tested this with an LLM client
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have documented all environment variables and configuration options

Additional context

  • Uses existing GitHub API client configuration
  • Implements proper error handling for rate limits and API errors
  • Returns JSON-formatted response for easy parsing by LLM clients

@monkeydaichan monkeydaichan force-pushed the feature/get-pr-review-comments branch from c629b9b to 46e54d7 Compare January 23, 2025 05:25
@monkeydaichan monkeydaichan deleted the feature/get-pr-review-comments branch January 23, 2025 05:57
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