Add iface linter

.golangci.next.reference.yml

@@ -1222,6 +1222,21 @@ linters-settings:
     # Default: false
     var-require-grouping: true
 
+  iface:
+    # By default, set to empty.
+    # Empty means that all analyzers are enabled.
+    # Default: []
+    enable:
+      - unused
+      - identical
+      - opaque
+    settings:
+      unused:
+        # List of packages path to exclude from the check.
+        # Default: []
+        exclude:
+          - github.com/example/log
+
   importas:
     # Do not allow unaliased imports of aliased packages.
     # Default: false
@@ -2659,6 +2674,7 @@ linters:
     - gosmopolitan
     - govet
     - grouper
+    - iface
     - importas
     - inamedparam
     - ineffassign
@@ -2775,6 +2791,7 @@ linters:
     - gosmopolitan
     - govet
     - grouper
+    - iface
     - importas
     - inamedparam
     - ineffassign

go.mod

@@ -116,6 +116,7 @@ require (
 	github.com/ultraware/funlen v0.1.0
 	github.com/ultraware/whitespace v0.1.1
 	github.com/uudashr/gocognit v1.1.3
+	github.com/uudashr/iface v1.2.0
 	github.com/valyala/quicktemplate v1.8.0
 	github.com/xen0n/gosmopolitan v1.2.2
 	github.com/yagipy/maintidx v1.0.0

jsonschema/golangci.next.jsonschema.json

@@ -293,6 +293,13 @@
         "waitgroup-by-value"
       ]
     },
+    "iface-analyzers": {
+      "enum": [
+        "unused",
+        "identical",
+        "opaque"
+      ]
+    },
     "linters": {
       "$comment": "anyOf with enum is used to allow auto completion of non-custom linters",
       "description": "Linters usable.",
@@ -354,6 +361,7 @@
             "gosmopolitan",
             "govet",
             "grouper",
+            "iface",
             "ifshort",
             "importas",
             "inamedparam",
@@ -1910,6 +1918,30 @@
             }
           }
         },
