feat: add usestdlibvars

Author: sashamelentyev

.golangci.reference.yml

@@ -1662,6 +1662,20 @@ linters-settings:
       # Default: true
       begin: false
 
+  usestdlibvars:
+    # Suggest the use of http.MethodXX
+    http-method: true
+    # Suggest the use of http.StatusXX
+    http-status-code: true
+    # Suggest the use of time.Weekday
+    time-weekday: false
+    # Suggest the use of time.Month
+    time-month: false
+    # Suggest the use of time.Layout
+    time-layout: false
+    # Suggest the use of crypto.Hash
+    crypto-hash: false
+
   unparam:
     # Inspect exported functions.
     #
@@ -1927,6 +1941,7 @@ linters:
     - unconvert
     - unparam
     - unused
+    - usestdlibvars
     - varcheck
     - varnamelen
     - wastedassign
@@ -2028,6 +2043,7 @@ linters:
     - unconvert
     - unparam
     - unused
+    - usestdlibvars
     - varcheck
     - varnamelen
     - wastedassign

go.mod

@@ -74,6 +74,7 @@ require (
 	github.com/ryancurrah/gomodguard v1.2.3
 	github.com/ryanrolds/sqlclosecheck v0.3.0
 	github.com/sanposhiho/wastedassign/v2 v2.0.6
+	github.com/sashamelentyev/usestdlibvars v1.7.0
 	github.com/securego/gosec/v2 v2.12.0
 	github.com/shazow/go-diff v0.0.0-20160112020656-b6b7b6733b8c
 	github.com/shirou/gopsutil/v3 v3.22.6

go.sum

@@ -588,6 +588,15 @@ type TenvSettings struct {
 	All bool `mapstructure:"all"`
 }
 
+type UseStdlibVarsSettings struct {
+	HTTPMethod     bool `mapstructure:"http-method"`
+	HTTPStatusCode bool `mapstructure:"http-status-code"`
+	TimeWeekday    bool `mapstructure:"time-weekday"`
+	TimeMonth      bool `mapstructure:"time-month"`
+	TimeLayout     bool `mapstructure:"time-layout"`
+	CryptoHash     bool `mapstructure:"crypto-hash"`
+}
+
 type UnparamSettings struct {
 	CheckExported bool `mapstructure:"check-exported"`
 	Algo          string

pkg/config/linters_settings.go

@@ -0,0 +1,32 @@
+package golinters
+
+import (
+	"github.com/sashamelentyev/usestdlibvars/pkg/analyzer"
+	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
+)
+
+func NewUseStdlibVars(cfg *config.UseStdlibVarsSettings) *goanalysis.Linter {
+	a := analyzer.New()
+
+	cfgMap := make(map[string]map[string]interface{})
+	if cfg != nil {
+		cfgMap[a.Name] = map[string]interface{}{
+			analyzer.HTTPMethodFlag:     cfg.HTTPMethod,
+			analyzer.HTTPStatusCodeFlag: cfg.HTTPStatusCode,
+			analyzer.TimeWeekdayFlag:    cfg.TimeWeekday,
+			analyzer.TimeMonthFlag:      cfg.TimeMonth,
+			analyzer.TimeLayoutFlag:     cfg.TimeLayout,
+			analyzer.CryptoHashFlag:     cfg.CryptoHash,
+		}
+	}
+
+	return goanalysis.NewLinter(
+		a.Name,
+		a.Doc,
+		[]*analysis.Analyzer{a},
+		cfgMap,
+	).WithLoadMode(goanalysis.LoadModeSyntax)
+}

pkg/golinters/usestdlibvars.go

@@ -164,6 +164,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 		thelperCfg          *config.ThelperSettings
 		unparamCfg          *config.UnparamSettings
 		unusedCfg           *config.StaticCheckSettings
+		usestdlibvars       *config.UseStdlibVarsSettings
 		varcheckCfg         *config.VarCheckSettings
 		varnamelenCfg       *config.VarnamelenSettings
 		whitespaceCfg       *config.WhitespaceSettings
@@ -766,6 +767,11 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 			WithChangeTypes().
 			WithURL("https://github.com/dominikh/go-tools/tree/master/unused"),
 
+		linter.NewConfig(golinters.NewUseStdlibVars(usestdlibvars)).
+			WithSince("v1.48.0").
+			WithPresets(linter.PresetStyle).
+			WithURL("https://github.com/sashamelentyev/usestdlibvars"),
+
 		linter.NewConfig(golinters.NewVarcheck(varcheckCfg)).
 			WithSince("v1.0.0").
 			WithLoadForGoAnalysis().

pkg/lint/lintersdb/manager.go

@@ -0,0 +1,89 @@
+//golangcitest:args -Eusestdlibvars
+package testdata
+
+import "net/http"
+
+func _200() {
+	_ = 200
+}
+
+func _200_1() {
+	var w http.ResponseWriter
+	w.WriteHeader(200) // ERROR `can use http.StatusOK instead "200"`
+}
+
+func sunday() {
+	_ = "Sunday" // ERROR `can use time.Sunday.String\(\) instead "Sunday"`
+}
+
+func monday() {
+	_ = "Monday" // ERROR `can use time.Monday.String\(\) instead "Monday"`
+}
+
+func tuesday() {
+	_ = "Tuesday" // ERROR `can use time.Tuesday.String\(\) instead "Tuesday"`
+}
+
+func wednesday() {
+	_ = "Wednesday" // ERROR `can use time.Wednesday.String\(\) instead "Wednesday"`
+}
+
+func thursday() {
+	_ = "Thursday" // ERROR `can use time.Thursday.String\(\) instead "Thursday"`
+}
+
+func friday() {
+	_ = "Friday" // ERROR `can use time.Friday.String\(\) instead "Friday"`
+}
+
+func saturday() {
+	_ = "Saturday" // ERROR `can use time.Saturday.String\(\) instead "Saturday"`
+}
+
+func january() {
+	_ = "January" // ERROR `can use time.January.String\(\) instead "January"`
+}
+
+func february() {
+	_ = "February" // ERROR `can use time.February.String\(\) instead "February"`
+}
+
+func march() {
+	_ = "March" // ERROR `can use time.March.String\(\) instead "March"`
+}
+
+func april() {
+	_ = "April" // ERROR `can use time.April.String\(\) instead "April"`
+}
+
+func may() {
+	_ = "May" // ERROR `can use time.May.String\(\) instead "May"`
+}
+
+func june() {
+	_ = "June" // ERROR `can use time.June.String\(\) instead "June"`
+}
+
+func july() {
+	_ = "July" // ERROR `can use time.July.String\(\) instead "July"`
+}
+
+func august() {
+	_ = "August" // ERROR `can use time.August.String\(\) instead "August"`
+}
+
+func september() {
+	_ = "September" // ERROR `can use time.September.String\(\) instead "September"`
+}
+
+func october() {
+	_ = "October" // ERROR `can use time.October.String\(\) instead "October"`
+}
+
+func november() {
+	_ = "November" // ERROR `can use time.November.String\(\) instead "November"`
+}
+
+func december() {
+	_ = "December" // ERROR `can use time.December.String\(\) instead "December"`
+}