fix: deduplicate typecheck errors (#5864)

Author: ldez

pkg/goanalysis/pkgerrors/errors.go

@@ -20,6 +20,7 @@ func (e *IllTypedError) Error() string {
 
 func BuildIssuesFromIllTypedError(errs []error, lintCtx *linter.Context) ([]*result.Issue, error) {
 	var issues []*result.Issue
+
 	uniqReportedIssues := map[string]bool{}
 
 	var other error
@@ -39,9 +40,17 @@ func BuildIssuesFromIllTypedError(errs []error, lintCtx *linter.Context) ([]*res
 				if uniqReportedIssues[err.Msg] {
 					continue
 				}
+
 				uniqReportedIssues[err.Msg] = true
 				lintCtx.Log.Errorf("typechecking error: %s", err.Msg)
 			} else {
+				key := fmt.Sprintf("%s.%d.%d.%s", issue.FilePath(), issue.Line(), issue.Column(), issue.Text)
+				if uniqReportedIssues[key] {
+					continue
+				}
+
+				uniqReportedIssues[key] = true
+
 				issue.Pkg = ill.Pkg // to save to cache later
 				issues = append(issues, issue)
 			}

pkg/lint/runner.go

@@ -232,7 +232,7 @@ func (r *Runner) processIssues(issues []*result.Issue, sw *timeutils.Stopwatch,
 		})
 
 		if err != nil {
-			r.Log.Warnf("Can't process result by %s processor: %s", p.Name(), err)
+			r.Log.Warnf("Can't process results by %s processor: %s", p.Name(), err)
 		} else {
 			stat := statPerProcessor[p.Name()]
 			stat.inCount += len(issues)