build/linters: Bump Go to v1.26 and golangci-lint to v2.9.0 (thrasher-corp#2166)

* build/linters: Bump Go to v1.26 and golangci-lint to v2.9.0

* refactor: Update UUID generation in tests to handle errors and improve logging function names

* refactor: Correct typo in assertion message for non-existent file check

Author: thrasher-

.devcontainer/Dockerfile

@@ -1,8 +1,8 @@
 ## Dev Container base image
 # See: https://github.com/devcontainers/images/tree/main/src/go
 # Use the published "dev-" tagged image for the latest Go version.
-# See MCR tags: dev-1.25, dev-1.25-bookworm
-ARG VARIANT=dev-1.25-bookworm
+# See MCR tags: dev-1.26, dev-1.26-trixie
+ARG VARIANT=dev-1.26-trixie
 FROM mcr.microsoft.com/devcontainers/go:${VARIANT}
 
 USER root

.devcontainer/post_create.sh

@@ -19,17 +19,14 @@ install_go_tool github.com/grpc-ecosystem/grpc-gateway/v2/protoc-gen-grpc-gatewa
 install_go_tool github.com/grpc-ecosystem/grpc-gateway/v2/protoc-gen-openapiv2@latest
 
 # Linting
-install_go_tool github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.4.0
+install_go_tool github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.9.0
 
 # Formatting
 install_go_tool mvdan.cc/gofumpt@latest
 
-# Go modernise enforcer
-install_go_tool golang.org/x/tools/gopls/internal/analysis/modernize/cmd/modernize@latest
-
 log "Installed tools:"
 
-for t in protoc-gen-go protoc-gen-go-grpc buf protoc-gen-grpc-gateway protoc-gen-openapiv2 golangci-lint gofumpt modernize; do
+for t in protoc-gen-go protoc-gen-go-grpc buf protoc-gen-grpc-gateway protoc-gen-openapiv2 golangci-lint gofumpt; do
   command -v "$t" || true
 done
 

.github/workflows/amd64.Dockerfile

@@ -1,4 +1,4 @@
-FROM golang:1.25
+FROM golang:1.26
 
 # Install GCC with multi-architecture support (needed for SQLite library)
 RUN apt-get update && apt-get install -y gcc-multilib && rm -rf /var/lib/apt/lists/*

.github/workflows/config-versions-lint.yml

@@ -1,7 +1,7 @@
 name: configs-versions-lint
 on: [push, pull_request]
 env:
-  GO_VERSION: 1.25.x
+  GO_VERSION: 1.26.x
 jobs:
   lint:
     name: config versions lint

.github/workflows/linter.yml

@@ -8,8 +8,8 @@ jobs:
       - uses: actions/checkout@v6
       - uses: actions/setup-go@v6
         with:
-          go-version: '1.25.x'
+          go-version: '1.26.x'
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v9
         with:
-          version: v2.4.0
+          version: v2.9.0

.github/workflows/misc.yml

@@ -9,7 +9,7 @@ jobs:
     - name: Setup Go
       uses: actions/setup-go@v6
       with:
-          go-version: 1.25.x
+          go-version: 1.26.x
 
     - name: Check for currency.NewPair(BTC, USD) used instead of currency.NewBTCUSD
       run: |
@@ -83,9 +83,4 @@ jobs:
           fi
         done
         
-    - name: Check Go modernise tool issues
-      run: |
-        pkgs=$(go list ./... | grep -vE '/gctrpc$|/backtester/btrpc$|/database/models/')
-        go run golang.org/x/tools/gopls/internal/analysis/modernize/cmd/modernize@latest -test $pkgs
-    
 

.github/workflows/proto-lint.yml

@@ -14,7 +14,7 @@ jobs:
       - name: Setup Go
         uses: actions/setup-go@v6
         with:
-          go-version: 1.25.x
+          go-version: 1.26.x
 
       - name: Setup build depends
         run: |

.github/workflows/tests.yml

@@ -1,7 +1,7 @@
 on: [push, pull_request]
 name: CI
 env:
-  GO_VERSION: 1.25.x
+  GO_VERSION: 1.26.x
   GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 jobs:
   backend:

.golangci.yml

@@ -52,6 +52,7 @@ linters:
     - goconst
     - gocritic
 #   - gocyclo
+#   - godoclint
 #   - godot
 #   - godox
 #   - goheader
@@ -65,6 +66,7 @@ linters:
 #   - importas
 #   - inamedparam
 #   - interfacebloat
+#   - iotamixing
     - intrange
 #   - ireturn
 #   - lll
@@ -74,6 +76,7 @@ linters:
     - mirror
     - misspell
 #   - mnd
+    - modernize
 #   - musttag
 #   - nakedret
 #   - nestif
@@ -104,6 +107,7 @@ linters:
     - testableexamples
     - testifylint
 #   - testpackage
+#   - unqueryvet
     - thelper
     - tparallel
     - unconvert
@@ -132,6 +136,9 @@ linters:
               desc: Please use the custom JSON package github.com/thrasher-corp/gocryptotrader/encoding/json instead of encoding/json to allow sonic/json switching.
     goconst:
       min-occurrences: 6
+    gosec:
+      excludes:
+        - G602 # False positives, see https://github.com/golangci/golangci-lint/issues/6334
     gocritic:
       disabled-checks:
         - wrapperFunc
@@ -182,6 +189,9 @@ linters:
       - linters:
           - revive
         text: "var-naming: avoid meaningless package names"
+      - linters:
+          - revive
+        text: "var-naming: avoid package names that conflict with Go standard library package names"
     paths:
       - vendor
       - web/

Dockerfile

@@ -1,4 +1,4 @@
-FROM golang:1.25 as build
+FROM golang:1.26 as build
 WORKDIR /go/src/github.com/thrasher-corp/gocryptotrader
 COPY . .
 RUN GO111MODULE=on go mod vendor