Feat dialtesting browser test integrate

.github/workflows/go.yml

@@ -22,11 +22,11 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v4
       with:
-        go-version: '1.19'
+        go-version: '1.25'
 
     - name: Set up lint
-      run: go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.46.2
-      #run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.46.2
+      run: go install github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.4.0
+      #run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v2.4.0
 
     - name: Lint
       run: make lint LINT_FIX=false

.golangci.toml

@@ -1,146 +1,160 @@
+version = '2'
+
 [run]
-timeout = "10m"
+go = '1.25'
 issues-exit-code = 1
-skip-files = ["packrd"]
-go = "1.19"
-skip-dirs = [
-	# TODO: we should add lint check on the package
-	"internal/obfuscate",
-	"pprofparser",
-]
-
-[linters-settings]
 
-[linters-settings.govet]
-check-shadowing = false
+[linters]
+default = 'all'
+disable = [
+  'containedctx',
+  'contextcheck',
+  'cyclop',
+  'dupl',
+  'err113',
+  'errchkjson',
+  'exhaustruct',
+  'forcetypeassert',
+  'gochecknoglobals',
+  'gocognit',
+  'gocyclo',
+  'gomoddirectives',
+  'ireturn',
+  'maintidx',
+  'mnd',
+  'nakedret',
+  'nestif',
+  'nilnil',
+  'nlreturn',
+  'noctx',
+  'nolintlint',
+  'nonamedreturns',
+  'paralleltest',
+  'prealloc',
+  'revive',
+  'tagliatelle',
+  'testpackage',
+  'varnamelen',
+  'wrapcheck',
+  'wsl',
+
+  # Keep the Go 1.25 CI migration focused on toolchain compatibility. These
+  # linters are newly enabled or materially stricter in golangci-lint v2 and
+  # would require broad unrelated code churn across the repository.
+  'bodyclose',
+  'canonicalheader',
+  'depguard',
+  'dupword',
+  'embeddedstructfieldcheck',
+  'errcheck',
+  'errname',
+  'errorlint',
+  'exptostd',
+  'funcorder',
+  'goconst',
+  'gocritic',
+  'gosec',
+  'gosmopolitan',
+  'inamedparam',
+  'interfacebloat',
+  'intrange',
+  'lll',
+  'musttag',
+  'nilerr',
+  'noinlineerr',
+  'nosprintfhostport',
+  'perfsprint',
+  'recvcheck',
+  'staticcheck',
+  'tagalign',
+  'testifylint',
+  'unconvert',
+  'unparam',
+  'usestdlibvars',
+  'usetesting',
+  'wastedassign',
+  'wsl_v5'
+]
 
-[linters-settings.golint]
-min-confidence = 0.0
+[linters.settings]
+[linters.settings.forbidigo]
+[[linters.settings.forbidigo.forbid]]
+pattern = '^print(ln)?$'
 
-[linters-settings.gocyclo]
-min-complexity = 14.0
+[[linters.settings.forbidigo.forbid]]
+pattern = '^spew\.Print(f|ln)?$'
 
-[linters-settings.gocognit]
-min-complexity = 14.0
+[[linters.settings.forbidigo.forbid]]
+pattern = '^spew\.Dump$'
 
-[linters-settings.cyclo]
-min-complexity = 14.0
+[linters.settings.funlen]
+lines = 230
+statements = 150
 
-[linters-settings.fieldalignment]
-suggest-new = true
+[linters.settings.gocognit]
+min-complexity = 14
 
-[linters-settings.goconst]
-min-len = 3.0
-min-occurrences = 4.0
+[linters.settings.goconst]
+min-len = 3
+min-occurrences = 4
 
-[linters-settings.misspell]
-locale = "US"
+[linters.settings.gocyclo]
+min-complexity = 14
 
-[linters-settings.funlen]
-lines = 230      # default 60
-statements = 150 # default 40
+[linters.settings.godox]
+keywords = [
+  'FIXME'
+]
 
-[linters-settings.forbidigo]
-forbid = ['^print(ln)?$', '^spew\.Print(f|ln)?$', '^spew\.Dump$']
+[linters.settings.gomoddirectives]
+replace-allow-list = [
+  'github.com/abbot/go-http-auth',
+  'github.com/go-check/check',
+  'github.com/gorilla/mux',
+  'github.com/mailgun/minheap',
+  'github.com/mailgun/multibuf'
+]
 
-[linters-settings.depguard]
-list-type = "blacklist"
-include-go-root = false
-packages = ["github.com/pkg/errors"]
+[linters.settings.lll]
+line-length = 150
+tab-width = 2
 
-[linters-settings.godox]
-keywords = ["FIXME"]
+[linters.settings.misspell]
+locale = 'US'
 
-[linters-settings.wsl]
+[linters.settings.wsl]
 allow-assign-and-anything = true
 
