reverts migration google/go-github GetProjectItem due to bug with the underlying library implementation

Author: stephenotalora

pkg/github/projects.go

@@ -6,11 +6,14 @@ import (
 	"fmt"
 	"io"
 	"net/http"
+	"net/url"
+	"reflect"
 	"strings"
 
 	ghErrors "github.com/github/github-mcp-server/pkg/errors"
 	"github.com/github/github-mcp-server/pkg/translations"
 	"github.com/google/go-github/v77/github"
+	"github.com/google/go-querystring/query"
 	"github.com/mark3labs/mcp-go/mcp"
 	"github.com/mark3labs/mcp-go/server"
 )
@@ -410,12 +413,17 @@ func ListProjectItems(getClient GetClientFn, t translations.TranslationHelperFun
 
 			var resp *github.Response
 			var projectItems []*github.ProjectV2Item
+			var queryPtr *string
+
+			if queryStr != "" {
+				queryPtr = &queryStr
+			}
 
 			opts := &github.ListProjectItemsOptions{
 				Fields: fields,
 				ListProjectsOptions: github.ListProjectsOptions{
 					ListProjectsPaginationOptions: github.ListProjectsPaginationOptions{PerPage: &perPage},
-					Query:                         &queryStr,
+					Query:                         queryPtr,
 				},
 			}
 
@@ -506,21 +514,32 @@ func GetProjectItem(getClient GetClientFn, t translations.TranslationHelperFunc)
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 
-			opts := &github.GetProjectItemOptions{}
+			var url string
+			if ownerType == "org" {
+				url = fmt.Sprintf("orgs/%s/projectsV2/%d/items/%d", owner, projectNumber, itemID)
+			} else {
+				url = fmt.Sprintf("users/%s/projectsV2/%d/items/%d", owner, projectNumber, itemID)
+			}
+
+			opts := fieldSelectionOptions{}
 
 			if len(fields) > 0 {
 				opts.Fields = fields
 			}
 
-			var resp *github.Response
-			var projectItem *github.ProjectV2Item
+			url, err = addOptions(url, opts)
+			if err != nil {
+				return mcp.NewToolResultError(err.Error()), nil
+			}
 
-			if ownerType == "org" {
-				projectItem, resp, err = client.Projects.GetOrganizationProjectItem(ctx, owner, projectNumber, itemID, opts)
-			} else {
-				projectItem, resp, err = client.Projects.GetUserProjectItem(ctx, owner, projectNumber, itemID, opts)
+			projectItem := projectV2Item{}
+
+			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, &projectItem)
 			if err != nil {
 				return ghErrors.NewGitHubAPIErrorResponse(ctx,
 					"failed to get project item",
@@ -828,6 +847,44 @@ func DeleteProjectItem(getClient GetClientFn, t translations.TranslationHelperFu
 		}
 }
 
+type fieldSelectionOptions struct {
+	// Specific list of field IDs to include in the response. If not provided, only the title field is included.
+	// Example: fields=102589,985201,169875 or fields[]=102589&fields[]=985201&fields[]=169875
+	Fields []int64 `url:"fields,omitempty,comma"`
+}
+
+// addOptions adds the parameters in opts as URL query parameters to s. opts
+// must be a struct whose fields may contain "url" tags.
+func addOptions(s string, opts any) (string, error) {
+	v := reflect.ValueOf(opts)
+	if v.Kind() == reflect.Ptr && v.IsNil() {
+		return s, nil
+	}
+
+	origURL, err := url.Parse(s)
+	if err != nil {
+		return s, err
+	}
+
+	origValues := origURL.Query()
+
+	// Use the github.com/google/go-querystring library to parse the struct
+	newValues, err := query.Values(opts)
+	if err != nil {
+		return s, err
+	}
+
+	// Merge the values
+	for key, values := range newValues {
+		for _, value := range values {
+			origValues.Add(key, value)
+		}
+	}
+
+	origURL.RawQuery = origValues.Encode()
+	return origURL.String(), nil
+}
+
 type updateProjectItemPayload struct {
 	Fields []updateProjectItem `json:"fields"`
 }