Update limit assertions and comments

lumaxis · web-flow · commit 2aa02b25f91a · 2025-08-05T08:50:57.000Z
diff --git a/README.md b/README.md
@@ -531,8 +531,8 @@ The following sets of tools are available (all are on by default):
   - `before`: Cursor for backward pagination (use with last) (string, optional)
   - `includeClosedPrs`: Include closed/merged pull requests in results (default: false) (boolean, optional)
   - `issue_numbers`: Array of issue numbers within the specified repository (number[], required)
-  - `last`: Number of results from end for backward pagination (max: 100) (number, optional)
-  - `limit`: Maximum number of closing PRs to return per issue (default: 10, max: 100) (number, optional)
+  - `last`: Number of results from end for backward pagination (max: 250) (number, optional)
+  - `limit`: Maximum number of closing PRs to return per issue (default: 100, max: 250) (number, optional)
   - `orderByState`: Order results by pull request state (default: false) (boolean, optional)
   - `owner`: The owner of the repository (string, required)
   - `repo`: The name of the repository (string, required)
diff --git a/e2e/e2e_test.go b/e2e/e2e_test.go
@@ -1674,7 +1674,7 @@ func TestFindClosingPullRequests(t *testing.T) {
 			owner:        "octocat",
 			repo:         "Hello-World",
 			issueNumbers: []int{1},
-			limit:        150,
+			limit:        251,
 			expectError:  true,
 		},
 	}
@@ -2167,21 +2167,21 @@ func TestFindClosingPullRequestsGraphQLParameters(t *testing.T) {
 				repo:         "vscode",
 				issueNumbers: []int{1},
 				args: map[string]any{
-					"limit": 150,
+					"limit": 251,
 				},
 				expectError: true,
-				description: "Should reject limit greater than 100",
+				description: "Should reject limit greater than 250",
 			},
 			{
 				name:         "Invalid last range - too high",
 				owner:        "microsoft",
 				repo:         "vscode",
 				issueNumbers: []int{1},
 				args: map[string]any{
-					"last": 150,
+					"last": 251,
 				},
 				expectError: true,
-				description: "Should reject last greater than 100",
+				description: "Should reject last greater than 250",
 			},
 		}
 
diff --git a/pkg/github/issues.go b/pkg/github/issues.go
@@ -1386,7 +1386,7 @@ func FindClosingPullRequests(getGQLClient GetGQLClientFn, t translations.Transla
 				),
 			),
 			mcp.WithNumber("limit",
-				mcp.Description("Maximum number of closing PRs to return per issue (default: 10, max: 100)"),
+				mcp.Description("Maximum number of closing PRs to return per issue (default: 100, max: 250)"),
 			),
 			mcp.WithBoolean("includeClosedPrs",
 				mcp.Description("Include closed/merged pull requests in results (default: false)"),
@@ -1404,7 +1404,7 @@ func FindClosingPullRequests(getGQLClient GetGQLClientFn, t translations.Transla
 				mcp.Description("Cursor for backward pagination (use with last)"),
 			),
 			mcp.WithNumber("last",
-				mcp.Description("Number of results from end for backward pagination (max: 100)"),
+				mcp.Description("Number of results from end for backward pagination (max: 250)"),
 			),
 		),
 		func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) {
