chore: update implementation

Author: ldez

.golangci.next.reference.yml

@@ -2121,42 +2121,46 @@ linters:
       # List of analyzers to disable.
       # By default, all analyzers are enabled.
       disable:
-        # Replace interface{} with any.
-        - any
-        # Replace for-range over b.N with b.Loop.
-        - bloop
-        # Replace []byte(fmt.Sprintf) with fmt.Appendf.
-        - fmtappendf
-        # Remove redundant re-declaration of loop variables.
-        - forvar
-        # Replace explicit loops over maps with calls to maps package.
-        - mapsloop
-        # Replace if/else statements with calls to min or max.
-        - minmax
-        # Simplify code by using go1.26's new(expr).
-        - newexpr
-        # Suggest replacing omitempty with omitzero for struct fields.
-        - omitzero
-        # Replace 3-clause for loops with for-range over integers.
-        - rangeint
-        # Replace reflect.TypeOf(x) with TypeFor[T]().
-        - reflecttypefor
-        # Replace loops with slices.Contains or slices.ContainsFunc.
-        - slicescontains
-        # Replace sort.Slice with slices.Sort for basic types.
-        - slicessort
-        # Use iterators instead of Len/At-style APIs.
-        - stditerators
-        # Replace HasPrefix/TrimPrefix with CutPrefix.
-        - stringscutprefix
-        # Replace ranging over Split/Fields with SplitSeq/FieldsSeq.
-        - stringsseq
-        # Replace += with strings.Builder.
-        - stringsbuilder
-        # Replace context.WithCancel with t.Context in tests.
-        - testingcontext
-        # Replace wg.Add(1)/go/wg.Done() with wg.Go.
-        - waitgroup
+      # Replace interface{} with any.
+      - any
+      # Replace for-range over b.N with b.Loop.
+      - bloop
+      # Replace []byte(fmt.Sprintf) with fmt.Appendf.
+      - fmtappendf
+      # Remove redundant re-declaration of loop variables.
+      - forvar
+      # Replace explicit loops over maps with calls to maps package.
+      - mapsloop
+      # Replace if/else statements with calls to min or max.
+      - minmax
+      # Simplify code by using go1.26's new(expr).
+      - newexpr
+      # Suggest replacing omitempty with omitzero for struct fields.
+      - omitzero
+      # Remove obsolete //+build comments.
+      - plusbuild
+      # Replace 3-clause for loops with for-range over integers.
+      - rangeint
+      # Replace reflect.TypeOf(x) with TypeFor[T]().
+      - reflecttypefor
+      # Replace loops with slices.Contains or slices.ContainsFunc.
+      - slicescontains
+      # Replace sort.Slice with slices.Sort for basic types.
+      - slicessort
+      # Use iterators instead of Len/At-style APIs.
+      - stditerators
+      # Replace strings.Index etc. with strings.Cut.
+      - stringscut
+      # Replace HasPrefix/TrimPrefix with CutPrefix.
+      - stringscutprefix
+      # Replace ranging over Split/Fields with SplitSeq/FieldsSeq.
+      - stringsseq
+      # Replace += with strings.Builder.
+      - stringsbuilder
+      # Replace context.WithCancel with t.Context in tests.
+      - testingcontext
+      # Replace wg.Add(1)/go/wg.Done() with wg.Go.
+      - waitgroup
 
     musttag:
       # A set of custom functions to check in addition to the builtin ones.

jsonschema/golangci.next.jsonschema.json

@@ -717,11 +717,13 @@
         "minmax",
         "newexpr",
         "omitzero",
+        "plusbuild",
         "rangeint",
         "reflecttypefor",
         "slicescontains",
         "slicessort",
         "stditerators",
+        "stringscut",
         "stringscutprefix",
         "stringsseq",
         "stringsbuilder",

pkg/golinters/modernize/testdata/fix/in/reflecttypefor.go

@@ -17,7 +17,7 @@ var (
 	_ = reflect.TypeOf((*error)(nil))     // want "reflect.TypeOf call can be simplified using TypeFor"
 	_ = reflect.TypeOf(io.Reader(nil))    // nope (likely a mistake)
 	_ = reflect.TypeOf((*io.Reader)(nil)) // want "reflect.TypeOf call can be simplified using TypeFor"
-	_ = reflect.TypeOf(*new(time.Time))   // nope (false negative of noEffects)
+	_ = reflect.TypeOf(*new(time.Time))   // want "reflect.TypeOf call can be simplified using TypeFor"
 	_ = reflect.TypeOf(time.Time{})       // want "reflect.TypeOf call can be simplified using TypeFor"
 	_ = reflect.TypeOf(time.Duration(0))  // want "reflect.TypeOf call can be simplified using TypeFor"
 )

pkg/golinters/modernize/testdata/fix/in/slicescontains.go

@@ -52,7 +52,7 @@ func assignTrueBreak(slice []int, needle int) {
 	print(found)
 }
 
