- Notifications
You must be signed in to change notification settings - Fork2.7k
Remove update_project_item tool#1167
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 removes theupdate_project_item tool from the GitHub MCP Server, streamlining the project management functionality to focus on add and delete operations only.
- Removes the
UpdateProjectItemfunction and its registration - Removes comprehensive test coverage for the update functionality
- Updates documentation to reflect the removal
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pkg/github/tools.go | Removes registration of UpdateProjectItem tool from the toolset |
| pkg/github/projects.go | Removes the complete UpdateProjectItem function implementation |
| pkg/github/projects_test.go | Removes all test cases for UpdateProjectItem functionality |
| go.mod | Changes go-querystring dependency from indirect to direct |
| README.md | Removes update_project_item tool documentation |
Tip: Customize your code reviews with copilot-instructions.md.Create the file orlearn how to get started.
pkg/github/projects.go Outdated
| typeupdateProjectItemOptionsstruct { | ||
| Fields []*newProjectV2Field`json:"fields,omitempty"` | ||
| } |
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.
TheupdateProjectItemOptions struct is no longer used after removing theUpdateProjectItem function. This struct should be removed to avoid dead code.
See below for a potential fix:
Copilot uses AI. Check for mistakes.
…cts-tool' into remove-update-projects-tool
6d01897 intogithub:mainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Delete tool that is failing due to schema issues
Old issue with json array typemodelcontextprotocol/modelcontextprotocol#410