github/github-mcp-serverPublic

NotificationsYou must be signed in to change notification settings
Fork2.7k
Star23.4k

Add update project item tool#1194

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

Jump to bottom

Merged

JoannaaKL merged 18 commits intogithub:mainfromJoannaaKL:add-update_project_item-tool

Oct 9, 2025

Merged

Add update project item tool#1194

JoannaaKL merged 18 commits intogithub:mainfromJoannaaKL:add-update_project_item-tool

Oct 9, 2025

+619 −62

Conversation

Copy link

Contributor

JoannaaKL commentedOct 8, 2025•
edited
Loading

This pr:

adds tool to update project item for an user or org
formats all project tools to follow formatting standard
Closes:Add support for Projects #44 (comment)

JoannaaKL added5 commits

October 8, 2025 16:45

Add update project item tool

6d77cde

Update docs

b221d70

Add tests and tool snaps

6ed0475

Merge branch 'main' into add-update_project_item-tool

554bccf

Remove unnecessary comments

2bf55a6

JoannaaKL marked this pull request as ready for review

October 9, 2025 08:00

JoannaaKL requested a review froma team as acode owner

October 9, 2025 08:00

CopilotAI review requested due to automatic review settings

October 9, 2025 08:00

CopilotAI reviewed

Oct 9, 2025

View reviewed changes

Copy link

Contributor

CopilotAI 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 a new tool to update project items for users or organizations, addressing issue#44. The implementation follows the existing pattern of project item management tools by adding theUpdateProjectItem function with comprehensive validation and testing.

AddUpdateProjectItem tool function with field update capabilities
Implement comprehensive test coverage with error handling scenarios
Update project item data structures to support additional fields

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
pkg/github/tools.go	Registers the new UpdateProjectItem tool in the write toolset
pkg/github/projects.go	Implements UpdateProjectItem function with validation and API call logic
pkg/github/projects_test.go	Adds comprehensive test suite covering success and error scenarios
pkg/github/minimal_types.go	Extends MinimalProjectItem struct with Title and Description fields
pkg/github/toolsnaps/update_project_item.snap	Tool schema snapshot for testing validation
README.md	Documents the new update_project_item tool parameters

_{Tip: Customize your code reviews with copilot-instructions.md.Create the file orlearn how to get started.}

pkg/github/projects.go OutdatedShow resolvedHide resolved

JoannaaKL added7 commits

October 9, 2025 10:52

Formatting and fixes

5447a1d

Merge remote-tracking branch 'refs/remotes/origin/add-update_project_…

aa4d088

…item-tool' into add-update_project_item-tool

Merge branch 'main' into add-update_project_item-tool

a01042f

Extract error messages to const

f3414a9

Merge remote-tracking branch 'refs/remotes/origin/add-update_project_…

3f37f45

…item-tool' into add-update_project_item-tool

Fix json tag

eb4d7c4

Rename field

d835623

JoannaaKL force-pushed theadd-update_project_item-tool branch fromc6415d1 tod835623Compare

October 9, 2025 09:09

Copy link

Contributor

tonytrg commentedOct 9, 2025•
edited
Loading

I am having a hard time understandingnew_field, not too familiar with the api of projects but it seems hard to understand to me only from the mcps side of description.

https://docs.github.com/en/rest/projects/items?apiVersion=2022-11-28#update-project-item-for-organization--parameters

fields array of objects Required
A list of field updates to apply.

id integer Required
The ID of the project field to update.

value null or string or number Required
The new value for the field:

For text, number, and date fields, provide the new value directly.
For single select and iteration fields, provide the ID of the option or iteration.
To clear the field, set this to null.

tonytrg reviewed

Oct 9, 2025

View reviewed changes

