Merge pull request #197 from buildkite/PS-1515/Fix-pagination

Author: mcncl

pkg/buildkite/artifacts.go

@@ -91,6 +91,7 @@ func ListArtifactsForBuild(client ArtifactsClient) (tool mcp.Tool, handler serve
 			mcp.WithString("build_number",
 				mcp.Required(),
 			),
+			withPagination(),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{
 				Title:        "Build Artifact List",
 				ReadOnlyHint: mcp.ToBoolPtr(true),
@@ -171,6 +172,7 @@ func ListArtifactsForJob(client ArtifactsClient) (tool mcp.Tool, handler server.
 			mcp.WithString("job_id",
 				mcp.Required(),
 			),
+			withPagination(),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{
 				Title:        "Job Artifact List",
 				ReadOnlyHint: mcp.ToBoolPtr(true),

pkg/buildkite/buildkite.go

@@ -18,7 +18,7 @@ type PaginatedResult[T any] struct {
 
 func optionalPaginationParams(r mcp.CallToolRequest) (buildkite.ListOptions, error) {
 	page := r.GetInt("page", 1)
-	perPage := r.GetInt("perPage", 1)
+	perPage := r.GetInt("perPage", 100)
 	return buildkite.ListOptions{
 		Page:    page,
 		PerPage: perPage,

pkg/buildkite/buildkite_test.go

@@ -28,13 +28,13 @@ func Test_optionalPaginationParams(t *testing.T) {
 			expectErr: false,
 		},
 		{
-			name: "missing pagination parameters should use new defaults (1 per page)",
+			name: "missing pagination parameters should use new defaults (100 per page)",
 			args: map[string]any{
 				"name": "test-name",
 			},
 			expected: buildkite.ListOptions{
 				Page:    1,
-				PerPage: 1,
+				PerPage: 100,
 			},
 			expectErr: false,
 		},