updated go and lint version

Author: Akshay2191

.github/workflows/ci.yml

@@ -95,7 +95,7 @@ jobs:
       - name: Lint Go
         uses: golangci/golangci-lint-action@1e7e51e771db61008b38414a730f564565cf7c20 # v9.2.0
         with:
-          version: v2.4.0
+          version: v2.10.0
           skip-cache: true
 
   vulnerability-scan:

Makefile.tools

@@ -1,6 +1,6 @@
 OAPICODEGEN	    = github.com/deepmap/oapi-codegen/v2/cmd/oapi-codegen@v2.1.0
 LEFTHOOK	    = github.com/evilmartians/lefthook@v1.6.9
-GOLANGCILINT    = github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.4.0
+GOLANGCILINT    = github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.10.0
 PROTOCGENGO	    = google.golang.org/protobuf/cmd/protoc-gen-go@v1.32.0
 GOFUMPT	        = mvdan.cc/gofumpt@v0.6.0
 COUNTERFEITER   = github.com/maxbrunsfeld/counterfeiter/v6@v6.8.1

go.mod

@@ -1,8 +1,6 @@
 module github.com/nginx/agent/v3
 
-go 1.24.2
-
-toolchain go1.24.13
+go 1.25.7
 
 require (
 	buf.build/gen/go/bufbuild/protovalidate/protocolbuffers/go v1.36.9-20250912141014-52f32327d4b0.1

internal/collector/containermetricsreceiver/internal/scraper/cpuscraper/internal/cgroup/cpu_test.go

@@ -13,7 +13,8 @@ import (
 	"strconv"
 	"testing"
 
-	"github.com/stretchr/testify/assert"
+	"github.com/pkg/errors"
+	"github.com/stretchr/testify/require"
 )
 
 func TestCollectCPUStats(t *testing.T) {
@@ -79,10 +80,18 @@ func TestCollectCPUStats(t *testing.T) {
 			cpuStat, err := cgroupCPUSource.collectCPUStats(ctx)
 
 			// Assert error
-			assert.IsType(tt, test.errorType, err)
+			if test.errorType == nil {
+				require.NoError(tt, err)
+			} else {
+				require.Error(tt, err)
+				// satisfy the linter's requirement for a more specific check than IsType.
+				require.Condition(tt, func() bool {
+					return errors.As(err, &test.errorType)
+				}, "Error should be of type %T", test.errorType)
+			}
 
 			// Assert result
-			assert.Equal(tt, test.cpuStat, cpuStat)
+			require.Equal(tt, test.cpuStat, cpuStat)
 		})
 	}
 }

internal/collector/containermetricsreceiver/internal/scraper/memoryscraper/internal/cgroup/memory_test.go

@@ -13,8 +13,9 @@ import (
 	"strconv"
 	"testing"
 
+	"github.com/pkg/errors"
 	"github.com/shirou/gopsutil/v4/mem"
-	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 
 func TestVirtualMemoryStat(t *testing.T) {
@@ -115,10 +116,18 @@ func TestVirtualMemoryStat(t *testing.T) {
 			virtualMemoryStat, err := cgroupCPUSource.VirtualMemoryStat()
 
 			// Assert error
-			assert.IsType(tt, test.errorType, err)
+			if test.errorType == nil {
+				require.NoError(tt, err)
+			} else {
+				require.Error(tt, err)
+				// satisfy the linter's requirement for a more specific check than IsType.
+				require.Condition(tt, func() bool {
+					return errors.As(err, &test.errorType)
+				}, "Error should be of type %T", test.errorType)
+			}
 
 			// Assert result
-			assert.Equal(tt, test.virtualMemoryStat, *virtualMemoryStat)
+			require.Equal(tt, test.virtualMemoryStat, *virtualMemoryStat)
 		})
 	}
 }

internal/collector/nginxreceiver/internal/scraper/accesslog/nginx_log_scraper.go

@@ -74,7 +74,7 @@ func NewScraper(
 	mb := metadata.NewMetricsBuilder(cfg.MetricsBuilderConfig, settings)
 	rb := mb.NewResourceBuilder()
 
-	operators := make([]operator.Config, 0)
+	operators := make([]operator.Config, 0, len(cfg.AccessLogs))
 
 	for _, accessLog := range cfg.AccessLogs {
 		logger.Info("Adding access log file operator", zap.String("file_path", accessLog.FilePath))

internal/config/config.go

@@ -1407,7 +1407,7 @@ func resolveExtensions() Extensions {
 
 func updateHeaders(headers []Header) []Header {
 	var err error
-	newHeaders := []Header{}
+	newHeaders := make([]Header, 0, len(headers))
 
 	for _, header := range headers {
 		value := header.Value

internal/config/config_test.go

@@ -977,7 +977,7 @@ func TestResolveExtensions(t *testing.T) {
 			extension := resolveExtensions()
 			require.NotNil(t, extension)
 
-			var result []string
+			result := make([]string, 0, len(extension.HeadersSetter.Headers))
 			for _, header := range extension.HeadersSetter.Headers {
 				result = append(result, header.Value)
 			}

internal/grpc/grpc_test.go

@@ -350,10 +350,15 @@ func validateError(t *testing.T, validationError error, isErrorExpected bool) {
 
 func Test_ValidateGrpcError(t *testing.T) {
 	result := ValidateGrpcError(TestError{})
-	assert.IsType(t, TestError{}, result)
+	if result == nil {
+		t.Fatalf("expected an error, got nil")
+	}
+	var te TestError
+	require.ErrorAs(t, result, &te)
 
 	result = ValidateGrpcError(status.Errorf(codes.InvalidArgument, "error"))
-	assert.IsType(t, &backoff.PermanentError{}, result)
+	var pe *backoff.PermanentError
+	require.ErrorAs(t, result, &pe)
 }
 
 func Test_transportCredentials(t *testing.T) {

internal/watcher/credentials/credential_watcher_service_test.go

@@ -104,7 +104,7 @@ func TestCredentialWatcherService_addWatcher(t *testing.T) {
 }
 
 func TestCredentialWatcherService_watchFiles(t *testing.T) {
-	var files []string
+	files := make([]string, 0, 3)
 
 	ctx := context.Background()
 	cws := NewCredentialWatcherService(types.AgentConfig(), model.Command)