chore(ci): GOEXPERIMENT=jsonv2

lvan100 · lianghuan · commit 5b844bd76580 · 2026-02-28T12:30:07.000+08:00
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -10,18 +10,24 @@ jobs:
   lint:
     name: Run golangci-lint
     runs-on: ubuntu-latest
+
+    env:
+      GOEXPERIMENT: jsonv2
+
     steps:
       - name: Checkout code
-        uses: actions/checkout@v4
+        uses: actions/checkout@v6.0.2
 
       - name: Print All environment variables
         run: env | sort
 
       - name: Set up Go
-        uses: actions/setup-go@v5
+        uses: actions/setup-go@v6.3.0
+        with:
+          go-version: '1.25'
 
       - name: Install golangci-lint
-        uses: golangci/golangci-lint-action@v7
+        uses: golangci/golangci-lint-action@v9.2.0
         with:
           version: latest
 
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -10,15 +10,21 @@ jobs:
   test:
     name: Run tests and collect coverage
     runs-on: ubuntu-latest
+
+    env:
+      GOEXPERIMENT: jsonv2
+
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v6.0.2
 
       - name: Print All environment variables
         run: env | sort
 
       - name: Set up Go
-        uses: actions/setup-go@v5
+        uses: actions/setup-go@v6.3.0
+        with:
+          go-version: '1.25'
 
       - name: Install dependencies
         run: go mod download
@@ -27,7 +33,7 @@ jobs:
         run: go test -count=1 -coverprofile=coverage.txt ./...
 
       - name: Upload results to Codecov
-        uses: codecov/codecov-action@v5
+        uses: codecov/codecov-action@v5.5.2
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
-          slug: go-spring/stdlib
+          slug: go-spring/stdlib
diff --git a/funcutil/funcutil_test.go b/funcutil/funcutil_test.go
@@ -33,14 +33,18 @@ func TestFuncName(t *testing.T) {
 	assert.That(t, funcutil.FuncName((*receiver).ptrFnWithArgs)).Equal("funcutil_test.(*receiver).ptrFnWithArgs")
 }
 
+// nolint: unused
 func fnNoArgs() {}
 
+// nolint: unused
 func fnWithArgs(i int) {}
 
 type receiver struct{}
 
+// nolint: unused
 func (r *receiver) ptrFnNoArgs() {}
 
