Add funcorder linter

.golangci.next.reference.yml

@@ -43,6 +43,7 @@ linters:
     - fatcontext
     - forbidigo
     - forcetypeassert
+    - funcorder
     - funlen
     - ginkgolinter
     - gocheckcompilerdirectives
@@ -150,6 +151,7 @@ linters:
     - fatcontext
     - forbidigo
     - forcetypeassert
+    - funcorder
     - funlen
     - ginkgolinter
     - gocheckcompilerdirectives
@@ -529,6 +531,12 @@ linters:
       # Default: false
       analyze-types: true
 
+    funcorder:
+      # Enable/disable feature to check constructors are placed after struct declaration
+      constructor: true
+      # Enable/disable feature to check whether the exported struct's methods are placed before the non-exported
+      struct-method: true
+
     funlen:
       # Checks the number of lines in a function.
       # If lower than 0, disable the check.

go.mod

@@ -73,6 +73,7 @@ require (
 	github.com/ldez/usetesting v0.4.2
 	github.com/leonklingele/grouper v1.1.2
 	github.com/macabu/inamedparam v0.2.0
+	github.com/manuelarte/funcorder v0.2.0
 	github.com/maratori/testableexamples v1.0.0
 	github.com/maratori/testpackage v1.1.1
 	github.com/matoous/godox v1.1.0

pkg/config/linters_settings.go

@@ -217,6 +217,7 @@ type LintersSettings struct {
 	Exhaustruct     ExhaustructSettings     `mapstructure:"exhaustruct"`
 	Fatcontext      FatcontextSettings      `mapstructure:"fatcontext"`
 	Forbidigo       ForbidigoSettings       `mapstructure:"forbidigo"`
+	FuncOrder       FuncOrderSettings       `mapstructure:"funcorder"`
 	Funlen          FunlenSettings          `mapstructure:"funlen"`
 	GinkgoLinter    GinkgoLinterSettings    `mapstructure:"ginkgolinter"`
 	Gocognit        GocognitSettings        `mapstructure:"gocognit"`
@@ -420,6 +421,11 @@ type ForbidigoPattern struct {
 	Msg     string `yaml:"msg,omitempty" mapstructure:"msg,omitempty"`
 }
 
+type FuncOrderSettings struct {
+	Constructor  *bool `mapstructure:"constructor,omitempty"`
+	StructMethod *bool `mapstructure:"struct-method,omitempty"`
+}
+
 type FunlenSettings struct {
 	Lines          int  `mapstructure:"lines"`
 	Statements     int  `mapstructure:"statements"`

pkg/golinters/funcorder/funcorder.go

@@ -0,0 +1,39 @@
+package funcorder
+
+import (
+	"github.com/manuelarte/funcorder/analyzer"
+	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/v2/pkg/config"
+
+	"github.com/golangci/golangci-lint/v2/pkg/goanalysis"
+)
+
+func New(settings *config.FuncOrderSettings) *goanalysis.Linter {
+	a := analyzer.NewAnalyzer()
+
+	cfg := map[string]map[string]any{}
+
+	if settings != nil {
+		constructor := true
+		if settings.Constructor != nil {
+			constructor = *settings.Constructor
+		}
+		structMethod := true
+		if settings.StructMethod != nil {
+			structMethod = *settings.StructMethod
+		}
+
+		cfg[a.Name] = map[string]any{
+			"constructor":   constructor,
+			"struct-method": structMethod,
+		}
+	}
+
+	return goanalysis.NewLinter(
+		a.Name,
+		a.Doc,
+		[]*analysis.Analyzer{a},
+		cfg,
+	).WithLoadMode(goanalysis.LoadModeSyntax)
+}

pkg/golinters/funcorder/funcorder_integration_test.go

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

pkg/golinters/funcorder/testdata/funcorder.go

@@ -0,0 +1,40 @@
+//golangcitest:args -Efuncorder
+package testdata
+
+import "time"
+
+type MyStruct struct {
+	Name string
+}
+
+func (m MyStruct) lenName() int { // want `unexported method "lenName" for struct "MyStruct" should be placed after the exported method "SetName"`
+	return len(m.Name)
+}
+
+func (m MyStruct) GetName() string {
+	return m.Name
+}
+
+func (m *MyStruct) SetName(name string) {
+	m.Name = name
+}
+
+type MyStruct2 struct {
+	Name string
+}
+
+func (m MyStruct2) GetName() string {
+	return m.Name
+}
+
+func (m *MyStruct2) SetName(name string) {
+	m.Name = name
+}
+
+func NewMyStruct2() *MyStruct2 { // want `constructor "NewMyStruct2" for struct "MyStruct2" should be placed before struct method "GetName"`
+	return &MyStruct2{Name: "John"}
+}
+
+func NewTime() time.Time {
+	return time.Now()
+}

pkg/lint/lintersdb/builder_linter.go

@@ -29,6 +29,7 @@ import (
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/fatcontext"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/forbidigo"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/forcetypeassert"
+	"github.com/golangci/golangci-lint/v2/pkg/golinters/funcorder"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/funlen"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/gci"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/ginkgolinter"
@@ -254,6 +255,10 @@ func (LinterBuilder) Build(cfg *config.Config) ([]*linter.Config, error) {
 			WithLoadForGoAnalysis().
 			WithURL("https://github.com/gostaticanalysis/forcetypeassert"),
 
+		linter.NewConfig(funcorder.New(&cfg.Linters.Settings.FuncOrder)).
+			WithSince("v2.1.0").
+			WithURL("https://github.com/manuelarte/funcorder"),
+
 		linter.NewConfig(fatcontext.New(&cfg.Linters.Settings.Fatcontext)).
 			WithSince("v1.58.0").
 			WithLoadForGoAnalysis().