feat: migration command

.golangci.yml

@@ -171,6 +171,17 @@ linters:
         linters: [gosec]
         text: "G306: Expect WriteFile permissions to be 0600 or less"
 
+      # Related to migration command.
+      - path: pkg/commands/internal/migrate/two/
+        linters:
+          - lll
+
+      # Related to migration command.
+      - path: pkg/commands/internal/migrate/
+        linters:
+          - gocritic
+        text: "hugeParam:"
+
 formatters:
   enable:
     - gofmt

Makefile

@@ -60,6 +60,11 @@ fast_check_generated:
 	git checkout -- go.mod go.sum # can differ between go1.16 and go1.17
 	git diff --exit-code # check no changes
 
+# Migration
+
+clone_config:
+	go run ./pkg/commands/internal/migrate/cloner/
+
 # Benchmark
 
 # Benchmark with a local version

pkg/commands/internal/migrate/cloner/cloner.go

@@ -0,0 +1,195 @@
+package main
+
+import (
+	"bytes"
+	"fmt"
+	"go/ast"
+	"go/parser"
+	"go/printer"
+	"go/token"
+	"log"
+	"os"
+	"path/filepath"
+	"reflect"
+	"strings"
+
+	"golang.org/x/tools/imports"
+)
+
+const newPkgName = "versiontwo"
+
+const (
+	srcDir = "./pkg/config"
+	dstDir = "./pkg/commands/internal/migrate/versiontwo"
+)
+
+func main() {
+	stat, err := os.Stat(srcDir)
+	if err != nil {
+		log.Fatal(err)
+	}
+
+	if !stat.IsDir() {
+		log.Fatalf("%s is not a directory", srcDir)
+	}
+
+	_ = os.RemoveAll(dstDir)
+
+	err = processPackage(srcDir, dstDir)
+	if err != nil {
+		log.Fatalf("Processing package error: %v", err)
+	}
+}
+
+func processPackage(srcDir, dstDir string) error {
+	return filepath.Walk(srcDir, func(srcPath string, _ os.FileInfo, err error) error {
+		if err != nil {
+			return err
+		}
+
+		if skipFile(srcPath) {
+			return nil
+		}
+
+		fset := token.NewFileSet()
+
+		file, err := parser.ParseFile(fset, srcPath, nil, parser.AllErrors)
+		if err != nil {
+			return fmt.Errorf("parsing %s: %w", srcPath, err)
+		}
+
+		processFile(file)
+
+		return writeNewFile(fset, file, srcPath, dstDir)
+	})
+}
+
+func skipFile(path string) bool {
+	if !strings.HasSuffix(path, ".go") || strings.HasSuffix(path, "_test.go") {
+		return true
+	}
+
+	switch filepath.Base(path) {
+	case "base_loader.go", "loader.go":
+		return true
+	default:
+		return false
+	}
+}
+
+func processFile(file *ast.File) {
+	file.Name.Name = newPkgName
+
+	var newDecls []ast.Decl
+	for _, decl := range file.Decls {
+		d, ok := decl.(*ast.GenDecl)
+		if !ok {
+			continue
+		}
+
+		switch d.Tok {
+		case token.CONST, token.VAR:
+			continue
+		case token.TYPE:
+			for _, spec := range d.Specs {
+				typeSpec, ok := spec.(*ast.TypeSpec)
+				if !ok {
+					continue
+				}
+
+				structType, ok := typeSpec.Type.(*ast.StructType)
+				if !ok {
+					continue
+				}
+
+				processStructFields(structType)
+			}
+		default:
+			// noop
+		}
+
+		newDecls = append(newDecls, decl)
+	}
+
+	file.Decls = newDecls
+}
+
+func processStructFields(structType *ast.StructType) {
+	var newFields []*ast.Field
+
+	for _, field := range structType.Fields.List {
+		if len(field.Names) > 0 && !field.Names[0].IsExported() {
+			continue
+		}
+
+		if field.Tag == nil {
+			continue
+		}
+
+		field.Type = convertType(field.Type)
+		field.Tag.Value = convertStructTag(field.Tag.Value)
+
+		newFields = append(newFields, field)
+	}
+
+	structType.Fields.List = newFields
+}
+
+func convertType(expr ast.Expr) ast.Expr {
+	ident, ok := expr.(*ast.Ident)
+	if !ok {
+		return expr
+	}
+
+	switch ident.Name {
+	case "bool", "string", "int", "int8", "int16", "int32", "int64", "float32", "float64":
+		return &ast.StarExpr{X: ident}
+
+	default:
+		return expr
+	}
+}
+
+func convertStructTag(value string) string {
+	structTag := reflect.StructTag(strings.Trim(value, "`"))
+
+	key := structTag.Get("mapstructure")
+
+	if key == ",squash" {
+		return wrapStructTag(`yaml:",inline"`)
+	}
+
+	return wrapStructTag(fmt.Sprintf(`yaml:"%[1]s,omitempty" toml:"%[1]s,omitempty"`, key))
+}
+
+func wrapStructTag(s string) string {
+	return "`" + s + "`"
+}
+
+func writeNewFile(fset *token.FileSet, file *ast.File, srcPath, dstDir string) error {
+	var buf bytes.Buffer
+
+	buf.WriteString("// Code generated by pkg/commands/internal/migrate/cloner/cloner.go. DO NOT EDIT.\n\n")
+
+	err := printer.Fprint(&buf, fset, file)
+	if err != nil {
+		return fmt.Errorf("printing %s: %w", srcPath, err)
+	}
+
+	dstPath := filepath.Join(dstDir, filepath.Base(srcPath))
+
+	_ = os.MkdirAll(filepath.Dir(dstPath), os.ModePerm)
+
+	formatted, err := imports.Process(dstPath, buf.Bytes(), nil)
+	if err != nil {
+		return fmt.Errorf("formatting %s: %w", dstPath, err)
+	}
+
+	//nolint:gosec,mnd // The permission is right.
+	err = os.WriteFile(dstPath, formatted, 0o644)
+	if err != nil {
+		return fmt.Errorf("writing file %s: %w", dstPath, err)
+	}
+
+	return nil
+}

pkg/commands/internal/migrate/migrate.go

@@ -0,0 +1,19 @@
+package migrate
+
+import (
+	"github.com/golangci/golangci-lint/pkg/commands/internal/migrate/ptr"
+	"github.com/golangci/golangci-lint/pkg/commands/internal/migrate/versionone"
+	"github.com/golangci/golangci-lint/pkg/commands/internal/migrate/versiontwo"
+)
+
+func ToConfig(old *versionone.Config) *versiontwo.Config {
+	return &versiontwo.Config{
+		Version:    ptr.Pointer("2"),
+		Linters:    toLinters(old),
+		Formatters: toFormatters(old),
+		Issues:     toIssues(old),
+		Output:     toOutput(old),
+		Severity:   toSeverity(old),
+		Run:        toRun(old),
+	}
+}

pkg/commands/internal/migrate/migrate_formatters.go

@@ -0,0 +1,105 @@
+package migrate
+
+import (
+	"slices"
+	"strings"
+
+	"github.com/golangci/golangci-lint/pkg/commands/internal/migrate/ptr"
+	"github.com/golangci/golangci-lint/pkg/commands/internal/migrate/versionone"
+	"github.com/golangci/golangci-lint/pkg/commands/internal/migrate/versiontwo"
+)
+
+func toFormatters(old *versionone.Config) versiontwo.Formatters {
+	enable := ProcessEffectiveFormatters(old.Linters)
+
+	var paths []string
+	if len(enable) > 0 {
+		paths = slices.Concat(old.Issues.ExcludeFiles, old.Issues.ExcludeDirs)
+
+		if old.Issues.UseDefaultExcludeDirs == nil || ptr.Deref(old.Issues.UseDefaultExcludeDirs) {
+			paths = append(paths, "examples$")
+		}
+	}
+
+	paths = append(paths, toFormattersPathsFromRules(old.Issues)...)
+
+	return versiontwo.Formatters{
+		Enable: enable,
+		Settings: versiontwo.FormatterSettings{
+			Gci:       toGciSettings(old.LintersSettings.Gci),
+			GoFmt:     toGoFmtSettings(old.LintersSettings.GoFmt),
+			GoFumpt:   toGoFumptSettings(old.LintersSettings.GoFumpt),
+			GoImports: toGoImportsSettings(old.LintersSettings.GoImports),
+		},
+		Exclusions: versiontwo.FormatterExclusions{
+			Generated: toExclusionGenerated(old.Issues.ExcludeGenerated),
+			Paths:     paths,
+		},
+	}
+}
+
+func toFormattersPathsFromRules(old versionone.Issues) []string {
+	var results []string
+
+	for _, rule := range old.ExcludeRules {
+		allNames := convertStaticcheckLinterNames(convertAlternativeNames(rule.Linters))
+
+		names := onlyFormatterNames(allNames)
+		if len(names) == 0 {
+			continue
+		}
+
+		if ptr.Deref(rule.Path) == "" {
+			continue
+		}
+
+		results = append(results, ptr.Deref(rule.Path))
+	}
+
+	return results
+}
+
+func toGciSettings(old versionone.GciSettings) versiontwo.GciSettings {
+	return versiontwo.GciSettings{
+		Sections:         old.Sections,
+		NoInlineComments: old.NoInlineComments,
+		NoPrefixComments: old.NoPrefixComments,
+		CustomOrder:      old.CustomOrder,
+		NoLexOrder:       old.NoLexOrder,
+	}
+}
+
+func toGoFmtSettings(old versionone.GoFmtSettings) versiontwo.GoFmtSettings {
+	settings := versiontwo.GoFmtSettings{
+		Simplify: old.Simplify,
+	}
+
+	for _, rule := range old.RewriteRules {
+		settings.RewriteRules = append(settings.RewriteRules, versiontwo.GoFmtRewriteRule{
+			Pattern:     rule.Pattern,
+			Replacement: rule.Replacement,
+		})
+	}
+
+	return settings
+}
+
+func toGoFumptSettings(old versionone.GoFumptSettings) versiontwo.GoFumptSettings {
+	return versiontwo.GoFumptSettings{
+		ModulePath: old.ModulePath,
+		ExtraRules: old.ExtraRules,
+	}
+}
+
+func toGoImportsSettings(old versionone.GoImportsSettings) versiontwo.GoImportsSettings {
+	var localPrefixes []string
+
+	prefixes := ptr.Deref(old.LocalPrefixes)
+	if prefixes != "" {
+		localPrefixes = strings.Split(prefixes, ",")
+	}
+
+	return versiontwo.GoImportsSettings{
+		LocalPrefixes: localPrefixes,
+	}
+}

pkg/commands/internal/migrate/migrate_issues.go

@@ -0,0 +1,20 @@
+package migrate
+
+import (
+	"github.com/golangci/golangci-lint/pkg/commands/internal/migrate/versionone"
+	"github.com/golangci/golangci-lint/pkg/commands/internal/migrate/versiontwo"
+)
+
+func toIssues(old *versionone.Config) versiontwo.Issues {
+	return versiontwo.Issues{
+		MaxIssuesPerLinter: old.Issues.MaxIssuesPerLinter,
+		MaxSameIssues:      old.Issues.MaxSameIssues,
+		UniqByLine:         old.Issues.UniqByLine,
+		DiffFromRevision:   old.Issues.DiffFromRevision,
+		DiffFromMergeBase:  old.Issues.DiffFromMergeBase,
+		DiffPatchFilePath:  old.Issues.DiffPatchFilePath,
+		WholeFiles:         old.Issues.WholeFiles,
+		Diff:               old.Issues.Diff,
+		NeedFix:            old.Issues.NeedFix,
+	}
+}