chore: remove unused get_jobs tool and related tests

Author: wolfeidau

pkg/buildkite/jobs.go

@@ -2,11 +2,7 @@ package buildkite
 
 import (
 	"context"
-	"encoding/json"
 	"errors"
-	"fmt"
-	"io"
-	"net/http"
 
 	"github.com/buildkite/buildkite-mcp-server/pkg/trace"
 	"github.com/buildkite/go-buildkite/v4"
@@ -18,17 +14,6 @@ type JobsClient interface {
 	UnblockJob(ctx context.Context, org string, pipeline string, buildNumber string, jobID string, opt *buildkite.JobUnblockOptions) (buildkite.Job, *buildkite.Response, error)
 }
 
-// GetJobsArgs struct for typed parameters
-type GetJobsArgs struct {
-	OrgSlug      string `json:"org_slug"`
-	PipelineSlug string `json:"pipeline_slug"`
-	BuildNumber  string `json:"build_number"`
-	JobState     string `json:"job_state"`
-	IncludeAgent bool   `json:"include_agent"`
-	Page         int    `json:"page"`
-	PerPage      int    `json:"perPage"`
-}
-
 // GetJobLogsArgs struct for typed parameters
 type GetJobLogsArgs struct {
 	OrgSlug      string `json:"org_slug"`
@@ -46,127 +31,6 @@ type UnblockJobArgs struct {
 	Fields       map[string]string `json:"fields,omitempty"`
 }
 
-func GetJobs(client BuildsClient) (tool mcp.Tool, handler mcp.TypedToolHandlerFunc[GetJobsArgs], scopes []string) {
-	return mcp.NewTool("get_jobs",
-			mcp.WithDescription("Get all jobs for a specific build including their state, timing, commands, and execution details"),
-			mcp.WithString("org_slug",
-				mcp.Required(),
-			),
-			mcp.WithString("pipeline_slug",
-				mcp.Required(),
-			),
-			mcp.WithString("build_number",
-				mcp.Required(),
-			),
-			mcp.WithString("job_state",
-				mcp.Description("Filter jobs by state. Supports actual states (scheduled, running, passed, failed, canceled, skipped, etc.)"),
-			),
-			mcp.WithBoolean("include_agent",
-				mcp.Description("Include detailed agent information in the response. When false (default), only agent ID is included to reduce response size."),
-			),
-			mcp.WithNumber("page",
-				mcp.Description("Page number for pagination (min 1)"),
-				mcp.Min(1),
-			),
-			mcp.WithNumber("perPage",
-				mcp.Description("Results per page for pagination (min 1, max 50)"),
-				mcp.Min(1),
-				mcp.Max(50),
-			),
-			mcp.WithToolAnnotation(mcp.ToolAnnotation{
-				Title:        "Get Jobs",
-				ReadOnlyHint: mcp.ToBoolPtr(true),
-			}),
-		),
-		func(ctx context.Context, request mcp.CallToolRequest, args GetJobsArgs) (*mcp.CallToolResult, error) {
-			ctx, span := trace.Start(ctx, "buildkite.GetJobs")
-			defer span.End()
-
-			// Validate required parameters
-			if args.OrgSlug == "" {
-				return mcp.NewToolResultError("org_slug parameter is required"), nil
-			}
-			if args.PipelineSlug == "" {
-				return mcp.NewToolResultError("pipeline_slug parameter is required"), nil
-			}
-			if args.BuildNumber == "" {
-				return mcp.NewToolResultError("build_number parameter is required"), nil
-			}
-
-			// Set defaults for pagination
-			page := args.Page
-			if page == 0 {
-				page = 1
-			}
-			perPage := args.PerPage
-			if perPage == 0 {
-				perPage = 30
-			}
-
-			paginationParams := ClientSidePaginationParams{
-				Page:    page,
-				PerPage: perPage,
-			}
-
-			span.SetAttributes(
-				attribute.String("org_slug", args.OrgSlug),
-				attribute.String("pipeline_slug", args.PipelineSlug),
-				attribute.String("build_number", args.BuildNumber),
-				attribute.String("job_state", args.JobState),
-				attribute.Bool("include_agent", args.IncludeAgent),
-				attribute.Int("page", paginationParams.Page),
-				attribute.Int("per_page", paginationParams.PerPage),
-			)
-
-			build, resp, err := client.Get(ctx, args.OrgSlug, args.PipelineSlug, args.BuildNumber, &buildkite.BuildGetOptions{})
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
-
-			if resp.StatusCode != http.StatusOK {
-				body, err := io.ReadAll(resp.Body)
-				if err != nil {
-					return nil, fmt.Errorf("failed to read response body: %w", err)
-				}
-				return mcp.NewToolResultError(fmt.Sprintf("failed to get build: %s", string(body))), nil
-			}
-
-			jobs := build.Jobs
-
-			// Filter jobs by state if specified
-			if args.JobState != "" {
-				filteredJobs := make([]buildkite.Job, 0)
-				for _, job := range build.Jobs {
-					if job.State == args.JobState {
-						filteredJobs = append(filteredJobs, job)
-					}
-				}
-				jobs = filteredJobs
-			}
-
-			// Remove agent details if not requested to reduce response size, but keep agent ID
-			if !args.IncludeAgent {
-				jobsWithoutAgent := make([]buildkite.Job, len(jobs))
-				for i, job := range jobs {
-					jobCopy := job
-					// Keep only the agent ID, remove all other verbose agent details
-					jobCopy.Agent = buildkite.Agent{ID: job.Agent.ID}
-					jobsWithoutAgent[i] = jobCopy
-				}
-				jobs = jobsWithoutAgent
-			}
-
-			// Always apply client-side pagination
-			result := applyClientSidePagination(jobs, paginationParams)
-			r, err := json.Marshal(&result)
-			if err != nil {
-				return nil, fmt.Errorf("failed to marshal jobs: %w", err)
-			}
-
-			return mcp.NewToolResultText(string(r)), nil
-		}, []string{"read_builds"}
-}
-
 func UnblockJob(client JobsClient) (tool mcp.Tool, handler mcp.TypedToolHandlerFunc[UnblockJobArgs], scopes []string) {
 	return mcp.NewTool("unblock_job",
 			mcp.WithDescription("Unblock a blocked job in a Buildkite build to allow it to continue execution"),