chore: remove golangci-lint mode

Author: ldez

protogetter.go

@@ -16,13 +16,6 @@ import (
 	"golang.org/x/tools/go/ast/inspector"
 )
 
-type Mode int
-
-const (
-	StandaloneMode Mode = iota
-	GolangciLintMode
-)
-
 const msgFormat = "avoid direct access to proto field %s, use %s instead"
 
 func NewAnalyzer(cfg *Config) *analysis.Analyzer {
@@ -35,7 +28,7 @@ func NewAnalyzer(cfg *Config) *analysis.Analyzer {
 		Doc:   "Reports direct reads from proto message fields when getters should be used",
 		Flags: flags(cfg),
 		Run: func(pass *analysis.Pass) (any, error) {
-			_, err := Run(pass, cfg)
+			err := Run(pass, cfg)
 			return nil, err
 		},
 	}
@@ -62,14 +55,13 @@ func flags(opts *Config) flag.FlagSet {
 }
 
 type Config struct {
-	Mode                    Mode // Zero value is StandaloneMode.
 	SkipGeneratedBy         []string
 	SkipFiles               []string
 	SkipAnyGenerated        bool
 	ReplaceFirstArgInAppend bool
 }
 
-func Run(pass *analysis.Pass, cfg *Config) ([]Issue, error) {
+func Run(pass *analysis.Pass, cfg *Config) error {
 	skipGeneratedBy := make([]string, 0, len(cfg.SkipGeneratedBy)+3)
 	// Always skip files generated by protoc-gen-go, protoc-gen-go-grpc and protoc-gen-grpc-gateway.
 	skipGeneratedBy = append(skipGeneratedBy, "protoc-gen-go", "protoc-gen-go-grpc", "protoc-gen-grpc-gateway")
@@ -90,7 +82,7 @@ func Run(pass *analysis.Pass, cfg *Config) ([]Issue, error) {
 
 		compile, err := glob.Compile(s)
 		if err != nil {
-			return nil, fmt.Errorf("invalid glob pattern: %w", err)
+			return fmt.Errorf("invalid glob pattern: %w", err)
 		}
 
 		skipFilesGlobPatterns = append(skipFilesGlobPatterns, compile)
@@ -124,24 +116,16 @@ func Run(pass *analysis.Pass, cfg *Config) ([]Issue, error) {
 
 	ins := inspector.New(files)
 
-	var issues []Issue
-
 	filter := NewPosFilter()
 	ins.Preorder(nodeTypes, func(node ast.Node) {
 		report := analyse(pass, filter, node, cfg)
 		if report == nil {
 			return
 		}
-
-		switch cfg.Mode {
-		case StandaloneMode:
-			pass.Report(report.ToDiagReport())
-		case GolangciLintMode:
-			issues = append(issues, report.ToIssue(pass.Fset))
-		}
+		pass.Report(report.ToDiagReport())
 	})
 
-	return issues, nil
+	return nil
 }
 
 func analyse(pass *analysis.Pass, filter *PosFilter, n ast.Node, cfg *Config) *Report {
@@ -185,19 +169,6 @@ func analyse(pass *analysis.Pass, filter *PosFilter, n ast.Node, cfg *Config) *R
 	}
 }
 
-// Issue is used to integrate with golangci-lint's inline auto fix.
-type Issue struct {
-	Pos       token.Position
-	Message   string
-	InlineFix InlineFix
-}
-
-type InlineFix struct {
-	StartCol  int // zero-based
-	Length    int
-	NewString string
-}
-
 type Report struct {
 	node   ast.Node
 	result *Result
@@ -225,27 +196,13 @@ func (r *Report) ToDiagReport() analysis.Diagnostic {
 	}
 }
 
-func (r *Report) ToIssue(fset *token.FileSet) Issue {
-	msg := fmt.Sprintf(msgFormat, r.result.From, r.result.To)
-	return Issue{
-		Pos:     fset.Position(r.node.Pos()),
-		Message: msg,
-		InlineFix: InlineFix{
-			StartCol:  fset.Position(r.node.Pos()).Column - 1,
-			Length:    len(r.result.From),
-			NewString: r.result.To,
-		},
-	}
-}
-
 func skipGeneratedFile(f *ast.File, prefixes []string, skipAny bool) bool {
 	if len(f.Comments) == 0 {
 		return false
 	}
 	firstComment := f.Comments[0].Text()
 
-	// https://golang.org/s/generatedcode
-	if skipAny && strings.HasPrefix(firstComment, "Code generated") {
+	if skipAny && ast.IsGenerated(f) {
 		return true
 	}