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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants