Remove unused per_page

Author: JoannaaKL

README.md

@@ -667,7 +667,6 @@ The following sets of tools are available (all are on by default):
   - `field_id`: The field's id. (number, required)
   - `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`: Number of results per page (max 100, default: 30) (number, optional)
   - `project_number`: The project's number. (number, required)
 
 - **get_project_item** - Get project item

pkg/github/__toolsnaps__/get_project_field.snap

@@ -22,10 +22,6 @@
         ],
         "type": "string"
       },
-      "per_page": {
-        "description": "Number of results per page (max 100, default: 30)",
-        "type": "number"
-      },
       "project_number": {
         "description": "The project's number.",
         "type": "number"

pkg/github/projects.go

@@ -255,7 +255,6 @@ func GetProjectField(getClient GetClientFn, t translations.TranslationHelperFunc
 			mcp.WithString("owner", mcp.Required(), mcp.Description("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.")),
 			mcp.WithNumber("project_number", mcp.Required(), mcp.Description("The project's number.")),
 			mcp.WithNumber("field_id", mcp.Required(), mcp.Description("The field's id.")),
-			mcp.WithNumber("per_page", mcp.Description("Number of results per page (max 100, default: 30)")),
 		), func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 			owner, err := RequiredParam[string](req, "owner")
 			if err != nil {
@@ -273,10 +272,6 @@ 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
@@ -289,11 +284,6 @@ func GetProjectField(getClient GetClientFn, t translations.TranslationHelperFunc
 				url = fmt.Sprintf("users/%s/projectsV2/%d/fields/%d", owner, projectNumber, fieldID)
 			}
 
-			opts := listProjectsOptions{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

@@ -451,7 +451,6 @@ func Test_GetProjectField(t *testing.T) {
 	assert.Contains(t, tool.InputSchema.Properties, "owner")
 	assert.Contains(t, tool.InputSchema.Properties, "project_number")
 	assert.Contains(t, tool.InputSchema.Properties, "field_id")
-	assert.Contains(t, tool.InputSchema.Properties, "per_page")
 	assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner_type", "owner", "project_number", "field_id"})
 
 	orgField := map[string]any{"id": 101, "name": "Status", "dataType": "single_select"}