Add Project Issue Field updates

Resolve attached Issue Fields through project field names and dispatch singular updates through the Issue Field mutation while keeping batch writes explicit.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 1d574277-3735-4356-a753-8354aa6b537c

Author: zwick

README.md

@@ -1133,7 +1133,7 @@ The following sets of tools are available:
   - `status`: The status of the project. Used for 'create_project_status_update' method. (string, optional)
   - `target_date`: The target date of the status update in YYYY-MM-DD format. Used for 'create_project_status_update' method. (string, optional)
   - `title`: The project title. Required for 'create_project' method. (string, optional)
-  - `updated_field`: The field/value to apply, using {"id": 123, "value": ...} or {"name": "Status", "value": ...}; null clears the field. Required for 'update_project_item' and 'update_project_items', where one top-level field/value applies to every item in a batch. For 'update_project_item' SINGLE_SELECT fields, the name form accepts option names; the ID form expects an option ID. (object, optional)
+  - `updated_field`: The field/value to apply, using {"id": 123, "value": ...} or {"name": "Status", "value": ...}; null clears the field. Required for 'update_project_item' and 'update_project_items', where one top-level field/value applies to every item in a batch. For 'update_project_item', the name form supports attached Issue Fields on Issue items and accepts SINGLE_SELECT option names case-insensitively. Attached Issue Fields are not supported by 'update_project_items'; use singular 'update_project_item'. The ID form addresses standard Project fields and expects an option ID for SINGLE_SELECT. (object, optional)
 
 </details>
 

pkg/github/__toolsnaps__/projects_write.snap

