feat: add modernize suite

Author: ldez

.golangci.next.reference.yml

@@ -85,6 +85,7 @@ linters:
     - mirror
     - misspell
     - mnd
+    - modernize
     - musttag
     - nakedret
     - nestif
@@ -199,6 +200,7 @@ linters:
     - mirror
     - misspell
     - mnd
+    - modernize
     - musttag
     - nakedret
     - nestif
@@ -2106,6 +2108,45 @@ linters:
         - '^math\.'
         - '^http\.StatusText$'
 
+    modernize:
+      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
+
     musttag:
       # A set of custom functions to check in addition to the builtin ones.
       # Default: json, xml, gopkg.in/yaml.v3, BurntSushi/toml, mitchellh/mapstructure, jmoiron/sqlx

jsonschema/golangci.next.jsonschema.json

@@ -706,6 +706,28 @@
         "header"
       ]
     },
+    "modernize-analyzers": {
+      "enum": [
+        "any",
+        "bloop",
+        "fmtappendf",
+        "forvar",
+        "mapsloop",
+        "minmax",
+        "newexpr",
+        "omitzero",
+        "rangeint",
+        "reflecttypefor",
+        "slicescontains",
+        "slicessort",
+        "stditerators",
+        "stringscutprefix",
+        "stringsseq",
+        "stringsbuilder",
+        "testingcontext",
+        "waitgroup"
+      ]
+    },
     "wsl-checks": {
       "enum": [
         "assign",
@@ -835,6 +857,7 @@
             "mirror",
             "misspell",
             "mnd",
+            "modernize",
             "musttag",
             "nakedret",
             "nestif",
@@ -2829,6 +2852,19 @@
             }
           }
         },
+        "modernizeSettings": {
+          "type": "object",
+          "additionalProperties": false,
+          "properties": {
+            "disable": {
+              "description": "List of analyzers to disable.",
+              "type": "array",
+              "items": {
+                "$ref": "#/definitions/modernize-analyzers"
+              }
+            }
+          }
+        },
         "nolintlintSettings": {
           "type": "object",
           "additionalProperties": false,
@@ -4747,6 +4783,9 @@
             "mnd": {
               "$ref": "#/definitions/settings/definitions/mndSettings"
             },
+            "modernize": {
+              "$ref": "#/definitions/settings/definitions/modernizeSettings"
+            },
             "nolintlint":{
               "$ref": "#/definitions/settings/definitions/nolintlintSettings"
             },

pkg/config/linters_settings.go

@@ -269,6 +269,7 @@ type LintersSettings struct {
 	Makezero                 MakezeroSettings                 `mapstructure:"makezero"`
 	Misspell                 MisspellSettings                 `mapstructure:"misspell"`
 	Mnd                      MndSettings                      `mapstructure:"mnd"`
+	Modernize                ModernizeSettings                `mapstructure:"modernize"`
 	MustTag                  MustTagSettings                  `mapstructure:"musttag"`
 	Nakedret                 NakedretSettings                 `mapstructure:"nakedret"`
 	Nestif                   NestifSettings                   `mapstructure:"nestif"`
@@ -758,6 +759,10 @@ type MndSettings struct {
 	IgnoredFunctions []string `mapstructure:"ignored-functions"`
 }
 
+type ModernizeSettings struct {
+	Disable []string `mapstructure:"disable"`
+}
+
 type NoLintLintSettings struct {
 	RequireExplanation bool     `mapstructure:"require-explanation"`
 	RequireSpecific    bool     `mapstructure:"require-specific"`

pkg/golinters/modernize/modernize.go

@@ -0,0 +1,34 @@
+package modernize
+
+import (
+	"slices"
+
+	"github.com/golangci/golangci-lint/v2/pkg/config"
+	"github.com/golangci/golangci-lint/v2/pkg/goanalysis"
+
+	"golang.org/x/tools/go/analysis"
+	"golang.org/x/tools/go/analysis/passes/modernize"
+)
+
+func New(settings *config.ModernizeSettings) *goanalysis.Linter {
+	var analyzers []*analysis.Analyzer
+
+	if settings == nil {
+		analyzers = modernize.Suite
+	} else {
+		for _, analyzer := range modernize.Suite {
+			if slices.Contains(settings.Disable, analyzer.Name) {
+				continue
+			}
+
+			analyzers = append(analyzers, analyzer)
+		}
+	}
+
+	return goanalysis.NewLinter(
+		"modernize",
+		"A suite of analyzers that suggest simplifications to Go code, using modern language and library features.",
+		analyzers,
+		nil).
+		WithLoadMode(goanalysis.LoadModeTypesInfo)
+}

pkg/golinters/modernize/modernize_integration_test.go

@@ -0,0 +1,19 @@
+package modernize
+
+import (
+	"testing"
+
+	"github.com/golangci/golangci-lint/v2/test/testshared/integration"
+)
+
+func TestFromTestdata(t *testing.T) {
+	integration.RunTestdata(t)
+}
+
+func TestFix(t *testing.T) {
+	integration.RunFix(t)
+}
+
+func TestFixPathPrefix(t *testing.T) {
+	integration.RunFixPathPrefix(t)
+}

pkg/golinters/modernize/testdata/modernize_any.go

@@ -0,0 +1,11 @@
+//golangcitest:args -Emodernize
+package testdata
+
+func _(x interface{}) {} // want "interface{} can be replaced by any"
+
+func _() {
+	var x interface{} // want "interface{} can be replaced by any"
+	const any = 1
+	var y interface{} // nope: any is shadowed here
+	_, _ = x, y
+}

pkg/golinters/modernize/testdata/modernize_custom.go

@@ -0,0 +1,13 @@
+//golangcitest:args -Emodernize
+//golangcitest:config_path testdata/modernize_custom.yml
+//golangcitest:expected_exitcode 0
+package testdata
+
+func _(x interface{}) {}
+
+func _() {
+	var x interface{}
+	const any = 1
+	var y interface{} // nope: any is shadowed here
+	_, _ = x, y
+}

pkg/golinters/modernize/testdata/modernize_custom.yml

@@ -0,0 +1,7 @@
+version: "2"
+
+linters:
+  settings:
+    modernize:
+      disable:
+        - any

pkg/lint/lintersdb/builder_linter.go

@@ -73,6 +73,7 @@ import (
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/mirror"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/misspell"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/mnd"
+	"github.com/golangci/golangci-lint/v2/pkg/golinters/modernize"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/musttag"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/nakedret"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/nestif"
@@ -382,6 +383,11 @@ func (LinterBuilder) Build(cfg *config.Config) ([]*linter.Config, error) {
 			WithSince("v1.22.0").
 			WithURL("https://github.com/tommy-muehle/go-mnd"),
 
+		linter.NewConfig(modernize.New(&cfg.Linters.Settings.Modernize)).
+			WithSince("v2.6.0").
+			WithLoadForGoAnalysis().
+			WithURL("https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize"),
+
 		linter.NewConfig(gomoddirectives.New(&cfg.Linters.Settings.GoModDirectives)).
 			WithSince("v1.39.0").
 			WithURL("https://github.com/ldez/gomoddirectives"),