Data types

Author: JoannaaKL

pkg/github/projects.go

@@ -185,7 +185,7 @@ func ListProjectFields(getClient GetClientFn, t translations.TranslationHelperFu
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
-			projectNumber, err := RequiredParam[string](req, "projectNumber")
+			projectNumber, err := RequiredParam[int](req, "projectNumber")
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
@@ -200,14 +200,13 @@ func ListProjectFields(getClient GetClientFn, t translations.TranslationHelperFu
 
 			var url string
 			if ownerType == "org" {
-				url = fmt.Sprintf("orgs/%s/projectsV2/%s/fields", owner, projectNumber)
+				url = fmt.Sprintf("orgs/%s/projectsV2/%d/fields", owner, projectNumber)
 			} else {
-				url = fmt.Sprintf("users/%s/projectsV2/%s/fields", owner, projectNumber)
+				url = fmt.Sprintf("users/%s/projectsV2/%d/fields", owner, projectNumber)
 			}
 			projectFields := []projectV2Field{}
 
 			opts := listProjectsOptions{PerPage: perPage}
-
 			if perPage > 0 {
 				opts.PerPage = perPage
 			}
@@ -265,14 +264,18 @@ func GetProjectField(getClient GetClientFn, t translations.TranslationHelperFunc
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
-			projectNumber, err := RequiredParam[int64](req, "projectNumber")
+			projectNumber, err := RequiredParam[int](req, "projectNumber")
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 			fieldID, err := RequiredParam[int64](req, "field_id")
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
+			perPage, err := OptionalIntParamWithDefault(req, "per_page", 30)
+			if err != nil {
+				return mcp.NewToolResultError(err.Error()), nil
+			}
 			client, err := getClient(ctx)
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
@@ -284,6 +287,15 @@ func GetProjectField(getClient GetClientFn, t translations.TranslationHelperFunc
 			} else {
 				url = fmt.Sprintf("users/%s/projectsV2/%d/fields/%d", owner, projectNumber, fieldID)
 			}
+
+			opts := listProjectsOptions{PerPage: perPage}
+			if perPage > 0 {
+				opts.PerPage = perPage
+			}
+			url, err = addOptions(url, opts)
+			if err != nil {
+				return nil, fmt.Errorf("failed to add options to request: %w", err)
+			}
 			projectField := projectV2Field{}
 
 			httpRequest, err := client.NewRequest("GET", url, nil)

pkg/github/projects_test.go

@@ -330,7 +330,7 @@ func Test_ListProjectFields(t *testing.T) {
 			requestArgs: map[string]interface{}{
 				"owner":         "octo-org",
 				"owner_type":    "org",
-				"projectNumber": "123",
+				"projectNumber": 123,
 			},
 			expectedLength: 1,
 		},
@@ -354,7 +354,7 @@ func Test_ListProjectFields(t *testing.T) {
 			requestArgs: map[string]interface{}{
 				"owner":         "octocat",
 				"owner_type":    "user",
-				"projectNumber": "456",
+				"projectNumber": 456,
 				"per_page":      float64(50),
 			},
 			expectedLength: 1,
@@ -370,7 +370,7 @@ func Test_ListProjectFields(t *testing.T) {
 			requestArgs: map[string]interface{}{
 				"owner":         "octo-org",
 				"owner_type":    "org",
-				"projectNumber": "789",
+				"projectNumber": 789,
 			},
 			expectError:    true,
 			expectedErrMsg: "failed to list projects",
@@ -380,7 +380,7 @@ func Test_ListProjectFields(t *testing.T) {
 			mockedClient: mock.NewMockedHTTPClient(),
 			requestArgs: map[string]interface{}{
 				"owner_type":    "org",
-				"projectNumber": "10",
+				"projectNumber": 10,
 			},
 			expectError: true,
 		},
@@ -389,7 +389,7 @@ func Test_ListProjectFields(t *testing.T) {
 			mockedClient: mock.NewMockedHTTPClient(),
 			requestArgs: map[string]interface{}{
 				"owner":         "octo-org",
-				"projectNumber": "10",
+				"projectNumber": 10,
 			},
 			expectError: true,
 		},
@@ -476,7 +476,7 @@ func Test_GetProjectField(t *testing.T) {
 			requestArgs: map[string]any{
 				"owner":         "octo-org",
 				"owner_type":    "org",
-				"projectNumber": int64(123),
+				"projectNumber": 123,
 				"field_id":      int64(101),
 			},
 			expectedID: 101,
@@ -492,7 +492,7 @@ func Test_GetProjectField(t *testing.T) {
 			requestArgs: map[string]any{
 				"owner":         "octocat",
 				"owner_type":    "user",
-				"projectNumber": int64(456),
+				"projectNumber": 456,
 				"field_id":      int64(202),
 			},
 			expectedID: 202,
@@ -508,7 +508,7 @@ func Test_GetProjectField(t *testing.T) {
 			requestArgs: map[string]any{
 				"owner":         "octo-org",
 				"owner_type":    "org",
-				"projectNumber": int64(789),
+				"projectNumber": 789,
 				"field_id":      int64(303),
 			},
 			expectError:    true,
@@ -519,7 +519,7 @@ func Test_GetProjectField(t *testing.T) {
 			mockedClient: mock.NewMockedHTTPClient(),
 			requestArgs: map[string]any{
 				"owner_type":    "org",
-				"projectNumber": int64(10),
+				"projectNumber": 10,
 				"field_id":      int64(1),
 			},
 			expectError: true,
@@ -529,7 +529,7 @@ func Test_GetProjectField(t *testing.T) {
 			mockedClient: mock.NewMockedHTTPClient(),
 			requestArgs: map[string]any{
 				"owner":         "octo-org",
-				"projectNumber": int64(10),
+				"projectNumber": 10,
 				"field_id":      int64(1),
 			},
 			expectError: true,
@@ -550,7 +550,7 @@ func Test_GetProjectField(t *testing.T) {
 			requestArgs: map[string]any{
 				"owner":         "octo-org",
 				"owner_type":    "org",
-				"projectNumber": int64(10),
+				"projectNumber": 10,
 			},
 			expectError: true,
 		},

pkg/github/tools.go

@@ -195,6 +195,7 @@ func DefaultToolsetGroup(readOnly bool, getClient GetClientFn, getGQLClient GetG
 			toolsets.NewServerTool(ListProjects(getClient, t)),
 			toolsets.NewServerTool(GetProject(getClient, t)),
 			toolsets.NewServerTool(ListProjectFields(getClient, t)),
+			toolsets.NewServerTool(GetProjectField(getClient, t)),
 		)
 
 	// Add toolsets to the group