@@ -186,7 +186,7 @@
         "type": "string"
       },
       "updated_field": {
-        "description": "The field/value to apply, using {\"id\": 123, \"value\": ...} or {\"name\": \"Status\", \"value\": ...}; null clears the field. Required for 'update_project_item' and 'update_project_items', where one top-level field/value applies to every item in a batch. For 'update_project_item' SINGLE_SELECT fields, the name form accepts option names; the ID form expects an option ID.",
+        "description": "The field/value to apply, using {\"id\": 123, \"value\": ...} or {\"name\": \"Status\", \"value\": ...}; null clears the field. Required for 'update_project_item' and 'update_project_items', where one top-level field/value applies to every item in a batch. For 'update_project_item', the name form supports attached Issue Fields on Issue items and accepts SINGLE_SELECT option names case-insensitively. Attached Issue Fields are not supported by 'update_project_items'; use singular 'update_project_item'. The ID form addresses standard Project fields and expects an option ID for SINGLE_SELECT.",
         "oneOf": [
           {
             "additionalProperties": false,

pkg/github/granular_tools_test.go

@@ -2428,7 +2428,7 @@ func TestGranularSetIssueFields(t *testing.T) {
 		require.False(t, result.IsError, getTextResult(t, result).Text)
 		// The last request captured is the mutation; the preceding issue ID
 		// query does not require the feature flag.
-		assert.Equal(t, "update_issue_suggestions", spy.captured.Get(headers.GraphQLFeaturesHeader))
+		assert.Equal(t, "issue_fields, repo_issue_fields, update_issue_suggestions", spy.captured.Get(headers.GraphQLFeaturesHeader))
 	})
 }
 

pkg/github/issues_granular.go

@@ -1263,6 +1263,47 @@ type IssueFieldCreateOrUpdateInput struct {
 	Suggest              *githubv4.Boolean `json:"suggest,omitempty"`
 }
 
+type setIssueFieldValueMutation struct {
+	SetIssueFieldValue struct {
+		Issue struct {
+			ID     githubv4.ID
+			Number githubv4.Int
+			URL    githubv4.String
+		}
+		IssueFieldValues []struct {
+			TextValue struct {
+				Value string
+			} `graphql:"... on IssueFieldTextValue"`
+			SingleSelectValue struct {
+				Name string
+			} `graphql:"... on IssueFieldSingleSelectValue"`
+			DateValue struct {
+				Value string
+			} `graphql:"... on IssueFieldDateValue"`
+			NumberValue struct {
+				Value float64
+			} `graphql:"... on IssueFieldNumberValue"`
+		}
+	} `graphql:"setIssueFieldValue(input: $input)"`
+}
+
+// SetIssueFieldValues applies typed Issue Field values to an issue node.
+func SetIssueFieldValues(ctx context.Context, gqlClient *githubv4.Client, issueID githubv4.ID, issueFields []IssueFieldCreateOrUpdateInput) (MinimalResponse, error) {
+	var mutation setIssueFieldValueMutation
+	input := SetIssueFieldValueInput{
+		IssueID:     issueID,
+		IssueFields: issueFields,
+	}
+	ctx = ghcontext.WithGraphQLFeatures(ctx, "issue_fields", "repo_issue_fields", "update_issue_suggestions")
+	if err := gqlClient.Mutate(ctx, &mutation, input, nil); err != nil {
+		return MinimalResponse{}, err
+	}
+	return MinimalResponse{
+		ID:  fmt.Sprintf("%v", mutation.SetIssueFieldValue.Issue.ID),
+		URL: string(mutation.SetIssueFieldValue.Issue.URL),
+	}, nil
+}
+
 // GranularSetIssueFields creates a tool to set issue field values on an issue using GraphQL.
 func GranularSetIssueFields(t translations.TranslationHelperFunc) inventory.ServerTool {
 	st := NewTool(
@@ -1486,47 +1527,12 @@ func GranularSetIssueFields(t translations.TranslationHelperFunc) inventory.Serv
 				return ghErrors.NewGitHubGraphQLErrorResponse(ctx, "failed to get issue", err), nil, nil
 			}
 
-			// Execute the setIssueFieldValue mutation
-			var mutation struct {
-				SetIssueFieldValue struct {
-					Issue struct {
-						ID     githubv4.ID
-						Number githubv4.Int
-						URL    githubv4.String
-					}
-					IssueFieldValues []struct {
-						TextValue struct {
-							Value string
-						} `graphql:"... on IssueFieldTextValue"`
-						SingleSelectValue struct {
-							Name string
-						} `graphql:"... on IssueFieldSingleSelectValue"`
-						DateValue struct {
-							Value string
-						} `graphql:"... on IssueFieldDateValue"`
-						NumberValue struct {
-							Value float64
-						} `graphql:"... on IssueFieldNumberValue"`
-					}
-				} `graphql:"setIssueFieldValue(input: $input)"`
-			}
-
-			mutationInput := SetIssueFieldValueInput{
-				IssueID:     issueID,
-				IssueFields: issueFields,
-			}
-
-			// The rationale and suggest input fields on IssueFieldCreateOrUpdateInput
-			// are gated behind the update_issue_suggestions GraphQL feature flag.
-			ctxWithFeatures := ghcontext.WithGraphQLFeatures(ctx, "update_issue_suggestions")
-			if err := gqlClient.Mutate(ctxWithFeatures, &mutation, mutationInput, nil); err != nil {
+			response, err := SetIssueFieldValues(ctx, gqlClient, issueID, issueFields)
+			if err != nil {
 				return ghErrors.NewGitHubGraphQLErrorResponse(ctx, "failed to set issue field values", err), nil, nil
 			}
 
-			r, err := json.Marshal(MinimalResponse{
-				ID:  fmt.Sprintf("%v", mutation.SetIssueFieldValue.Issue.ID),
-				URL: string(mutation.SetIssueFieldValue.Issue.URL),
-			})
+			r, err := json.Marshal(response)
 			if err != nil {
 				return utils.NewToolResultErrorFromErr("failed to marshal response", err), nil, nil
 			}

pkg/github/projects.go

@@ -551,7 +551,7 @@ func projectUpdatedFieldSchema() *jsonschema.Schema {
 
 	return &jsonschema.Schema{
 		Type:        "object",
-		Description: "The field/value to apply, using {\"id\": 123, \"value\": ...} or {\"name\": \"Status\", \"value\": ...}; null clears the field. Required for 'update_project_item' and 'update_project_items', where one top-level field/value applies to every item in a batch. For 'update_project_item' SINGLE_SELECT fields, the name form accepts option names; the ID form expects an option ID.",
+		Description: "The field/value to apply, using {\"id\": 123, \"value\": ...} or {\"name\": \"Status\", \"value\": ...}; null clears the field. Required for 'update_project_item' and 'update_project_items', where one top-level field/value applies to every item in a batch. For 'update_project_item', the name form supports attached Issue Fields on Issue items and accepts SINGLE_SELECT option names case-insensitively. Attached Issue Fields are not supported by 'update_project_items'; use singular 'update_project_item'. The ID form addresses standard Project fields and expects an option ID for SINGLE_SELECT.",
 		OneOf: []*jsonschema.Schema{
 			variant([]string{"id", "value"}, map[string]*jsonschema.Schema{
 				"id": {
@@ -1191,7 +1191,7 @@ func getProjectField(ctx context.Context, client *github.Client, owner, ownerTyp
 	return utils.NewToolResultText(string(r)), nil, nil
 }
 
-func getProjectItem(ctx context.Context, client *github.Client, owner, ownerType string, projectNumber int, itemID int64, fields []int64) (*mcp.CallToolResult, any, error) {
+func fetchProjectItem(ctx context.Context, client *github.Client, owner, ownerType string, projectNumber int, itemID int64, fields []int64) (*github.ProjectV2Item, *github.Response, error) {
 	var resp *github.Response
 	var projectItem *github.ProjectV2Item
 	var opts *github.GetProjectItemOptions
@@ -1208,6 +1208,11 @@ func getProjectItem(ctx context.Context, client *github.Client, owner, ownerType
 	} else {
 		projectItem, resp, err = client.Projects.GetUserProjectItem(ctx, owner, projectNumber, itemID, opts)
 	}
+	return projectItem, resp, err
+}
+
+func getProjectItem(ctx context.Context, client *github.Client, owner, ownerType string, projectNumber int, itemID int64, fields []int64) (*mcp.CallToolResult, any, error) {
+	projectItem, resp, err := fetchProjectItem(ctx, client, owner, ownerType, projectNumber, itemID, fields)
 
 	if err != nil {
 		return ghErrors.NewGitHubAPIErrorResponse(ctx,
@@ -1235,7 +1240,7 @@ func getProjectItem(ctx context.Context, client *github.Client, owner, ownerType
 }
 
 func updateProjectItem(ctx context.Context, client *github.Client, gqlClient *githubv4.Client, owner, ownerType string, projectNumber int, itemID int64, fieldValue map[string]any) (*mcp.CallToolResult, any, error) {
-	updatePayload, err := buildUpdateProjectItem(ctx, gqlClient, owner, ownerType, projectNumber, fieldValue)
+	update, err := buildUpdateProjectItem(ctx, gqlClient, owner, ownerType, projectNumber, fieldValue)
 	if err != nil {
 		var structured *ghErrors.StructuredResolutionError
 		if errors.As(err, &structured) {
@@ -1244,13 +1249,44 @@ func updateProjectItem(ctx context.Context, client *github.Client, gqlClient *gi
 		return utils.NewToolResultError(err.Error()), nil, nil
 	}
 
+	if update.IssueField != nil {
+		projectItem, resp, fetchErr := fetchProjectItem(ctx, client, owner, ownerType, projectNumber, itemID, nil)
+		if fetchErr != nil {
+			return ghErrors.NewGitHubAPIErrorResponse(ctx, "failed to inspect project item content", resp, fetchErr), nil, nil
+		}
+		if resp != nil && resp.Body != nil {
+			defer func() { _ = resp.Body.Close() }()
+		}
+		if resp == nil || resp.StatusCode != http.StatusOK {
+			return utils.NewToolResultError("failed to inspect project item content"), nil, nil
+		}
+
+		issueID, resolveErr := projectItemIssueNodeID(projectItem)
+		if resolveErr != nil {
+			var structured *ghErrors.StructuredResolutionError
+			if errors.As(resolveErr, &structured) {
+				return ghErrors.NewStructuredResolutionErrorResponse(structured), nil, nil
+			}
+			return utils.NewToolResultError(resolveErr.Error()), nil, nil
+		}
+		response, mutationErr := SetIssueFieldValues(ctx, gqlClient, issueID, []IssueFieldCreateOrUpdateInput{*update.IssueField})
+		if mutationErr != nil {
+			return ghErrors.NewGitHubGraphQLErrorResponse(ctx, "failed to update Issue Field", mutationErr), nil, nil
+		}
+		r, marshalErr := json.Marshal(response)
+		if marshalErr != nil {
+			return nil, nil, fmt.Errorf("failed to marshal response: %w", marshalErr)
+		}
+		return utils.NewToolResultText(string(r)), nil, nil
+	}
+
 	var resp *github.Response
 	var updatedItem *github.ProjectV2Item
 
 	if ownerType == "org" {
-		updatedItem, resp, err = client.Projects.UpdateOrganizationProjectItem(ctx, owner, projectNumber, itemID, updatePayload)
+		updatedItem, resp, err = client.Projects.UpdateOrganizationProjectItem(ctx, owner, projectNumber, itemID, update.Project)
 	} else {
-		updatedItem, resp, err = client.Projects.UpdateUserProjectItem(ctx, owner, projectNumber, itemID, updatePayload)
+		updatedItem, resp, err = client.Projects.UpdateUserProjectItem(ctx, owner, projectNumber, itemID, update.Project)
 	}
 
 	if err != nil {
@@ -1277,6 +1313,36 @@ func updateProjectItem(ctx context.Context, client *github.Client, gqlClient *gi
 	return utils.NewToolResultText(string(r)), nil, nil
 }
 
+func projectItemIssueNodeID(item *github.ProjectV2Item) (githubv4.ID, error) {
+	if item == nil || item.ContentType == nil {
+		return "", ghErrors.NewStructuredResolutionError(
+			"issue_field_metadata_unavailable",
+			"",
+			"the project item response did not identify its content type; Issue Fields can only be updated on Issue items",
+			nil,
+		)
+	}
+
+	contentType := string(*item.ContentType)
+	if contentType != "Issue" {
+		return "", ghErrors.NewStructuredResolutionError(
+			"unsupported_item_type",
+			contentType,
+			"Issue Fields can only be updated on Issue project items, not pull requests or draft issues",
+			nil,
+		)
+	}
+	if item.Content == nil || item.Content.Issue == nil || item.Content.Issue.GetNodeID() == "" {
+		return "", ghErrors.NewStructuredResolutionError(
+			"issue_field_metadata_unavailable",
+			"Issue",
+			"the project item response did not include the underlying Issue node ID needed to update the Issue Field",
+			nil,
+		)
+	}
+	return githubv4.ID(item.Content.Issue.GetNodeID()), nil
+}
+
 func deleteProjectItem(ctx context.Context, client *github.Client, owner, ownerType string, projectNumber int, itemID int64) (*mcp.CallToolResult, any, error) {
 	var resp *github.Response
 	var err error
@@ -1614,8 +1680,13 @@ func validateAndConvertToInt64(value any) (int64, error) {
 	}
 }
 
-// buildUpdateProjectItem builds UpdateProjectItemOptions, resolving field names and SINGLE_SELECT option names server-side.
-func buildUpdateProjectItem(ctx context.Context, gqlClient *githubv4.Client, owner, ownerType string, projectNumber int, input map[string]any) (*github.UpdateProjectItemOptions, error) {
+type resolvedProjectItemUpdate struct {
+	Project    *github.UpdateProjectItemOptions
+	IssueField *IssueFieldCreateOrUpdateInput
+}
+
+// buildUpdateProjectItem resolves the target field and builds the matching Project or Issue Field write.
+func buildUpdateProjectItem(ctx context.Context, gqlClient *githubv4.Client, owner, ownerType string, projectNumber int, input map[string]any) (*resolvedProjectItemUpdate, error) {
 	if input == nil {
 		return nil, fmt.Errorf("updated_field must be an object")
 	}
@@ -1659,6 +1730,13 @@ func buildUpdateProjectItem(ctx context.Context, gqlClient *githubv4.Client, own
 		if err != nil {
 			return nil, err
 		}
+		if resolved.IsIssueField {
+			issueField, buildErr := buildIssueFieldUpdate(resolved, valueField)
+			if buildErr != nil {
+				return nil, buildErr
+			}
+			return &resolvedProjectItemUpdate{IssueField: issueField}, nil
+		}
 		parsedID, parseErr := parseInt64(resolved.ID)
 		if parseErr != nil {
 			return nil, fmt.Errorf("resolved field %q has non-numeric ID %q; pass updated_field.id directly", resolved.Name, resolved.ID)
@@ -1694,7 +1772,86 @@ func buildUpdateProjectItem(ctx context.Context, gqlClient *githubv4.Client, own
 		}},
 	}
 
-	return payload, nil
+	return &resolvedProjectItemUpdate{Project: payload}, nil
+}
+
+func buildIssueFieldUpdate(field *ResolvedField, raw any) (*IssueFieldCreateOrUpdateInput, error) {
+	if field == nil || field.IssueFieldNodeID == "" {
+		name := ""
+		if field != nil {
+			name = field.Name
+		}
+		return nil, ghErrors.NewStructuredResolutionError(
+			"issue_field_metadata_unavailable",
+			name,
+			"the attached Project field did not include the underlying Issue Field node ID; refresh field metadata and retry",
+			nil,
+		)
+	}
+
+	input := &IssueFieldCreateOrUpdateInput{FieldID: githubv4.ID(field.IssueFieldNodeID)}
+	if raw == nil {
+		deleteValue := githubv4.Boolean(true)
+		input.Delete = &deleteValue
+		return input, nil
+	}
+
+	invalidValue := func(hint string) (*IssueFieldCreateOrUpdateInput, error) {
+		return nil, ghErrors.NewStructuredResolutionError("invalid_field_value", field.Name, hint, nil)
+	}
+
+	switch field.DataType {
+	case "TEXT":
+		value, ok := raw.(string)
+		if !ok {
+			return invalidValue(fmt.Sprintf("Issue Field %q is TEXT; value must be a string or null to clear it", field.Name))
+		}
+		input.TextValue = githubv4.NewString(githubv4.String(value))
+	case "NUMBER":
+		value, ok := toFloat64(raw)
+		if !ok {
+			return invalidValue(fmt.Sprintf("Issue Field %q is NUMBER; value must be a finite number or null to clear it", field.Name))
+		}
+		number := githubv4.Float(value)
+		input.NumberValue = &number
+	case "DATE":
+		value, ok := raw.(string)
+		if !ok {
+			return invalidValue(fmt.Sprintf("Issue Field %q is DATE; value must be a YYYY-MM-DD string or null to clear it", field.Name))
+		}
+		if _, err := time.Parse("2006-01-02", value); err != nil {
+			return invalidValue(fmt.Sprintf("Issue Field %q is DATE; value %q must use YYYY-MM-DD format", field.Name, value))
+		}
+		input.DateValue = githubv4.NewString(githubv4.String(value))
+	case "SINGLE_SELECT":
+		value, ok := raw.(string)
+		if !ok || value == "" {
+			return invalidValue(fmt.Sprintf("Issue Field %q is SINGLE_SELECT; value must be a non-empty option name or ID, or null to clear it", field.Name))
+		}
+		optionID, err := resolveSingleSelectOptionByName(field, value)
+		if err != nil {
+			for _, option := range field.Options {
+				if option.ID == value {
+					optionID = value
+					err = nil
+					break
+				}
+			}
+			if err != nil {
+				return nil, err
+			}
+		}
+		id := githubv4.ID(optionID)
+		input.SingleSelectOptionID = &id
+	default:
+		return nil, ghErrors.NewStructuredResolutionError(
+			"unsupported_field_type",
+			field.Name,
+			fmt.Sprintf("Issue Field %q has unsupported data type %q; supported types are TEXT, NUMBER, DATE, and SINGLE_SELECT", field.Name, field.DataType),
+			nil,
+		)
+	}
+	return input, nil
 }
 
 func extractPaginationOptionsFromArgs(args map[string]any) (github.ListProjectsPaginationOptions, error) {

pkg/github/projects_batch.go

@@ -119,6 +119,12 @@ func updateProjectItemsBatch(ctx context.Context, client *github.Client, gqlClie
 	if fieldErr != nil {
 		return batchTopLevelError(fieldErr), nil, nil
 	}
+	if field.IsIssueField {
+		return utils.NewToolResultError(fmt.Sprintf(
+			"field %q is an attached Issue Field; update_project_items does not support Issue Fields because they are values on Issue content. Use singular update_project_item for each Issue item",
+			field.Name,
+		)), nil, nil
+	}
 
 	kind := batchMutationUpdate
 	var value githubv4.ProjectV2FieldValue