Remove update_project_item tool (#1167)

* Remove update_project_item tool

* Lint

Author: JoannaaKL

README.md

@@ -707,13 +707,6 @@ The following sets of tools are available (all are on by default):
   - `per_page`: Number of results per page (max 100, default: 30) (number, optional)
   - `query`: Filter projects by a search query (matches title and description) (string, optional)
 
-- **update_project_item** - Update project item
-  - `fields`: A list of field updates to apply. (array, required)
-  - `item_id`: The numeric ID of the project item to update (not the issue or pull request ID). (number, required)
-  - `owner`: If owner_type == user it is the handle for the GitHub user account. If owner_type == org it is the name of the organization. The name is not case sensitive. (string, required)
-  - `owner_type`: Owner type (string, required)
-  - `project_number`: The project's number. (number, required)
-
 </details>
 
 <details>

go.mod

@@ -31,7 +31,7 @@ require (
 	github.com/fsnotify/fsnotify v1.8.0 // indirect
 	github.com/go-viper/mapstructure/v2 v2.4.0
 	github.com/google/go-github/v71 v71.0.0 // indirect
-	github.com/google/go-querystring v1.1.0 // indirect
+	github.com/google/go-querystring v1.1.0
 	github.com/google/uuid v1.6.0 // indirect
 	github.com/gorilla/mux v1.8.0 // indirect
 	github.com/inconshreveable/mousetrap v1.1.0 // indirect

pkg/github/projects.go

@@ -621,138 +621,6 @@ func DeleteProjectItem(getClient GetClientFn, t translations.TranslationHelperFu
 		}
 }
 
-func UpdateProjectItem(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
-	return mcp.NewTool("update_project_item",
-			mcp.WithDescription(t("TOOL_UPDATE_PROJECT_ITEM_DESCRIPTION", "Update a specific Project item for a user or org")),
-			mcp.WithToolAnnotation(mcp.ToolAnnotation{Title: t("TOOL_UPDATE_PROJECT_ITEM_USER_TITLE", "Update project item"), ReadOnlyHint: ToBoolPtr(false)}),
-			mcp.WithString("owner_type", mcp.Required(), mcp.Description("Owner type"), mcp.Enum("user", "org")),
-			mcp.WithString("owner", mcp.Required(), mcp.Description("If owner_type == user it is the handle for the GitHub user account. If owner_type == org it is the name of the organization. The name is not case sensitive.")),
-			mcp.WithNumber("project_number", mcp.Required(), mcp.Description("The project's number.")),
-			mcp.WithNumber("item_id", mcp.Required(), mcp.Description("The numeric ID of the project item to update (not the issue or pull request ID).")),
-			mcp.WithArray("fields", mcp.Required(), mcp.Description("A list of field updates to apply.")),
-		), func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
-			owner, err := RequiredParam[string](req, "owner")
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
-			ownerType, err := RequiredParam[string](req, "owner_type")
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
-			projectNumber, err := RequiredInt(req, "project_number")
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
-			itemID, err := RequiredInt(req, "item_id")
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
-			client, err := getClient(ctx)
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
-			fieldsParam, ok := req.GetArguments()["fields"]
-			if !ok {
-				return mcp.NewToolResultError("missing required parameter: fields"), nil
-			}
-
-			rawFields, ok := fieldsParam.([]any)
-			if !ok {
-				return mcp.NewToolResultError("parameter fields must be an array of objects"), nil
-			}
-			if len(rawFields) == 0 {
-				return mcp.NewToolResultError("fields must contain at least one field update"), nil
-			}
-
-			var projectsURL string
-			if ownerType == "org" {
-				projectsURL = fmt.Sprintf("orgs/%s/projectsV2/%d/items/%d", owner, projectNumber, itemID)
-			} else {
-				projectsURL = fmt.Sprintf("users/%s/projectsV2/%d/items/%d", owner, projectNumber, itemID)
-			}
-
-			updateFields := make([]*newProjectV2Field, 0, len(rawFields))
-			for idx, rawField := range rawFields {
-				fieldMap, ok := rawField.(map[string]any)
-				if !ok {
-					return mcp.NewToolResultError(fmt.Sprintf("fields[%d] must be an object", idx)), nil
-				}
-
-				rawID, ok := fieldMap["id"]
-				if !ok {
-					return mcp.NewToolResultError(fmt.Sprintf("fields[%d] is missing 'id'", idx)), nil
-				}
-
-				var fieldID int64
-				switch v := rawID.(type) {
-				case float64:
-					fieldID = int64(v)
-				case int64:
-					fieldID = v
-				case json.Number:
-					n, convErr := v.Int64()
-					if convErr != nil {
-						return mcp.NewToolResultError(fmt.Sprintf("fields[%d].id must be a numeric value", idx)), nil
-					}
-					fieldID = n
-				default:
-					return mcp.NewToolResultError(fmt.Sprintf("fields[%d].id must be a numeric value", idx)), nil
-				}
-
-				value, ok := fieldMap["value"]
-				if !ok {
-					return mcp.NewToolResultError(fmt.Sprintf("fields[%d] is missing 'value'", idx)), nil
-				}
-
-				updateFields = append(updateFields, &newProjectV2Field{
-					ID:    github.Ptr(fieldID),
-					Value: value,
-				})
-			}
-
-			updateProjectItemOptions := &updateProjectItemOptions{Fields: updateFields}
-
-			httpRequest, err := client.NewRequest("PATCH", projectsURL, updateProjectItemOptions)
-			if err != nil {
-				return nil, fmt.Errorf("failed to create request: %w", err)
-			}
-
-			updatedItem := projectV2Item{}
-			resp, err := client.Do(ctx, httpRequest, &updatedItem)
-			if err != nil {
-				return ghErrors.NewGitHubAPIErrorResponse(ctx,
-					"failed to update a project item",
-					resp,
-					err,
-				), nil
-			}
-			defer func() { _ = resp.Body.Close() }()
-
-			if resp.StatusCode != http.StatusOK {
-				body, err := io.ReadAll(resp.Body)
-				if err != nil {
-					return nil, fmt.Errorf("failed to read response body: %w", err)
-				}
-				return mcp.NewToolResultError(fmt.Sprintf("failed to update a project item: %s", string(body))), nil
-			}
-			r, err := json.Marshal(convertToMinimalProjectItem(&updatedItem))
-			if err != nil {
-				return nil, fmt.Errorf("failed to marshal response: %w", err)
-			}
-
-			return mcp.NewToolResultText(string(r)), nil
-		}
-}
-
-type updateProjectItemOptions struct {
-	Fields []*newProjectV2Field `json:"fields,omitempty"`
-}
-
-type newProjectV2Field struct {
-	ID    *int64 `json:"id,omitempty"`
-	Value any    `json:"value,omitempty"`
-}
-
 type newProjectItem struct {
 	ID   int64  `json:"id,omitempty"` // Issue or Pull Request ID to add to the project.
 	Type string `json:"type,omitempty"`