-func assignFalseBreak(slice []int, needle int) { // TODO: treat this specially like booleanTrue
+func assignFalseBreak(slice []int, needle int) {
 	found := true
 	for _, elem := range slice { // want "Loop can be simplified using slices.Contains"
 		if elem == needle {

pkg/golinters/modernize/testdata/fix/out/forvar.go

@@ -5,37 +5,28 @@ package forvar
 func _(m map[int]int, s []int) {
 	// changed
 	for i := range s {
-		// want "copying variable is unneeded"
 		go f(i)
 	}
 	for _, v := range s {
-		// want "copying variable is unneeded"
 		go f(v)
 	}
 	for k, v := range m {
-		// want "copying variable is unneeded"
-		// want "copying variable is unneeded"
 		go f(k)
 		go f(v)
 	}
 	for k, v := range m {
-		// want "copying variable is unneeded"
-		// want "copying variable is unneeded"
 		go f(k)
 		go f(v)
 	}
 	for k, v := range m {
-		// want "copying variable is unneeded"
 		go f(k)
 		go f(v)
 	}
 	for k, v := range m {
-		// want "copying variable is unneeded"
 		go f(k)
 		go f(v)
 	}
 	for i := range s {
-		/* hi */ // want "copying variable is unneeded"
 		go f(i)
 	}
 	// nope

pkg/golinters/modernize/testdata/fix/out/reflecttypefor.go

@@ -17,7 +17,7 @@ var (
 	_ = reflect.TypeFor[*error]()        // want "reflect.TypeOf call can be simplified using TypeFor"
 	_ = reflect.TypeOf(io.Reader(nil))   // nope (likely a mistake)
 	_ = reflect.TypeFor[*io.Reader]()    // want "reflect.TypeOf call can be simplified using TypeFor"
-	_ = reflect.TypeOf(*new(time.Time))  // nope (false negative of noEffects)
+	_ = reflect.TypeFor[time.Time]()     // want "reflect.TypeOf call can be simplified using TypeFor"
 	_ = reflect.TypeFor[time.Time]()     // want "reflect.TypeOf call can be simplified using TypeFor"
 	_ = reflect.TypeFor[time.Duration]() // want "reflect.TypeOf call can be simplified using TypeFor"
 )

pkg/golinters/modernize/testdata/fix/out/slicescontains.go

@@ -38,11 +38,8 @@ func assignTrueBreak(slice []int, needle int) {
 	print(found)
 }
 
-func assignFalseBreak(slice []int, needle int) { // TODO: treat this specially like booleanTrue
-	found := true
-	if slices.Contains(slice, needle) {
-		found = false
-	}
+func assignFalseBreak(slice []int, needle int) {
+	found := !slices.Contains(slice, needle)
 	print(found)
 }
 

pkg/golinters/modernize/testdata/fix/out/stditerators.go

@@ -15,9 +15,9 @@ func _(scope *types.Scope) {
 		print(child)
 	}
 	{
-		const child = 0                        // shadowing of preferred name at def
-		for child0 := range scope.Children() { // want "NumChildren/Child loop can simplified using Scope.Children iteration"
-			print(child0)
+		const child = 0                       // shadowing of preferred name at def
+		for child := range scope.Children() { // want "NumChildren/Child loop can simplified using Scope.Children iteration"
+			print(child)
 		}
 	}
 	{

pkg/golinters/modernize/testdata/fix/out/waitgroup.go

@@ -12,17 +12,14 @@ import (
 // supported case for pattern 1.
 func _() {
 	var wg sync.WaitGroup
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	wg.Go(func() {
 		fmt.Println()
 	})
 
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	wg.Go(func() {
 	})
 
 	for range 10 {
-		// want "Goroutine creation can be simplified using WaitGroup.Go"
 		wg.Go(func() {
 			fmt.Println()
 		})
@@ -32,17 +29,14 @@ func _() {
 // supported case for pattern 2.
 func _() {
 	var wg sync.WaitGroup
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	wg.Go(func() {
 		fmt.Println()
 	})
 
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	wg.Go(func() {
 	})
 
 	for range 10 {
-		// want "Goroutine creation can be simplified using WaitGroup.Go"
 		wg.Go(func() {
 			fmt.Println()
 		})
@@ -52,19 +46,16 @@ func _() {
 // this function puts some wrong usages but waitgroup modernizer will still offer fixes.
 func _() {
 	var wg sync.WaitGroup
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	wg.Go(func() {
 		defer wg.Done()
 		fmt.Println()
 	})
 
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	wg.Go(func() {
 		fmt.Println()
 		wg.Done()
 	})
 
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	wg.Go(func() {
 		fmt.Println()
 		wg.Done()
@@ -156,20 +147,17 @@ type ServerContainer struct {
 
 func _() {
 	var s Server
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	s.wg.Go(func() {
 		print()
 	})
 
 	var sc ServerContainer
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	sc.serv.wg.Go(func() {
 		print()
 	})
 
 	var wg sync.WaitGroup
 	arr := [1]*sync.WaitGroup{&wg}
-	// want "Goroutine creation can be simplified using WaitGroup.Go"
 	arr[0].Go(func() {
 		print()
 	})

test/testshared/directives.go

@@ -149,7 +149,10 @@ func evaluateBuildTags(tb testing.TB, line string) bool {
 }
 
 func buildTagGoVersion(tag string) bool {
-	vRuntime, err := hcversion.NewVersion(strings.TrimPrefix(runtime.Version(), "go"))
+	// `runtime.Version()` returns the go version with extra info. (ex: go1.25.1 X:nodwarf5)
+	before, _, _ := strings.Cut(runtime.Version(), " ")
+
+	vRuntime, err := hcversion.NewVersion(strings.TrimPrefix(before, "go"))
 	if err != nil {
 		return false
 	}