feature: custom issue reporter

Author: butuzov

pkg/golinters/mirror.go

@@ -1,31 +1,70 @@
 package golinters
 
 import (
+	"sync"
+
 	"github.com/butuzov/mirror"
 	"golang.org/x/tools/go/analysis"
 
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
+	"github.com/golangci/golangci-lint/pkg/lint/linter"
+	"github.com/golangci/golangci-lint/pkg/result"
 )
 
 func NewMirror() *goanalysis.Linter {
+	var ( // Issue reporter related.
+		mu     sync.Mutex
+		issues []goanalysis.Issue
+	)
+
 	a := mirror.NewAnalyzer()
+	a.Run = func(pass *analysis.Pass) (any, error) {
+		// mirror only lints test files if the `--with-tests` flag is passed, so we
+		// pass the `with-tests` flag as true to the analyzer before running it. This
+		// can be turned off by using the regular golangci-lint flags such as `--tests`
+		// or `--skip-files` or can be disabled per linter via exclude rules (see
+		// https://github.com/golangci/golangci-lint/issues/2527#issuecomment-1023707262)
+		withTests := true
+		violations := mirror.Run(pass, withTests)
+
+		if len(violations) == 0 {
+			return nil, nil
+		}
 
-	// mirror only lints test files if the `--with-tests` flag is passed, so we
-	// pass the `with-tests` flag as true to the analyzer before running it. This
-	// can be turned off by using the regular golangci-lint flags such as `--tests`
-	// or `--skip-files` or can be disabled per linter via exclude rules (see
-	// https://github.com/golangci/golangci-lint/issues/2527#issuecomment-1023707262)
+		for i := range violations {
+			tmp := violations[i].Issue(pass)
 
-	cfg := map[string]map[string]any{
-		a.Name: {
-			"with-tests": true,
-		},
+			issue := result.Issue{
+				FromLinter: a.Name,
+				Text:       tmp.Message,
+				Pos:        tmp.Start,
+			}
+
+			if len(tmp.InlineFix) > 0 {
+				issue.Replacement = &result.Replacement{
+					Inline: &result.InlineFix{
+						StartCol:  tmp.Start.Column - 1,
+						Length:    len(tmp.Original),
+						NewString: tmp.InlineFix,
+					},
+				}
+			}
+			mu.Lock()
+			issues = append(issues, goanalysis.NewIssue(&issue, pass))
+			mu.Unlock()
+		}
+
+		return nil, nil
 	}
 
-	return goanalysis.NewLinter(
+	analyzer := goanalysis.NewLinter(
 		a.Name,
 		a.Doc,
 		[]*analysis.Analyzer{a},
-		cfg,
-	).WithLoadMode(goanalysis.LoadModeWholeProgram)
+		nil,
+	).WithIssuesReporter(func(*linter.Context) []goanalysis.Issue {
+		return issues
+	}).WithLoadMode(goanalysis.LoadModeTypesInfo)
+
+	return analyzer
 }