Skip to content

Conversation

@juruen
Copy link
Collaborator

@juruen juruen commented Mar 24, 2025

Context

This PR adds support for the update_issue tool.

Screenshot 2025-03-22 at 07 37 06
Screenshot 2025-03-22 at 07 37 17

Copilot AI review requested due to automatic review settings March 24, 2025 07:05
Copy link
Contributor

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 support for an update_issue tool for modifying issues in a GitHub repository.

  • Introduces the updateIssue function in pkg/github/issues.go.
  • Adds test coverage for the update_issue tool in pkg/github/issues_test.go.
  • Updates README.md with relevant tool documentation.
  • Registers the update_issue tool in pkg/github/server.go.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
pkg/github/issues.go Adds the updateIssue tool with parameters for issue updates.
pkg/github/issues_test.go Provides tests to validate the updateIssue tool behavior.
README.md Documents the new update_issue tool and its parameters.
pkg/github/server.go Registers the update_issue tool (and unintentionally duplicates createIssue).
Comments suppressed due to low confidence (1)

pkg/github/server.go:43

  • Duplicate registration of the createIssue tool is observed which may lead to unintended behavior. Removing the redundant call should resolve this issue.
s.AddTool(createIssue(client, t)) 

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon. Learn more

@juruen juruen force-pushed the juruen/update-issue-tool branch from 8e033fd to 5f5a0a5 Compare March 24, 2025 07:13
@juruen juruen merged commit faa2366 into main Mar 24, 2025
4 checks passed
@juruen juruen deleted the juruen/update-issue-tool branch March 24, 2025 12:25
DaleSeo pushed a commit to DaleSeo/github-mcp-server that referenced this pull request Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants