feat: add rowserr linter

Add rowserr a linter which checks for missing sql Rows.Err() calls
that supports generics.

Author: stevenh

.golangci.reference.yml

@@ -1636,6 +1636,14 @@ linters-settings:
         severity: warning
         disabled: false
 
+  rowserr:
+    # packages sets additional packages to check.
+    # The following know sql packages are always checked:
+    # - database/sql
+    # - github.com/jmoiron/sqlx
+    # Default: []
+    packages: []
+
   rowserrcheck:
     # database/sql is always checked
     # Default: []
@@ -2039,6 +2047,7 @@ linters:
     - promlinter
     - reassign
     - revive
+    - rowserr
     - rowserrcheck
     - scopelint
     - sqlclosecheck
@@ -2146,6 +2155,7 @@ linters:
     - promlinter
     - reassign
     - revive
+    - rowserr
     - rowserrcheck
     - scopelint
     - sqlclosecheck

go.mod

@@ -93,6 +93,7 @@ require (
 	github.com/spf13/viper v1.12.0
 	github.com/ssgreg/nlreturn/v2 v2.2.1
 	github.com/stbenjam/no-sprintf-host-port v0.1.1
+	github.com/stevenh/go-rowserr v0.2.0
 	github.com/stretchr/testify v1.8.1
 	github.com/tdakkota/asciicheck v0.1.1
 	github.com/tetafro/godot v1.4.11

go.sum

@@ -190,6 +190,7 @@ type LintersSettings struct {
 	Promlinter       PromlinterSettings
 	Reassign         ReassignSettings
 	Revive           ReviveSettings
+	RowsErr          RowsErrSettings
 	RowsErrCheck     RowsErrCheckSettings
 	Staticcheck      StaticCheckSettings
 	Structcheck      StructCheckSettings
@@ -600,6 +601,9 @@ type ReviveSettings struct {
 		Severity string
 	}
 }
+type RowsErrSettings struct {
+	Packages []string
+}
 
 type RowsErrCheckSettings struct {
 	Packages []string

pkg/config/linters_settings.go

@@ -0,0 +1,19 @@
+package golinters
+
+import (
+	"github.com/stevenh/go-rowserr/pkg/rowserr"
+	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
+)
+
+func NewRowsErr(settings *config.RowsErrSettings) *goanalysis.Linter {
+	a := rowserr.NewAnalyzer(settings.Packages...)
+	return goanalysis.NewLinter(
+		a.Name,
+		a.Doc,
+		[]*analysis.Analyzer{a},
+		nil,
+	).WithLoadMode(goanalysis.LoadModeTypesInfo)
+}

pkg/golinters/rowserr.go

@@ -158,6 +158,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 		promlinterCfg       *config.PromlinterSettings
 		reassignCfg         *config.ReassignSettings
 		reviveCfg           *config.ReviveSettings
+		rowserrCfg          *config.RowsErrSettings
 		rowserrcheckCfg     *config.RowsErrCheckSettings
 		staticcheckCfg      *config.StaticCheckSettings
 		structcheckCfg      *config.StructCheckSettings
@@ -234,6 +235,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 		promlinterCfg = &m.cfg.LintersSettings.Promlinter
 		reassignCfg = &m.cfg.LintersSettings.Reassign
 		reviveCfg = &m.cfg.LintersSettings.Revive
+		rowserrCfg = &m.cfg.LintersSettings.RowsErr
 		rowserrcheckCfg = &m.cfg.LintersSettings.RowsErrCheck
 		staticcheckCfg = &m.cfg.LintersSettings.Staticcheck
 		structcheckCfg = &m.cfg.LintersSettings.Structcheck
@@ -707,6 +709,12 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 			ConsiderSlow().
 			WithURL("https://github.com/mgechev/revive"),
 
+		linter.NewConfig(golinters.NewRowsErr(rowserrCfg)).
+			WithSince("v1.51.0").
+			WithLoadForGoAnalysis().
+			WithPresets(linter.PresetBugs, linter.PresetSQL).
+			WithURL("https://github.com/stevenh/go-rowserr"),
+
 		linter.NewConfig(golinters.NewRowsErrCheck(rowserrcheckCfg)).
 			WithSince("v1.23.0").
 			WithLoadForGoAnalysis().

pkg/lint/lintersdb/manager.go

@@ -0,0 +1,17 @@
+//golangcitest:args -Erowserr
+package testdata
+
+import (
+	"database/sql"
+)
+
+func RowsErrNotChecked(db *sql.DB) {
+	rows, err := db.Query("select id from tb") // want "rows.Err\\(\\) must be checked"
+	if err != nil {
+		// Handle error.
+	}
+
+	for rows.Next() {
+		// Handle row.
+	}
+}