+        "iface": {
+          "type": "object",
+          "additionalProperties": false,
+          "properties": {
+            "enable": {
+              "description": "Enable analyzers by name.",
+              "type": "array",
+              "items": {
+                "$ref": "#/definitions/iface-analyzers"
+              }
+            },
+            "settings": {
+              "type": "object",
+              "propertyNames": {
+                "$ref": "#/definitions/iface-analyzers"
+              },
+              "patternProperties": {
+                "^.*$": {
+                  "type": "object"
+                }
+              }
+            }
+          }
+        },
         "importas": {
           "type": "object",
           "additionalProperties": false,

pkg/config/linters_settings.go

@@ -233,6 +233,7 @@ type LintersSettings struct {
 	Gosmopolitan    GosmopolitanSettings
 	Govet           GovetSettings
 	Grouper         GrouperSettings
+	Iface           IfaceSettings
 	ImportAs        ImportAsSettings
 	Inamedparam     INamedParamSettings
 	InterfaceBloat  InterfaceBloatSettings
@@ -648,6 +649,11 @@ type GrouperSettings struct {
 	VarRequireGrouping        bool `mapstructure:"var-require-grouping"`
 }
 
+type IfaceSettings struct {
+	Enable   []string                  `mapstructure:"enable"`
+	Settings map[string]map[string]any `mapstructure:"settings"`
+}
+
 type ImportAsSettings struct {
 	Alias          []ImportAsAlias
 	NoUnaliased    bool `mapstructure:"no-unaliased"`

pkg/golinters/iface/iface.go

@@ -0,0 +1,60 @@
+package iface
+
+import (
+	"slices"
+	"strings"
+
+	"github.com/uudashr/iface/identical"
+	"github.com/uudashr/iface/opaque"
+	"github.com/uudashr/iface/unused"
+	"golang.org/x/exp/maps"
+	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/goanalysis"
+)
+
+func New(settings *config.IfaceSettings) *goanalysis.Linter {
+	var conf map[string]map[string]any
+	if settings != nil {
+		conf = settings.Settings
+	}
+
+	return goanalysis.NewLinter(
+		"iface",
+		"Detect the incorrect use of interfaces, helping developers avoid interface pollution.",
+		analyzersFromSettings(settings),
+		conf,
+	).WithLoadMode(goanalysis.LoadModeTypesInfo)
+}
+
+func analyzersFromSettings(settings *config.IfaceSettings) []*analysis.Analyzer {
+	allAnalyzers := map[string]*analysis.Analyzer{
+		"unused":    unused.Analyzer,
+		"identical": identical.Analyzer,
+		"opaque":    opaque.Analyzer,
+	}
+
+	if settings == nil || len(settings.Enable) == 0 {
+		analyzers := maps.Values(allAnalyzers)
+
+		// To have a deterministic order.
+		slices.SortFunc(analyzers, func(a *analysis.Analyzer, b *analysis.Analyzer) int {
+			return strings.Compare(a.Name, b.Name)
+		})
+
+		return analyzers
+	}
+
+	var analyzers []*analysis.Analyzer
+	for _, name := range uniqueNames(settings.Enable) {
+		analyzers = append(analyzers, allAnalyzers[name])
+	}
+
+	return analyzers
+}
+
+func uniqueNames(names []string) []string {
+	slices.Sort(names)
+	return slices.Compact(names)
+}

pkg/golinters/iface/iface_integration_test.go

@@ -0,0 +1,11 @@
+package iface
+
+import (
+	"testing"
+
+	"github.com/golangci/golangci-lint/test/testshared/integration"
+)
+
+func TestFromTestdata(t *testing.T) {
+	integration.RunTestdata(t)
+}

pkg/golinters/iface/testdata/iface.go

@@ -0,0 +1,64 @@
+//golangcitest:args -Eiface
+package testdata
+
+// identical
+
+type Pinger interface { // want "identical: interface Pinger contains identical methods or type constraints from another interface, causing redundancy"
+	Ping() error
+}
+
+type Healthcheck interface { // want "identical: interface Healthcheck contains identical methods or type constraints from another interface, causing redundancy"
+	Ping() error
+}
+
+// opaque
+
+type Server interface {
+	Serve() error
+}
+
+type server struct {
+	addr string
+}
+
+func (s server) Serve() error {
+	return nil
+}
+
+func NewServer(addr string) Server { // want "opaque: NewServer function return Server interface at the 1st result, abstract a single concrete implementation of \\*server"
+	return &server{addr: addr}
+}
+
+// unused
+
+type User struct {
+	ID   string
+	Name string
+}
+
+type UserRepository interface { // want "unused: interface UserRepository is declared but not used within the package"
+	UserOf(id string) (*User, error)
+}
+
+type UserRepositorySQL struct {
+}
+
+func (r *UserRepositorySQL) UserOf(id string) (*User, error) {
+	return nil, nil
+}
+
+type Granter interface {
+	Grant(permission string) error
+}
+
+func AllowAll(g Granter) error {
+	return g.Grant("all")
+}
+
+type Allower interface {
+	Allow(permission string) error
+}
+
+func Allow(x interface{}) {
+	_ = x.(Allower)
+}

pkg/golinters/iface/testdata/iface_identical.go

@@ -0,0 +1,65 @@
+//golangcitest:args -Eiface
+//golangcitest:config_path testdata/iface_identical.yml
+package testdata
+
+// identical
+
+type Pinger interface { // want "identical: interface Pinger contains identical methods or type constraints from another interface, causing redundancy"
+	Ping() error
+}
+
+type Healthcheck interface { // want "identical: interface Healthcheck contains identical methods or type constraints from another interface, causing redundancy"
+	Ping() error
+}
+
+// opaque
+
+type Server interface {
+	Serve() error
+}
+
+type server struct {
+	addr string
+}
+
+func (s server) Serve() error {
+	return nil
+}
+
+func NewServer(addr string) Server {
+	return &server{addr: addr}
+}
+
+// unused
+
+type User struct {
+	ID   string
+	Name string
+}
+
+type UserRepository interface {
+	UserOf(id string) (*User, error)
+}
+
+type UserRepositorySQL struct {
+}
+
+func (r *UserRepositorySQL) UserOf(id string) (*User, error) {
+	return nil, nil
+}
+
+type Granter interface {
+	Grant(permission string) error
+}
+
+func AllowAll(g Granter) error {
+	return g.Grant("all")
+}
+
+type Allower interface {
+	Allow(permission string) error
+}
+
+func Allow(x interface{}) {
+	_ = x.(Allower)
+}

pkg/golinters/iface/testdata/iface_identical.yml

@@ -0,0 +1,4 @@
+linters-settings:
+  iface:
+    enable:
+      - identical