Adjust for actual GitHub API limits

lumaxis · web-flow · commit 34b53dba96dc · 2025-08-04T16:16:59.000Z
diff --git a/pkg/github/issues.go b/pkg/github/issues.go
@@ -1356,7 +1356,8 @@ type ClosingPullRequest struct {
 
 const (
 	// DefaultClosingPRsLimit is the default number of closing PRs to return per issue
-	DefaultClosingPRsLimit = 10
+	// Aligned with GitHub GraphQL API default of 100 items per page
+	DefaultClosingPRsLimit = 100
 )
 
 // FindClosingPullRequests creates a tool to find pull requests that closed specific issues
@@ -1414,11 +1415,11 @@ func FindClosingPullRequests(getGQLClient GetGQLClientFn, t translations.Transla
 				limitExplicitlySet = true
 				if limitFloat, ok := limitParam.(float64); ok {
 					limit = int(limitFloat)
-					if limit <= 0 || limit > 100 {
-						return mcp.NewToolResultError("limit must be between 1 and 100 inclusive"), nil
+					if limit <= 0 || limit > 250 {
+						return mcp.NewToolResultError("limit must be between 1 and 250 inclusive (GitHub GraphQL API maximum)"), nil
 					}
 				} else {
-					return mcp.NewToolResultError("limit must be a number between 1 and 100"), nil
+					return mcp.NewToolResultError("limit must be a number between 1 and 250 (GitHub GraphQL API maximum)"), nil
 				}
 			}
 
@@ -1427,8 +1428,8 @@ func FindClosingPullRequests(getGQLClient GetGQLClientFn, t translations.Transla
 			if err != nil {
 				return mcp.NewToolResultError(fmt.Sprintf("last parameter error: %s", err.Error())), nil
 			}
-			if last != 0 && (last <= 0 || last > 100) {
-				return mcp.NewToolResultError("last must be between 1 and 100 inclusive for backward pagination"), nil
+			if last != 0 && (last <= 0 || last > 250) {
+				return mcp.NewToolResultError("last must be between 1 and 250 inclusive for backward pagination (GitHub GraphQL API maximum)"), nil
 			}
 
 			// Parse cursor parameters

Original file line number	Diff line number	Diff line change
`@@ -1356,7 +1356,8 @@ type ClosingPullRequest struct {`
`1356`	`1356`
`1357`	`1357`	`const (`
`1358`	`1358`	`// DefaultClosingPRsLimit is the default number of closing PRs to return per issue`
`1359`		`- DefaultClosingPRsLimit = 10`
	`1359`	`+ // Aligned with GitHub GraphQL API default of 100 items per page`
	`1360`	`+ DefaultClosingPRsLimit = 100`
`1360`	`1361`	`)`
`1361`	`1362`
`1362`	`1363`	`// FindClosingPullRequests creates a tool to find pull requests that closed specific issues`
`@@ -1414,11 +1415,11 @@ func FindClosingPullRequests(getGQLClient GetGQLClientFn, t translations.Transla`
`1414`	`1415`	`limitExplicitlySet = true`
`1415`	`1416`	`if limitFloat, ok := limitParam.(float64); ok {`
`1416`	`1417`	`limit = int(limitFloat)`
`1417`		`- if limit <= 0 \|\| limit > 100 {`
`1418`		`- return mcp.NewToolResultError("limit must be between 1 and 100 inclusive"), nil`
	`1418`	`+ if limit <= 0 \|\| limit > 250 {`
	`1419`	`+ return mcp.NewToolResultError("limit must be between 1 and 250 inclusive (GitHub GraphQL API maximum)"), nil`
`1419`	`1420`	`}`
`1420`	`1421`	`} else {`
`1421`		`- return mcp.NewToolResultError("limit must be a number between 1 and 100"), nil`
	`1422`	`+ return mcp.NewToolResultError("limit must be a number between 1 and 250 (GitHub GraphQL API maximum)"), nil`
`1422`	`1423`	`}`
`1423`	`1424`	`}`
`1424`	`1425`
`@@ -1427,8 +1428,8 @@ func FindClosingPullRequests(getGQLClient GetGQLClientFn, t translations.Transla`
`1427`	`1428`	`if err != nil {`
`1428`	`1429`	`return mcp.NewToolResultError(fmt.Sprintf("last parameter error: %s", err.Error())), nil`
`1429`	`1430`	`}`
`1430`		`- if last != 0 && (last <= 0 \|\| last > 100) {`
`1431`		`- return mcp.NewToolResultError("last must be between 1 and 100 inclusive for backward pagination"), nil`
	`1431`	`+ if last != 0 && (last <= 0 \|\| last > 250) {`
	`1432`	`+ return mcp.NewToolResultError("last must be between 1 and 250 inclusive for backward pagination (GitHub GraphQL API maximum)"), nil`
`1432`	`1433`	`}`
`1433`	`1434`
`1434`	`1435`	`// Parse cursor parameters`