pkg/github/projects.go

		),
		mcp.WithObject("new_field",
		mcp.Required(),
		mcp.Description("Object consisting of the ID of the project field to update and the new value for the field. To clear the field, set\"value\" to null. Example: {\"id\": 123456,\"value\":\"New Value\"}"),

Copy link

Contributor

tonytrgOct 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

seems like we are looking for a map struct.

Should it look something like this

mcp.Items(map[string]interface{}{"type": "string",},),

Copy link

Contributor

tonytrgOct 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I am not sure if we can rely on the llm to always understand and format a correct parseable struct

Copy link

Contributor

tonytrgOct 9, 2025•
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

ah ok i see it needs to be more flexible

mcp.WithObject("new_field",mcp.Required(),mcp.Description("Object consisting of the ID of the project field to update and the new value for the field. To clear the field, set \"value\" to null."),mcp.Properties(map[string]any{"id": map[string]any{"type":        "number","description": "The ID of the project field to update (required)",},"value": map[string]any{"description": "The new value for the field. Can be a string for text fields, a number for number/single_select/iteration fields, or null to clear the field (required)",},}),

would something like that help?

Copy link

Contributor

almaleksiaOct 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I'd propose we split it into 2 fields which is field_id and field_value

Copy link

ContributorAuthor

JoannaaKLOct 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

How the types would look like then? We can useWithNumber for the field_id but field value needs to stay an object

Copy link

ContributorAuthor

JoannaaKLOct 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

We can do it 👍

Copy link

Contributor

tonytrgOct 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

@almaleksia @JoannaaKL i updated the struct the one before was wrong

JoannaaKL added4 commits

October 9, 2025 11:53

Merge branch 'main' into add-update_project_item-tool

b8fc431

Update params

2032557

Update test

402eb90

Merge remote-tracking branch 'refs/remotes/origin/add-update_project_…

f48befc

…item-tool' into add-update_project_item-tool

JoannaaKL requested review fromalmaleksia andtonytrg

October 9, 2025 12:38

JoannaaKL force-pushed theadd-update_project_item-tool branch fromab0ae59 tof48befcCompare

October 9, 2025 14:26

almaleksia reviewed

Oct 9, 2025

View reviewed changes

pkg/github/projects.go OutdatedShow resolvedHide resolved

almaleksia reviewed

Oct 9, 2025

View reviewed changes

pkg/github/projects.go OutdatedShow resolvedHide resolved

JoannaaKL added2 commits

October 9, 2025 16:30

Update tool example

d1ba41f

.

fc68db5

Copy link

Contributor

tonytrg commentedOct 9, 2025•
edited
Loading

💯 thanks for making this complicated one work

note for future as i dont want to block this merge:

correct parsing of the iteration ids is missing right now, but as that actually should be part of the sdk we can fix that later on

tonytrg approved these changes

Oct 9, 2025

View reviewed changes

JoannaaKL merged commit5c61abe intogithub:main

Oct 9, 2025

10 checks passed

BrewTestBot mentioned this pull request

Oct 10, 2025

github-mcp-server 0.18.0Homebrew/homebrew-core#248127

Merged

Labels

None yet

3 participants

Movatterモバイル変換

Add update project item tool#1194

Add update project item tool#1194

Conversation

JoannaaKL commentedOct 8, 2025• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tonytrg commentedOct 9, 2025• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Uh oh!

tonytrgOct 9, 2025

Choose a reason for hiding this comment

Uh oh!

tonytrgOct 9, 2025

Choose a reason for hiding this comment

Uh oh!

tonytrgOct 9, 2025• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

almaleksiaOct 9, 2025

Choose a reason for hiding this comment

Uh oh!

JoannaaKLOct 9, 2025

Choose a reason for hiding this comment

Uh oh!

JoannaaKLOct 9, 2025

Choose a reason for hiding this comment

Uh oh!

tonytrgOct 9, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

tonytrg commentedOct 9, 2025• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

JoannaaKL commentedOct 8, 2025•
edited
Loading

tonytrg commentedOct 9, 2025•
edited
Loading

tonytrgOct 9, 2025•
edited
Loading

tonytrg commentedOct 9, 2025•
edited
Loading