feat: fatcontext settings

Author: Crocmagnon

.golangci.next.reference.yml

@@ -548,6 +548,12 @@ linters-settings:
     exclude:
       - '.+/cobra\.Command$'
 
+  fatcontext:
+    # Check for potential fat contexts in struct pointers.
+    # May generate false positives.
+    # Default: false
+    check-struct-pointers: true
+
   forbidigo:
     # Forbid the following identifiers (list of regexp).
     # Default: ["^(fmt\\.Print(|f|ln)|print|println)$"]

go.mod

@@ -11,7 +11,7 @@ require (
 	github.com/Antonboom/nilnil v1.0.1
 	github.com/Antonboom/testifylint v1.5.2
 	github.com/BurntSushi/toml v1.4.1-0.20240526193622-a339e1f7089c
-	github.com/Crocmagnon/fatcontext v0.6.0
+	github.com/Crocmagnon/fatcontext v0.7.0
 	github.com/Djarvur/go-err113 v0.0.0-20210108212216-aea10b59be24
 	github.com/GaijinEntertainment/go-exhaustruct/v3 v3.3.0
 	github.com/OpenPeeDeeP/depguard/v2 v2.2.0

go.sum

@@ -40,6 +40,9 @@ var defaultLintersSettings = LintersSettings{
 		ExplicitExhaustiveMap:      false,
 		ExplicitExhaustiveSwitch:   false,
 	},
+	Fatcontext: FatcontextSettings{
+		CheckStructPointers: false,
+	},
 	Forbidigo: ForbidigoSettings{
 		ExcludeGodocExamples: true,
 	},
@@ -223,6 +226,7 @@ type LintersSettings struct {
 	ErrorLint       ErrorLintSettings
 	Exhaustive      ExhaustiveSettings
 	Exhaustruct     ExhaustructSettings
+	Fatcontext      FatcontextSettings
 	Forbidigo       ForbidigoSettings
 	Funlen          FunlenSettings
 	Gci             GciSettings
@@ -430,6 +434,10 @@ type ExhaustructSettings struct {
 	Exclude []string `mapstructure:"exclude"`
 }
 
+type FatcontextSettings struct {
+	CheckStructPointers bool `mapstructure:"check-struct-pointers"`
+}
+
 type ForbidigoSettings struct {
 	Forbid               []ForbidigoPattern `mapstructure:"forbid"`
 	ExcludeGodocExamples bool               `mapstructure:"exclude-godoc-examples"`

pkg/config/linters_settings.go

@@ -4,16 +4,25 @@ import (
 	"github.com/Crocmagnon/fatcontext/pkg/analyzer"
 	"golang.org/x/tools/go/analysis"
 
+	"github.com/golangci/golangci-lint/pkg/config"
 	"github.com/golangci/golangci-lint/pkg/goanalysis"
 )
 
-func New() *goanalysis.Linter {
-	a := analyzer.Analyzer
+func New(settings *config.FatcontextSettings) *goanalysis.Linter {
+	a := analyzer.NewAnalyzer()
+
+	cfg := map[string]map[string]any{}
+
+	if settings != nil {
+		cfg[a.Name] = map[string]any{
+			analyzer.FlagCheckStructPointers: settings.CheckStructPointers,
+		}
+	}
 
 	return goanalysis.NewLinter(
 		a.Name,
 		a.Doc,
 		[]*analysis.Analyzer{a},
-		nil,
+		cfg,
 	).WithLoadMode(goanalysis.LoadModeTypesInfo)
 }

pkg/golinters/fatcontext/fatcontext.go

@@ -307,7 +307,7 @@ func (LinterBuilder) Build(cfg *config.Config) ([]*linter.Config, error) {
 			WithPresets(linter.PresetStyle).
 			WithURL("https://github.com/gostaticanalysis/forcetypeassert"),
 
-		linter.NewConfig(fatcontext.New()).
+		linter.NewConfig(fatcontext.New(&cfg.LintersSettings.Fatcontext)).
 			WithSince("v1.58.0").
 			WithPresets(linter.PresetPerformance).
 			WithLoadForGoAnalysis().