chore(deps): update golangci-lint version to support go1.25

Author: Fluf22

.github/actions/setup/action.yml

@@ -78,7 +78,7 @@ runs:
       if: ${{ inputs.language == 'go' }}
       shell: bash
       run: |
-        curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.60.3
+        curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v2.4.0
         echo "$(go env GOPATH)/bin" >> $GITHUB_PATH
         go install golang.org/x/tools/cmd/[email protected]
 

clients/algoliasearch-client-go/.golangci.bck.yml

@@ -0,0 +1,71 @@
+linters-settings:
+  govet:
+    enable-all: true
+    disable:
+      - fieldalignment
+
+  revive:
+    rules:
+      - name: var-naming
+        disabled: true
+
+linters:
+  enable-all: true
+
+  disable:
+    - godox
+    - bodyclose
+    - contextcheck
+    - interfacebloat
+    - gci
+    - gosmopolitan
+    - wsl
+    - varnamelen
+    - nlreturn
+    - err113
+    - gochecknoglobals
+    - exhaustruct
+    - exhaustive
+    - depguard
+    - lll
+    - forbidigo
+    - gochecknoinits
+    - cyclop
+    - errorlint
+    - gomnd
+    - tagliatelle
+    - nilnil
+    - stylecheck
+    - musttag
+    - errchkjson
+    - nonamedreturns
+    - inamedparam
+    - ineffassign
+    - dupword
+    - nestif
+    - goconst
+    - funlen
+    - dupl
+    - unparam
+    - gocognit
+    - forcetypeassert
+    - wastedassign
+    - gocyclo
+    - maintidx
+    - copyloopvar
+    - intrange
+    - canonicalheader
+    - mnd
+    - perfsprint
+    - containedctx
+
+    # Deprecated
+    - execinquery
+    - exportloopref
+
+issues:
+  exclude-generated: disable
+
+run:
+  concurrency: 2
+  timeout: 10m

clients/algoliasearch-client-go/.golangci.yml

@@ -1,71 +1,84 @@
-linters-settings:
-  govet:
-    enable-all: true
-    disable:
-      - fieldalignment
-
-  revive:
-    rules:
-      - name: var-naming
-        disabled: true
-
+version: "2"
+run:
+  concurrency: 2
 linters:
-  enable-all: true
-
+  default: all
   disable:
-    - godox
     - bodyclose
+    - canonicalheader
+    - containedctx
     - contextcheck
-    - interfacebloat
-    - gci
-    - gosmopolitan
-    - wsl
-    - varnamelen
-    - nlreturn
+    - copyloopvar
+    - cyclop
+    - depguard
+    - dupl
+    - dupword
     - err113
-    - gochecknoglobals
-    - exhaustruct
+    - errchkjson
+    - errorlint
     - exhaustive
-    - depguard
-    - lll
+    - exhaustruct
     - forbidigo
-    - gochecknoinits
-    - cyclop
-    - errorlint
-    - gomnd
-    - tagliatelle
-    - nilnil
-    - stylecheck
-    - musttag
-    - errchkjson
-    - nonamedreturns
-    - inamedparam
-    - ineffassign
-    - dupword
-    - nestif
-    - goconst
+    - forcetypeassert
+    - funcorder
     - funlen
-    - dupl
-    - unparam
+    - gochecknoglobals
+    - gochecknoinits
     - gocognit
-    - forcetypeassert
-    - wastedassign
+    - goconst
     - gocyclo
-    - maintidx
-    - copyloopvar
+    - godox
+    - gosmopolitan
+    - inamedparam
+    - ineffassign
+    - interfacebloat
     - intrange
-    - canonicalheader
+    - lll
+    - maintidx
     - mnd
+    - musttag
+    - nestif
+    - nilnil
+    - nlreturn
+    - noctx
+    - noinlineerr
+    - nonamedreturns
     - perfsprint
-    - containedctx
-
-    # Deprecated
-    - execinquery
-    - exportloopref
-
-issues:
-  exclude-generated: disable
-
-run:
-  concurrency: 2
-  timeout: 10m
+    - recvcheck
+    - staticcheck
+    - tagliatelle
+    - unparam
+    - varnamelen
+    - wastedassign
+    - wsl
+  settings:
+    govet:
+      disable:
+        - fieldalignment
+      enable-all: true
+    revive:
+      rules:
+        - name: var-naming
+          disabled: true
+  exclusions:
+    generated: disable
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gofmt
+    - gofumpt
+    - goimports
+  exclusions:
+    generated: disable
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

clients/algoliasearch-client-go/algolia/debug/debug.go

