Add perfsprint linter (#3714)

Co-authored-by: Fernandez Ludovic <[email protected]>
Co-authored-by: Anton Telyshev <[email protected]>

.golangci.reference.yml

@@ -2314,6 +2314,7 @@ linters:
     - nosnakecase
     - nosprintfhostport
     - paralleltest
+    - perfsprint
     - prealloc
     - predeclared
     - promlinter
@@ -2433,6 +2434,7 @@ linters:
     - nosnakecase
     - nosprintfhostport
     - paralleltest
+    - perfsprint
     - prealloc
     - predeclared
     - promlinter

go.mod

@@ -27,6 +27,7 @@ require (
 	github.com/breml/errchkjson v0.3.6
 	github.com/butuzov/ireturn v0.2.1
 	github.com/butuzov/mirror v1.1.0
+	github.com/catenacyber/perfsprint v0.2.0
 	github.com/charithe/durationcheck v0.0.10
 	github.com/curioswitch/go-reassign v0.2.0
 	github.com/daixiang0/gci v0.11.2

pkg/golinters/perfsprint.go

@@ -0,0 +1,19 @@
+package golinters
+
+import (
+	"github.com/catenacyber/perfsprint/analyzer"
+	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
+)
+
+func NewPerfSprint() *goanalysis.Linter {
+	a := analyzer.Analyzer
+
+	return goanalysis.NewLinter(
+		a.Name,
+		a.Doc,
+		[]*analysis.Analyzer{a},
+		nil,
+	).WithLoadMode(goanalysis.LoadModeTypesInfo)
+}

pkg/lint/lintersdb/manager.go

@@ -712,6 +712,12 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 			WithPresets(linter.PresetStyle, linter.PresetTest).
 			WithURL("https://github.com/kunwardeep/paralleltest"),
 
+		linter.NewConfig(golinters.NewPerfSprint()).
+			WithSince("v1.55.0").
+			WithLoadForGoAnalysis().
+			WithPresets(linter.PresetPerformance).
+			WithURL("https://github.com/catenacyber/perfsprint"),
+
 		linter.NewConfig(golinters.NewPreAlloc(preallocCfg)).
 			WithSince("v1.19.0").
 			WithPresets(linter.PresetPerformance).

test/testdata/perfsprint.go

@@ -0,0 +1,49 @@
+//golangcitest:args -Eperfsprint
+package testdata
+
+import "fmt"
+
+func TestPerfsprint() {
+	var (
+		s   string
+		err error
+		b   bool
+		i   int
+		i64 int64
+		ui  uint
+	)
+
+	fmt.Sprintf("%s", s)           // want "fmt.Sprintf can be replaced with just using the string"
+	fmt.Sprint(s)                  // want "fmt.Sprint can be replaced with just using the string"
+	fmt.Sprintf("%s", err)         // want "fmt.Sprintf can be replaced with err.Error()"
+	fmt.Sprint(err)                // want "fmt.Sprint can be replaced with err.Error()"
+	fmt.Sprintf("%t", b)           // want "fmt.Sprintf can be replaced with faster strconv.FormatBool"
+	fmt.Sprint(b)                  // want "fmt.Sprint can be replaced with faster strconv.FormatBool"
+	fmt.Sprintf("%d", i)           // want "fmt.Sprintf can be replaced with faster strconv.Itoa"
+	fmt.Sprint(i)                  // want "fmt.Sprint can be replaced with faster strconv.Itoa"
+	fmt.Sprintf("%d", i64)         // want "fmt.Sprintf can be replaced with faster strconv.FormatInt"
+	fmt.Sprint(i64)                // want "fmt.Sprint can be replaced with faster strconv.FormatInt"
+	fmt.Sprintf("%d", ui)          // want "fmt.Sprintf can be replaced with faster strconv.FormatUint"
+	fmt.Sprint(ui)                 // want "fmt.Sprint can be replaced with faster strconv.FormatUint"
+	fmt.Sprintf("%x", []byte{'a'}) // want "fmt.Sprintf can be replaced with faster hex.EncodeToString"
+
+	fmt.Sprint("test", 42)
+	fmt.Sprint(42, 42)
+	fmt.Sprintf("test")
+	fmt.Sprintf("%v")
+	fmt.Sprintf("%d")
+	fmt.Sprintf("%d", 42, 42)
+	fmt.Sprintf("%#d", 42)
+	fmt.Sprintf("value %d", 42)
+	fmt.Sprintf("val%d", 42)
+	fmt.Sprintf("%s %v", "hello", "world")
+	fmt.Sprintf("%#v", 42)
+	fmt.Sprintf("%T", struct{ string }{})
+	fmt.Sprintf("%%v", 42)
+	fmt.Sprintf("%3d", 42)
+	fmt.Sprintf("% d", 42)
+	fmt.Sprintf("%-10d", 42)
+	fmt.Sprintf("%[2]d %[1]d\n", 11, 22)
+	fmt.Sprintf("%[3]*.[2]*[1]f", 12.0, 2, 6)
+	fmt.Sprintf("%d %d %#[1]x %#x", 16, 17)
+}