-[linters-settings.importas]
-corev1 = "k8s.io/api/core/v1"
-networkingv1beta1 = "k8s.io/api/networking/v1beta1"
-extensionsv1beta1 = "k8s.io/api/extensions/v1beta1"
-metav1 = "k8s.io/apimachinery/pkg/apis/meta/v1"
-kubeerror = "k8s.io/apimachinery/pkg/api/errors"
-
-[linters-settings.gomoddirectives]
-replace-allow-list = [
-  "github.com/abbot/go-http-auth",
-  "github.com/go-check/check",
-  "github.com/gorilla/mux",
-  "github.com/mailgun/minheap",
-  "github.com/mailgun/multibuf",
+[linters.exclusions]
+generated = 'lax'
+paths = [
+  'packrd',
+  'internal/obfuscate',
+  'pprofparser',
+  'third_party$',
+  'builtin$',
+  'examples$'
 ]
 
-[linters-settings.lll]
-line-length = 150
-tab-width = 2
-
-[linters]
-enable-all = true
-disable = [
-  # 权且放开他们
-	"nakedret",
-  "testpackage",      # Too strict
-  "wrapcheck",        # 不便于错误处理
-  "tagliatelle",      # 跟现有 json tag 命名方式
-  "paralleltest",     # 可开启，改动范围较大
-  "noctx",            # 要求 HTTP 请求都用 context 形式，改动较大
-  "nlreturn",         # 要求 return 语句前有一个空行
-  "gomnd",            # 不放过任何一个魔术数
-  "wsl",              # 更好代码分段
-  "prealloc",         # Too many false-positive.
-  "nestif",           # Too many false-positive.
-  "goerr113",         # 不能 fmt.Errorf/errors.New
-  "gochecknoglobals", # 不能搞全局变量
-  "exhaustivestruct", # 结构体初始化字段是否完整
-  "golint",           # Too strict
-  "interfacer",
-  "scopelint",        # obsoluted: https://github.com/kyoh86/scopelint#obsoleted
-
-
-  # 代码太复杂
-  "gocognit",
-  "gocyclo",
-
-  "dupl",     # 还不允许有相似代码
-
-  "cyclop",
-  "gomoddirectives", # used `replace' in go.mod
-  "nolintlint",
-  "revive",
-  "exhaustruct",     # [升级 Go1.18 后加入] 要求结构体每次务必每个字段都要填
-  "varnamelen",      # [升级 Go1.18 后加入] 变量名长度检查
-  "nonamedreturns",  # [升级 Go1.18 后加入] 不允许函数有名返回
-  "forcetypeassert", # [升级 Go1.18 后加入] 强制断言检查
-  "gci",             # [升级 Go1.18 后加入] Tab 键检查
-  "maintidx",        # [升级 Go1.18 后加入] 函数长度检查
-  "containedctx",    # [升级 Go1.18 后加入] Go 不推荐把 context 放在结构体中：https://go.dev/blog/context-and-structs
-  "ireturn",         # [升级 Go1.18 后加入] 函数 interface 返回检查
-  "contextcheck",    # [升级 Go1.18 后加入] 函数参数必须传入 context
-  "errchkjson",      # [升级 Go1.18 后加入] json 序列化/反序列化必须检查
-  "nilnil",          # [升级 Go1.18 后加入] 函数返回 nil, nil 检查
+[[linters.exclusions.rules]]
+linters = [
+  'deadcode',
+  'errcheck',
+  'fieldalignment',
+  'funlen',
+  'goconst',
+  'godot',
+  'gosec',
+  'lll',
+  'maligned',
+  'perfsprint',
+  'staticcheck',
+  'unparam',
+  'unused',
+  'varcheck',
+  'wsl'
 ]
+path = '(.+)_test.go'
 
 [issues]
-exclude-use-default = false
-max-per-linter = 0
 max-same-issues = 0
-exclude = []
-
-[[issues.exclude-rules]]
-path = "(.+)_test.go"
-linters = [
-	"perfsprint",
-	"errcheck",
-	"goconst",
-	"funlen",
-	"godot",
-	"lll",
-	"gosec",
-	"stylecheck",
-	"wsl",
-	"unused",
-	"deadcode",
-	"unparam",
-	"fieldalignment",
-	"maligned",
-	"varcheck"]

AGENTS.md

@@ -31,7 +31,7 @@ This document provides guidelines for AI agents working on the GuanceCloud CLI U
 ## Code Style Guidelines
 
 ### General
-- **Go Version**: 1.19+
+- **Go Version**: 1.25+
 - **License Header**: All files must include the MIT license header (see existing files)
 - **Line Length**: Maximum 150 characters (configured in .golangci.toml)
 - **Complexity**: Maximum cyclomatic complexity of 14
@@ -132,4 +132,4 @@ The project uses golangci-lint with custom configuration (.golangci.toml):
 - Follow the existing code patterns and conventions
 - Check .golangci.toml for specific linter configurations
 - Use the Makefile commands for standard operations
-- Test coverage is important - maintain or improve coverage
+- Test coverage is important - maintain or improve coverage

aggregate/tail-sampling.go

@@ -367,7 +367,7 @@ func (t *TailSamplingConfigs) Init() error {
 	}
 
 	if len(errs) > 0 {
-		return fmt.Errorf(strings.Join(errs, "; "))
+		return fmt.Errorf("%s", strings.Join(errs, "; "))
 	}
 
 	return nil