Skip to content

Add github get issue tool #6

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

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

DuongQuangHungEA
Copy link

Add get_issue tool in mcp server. Put GITHUB_TOKEN to Authorization header as optional

DuongQuangHungEA

This comment was marked as resolved.

DuongQuangHungEA

This comment was marked as resolved.

DuongQuangHungEA and others added 2 commits May 2, 2025 14:27
- Implements a new GitHub tool to retrieve the list of files changed in a pull request
- Provides helpful formatting with status, additions, deletions, and summaries
- Includes comprehensive unit and integration tests
- Follows the same pattern as existing GitHub tools

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
DuongQuangHungEA

This comment was marked as resolved.

DuongQuangHungEA and others added 3 commits May 5, 2025 10:27
- Implement list_pull_requests_tool for fetching PRs from GitHub repositories
- Add handler function to process API requests and format results
- Create unit and integration tests for the new tool
- Add tool to module exports

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
DuongQuangHungEA

This comment was marked as resolved.

Copy link

@ThangDPham ThangDPham left a comment

Choose a reason for hiding this comment

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

General review comments on the addition of GitHub tools. Overall, this looks like a solid addition introducing useful GitHub functionality.

Copy link

@ThangDPham ThangDPham left a comment

Choose a reason for hiding this comment

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

LGTM! This PR adds valuable GitHub integration tools. The code is well-structured with good error handling and tests. One suggestion regarding inline comments.

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.

2 participants