Resolve project view fields by name

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

Copilot-Session: c6f8ede6-efee-4191-900d-59a1bb0af000

Author: zwick

README.md

@@ -1141,7 +1141,8 @@ The following sets of tools are available:
   - `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)
   - `view_id`: Project view node ID for update or delete; must belong to owner/project_number. (string, optional)
-  - `visible_fields`: Field database IDs for table or board creation; unsupported for roadmap. (string[], optional)
+  - `visible_field_names`: Field names for table or board creation; mutually exclusive with visible_fields. (string[], optional)
+  - `visible_fields`: Field database IDs for table or board creation; mutually exclusive with visible_field_names. (string[], optional)
 
 </details>
 

pkg/github/__toolsnaps__/projects_write.snap

@@ -249,8 +249,15 @@
         "description": "Project view node ID for update or delete; must belong to owner/project_number.",
         "type": "string"
       },
+      "visible_field_names": {
+        "description": "Field names for table or board creation; mutually exclusive with visible_fields.",
+        "items": {
+          "type": "string"
+        },
+        "type": "array"
+      },
       "visible_fields": {
-        "description": "Field database IDs for table or board creation; unsupported for roadmap.",
+        "description": "Field database IDs for table or board creation; mutually exclusive with visible_field_names.",
         "items": {
           "type": "string"
         },

pkg/github/projects.go

@@ -757,7 +757,14 @@ func ProjectsWrite(t translations.TranslationHelperFunc) inventory.ServerTool {
 					},
 					"visible_fields": {
 						Type:        "array",
-						Description: "Field database IDs for table or board creation; unsupported for roadmap.",
+						Description: "Field database IDs for table or board creation; mutually exclusive with visible_field_names.",
+						Items: &jsonschema.Schema{
+							Type: "string",
+						},
+					},
+					"visible_field_names": {
+						Type:        "array",
+						Description: "Field names for table or board creation; mutually exclusive with visible_fields.",
 						Items: &jsonschema.Schema{
 							Type: "string",
 						},
@@ -862,13 +869,12 @@ func ProjectsWrite(t translations.TranslationHelperFunc) inventory.ServerTool {
 				return utils.NewToolResultError(err.Error()), nil, nil
 			}
 
-			gqlClient, err := deps.GetGQLClient(ctx)
-			if err != nil {
-				return utils.NewToolResultError(err.Error()), nil, nil
-			}
-
 			// create_project does not require project_number or a REST client
 			if method == projectsMethodCreateProject {
+				gqlClient, gqlErr := deps.GetGQLClient(ctx)
+				if gqlErr != nil {
+					return utils.NewToolResultError(gqlErr.Error()), nil, nil
+				}
 				return createProject(ctx, gqlClient, owner, ownerType, args)
 			}
 
@@ -890,6 +896,26 @@ func ProjectsWrite(t translations.TranslationHelperFunc) inventory.ServerTool {
 				}
 			}
 
+			if method == projectsMethodCreateProjectView {
+				visibleFieldNames, namesErr := OptionalStringArrayParam(args, "visible_field_names")
+				if namesErr != nil {
+					return utils.NewToolResultError(namesErr.Error()), nil, nil
+				}
+				var gqlClient *githubv4.Client
+				if len(visibleFieldNames) > 0 {
+					gqlClient, err = deps.GetGQLClient(ctx)
+					if err != nil {
+						return utils.NewToolResultError(err.Error()), nil, nil
+					}
+				}
+				return createProjectView(ctx, client, gqlClient, args, owner, ownerType, projectNumber, visibleFieldNames)
+			}
+
+			gqlClient, err := deps.GetGQLClient(ctx)
+			if err != nil {
+				return utils.NewToolResultError(err.Error()), nil, nil
+			}
+
 			switch method {
 			case projectsMethodAddProjectItem:
 				itemType, err := RequiredParam[string](args, "item_type")
@@ -980,8 +1006,6 @@ func ProjectsWrite(t translations.TranslationHelperFunc) inventory.ServerTool {
 				return createProjectStatusUpdate(ctx, gqlClient, owner, ownerType, projectNumber, body, status, startDate, targetDate)
 			case projectsMethodCreateIterationField:
 				return createIterationField(ctx, gqlClient, owner, ownerType, projectNumber, args)
-			case projectsMethodCreateProjectView:
-				return createProjectView(ctx, client, args, owner, ownerType, projectNumber)
 			case projectsMethodUpdateProjectView:
 				return updateProjectView(ctx, gqlClient, args, owner, ownerType, projectNumber)
 			case projectsMethodDeleteProjectView:
@@ -1891,7 +1915,7 @@ func getProjectView(ctx context.Context, gqlClient *githubv4.Client, viewID stri
 	return utils.NewToolResultText(string(result)), !bool(query.Node.ProjectView.Project.Public), nil, nil
 }
 
-func createProjectView(ctx context.Context, client *github.Client, args map[string]any, owner, ownerType string, projectNumber int) (*mcp.CallToolResult, any, error) {
+func createProjectView(ctx context.Context, client *github.Client, gqlClient *githubv4.Client, args map[string]any, owner, ownerType string, projectNumber int, visibleFieldNames []string) (*mcp.CallToolResult, any, error) {
 	name, err := RequiredParam[string](args, "name")
 	if err != nil {
 		return utils.NewToolResultError(err.Error()), nil, nil
@@ -1915,8 +1939,22 @@ func createProjectView(ctx context.Context, client *github.Client, args map[stri
 	if err != nil {
 		return utils.NewToolResultError(err.Error()), nil, nil
 	}
+	if len(visibleFields) > 0 && len(visibleFieldNames) > 0 {
+		return utils.NewToolResultError("provide either 'visible_fields' or 'visible_field_names', not both"), nil, nil
+	}
+	if len(visibleFieldNames) > 0 {
+		resolvedIDs, resolveErr := resolveFieldNamesToIDs(ctx, gqlClient, owner, ownerType, projectNumber, visibleFieldNames)
+		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
+		}
+		visibleFields = resolvedIDs
+	}
 	if layout == githubv4.ProjectV2ViewLayoutRoadmapLayout && len(visibleFields) > 0 {
-		return utils.NewToolResultError("visible_fields is not supported for roadmap views"), nil, nil
+		return utils.NewToolResultError("visible fields are not supported for roadmap views"), nil, nil
 	}
 
 	requestBody := CreateProjectV2ViewRequest{

pkg/github/projects_test.go

@@ -888,6 +888,7 @@ func Test_ProjectsWrite(t *testing.T) {
 	assert.Contains(t, inputSchema.Properties, "layout")
 	assert.Contains(t, inputSchema.Properties, "filter")
 	assert.Contains(t, inputSchema.Properties, "visible_fields")
+	assert.Contains(t, inputSchema.Properties, "visible_field_names")
 	assert.Contains(t, inputSchema.Properties["method"].Enum, projectsMethodCreateProjectView)
 	assert.Contains(t, inputSchema.Properties["method"].Enum, projectsMethodUpdateProjectView)
 	assert.Contains(t, inputSchema.Properties["method"].Enum, projectsMethodDeleteProjectView)