update types

Author: tmelliottjr

README.md

@@ -840,7 +840,7 @@ Options are:
 - **list_project_items** - List project items
   - `after`: Forward pagination cursor from previous pageInfo.nextCursor. (string, optional)
   - `before`: Backward pagination cursor from previous pageInfo.prevCursor (rare). (string, optional)
-  - `fields`: Field IDs to include (e.g. ["102589", "985201"]). CRITICAL: Always provide to get field values. Without this, only titles returned. Get IDs from list_project_fields first. (string[], optional)
+  - `fields`: Field IDs to include (e.g. ["102589", "985201"]). CRITICAL: Always provide to get field values. Without this, only titles returned. (string[], optional)
   - `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)
   - `per_page`: Results per page (max 50) (number, optional)

pkg/github/__toolsnaps__/list_project_items.snap

@@ -15,7 +15,7 @@
         "type": "string"
       },
       "fields": {
-        "description": "Field IDs to include (e.g. [\"102589\", \"985201\"]). CRITICAL: Always provide to get field values. Without this, only titles returned. Get IDs from list_project_fields first.",
+        "description": "Field IDs to include (e.g. [\"102589\", \"985201\"]). CRITICAL: Always provide to get field values. Without this, only titles returned.",
         "items": {
           "type": "string"
         },

pkg/github/projects.go

@@ -256,19 +256,30 @@ func ListProjectFields(getClient GetClientFn, t translations.TranslationHelperFu
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 
-			var resp *github.Response
-			var projectFields []*github.ProjectV2Field
-
 			opts := &github.ListProjectsOptions{
 				ListProjectsPaginationOptions: pagination,
 			}
 
+			var url string
 			if ownerType == "org" {
-				projectFields, resp, err = client.Projects.ListOrganizationProjectFields(ctx, owner, projectNumber, opts)
+				url = fmt.Sprintf("orgs/%s/projectsV2/%d/fields", owner, projectNumber)
 			} else {
-				projectFields, resp, err = client.Projects.ListUserProjectFields(ctx, owner, projectNumber, opts)
+				url = fmt.Sprintf("users/%s/projectsV2/%d/fields", owner, projectNumber)
 			}
 
+			url, err = addOptions(url, opts)
+			if err != nil {
+				return mcp.NewToolResultError(err.Error()), nil
+			}
+
+			httpRequest, err := client.NewRequest("GET", url, nil)
+			if err != nil {
+				return nil, fmt.Errorf("failed to create request: %w", err)
+			}
+
+			var projectFields []projectV2Field
+			resp, err := client.Do(ctx, httpRequest, &projectFields)
+
 			if err != nil {
 				return ghErrors.NewGitHubAPIErrorResponse(ctx,
 					"failed to list project fields",
@@ -401,7 +412,7 @@ func ListProjectItems(getClient GetClientFn, t translations.TranslationHelperFun
 				mcp.Description("Backward pagination cursor from previous pageInfo.prevCursor (rare)."),
 			),
 			mcp.WithArray("fields",
-				mcp.Description("Field IDs to include (e.g. [\"102589\", \"985201\"]). CRITICAL: Always provide to get field values. Without this, only titles returned. Get IDs from list_project_fields first."),
+				mcp.Description("Field IDs to include (e.g. [\"102589\", \"985201\"]). CRITICAL: Always provide to get field values. Without this, only titles returned."),
 				mcp.WithStringItems(),
 			),
 		), func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
@@ -441,7 +452,7 @@ func ListProjectItems(getClient GetClientFn, t translations.TranslationHelperFun
 			}
 
 			var resp *github.Response
-			var projectItems []*github.ProjectV2Item
+			var projectItems []projectV2Item
 			var queryPtr *string
 
 			if queryStr != "" {
@@ -456,12 +467,25 @@ func ListProjectItems(getClient GetClientFn, t translations.TranslationHelperFun
 				},
 			}
 
+			var url string
 			if ownerType == "org" {
-				projectItems, resp, err = client.Projects.ListOrganizationProjectItems(ctx, owner, projectNumber, opts)
+				url = fmt.Sprintf("orgs/%s/projectsV2/%d/items", owner, projectNumber)
 			} else {
-				projectItems, resp, err = client.Projects.ListUserProjectItems(ctx, owner, projectNumber, opts)
+				url = fmt.Sprintf("users/%s/projectsV2/%d/items", owner, projectNumber)
+			}
+
+			url, err = addOptions(url, opts)
+			if err != nil {
+				return mcp.NewToolResultError(err.Error()), nil
 			}
 
+			httpRequest, err := client.NewRequest("GET", url, nil)
+			if err != nil {
+				return nil, fmt.Errorf("failed to create request: %w", err)
+			}
+
+			resp, err = client.Do(ctx, httpRequest, &projectItems)
+
 			if err != nil {
 				return ghErrors.NewGitHubAPIErrorResponse(ctx,
 					ProjectListFailedError,
@@ -920,6 +944,18 @@ type pageInfo struct {
 	PrevCursor      string `json:"prevCursor,omitempty"`
 }
 
+type projectV2Field struct {
+	ID            *int64            `json:"id,omitempty"`
+	NodeID        *string           `json:"node_id,omitempty"`
+	Name          *string           `json:"name,omitempty"`
+	DataType      *string           `json:"data_type,omitempty"`
+	ProjectURL    *string           `json:"project_url,omitempty"`
+	Options       []any             `json:"options,omitempty"`
+	Configuration any               `json:"configuration,omitempty"`
+	CreatedAt     *github.Timestamp `json:"created_at,omitempty"`
+	UpdatedAt     *github.Timestamp `json:"updated_at,omitempty"`
+}
+
 func toNewProjectType(projType string) string {
 	switch strings.ToLower(projType) {
 	case "issue":
@@ -983,11 +1019,20 @@ func extractPaginationOptions(request mcp.CallToolRequest) (github.ListProjectsP
 		return github.ListProjectsPaginationOptions{}, err
 	}
 
-	return github.ListProjectsPaginationOptions{
+	opts := github.ListProjectsPaginationOptions{
 		PerPage: &perPage,
-		After:   &after,
-		Before:  &before,
-	}, nil
+	}
+
+	// Only set After/Before if they have non-empty values
+	if after != "" {
+		opts.After = &after
+	}
+
+	if before != "" {
+		opts.Before = &before
+	}
+
+	return opts, nil
 }
 
 // addOptions adds the parameters in opts as URL query parameters to s. opts