Stop requesting all deploys for a service (#6)

GitOrigin-RevId: fd75437

Author: aaijazi

README.md

@@ -283,8 +283,8 @@ feedback or would like to report a bug or feature request, please create a GitHu
    - Verify Render.com API status
 
 2. **Authorization Errors**
-   - Ensure your API key has the necessary permissions
    - Check if your API key has expired
+   - Ensure your API key is still valid and has not been revoked
 
 3. **Service Creation Failures**
    - Verify repository URLs are accessible

pkg/deploy/repo.go

@@ -16,21 +16,8 @@ func NewRepo(c *client.ClientWithResponses) *Repo {
 	}
 }
 
-type ListDeploysParams struct {
-	*client.ListDeploysParams
-	serviceId string
-}
-
-func (r *Repo) ListDeploys(ctx context.Context, serviceId string, params *client.ListDeploysParams) ([]*client.Deploy, error) {
-	listDeploysParams := &ListDeploysParams{
-		ListDeploysParams: params,
-		serviceId:         serviceId,
-	}
-	return client.ListAll(ctx, listDeploysParams, r.listDeploysPage)
-}
-
-func (r *Repo) listDeploysPage(ctx context.Context, params *ListDeploysParams) ([]*client.Deploy, *client.Cursor, error) {
-	resp, err := r.client.ListDeploysWithResponse(ctx, params.serviceId, params.ListDeploysParams)
+func (r *Repo) ListDeploys(ctx context.Context, serviceId string, params *client.ListDeploysParams) ([]*client.Deploy, *client.Cursor, error) {
+	resp, err := r.client.ListDeploysWithResponse(ctx, serviceId, params)
 	if err != nil {
 		return nil, nil, err
 	}

pkg/deploy/tools.go

@@ -7,6 +7,7 @@ import (
 	"github.com/mark3labs/mcp-go/mcp"
 	"github.com/mark3labs/mcp-go/server"
 	"github.com/render-oss/render-mcp-server/pkg/client"
+	"github.com/render-oss/render-mcp-server/pkg/pointers"
 	"github.com/render-oss/render-mcp-server/pkg/validate"
 )
 
@@ -33,14 +34,41 @@ func listDeploys(deployRepo *Repo) server.ServerTool {
 				mcp.Required(),
 				mcp.Description("The ID of the service to get deployments for"),
 			),
+			mcp.WithNumber("limit",
+				mcp.Description("The maximum number of deploys to return in a single page. To fetch "+
+					"additional pages of results, set the cursor to the last deploy in the previous page. "+
+					"It should be rare to need to set this value greater than 20."),
+				mcp.DefaultNumber(10),
+				mcp.Min(1),
+				mcp.Max(100),
+			),
+			mcp.WithString("cursor",
+				mcp.Description("A unique string that corresponds to a position in the result list. "+
+					"If provided, the endpoint returns results that appear after the corresponding position. "+
+					"To fetch the first page of results, set to the empty string."),
+				mcp.DefaultString(""),
+			),
 		),
 		Handler: func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 			serviceId, err := validate.RequiredToolParam[string](request, "serviceId")
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 
-			deploys, err := deployRepo.ListDeploys(ctx, serviceId, &client.ListDeploysParams{})
+			params := &client.ListDeploysParams{}
+			if limit, ok, err := validate.OptionalToolParam[float64](request, "limit"); err != nil {
+				return mcp.NewToolResultError(err.Error()), nil
+			} else if ok {
+				params.Limit = pointers.From(int(limit))
+			}
+
+			if cursor, ok, err := validate.OptionalToolParam[string](request, "cursor"); err != nil {
+				return mcp.NewToolResultError(err.Error()), nil
+			} else if ok {
+				params.Cursor = &cursor
+			}
+
+			deploys, cursor, err := deployRepo.ListDeploys(ctx, serviceId, params)
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
@@ -49,8 +77,15 @@ func listDeploys(deployRepo *Repo) server.ServerTool {
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
+			respText := string(respJSON) + "\n\n cursor: "
 
-			return mcp.NewToolResultText(string(respJSON)), nil
+			if cursor == nil {
+				respText += `""`
+			} else {
+				respText += *cursor
+			}
+
+			return mcp.NewToolResultText(respText), nil
 		},
 	}
 }

pkg/logs/tools.go

@@ -233,9 +233,6 @@ func listLogLabelValues(logRepo *LogRepo) server.ServerTool {
 		Tool: mcp.NewTool("list_log_label_values",
 			mcp.WithDescription("List all values for a given log label in the logs matching the provided filters. "+
 				"This can be used to discover what values are available for filtering logs using the list_logs tool. "+
-				"Logs are paginated by start and end timestamps. "+
-				"There are more logs to fetch if hasMore is true in the response. "+
-				"Provide the nextStartTime and nextEndTime timestamps as the startTime and endTime query parameters to fetch the next page of logs. "+
 				"You can query for logs across multiple resources, but all resources must be in the same region and belong to the same owner.",
 			),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{