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[int64](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
 			}
@@ -273,6 +272,10 @@ func GetProjectField(getClient GetClientFn, t translations.TranslationHelperFunc
 			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": int64(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": int64(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": int64(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": int64(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": int64(10),
 			},
 			expectError: true,
 		},