- Notifications
You must be signed in to change notification settings - Fork897
Add tool for getting a single commit (includes stats, files)#216
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR adds a new tool for retrieving detailed information about a single GitHub commit, including associated stats and file changes.
- Registers the new "get_commit" tool in the server configuration.
- Implements the tool in pkg/github/repositories.go with support for pagination.
- Adds corresponding tests in pkg/github/repositories_test.go and updates the README.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pkg/github/server.go | Added tool registration for get_commit |
| pkg/github/repositories_test.go | Added tests to verify proper behavior of the new get_commit tool |
| pkg/github/repositories.go | Implemented the get_commit tool with error handling and pagination |
| README.md | Updated documentation to include the new get_commit tool |
Comments suppressed due to low confidence (1)
pkg/github/repositories.go:18
- [nitpick] The translation key 'TOOL_GET_COMMITS_DESCRIPTION' uses the plural 'commits' while the tool is named 'get_commit'. Consider updating the key for consistency.
mcp.WithDescription(t("TOOL_GET_COMMITS_DESCRIPTION", "Get details for a commit from a GitHub repository")),There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGMT! 🚀
Thank you!
919a10c intomainUh oh!
There was an error while loading.Please reload this page.
While we offer tools to list repo commits today, from an LLM perspective this relies heavily on the accuracy of the commit message to glean any meaning from it. A natural followup may be to dig deeper into a specific commit.
The existing "get commit" endpoint offered by our API allows for more detailed information, including therelative size of the change as well as theactual file changes.
Some similar data is available via
getPullRequestFilesbut this allows more granular/targeted querying.