-
Notifications
You must be signed in to change notification settings - Fork 881
Added get_issue_events , get_issue_timeline and get_issue_event tools #211
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
base: main
Are you sure you want to change the base?
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 three new tools to handle GitHub issue events and timeline retrieval with associated tests and documentation updates.
- Added tool registrations in the server to support get_issue_timeline, get_issue_events, and get_issue_event.
- Implemented the corresponding functions in pkg/github/issues.go and added comprehensive tests in pkg/github/issues_test.go.
- Updated README.md to document the new tools.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
File | Description |
---|---|
pkg/github/server.go | Registers new tools and adds a helper function for int64 parameter conversion. |
pkg/github/issues_test.go | Provides tests for the new tools covering success and error scenarios. |
pkg/github/issues.go | Implements tool functions for issue timeline, events, and a single event retrieval. |
README.md | Updates documentation for the new tools with expected input parameters. |
Comments suppressed due to low confidence (2)
README.md:167
- The parameter name 'perPage' in the documentation does not match the tool definition which uses 'per_page'. Consider updating the README to use consistent naming.
- `perPage`: Results per page (number, optional)
pkg/github/issues_test.go:1434
- [nitpick] The test for 'get_issue_event' uses an event_id value of 42 while the expected mock event has a different ID (17196710688). Consider aligning the event_id values to avoid potential confusion.
"event_id": float64(42),
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.
I think with the pagination helper and the annotations this is good to merge. The code is great, and I'm curious to see what models do with the tools!
Thanks again @artemsaveliev
Implements 3 tools dealing with issue events and timeline: