Skip to content

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

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

Conversation

artemsaveliev
Copy link

Implements 3 tools dealing with issue events and timeline:

@Copilot Copilot AI review requested due to automatic review settings April 10, 2025 06:45
Copy link
Contributor

@Copilot Copilot AI left a 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),

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants