feat: deprecate usage of linter alternative names

Author: ldez

pkg/lint/lintersdb/validator.go

@@ -3,10 +3,12 @@ package lintersdb
 import (
 	"errors"
 	"fmt"
+	"os"
 	"slices"
 	"strings"
 
 	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/logutils"
 )
 
 type Validator struct {
@@ -28,6 +30,7 @@ func (v Validator) Validate(cfg *config.Config) error {
 	validators := []func(cfg *config.Linters) error{
 		v.validateLintersNames,
 		v.validatePresets,
+		v.alternativeNamesDeprecation,
 	}
 
 	for _, v := range validators {
@@ -75,3 +78,35 @@ func (v Validator) validatePresets(cfg *config.Linters) error {
 
 	return nil
 }
+
+func (v Validator) alternativeNamesDeprecation(cfg *config.Linters) error {
+	if v.m.cfg.InternalTest || v.m.cfg.InternalCmdTest || os.Getenv(logutils.EnvTestRun) == "1" {
+		return nil
+	}
+
+	altNames := map[string][]string{}
+	for _, lc := range v.m.GetAllSupportedLinterConfigs() {
+		for _, alt := range lc.AlternativeNames {
+			altNames[alt] = append(altNames[alt], lc.Name())
+		}
+	}
+
+	var names []string
+	names = append(names, cfg.Enable...)
+	names = append(names, cfg.Disable...)
+
+	for _, name := range names {
+		lc, ok := altNames[name]
+		if !ok {
+			continue
+		}
+
+		if len(lc) > 1 {
+			v.m.log.Warnf("The linter name %q is deprecated. It has been splited into: %s.", name, strings.Join(lc, ", "))
+		} else {
+			v.m.log.Warnf("The linter name %q is deprecated. It has been renamed to: %s.", name, lc[0])
+		}
+	}
+
+	return nil
+}

pkg/lint/lintersdb/validator_test.go

@@ -6,6 +6,7 @@ import (
 	"github.com/stretchr/testify/require"
 
 	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/logutils"
 )
 
 type validateErrorTestCase struct {
@@ -215,3 +216,25 @@ func TestValidator_validatePresets_error(t *testing.T) {
 		})
 	}
 }
+
+func TestValidator_alternativeNamesDeprecation(t *testing.T) {
+	log := logutils.NewMockLog()
+	log.On("Warnf", "The linter name %q is deprecated. It has been renamed to: %s.", "vet", "govet")
+	log.On("Warnf", "The linter name %q is deprecated. It has been renamed to: %s.", "vetshadow", "govet")
+	log.On("Warnf", "The linter name %q is deprecated. It has been renamed to: %s.", "logrlint", "loggercheck")
+	log.On("Warnf", "The linter name %q is deprecated. It has been splited into: %s.", "megacheck", "gosimple, staticcheck, unused")
+	log.On("Warnf", "The linter name %q is deprecated. It has been renamed to: %s.", "gas", "gosec")
+
+	m, err := NewManager(log, nil, NewLinterBuilder())
+	require.NoError(t, err)
+
+	v := NewValidator(m)
+
+	cfg := &config.Linters{
+		Enable:  []string{"vet", "vetshadow", "logrlint"},
+		Disable: []string{"megacheck", "gas"},
+	}
+
+	err = v.alternativeNamesDeprecation(cfg)
+	require.NoError(t, err)
+}