Merge pull request #272 from buildkite/chore_go_buildkite_v5

chore(deps): upgrade go-buildkite to v5 which fixes presence issues with updates

Author: wolfeidau

AGENT.md

@@ -13,7 +13,7 @@
 ## Architecture
 - **Main binary**: `cmd/buildkite-mcp-server/main.go` - MCP server for Buildkite API access
 - **Core packages**: `pkg/buildkite/` - API wrappers and tool handlers, `internal/commands/` - CLI commands, `pkg/toolsets/` - tool registry, `pkg/server/` - MCP server setup
-- **Key dependencies**: `github.com/modelcontextprotocol/go-sdk` (MCP protocol), `github.com/buildkite/go-buildkite/v4` (API client)
+- **Key dependencies**: `github.com/modelcontextprotocol/go-sdk` (MCP protocol), `github.com/buildkite/go-buildkite/v5` (API client)
 - **Configuration**: Environment variables (BUILDKITE_API_TOKEN, OTEL tracing)
 - **CI/CD**: `buildkite` organization, `buildkite-mcp-server` pipeline slug for build and test (`.buildkite/pipeline.yml`), `buildkite-mcp-server-release` pipeline slug for releases (`.buildkite/pipeline.release.yml`)
 

cmd/buildkite-mcp-server/main.go