@@ -30,8 +30,11 @@ func Display(input any) {
 	if !debug {
 		return
 	}
+
 	start := time.Now()
+
 	var msg string
+
 	switch v := input.(type) {
 	case *http.Request:
 		msg = debugRequest(v)
@@ -40,6 +43,7 @@ func Display(input any) {
 	default:
 		msg = fmt.Sprintf("do not know how to display %#v", v)
 	}
+
 	Println(msg)
 	fmt.Printf("took %s\n", time.Since(start))
 }
@@ -52,6 +56,7 @@ func Printf(format string, a ...any) {
 	if !debug {
 		return
 	}
+
 	msg := fmt.Sprintf(format, a...)
 	fmt.Printf("> ALGOLIA DEBUG: %s", msg)
 }

clients/algoliasearch-client-go/algolia/debug/utils.go

@@ -16,11 +16,14 @@ func copyReadCloser(r io.ReadCloser) (io.ReadCloser, string) {
 	if r == nil {
 		return nil, ""
 	}
+
 	data, err := io.ReadAll(r)
 	_ = r.Close()
+
 	if err != nil {
 		return nil, ""
 	}
+
 	return io.NopCloser(bytes.NewReader(data)), string(data)
 }
 
@@ -29,10 +32,12 @@ func decodeGzipContent(in string) (string, error) {
 	if err != nil {
 		return in, fmt.Errorf("cannot open content with gzip.Reader: %w", err)
 	}
+
 	out, err := io.ReadAll(gr)
 	if err != nil {
 		return in, fmt.Errorf("cannot read content from gzip.Reader: %w", err)
 	}
+
 	return string(out), nil
 }
 
@@ -58,10 +63,12 @@ func extractBody(body io.ReadCloser, c compression.Compression) (io.ReadCloser,
 
 func prettyPrintJSON(input string) string {
 	var b bytes.Buffer
+
 	err := json.Indent(&b, []byte(input), "\t", "  ")
 	if err != nil {
 		return input
 	}
+
 	return strings.TrimSuffix(b.String(), "\n")
 }
 
@@ -88,8 +95,10 @@ func debugRequest(req *http.Request) string {
 		if strings.Contains(strings.ToLower(k), "algolia") {
 			str = strings.Repeat("*", len(str))
 		}
+
 		msg += fmt.Sprintf("\theader=%s:%q\n", k, str)
 	}
+
 	msg += fmt.Sprintf("\tbody=\n\t%s\n", prettyPrintJSON(body))
 
 	return msg
@@ -101,6 +110,7 @@ func debugResponse(res *http.Response) string {
 	}
 
 	var body string
+
 	res.Body, body = extractBody(res.Body, compression.NONE)
 
 	msg := "> ALGOLIA DEBUG response:\n"

clients/algoliasearch-client-go/algolia/errs/no_more_host_to_try_err.go

@@ -25,6 +25,7 @@ func (e *NoMoreHostToTryError) Error() string {
 	if len(e.intermediateNetworkErrors) > 0 {
 		return fmt.Errorf("all hosts have been contacted unsuccessfully, it can either be a server or a network error or wrong appID/key credentials were used. %w", errors.Join(e.intermediateNetworkErrors...)).Error()
 	}
+
 	return "all hosts have been contacted unsuccessfully, it can either be a server or a network error or wrong appID/key credentials were used. You can use 'ExposeIntermediateNetworkErrors: true' in the config to investigate."
 }
 

clients/algoliasearch-client-go/algolia/transport/retry_strategy.go

@@ -29,6 +29,7 @@ type Host struct {
 
 type RetryStrategy struct {
 	sync.RWMutex
+
 	hosts        []StatefulHost
 	writeTimeout time.Duration
 	readTimeout  time.Duration
@@ -107,7 +108,7 @@ func (s *RetryStrategy) Decide(h Host, code int, err error) Outcome {
 		return Retry
 	}
 
-	if !(isZero(code) || is4xx(code) || is2xx(code)) || isNetworkError(err) {
+	if (!isZero(code) && !is4xx(code) && !is2xx(code)) || isNetworkError(err) {
 		s.markDown(h)
 		return Retry
 	}
@@ -146,6 +147,7 @@ func isNetworkError(err error) bool {
 	if err == nil {
 		return false
 	}
+
 	_, ok := err.(net.Error)
 	// We need to ensure that the error is a net.Error but not a
 	// context.DeadlineExceeded error (which is actually a net.Error), because
@@ -157,6 +159,7 @@ func isTimeoutError(err error) bool {
 	if err == nil {
 		return false
 	}
+
 	return strings.Contains(err.Error(), context.DeadlineExceeded.Error())
 }