fix: return project item id usable for updates

Author: njoppi2

pkg/github/projects.go

@@ -6,6 +6,7 @@ import (
 	"fmt"
 	"io"
 	"net/http"
+	"strconv"
 	"time"
 
 	ghErrors "github.com/github/github-mcp-server/pkg/errors"
@@ -1084,7 +1085,8 @@ func addProjectItem(ctx context.Context, gqlClient *githubv4.Client, owner, owne
 	var mutation struct {
 		AddProjectV2ItemByID struct {
 			Item struct {
-				ID githubv4.ID
+				ID             githubv4.ID
+				FullDatabaseID string `graphql:"fullDatabaseId"`
 			}
 		} `graphql:"addProjectV2ItemById(input: $input)"`
 	}
@@ -1110,6 +1112,12 @@ func addProjectItem(ctx context.Context, gqlClient *githubv4.Client, owner, owne
 		"id":      mutation.AddProjectV2ItemByID.Item.ID,
 		"message": fmt.Sprintf("Successfully added %s %s/%s#%d to project %s/%d", itemType, itemOwner, itemRepo, itemNumber, owner, projectNumber),
 	}
+	if fullDatabaseID := mutation.AddProjectV2ItemByID.Item.FullDatabaseID; fullDatabaseID != "" {
+		result["full_database_id"] = fullDatabaseID
+		if itemID, err := strconv.ParseInt(fullDatabaseID, 10, 64); err == nil {
+			result["item_id"] = itemID
+		}
+	}
 
 	r, err := json.Marshal(result)
 	if err != nil {

pkg/github/projects_test.go

@@ -486,7 +486,8 @@ func Test_ProjectsWrite_AddProjectItem(t *testing.T) {
 				struct {
 					AddProjectV2ItemByID struct {
 						Item struct {
-							ID githubv4.ID
+							ID             githubv4.ID
+							FullDatabaseID string `graphql:"fullDatabaseId"`
 						}
 					} `graphql:"addProjectV2ItemById(input: $input)"`
 				}{},
@@ -498,7 +499,8 @@ func Test_ProjectsWrite_AddProjectItem(t *testing.T) {
 				githubv4mock.DataResponse(map[string]any{
 					"addProjectV2ItemById": map[string]any{
 						"item": map[string]any{
-							"id": "PVTI_item1",
+							"id":             "PVTI_item1",
+							"fullDatabaseId": "1001",
 						},
 					},
 				}),
@@ -530,6 +532,8 @@ func Test_ProjectsWrite_AddProjectItem(t *testing.T) {
 		err = json.Unmarshal([]byte(textContent.Text), &response)
 		require.NoError(t, err)
 		assert.NotNil(t, response["id"])
+		assert.Equal(t, float64(1001), response["item_id"])
+		assert.Equal(t, "1001", response["full_database_id"])
 		assert.Contains(t, response["message"], "Successfully added")
 	})
 
@@ -583,7 +587,8 @@ func Test_ProjectsWrite_AddProjectItem(t *testing.T) {
 				struct {
 					AddProjectV2ItemByID struct {
 						Item struct {
-							ID githubv4.ID
+							ID             githubv4.ID
+							FullDatabaseID string `graphql:"fullDatabaseId"`
 						}
 					} `graphql:"addProjectV2ItemById(input: $input)"`
 				}{},
@@ -595,7 +600,8 @@ func Test_ProjectsWrite_AddProjectItem(t *testing.T) {
 				githubv4mock.DataResponse(map[string]any{
 					"addProjectV2ItemById": map[string]any{
 						"item": map[string]any{
-							"id": "PVTI_item2",
+							"id":             "PVTI_item2",
+							"fullDatabaseId": "1002",
 						},
 					},
 				}),
@@ -627,6 +633,8 @@ func Test_ProjectsWrite_AddProjectItem(t *testing.T) {
 		err = json.Unmarshal([]byte(textContent.Text), &response)
 		require.NoError(t, err)
 		assert.NotNil(t, response["id"])
+		assert.Equal(t, float64(1002), response["item_id"])
+		assert.Equal(t, "1002", response["full_database_id"])
 		assert.Contains(t, response["message"], "Successfully added")
 	})