@@ -10,7 +10,7 @@ import (
 	buildkitelogs "github.com/buildkite/buildkite-logs"
 	"github.com/buildkite/buildkite-mcp-server/internal/commands"
 	"github.com/buildkite/buildkite-mcp-server/pkg/trace"
-	gobuildkite "github.com/buildkite/go-buildkite/v4"
+	gobuildkite "github.com/buildkite/go-buildkite/v5"
 	"github.com/mattn/go-isatty"
 	"github.com/rs/zerolog"
 	"github.com/rs/zerolog/log"

go.mod

@@ -4,8 +4,8 @@ go 1.25.0
 
 require (
 	github.com/alecthomas/kong v1.15.0
-	github.com/buildkite/buildkite-logs v0.8.1
-	github.com/buildkite/go-buildkite/v4 v4.21.0
+	github.com/buildkite/buildkite-logs v0.9.1
+	github.com/buildkite/go-buildkite/v5 v5.0.0
 	github.com/google/jsonschema-go v0.4.3
 	github.com/mattn/go-isatty v0.0.22
 	github.com/microcosm-cc/bluemonday v1.0.27

go.sum

@@ -72,10 +72,10 @@ github.com/aws/smithy-go v1.25.1 h1:J8ERsGSU7d+aCmdQur5Txg6bVoYelvQJgtZehD12GkI=
 github.com/aws/smithy-go v1.25.1/go.mod h1:YE2RhdIuDbA5E5bTdciG9KrW3+TiEONeUWCqxX9i1Fc=
 github.com/aymerick/douceur v0.2.0 h1:Mv+mAeH1Q+n9Fr+oyamOlAkUNPWPlA8PPGR0QAaYuPk=
 github.com/aymerick/douceur v0.2.0/go.mod h1:wlT5vV2O3h55X9m7iVYN0TBM0NH/MmbLnd30/FjWUq4=
-github.com/buildkite/buildkite-logs v0.8.1 h1:cJ9K46Td2FH7M+CjDNM5CN2BHNmztgJu8CeyYOTu6jE=
-github.com/buildkite/buildkite-logs v0.8.1/go.mod h1:ORrtYBhFeN8ETv9Hi8ylugemCKjX+yfpAJslrbWVyDc=
-github.com/buildkite/go-buildkite/v4 v4.21.0 h1:H/aTrPexE035u269mCTJ6v8dHB6oVgWjETc9bEMKgQM=
-github.com/buildkite/go-buildkite/v4 v4.21.0/go.mod h1:t/M4DUcs7qyebtzm3nkyZ1zUB/svWnKtR+uRU2Ca8tQ=
+github.com/buildkite/buildkite-logs v0.9.1 h1:JWzXaXpm4iju/xjPLw16Q80wHiYz4ne6UU+/DZ1PPuY=
+github.com/buildkite/buildkite-logs v0.9.1/go.mod h1:sW+bJPZVtkbXAkkmdyJMnX9y/npWOJJsU+rz7e/zsSg=
+github.com/buildkite/go-buildkite/v5 v5.0.0 h1:1pmSzO5dGQm7/GhB/hW8siNNDkdQSdeAm+CdKlX5enU=
+github.com/buildkite/go-buildkite/v5 v5.0.0/go.mod h1:wsKNPQmX0vAAR+RcGLyDJkSTsHXrHpnwSURaMckM7Wg=
 github.com/cenkalti/backoff v2.2.1+incompatible h1:tNowT99t7UNflLxfYYSlKYsBpXdEet03Pg2g16Swow4=
 github.com/cenkalti/backoff v2.2.1+incompatible/go.mod h1:90ReRw6GdpyfrHakVjL/QHaoyV4aDUVVkXQJJJ3NXXM=
 github.com/cenkalti/backoff/v5 v5.0.3 h1:ZN+IMa753KfX5hd8vVaMixjnqRZ3y8CuJKRKj1xcsSM=

internal/commands/command.go

@@ -7,7 +7,7 @@ import (
 	"runtime"
 
 	buildkitelogs "github.com/buildkite/buildkite-logs"
-	gobuildkite "github.com/buildkite/go-buildkite/v4"
+	gobuildkite "github.com/buildkite/go-buildkite/v5"
 	"github.com/rs/zerolog/log"
 )
 

pkg/buildkite/access_token.go

@@ -4,7 +4,7 @@ import (
 	"context"
 
 	"github.com/buildkite/buildkite-mcp-server/pkg/trace"
-	"github.com/buildkite/go-buildkite/v4"
+	"github.com/buildkite/go-buildkite/v5"
 	"github.com/modelcontextprotocol/go-sdk/mcp"
 )
 

pkg/buildkite/access_token_test.go

@@ -8,7 +8,7 @@ import (
 	"testing"
 	"time"
 
-	"github.com/buildkite/go-buildkite/v4"
+	"github.com/buildkite/go-buildkite/v5"
 	"github.com/stretchr/testify/require"
 )
 
@@ -64,5 +64,5 @@ func TestAccessToken(t *testing.T) {
 
 	textContent := getTextResult(t, result)
 
-	assert.JSONEq(`{"uuid":"123","scopes":["read_build","read_pipeline"],"description":"Test token","created_at":"2023-01-01T00:00:00Z","user":{"name":"Test User","email":"test@example.com"}}`, textContent.Text)
+	assert.JSONEq(`{"uuid":"123","scopes":["read_build","read_pipeline"],"description":"Test token","created_at":"2023-01-01T00:00:00Z","expires_at":null,"user":{"name":"Test User","email":"test@example.com"}}`, textContent.Text)
 }

pkg/buildkite/annotations.go

@@ -6,7 +6,7 @@ import (
 
 	"github.com/buildkite/buildkite-mcp-server/pkg/trace"
 	"github.com/buildkite/buildkite-mcp-server/pkg/utils"
-	"github.com/buildkite/go-buildkite/v4"
+	"github.com/buildkite/go-buildkite/v5"
 	"github.com/modelcontextprotocol/go-sdk/mcp"
 	"go.opentelemetry.io/otel/attribute"
 )

pkg/buildkite/annotations_test.go

@@ -5,7 +5,7 @@ import (
 	"net/http"
 	"testing"
 
-	"github.com/buildkite/go-buildkite/v4"
+	"github.com/buildkite/go-buildkite/v5"
 	"github.com/stretchr/testify/require"
 )
 

pkg/buildkite/artifacts.go

@@ -12,7 +12,7 @@ import (
 	"github.com/buildkite/buildkite-mcp-server/pkg/tokens"
 	"github.com/buildkite/buildkite-mcp-server/pkg/trace"
 	"github.com/buildkite/buildkite-mcp-server/pkg/utils"
-	"github.com/buildkite/go-buildkite/v4"
+	"github.com/buildkite/go-buildkite/v5"
 	"github.com/modelcontextprotocol/go-sdk/mcp"
 	"go.opentelemetry.io/otel/attribute"
 )