+// nolint: unused
 func (r *receiver) ptrFnWithArgs(i int) {}
 
 func TestFileLine(t *testing.T) {
@@ -53,25 +57,25 @@ func TestFileLine(t *testing.T) {
 		{
 			fnNoArgs,
 			"funcutil/funcutil_test.go",
-			36,
+			37,
 			"funcutil_test.fnNoArgs",
 		},
 		{
 			fnWithArgs,
 			"funcutil/funcutil_test.go",
-			38,
+			40,
 			"funcutil_test.fnWithArgs",
 		},
 		{
 			(*receiver).ptrFnNoArgs,
 			"funcutil/funcutil_test.go",
-			42,
+			45,
 			"funcutil_test.(*receiver).ptrFnNoArgs",
 		},
 		{
 			(*receiver).ptrFnWithArgs,
 			"funcutil/funcutil_test.go",
-			44,
+			48,
 			"funcutil_test.(*receiver).ptrFnWithArgs",
 		},
 	}
diff --git a/goutil/goutil_test.go b/goutil/goutil_test.go
@@ -173,6 +173,7 @@ func TestGoValue(t *testing.T) {
 	t.Run("context value inheritance with withoutCancel", func(t *testing.T) {
 		key := "test_key"
 		value := "test_value"
+		// nolint: staticcheck
 		ctx := context.WithValue(context.Background(), key, value)
 		ctx, cancel := context.WithCancel(ctx)
 		cancel()
diff --git a/httpclt/httpclt_test.go b/httpclt/httpclt_test.go
@@ -72,10 +72,7 @@ func (r *HelloResponse) DecodeJSON(d jsonflow.Decoder) (err error) {
 		return err
 	}
 
-	for {
-		if d.PeekKind() == '}' {
-			break
-		}
+	for d.PeekKind() != '}' {
 
 		var key string
 		key, err = jsonflow.DecodeString(d)
@@ -112,6 +109,7 @@ func (c *HelloClient) Hello(ctx context.Context, req *HelloRequest, opts ...http
 		Schema:  "http",
 		Method:  "GET",
 		Pattern: "/v1/hello",
+		// nolint: staticcheck
 		RawPath: fmt.Sprintf("/v1/hello"),
 		Query:   req,
 		Header: http.Header{
diff --git a/httpsvr/httpsvr_test.go b/httpsvr/httpsvr_test.go
@@ -45,7 +45,7 @@ func NewHelloRequest() *HelloRequest {
 func (x *HelloRequest) Bind(r *http.Request) (err error) {
 	values, parseErr := url.ParseQuery(r.URL.RawQuery)
 	if parseErr != nil {
-		err = errutil.Explain(err, "parse query error: %w", parseErr)
+		err = errutil.Explain(err, "parse query error: %s", parseErr)
 		return
 	}
 
@@ -58,7 +58,7 @@ func (x *HelloRequest) Bind(r *http.Request) (err error) {
 		if len(v) == 1 {
 			x.Message = v[0]
 		} else {
-			err = errutil.Stack(err, "invalid value for \"message\"")
+			err = errutil.Explain(err, "invalid value for \"message\"")
 		}
 	}
 	if !hasMessage {
@@ -69,7 +69,7 @@ func (x *HelloRequest) Bind(r *http.Request) (err error) {
 
 func (x *HelloRequest) Validate() (err error) {
 	if validateErr := x.HelloRequestBody.Validate(); validateErr != nil {
-		err = errutil.Stack(err, "validate failed on \"HelloRequest\": %w", validateErr)
+		err = errutil.Stack(err, "validate failed on \"HelloRequest\": %s", validateErr)
 	}
 	return
 }
@@ -82,10 +82,7 @@ func (x *HelloRequestBody) DecodeJSON(d jsonflow.Decoder) (err error) {
 		return err
 	}
 
-	for {
-		if d.PeekKind() == '}' {
-			break
-		}
+	for d.PeekKind() != '}' {
 
 		var key string
 		key, err = jsonflow.DecodeString(d)
@@ -172,9 +169,9 @@ func TestHello(t *testing.T) {
 	if err != nil {
 		panic(err)
 	}
-	resp.Body.Close()
+	_ = resp.Body.Close()
 	fmt.Println(string(b))
-	svr.Shutdown(t.Context())
+	_ = svr.Shutdown(t.Context())
 }
 
 func TestStream(t *testing.T) {
@@ -195,7 +192,7 @@ func TestStream(t *testing.T) {
 	if err != nil {
 		panic(err)
 	}
-	resp.Body.Close()
+	_ = resp.Body.Close()
 	fmt.Print(string(b))
-	svr.Shutdown(t.Context())
+	_ = svr.Shutdown(t.Context())
 }
diff --git a/iterutil/iterutil_test.go b/iterutil/iterutil_test.go
@@ -37,14 +37,14 @@ func BenchmarkRanges(b *testing.B) {
 
 	b.Run("for", func(b *testing.B) {
 		for b.Loop() {
-			fibonacci(N)
+			_ = fibonacci(N)
 		}
 	})
 
 	b.Run("loop", func(b *testing.B) {
 		for b.Loop() {
 			Times(5, func(i int) {
-				fibonacci(N)
+				_ = fibonacci(N)
 			})
 		}
 	})
diff --git a/jsonflow/decode.go b/jsonflow/decode.go
@@ -335,10 +335,7 @@ func DecodeArray[T any](
 		case '[':
 			_, _, _ = d.ReadToken()
 			v := make([]T, 0)
-			for {
-				if d.PeekKind() == ']' {
-					break
-				}
+			for d.PeekKind() != ']' {
 				i, err := parseFn(d)
 				if err != nil {
 					return nil, err
@@ -372,10 +369,7 @@ func DecodeMap[K comparable, V any](
 		case '{':
 			_, _, _ = d.ReadToken()
 			m := make(map[K]V)
-			for {
-				if d.PeekKind() == '}' {
-					break
-				}
+			for d.PeekKind() != '}' {
 				key, err := parseKeyFn(d)
 				if err != nil {
 					return nil, err
diff --git a/jsonflow/decode_test.go b/jsonflow/decode_test.go
@@ -862,10 +862,7 @@ func (b *TestObject) DecodeJSON(d Decoder) error {
 		foundInt bool
 	)
 
-	for {
-		if d.PeekKind() == '}' {
-			break
-		}
+	for d.PeekKind() != '}' {
 		key, err := DecodeString(d)
 		if err != nil {
 			return err
diff --git a/mathutil/overflow.go b/mathutil/overflow.go
@@ -33,7 +33,6 @@ func OverflowInt[T ~int | ~int8 | ~int16 | ~int32 | ~int64](v int64) bool {
 	case int32:
 		return v > math.MaxInt32 || v < math.MinInt32
 	case int64:
-		return v > math.MaxInt64 || v < math.MinInt64
 	}
 	return false
 }
diff --git a/scripts/update-workflow-versions.sh b/scripts/update-workflow-versions.sh
diff --git a/testing/internal/assert.go b/testing/internal/assert.go
diff --git a/typeutil/typeutil_test.go b/typeutil/typeutil_test.go

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ func NewHelloRequest() *HelloRequest {`
`45`	`45`	`func (x HelloRequest) Bind(r http.Request) (err error) {`
`46`	`46`	`values, parseErr := url.ParseQuery(r.URL.RawQuery)`
`47`	`47`	`if parseErr != nil {`
`48`		`- err = errutil.Explain(err, "parse query error: %w", parseErr)`
	`48`	`+ err = errutil.Explain(err, "parse query error: %s", parseErr)`
`49`	`49`	`return`
`50`	`50`	`}`
`51`	`51`
`@@ -58,7 +58,7 @@ func (x HelloRequest) Bind(r http.Request) (err error) {`
`58`	`58`	`if len(v) == 1 {`
`59`	`59`	`x.Message = v[0]`
`60`	`60`	`} else {`
`61`		`- err = errutil.Stack(err, "invalid value for \"message\"")`
	`61`	`+ err = errutil.Explain(err, "invalid value for \"message\"")`
`62`	`62`	`}`
`63`	`63`	`}`
`64`	`64`	`if !hasMessage {`
`@@ -69,7 +69,7 @@ func (x HelloRequest) Bind(r http.Request) (err error) {`
`69`	`69`
`70`	`70`	`func (x *HelloRequest) Validate() (err error) {`
`71`	`71`	`if validateErr := x.HelloRequestBody.Validate(); validateErr != nil {`
`72`		`- err = errutil.Stack(err, "validate failed on \"HelloRequest\": %w", validateErr)`
	`72`	`+ err = errutil.Stack(err, "validate failed on \"HelloRequest\": %s", validateErr)`
`73`	`73`	`}`
`74`	`74`	`return`
`75`	`75`	`}`
`@@ -82,10 +82,7 @@ func (x *HelloRequestBody) DecodeJSON(d jsonflow.Decoder) (err error) {`
`82`	`82`	`return err`
`83`	`83`	`}`
`84`	`84`
`85`		`- for {`
`86`		`- if d.PeekKind() == '}' {`
`87`		`- break`
`88`		`- }`
	`85`	`+ for d.PeekKind() != '}' {`
`89`	`86`
`90`	`87`	`var key string`
`91`	`88`	`key, err = jsonflow.DecodeString(d)`
`@@ -172,9 +169,9 @@ func TestHello(t *testing.T) {`
`172`	`169`	`if err != nil {`
`173`	`170`	`panic(err)`
`174`	`171`	`}`
`175`		`- resp.Body.Close()`
	`172`	`+ _ = resp.Body.Close()`
`176`	`173`	`fmt.Println(string(b))`
`177`		`- svr.Shutdown(t.Context())`
	`174`	`+ _ = svr.Shutdown(t.Context())`
`178`	`175`	`}`
`179`	`176`
`180`	`177`	`func TestStream(t *testing.T) {`
`@@ -195,7 +192,7 @@ func TestStream(t *testing.T) {`
`195`	`192`	`if err != nil {`
`196`	`193`	`panic(err)`
`197`	`194`	`}`
`198`		`- resp.Body.Close()`
	`195`	`+ _ = resp.Body.Close()`
`199`	`196`	`fmt.Print(string(b))`
`200`		`- svr.Shutdown(t.Context())`
	`197`	`+ _ = svr.Shutdown(t.Context())`
`201`	`198`	`}`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,6 @@ func OverflowInt[T ~int \| ~int8 \| ~int16 \| ~int32 \| ~int64](v int64) bool {`
`33`	`33`	`case int32:`
`34`	`34`	`return v > math.MaxInt32 \|\| v < math.MinInt32`
`35`	`35`	`case int64:`
`36`		`- return v > math.MaxInt64 \|\| v < math.MinInt64`
`37`	`36`	`}`
`38`	`37`	`return false`
`39`	`38`	`}`