update go-github usage

Author: tmelliottjr

pkg/github/projects.go

@@ -308,39 +308,25 @@ Note: item_id for add is the issue/PR ID; for update/delete it's the project ite
 }
 
 func getProject(ctx context.Context, client *github.Client, owner string, ownerType string, projectNumber int) (*mcp.CallToolResult, error) {
-	var url string
+	var project *github.ProjectV2
+	var resp *github.Response
+	var err error
+
 	if ownerType == "org" {
-		url = fmt.Sprintf("orgs/%s/projectsV2/%d", owner, projectNumber)
+		project, resp, err = client.Projects.GetProjectForOrg(ctx, owner, projectNumber)
 	} else {
-		url = fmt.Sprintf("users/%s/projectsV2/%d", owner, projectNumber)
+		project, resp, err = client.Projects.GetProjectForUser(ctx, owner, projectNumber)
 	}
 
-	project := github.ProjectV2{}
-
-	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, &project)
 	if err != nil {
 		return ghErrors.NewGitHubAPIErrorResponse(ctx,
 			"failed to get project",
 			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 get project: %s", string(body))), nil
-	}
 
-	minimalProject := convertToMinimalProject(&project)
+	minimalProject := convertToMinimalProject(project)
 	r, err := json.Marshal(minimalProject)
 	if err != nil {
 		return nil, fmt.Errorf("failed to marshal response: %w", err)
@@ -350,15 +336,6 @@ func getProject(ctx context.Context, client *github.Client, owner string, ownerT
 }
 
 func listProjects(ctx context.Context, client *github.Client, owner string, ownerType string, queryStr string, pagination paginationOptions) (*mcp.CallToolResult, error) {
-	var url string
-	if ownerType == "org" {
-		url = fmt.Sprintf("orgs/%s/projectsV2", owner)
-	} else {
-		url = fmt.Sprintf("users/%s/projectsV2", owner)
-	}
-	projects := []github.ProjectV2{}
-	minimalProjects := []MinimalProject{}
-
 	opts := &github.ListProjectsOptions{
 		ListProjectsPaginationOptions: github.ListProjectsPaginationOptions{
 			PerPage: pagination.PerPage,
@@ -368,36 +345,27 @@ func listProjects(ctx context.Context, client *github.Client, owner string, owne
 		Query: queryStr,
 	}
 
-	url, err := addOptions(url, opts)
-	if err != nil {
-		return nil, fmt.Errorf("failed to add options to request: %w", err)
-	}
+	var projects []*github.ProjectV2
+	var resp *github.Response
+	var err error
 
-	httpRequest, err := client.NewRequest("GET", url, nil)
-	if err != nil {
-		return nil, fmt.Errorf("failed to create request: %w", err)
+	if ownerType == "org" {
+		projects, resp, err = client.Projects.ListProjectsForOrg(ctx, owner, opts)
+	} else {
+		projects, resp, err = client.Projects.ListProjectsForUser(ctx, owner, opts)
 	}
 
-	resp, err := client.Do(ctx, httpRequest, &projects)
 	if err != nil {
 		return ghErrors.NewGitHubAPIErrorResponse(ctx,
 			"failed to list projects",
 			resp,
 			err,
 		), nil
 	}
-	defer func() { _ = resp.Body.Close() }()
 
+	minimalProjects := []MinimalProject{}
 	for _, project := range projects {
-		minimalProjects = append(minimalProjects, *convertToMinimalProject(&project))
-	}
-
-	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 list projects: %s", string(body))), nil
+		minimalProjects = append(minimalProjects, *convertToMinimalProject(project))
 	}
 
 	// Create response with pagination info
@@ -422,7 +390,7 @@ func getProjectField(ctx context.Context, client *github.Client, owner string, o
 		url = fmt.Sprintf("users/%s/projectsV2/%d/fields/%d", owner, projectNumber, fieldID)
 	}
 
-	projectField := projectV2Field{}
+	var projectField github.ProjectV2Field
 
 	httpRequest, err := client.NewRequest("GET", url, nil)
 	if err != nil {
@@ -446,6 +414,7 @@ func getProjectField(ctx context.Context, client *github.Client, owner string, o
 		}
 		return mcp.NewToolResultError(fmt.Sprintf("failed to get project field: %s", string(body))), nil
 	}
+
 	r, err := json.Marshal(projectField)
 	if err != nil {
 		return nil, fmt.Errorf("failed to marshal response: %w", err)
@@ -461,7 +430,6 @@ func listProjectFields(ctx context.Context, client *github.Client, owner string,
 	} else {
 		url = fmt.Sprintf("users/%s/projectsV2/%d/fields", owner, projectNumber)
 	}
-	projectFields := []projectV2Field{}
 
 	type listProjectFieldsOptions struct {
 		paginationOptions
@@ -481,6 +449,7 @@ func listProjectFields(ctx context.Context, client *github.Client, owner string,
 		return nil, fmt.Errorf("failed to create request: %w", err)
 	}
 
+	var projectFields []*github.ProjectV2Field
 	resp, err := client.Do(ctx, httpRequest, &projectFields)
 	if err != nil {
 		return ghErrors.NewGitHubAPIErrorResponse(ctx,
@@ -756,22 +725,11 @@ type updateProjectItem struct {
 	Value any `json:"value"`
 }
 
-type projectV2Field struct {
-	ID        *int64            `json:"id,omitempty"`         // The unique identifier for this field.
-	NodeID    string            `json:"node_id,omitempty"`    // The GraphQL node ID for this field.
-	Name      string            `json:"name,omitempty"`       // The display name of the field.
-	DataType  string            `json:"data_type,omitempty"`  // The data type of the field (e.g., "text", "number", "date", "single_select", "multi_select").
-	URL       string            `json:"url,omitempty"`        // The API URL for this field.
-	Options   []*any            `json:"options,omitempty"`    // Available options for single_select and multi_select fields.
-	CreatedAt *github.Timestamp `json:"created_at,omitempty"` // The time when this field was created.
-	UpdatedAt *github.Timestamp `json:"updated_at,omitempty"` // The time when this field was last updated.
-}
-
 type projectV2ItemFieldValue struct {
-	ID       *int64      `json:"id,omitempty"`        // The unique identifier for this field.
-	Name     string      `json:"name,omitempty"`      // The display name of the field.
-	DataType string      `json:"data_type,omitempty"` // The data type of the field (e.g., "text", "number", "date", "single_select", "multi_select").
-	Value    interface{} `json:"value,omitempty"`     // The value of the field for a specific project item.
+	ID       *int64 `json:"id,omitempty"`        // The unique identifier for this field.
+	Name     string `json:"name,omitempty"`      // The display name of the field.
+	DataType string `json:"data_type,omitempty"` // The data type of the field (e.g., "text", "number", "date", "single_select", "multi_select").
+	Value    any    `json:"value,omitempty"`     // The value of the field for a specific project item.
 }
 
 type projectV2Item struct {