gci: fix issues and re-enable autofix

go.mod

@@ -20,7 +20,7 @@ require (
 	github.com/breml/errchkjson v0.3.0
 	github.com/butuzov/ireturn v0.1.1
 	github.com/charithe/durationcheck v0.0.9
-	github.com/daixiang0/gci v0.3.3
+	github.com/daixiang0/gci v0.3.4-0.20220530072138-bf921fe4731b
 	github.com/denis-tingaikin/go-header v0.4.3
 	github.com/esimonov/ifshort v1.0.4
 	github.com/fatih/color v1.13.0
@@ -164,6 +164,9 @@ require (
 	github.com/tklauser/numcpus v0.4.0 // indirect
 	github.com/valyala/bytebufferpool v1.0.0 // indirect
 	github.com/yusufpapurcu/wmi v1.2.2 // indirect
+	go.uber.org/atomic v1.7.0 // indirect
+	go.uber.org/multierr v1.6.0 // indirect
+	go.uber.org/zap v1.17.0 // indirect
 	golang.org/x/exp/typeparams v0.0.0-20220218215828-6cf2b201936e // indirect
 	golang.org/x/mod v0.6.0-dev.0.20220419223038-86c51ed26bb4 // indirect
 	golang.org/x/sync v0.0.0-20210220032951-036812b2e83c // indirect

pkg/golinters/gci.go

@@ -1,10 +1,11 @@
 package golinters
 
 import (
-	"fmt"
-	"strings"
+	"sync"
 
-	gci "github.com/daixiang0/gci/pkg/analyzer"
+	gcicfg "github.com/daixiang0/gci/pkg/configuration"
+	"github.com/daixiang0/gci/pkg/gci"
+	"github.com/pkg/errors"
 	"golang.org/x/tools/go/analysis"
 
 	"github.com/golangci/golangci-lint/pkg/config"
@@ -15,33 +16,64 @@ import (
 const gciName = "gci"
 
 func NewGci(settings *config.GciSettings) *goanalysis.Linter {
-	var linterCfg map[string]map[string]interface{}
-	if settings != nil {
-		cfg := map[string]interface{}{
-			gci.NoInlineCommentsFlag:  settings.NoInlineComments,
-			gci.NoPrefixCommentsFlag:  settings.NoPrefixComments,
-			gci.SectionsFlag:          strings.Join(settings.Sections, gci.SectionDelimiter),
-			gci.SectionSeparatorsFlag: strings.Join(settings.SectionSeparator, gci.SectionDelimiter),
-		}
+	var cfg *gci.GciConfiguration
+	var resIssues []goanalysis.Issue
+	var err error
 
-		if settings.LocalPrefixes != "" {
-			prefix := []string{"standard", "default", fmt.Sprintf("prefix(%s)", settings.LocalPrefixes)}
-			cfg[gci.SectionsFlag] = strings.Join(prefix, gci.SectionDelimiter)
-		}
+	analyzer := &analysis.Analyzer{
+		Name: gciName,
+		Doc:  goanalysis.TheOnlyanalyzerDoc,
+		Run:  goanalysis.DummyRun,
+	}
 
-		linterCfg = map[string]map[string]interface{}{
-			gci.Analyzer.Name: cfg,
+	if settings != nil {
+		strcfg := gci.GciStringConfiguration{
+			Cfg: gcicfg.FormatterConfiguration{
+				NoInlineComments: settings.NoInlineComments,
+				NoPrefixComments: settings.NoPrefixComments,
+			},
+			SectionStrings:          settings.Sections,
+			SectionSeparatorStrings: settings.SectionSeparator,
 		}
+		cfg, _ = strcfg.Parse()
 	}
 
 	return goanalysis.NewLinter(
 		gciName,
 		"Gci controls golang package import order and makes it always deterministic.",
-		[]*analysis.Analyzer{gci.Analyzer},
-		linterCfg,
+		[]*analysis.Analyzer{analyzer},
+		nil,
 	).WithContextSetter(func(lintCtx *linter.Context) {
-		if settings.LocalPrefixes != "" {
-			lintCtx.Log.Warnf("gci: `local-prefixes` is deprecated, use `sections` and `prefix(%s)` instead.", settings.LocalPrefixes)
+		analyzer.Run = func(pass *analysis.Pass) (interface{}, error) {
+			var fileNames []string
+			for _, f := range pass.Files {
+				pos := pass.Fset.PositionFor(f.Pos(), false)
+				fileNames = append(fileNames, pos.Filename)
+			}
+			var lock sync.Mutex
+			var diffs []string
+			err = gci.DiffFormattedFilesToArray(fileNames, *cfg, &diffs, &lock)
+			if err != nil {
+				return nil, err
+			}
+			for _, diff := range diffs {
+				if diff == "" {
+					continue
+				}
+
+				is, err := extractIssuesFromPatch(diff, lintCtx, gciName)
+				if err != nil {
+					return nil, errors.Wrapf(err, "can't extract issues from gci diff output %s", diff)
+				}
+
+				for i := range is {
+					resIssues = append(resIssues, goanalysis.NewIssue(&is[i], pass))
+				}
+			}
+
+			return nil, nil
 		}
+	}).WithIssuesReporter(func(*linter.Context) []goanalysis.Issue {
+		return resIssues
 	}).WithLoadMode(goanalysis.LoadModeSyntax)
 }

pkg/golinters/gofmt_common.go

@@ -207,9 +207,36 @@ func (p *hunkChangesParser) parse(h *diffpkg.Hunk) []Change {
 	return p.ret
 }
 
+func getErrorTextForGci(lintCtx *linter.Context) string {
+	text := "File is not `gci`-ed"
+	noInLineComments := lintCtx.Settings().Gci.NoInlineComments
+	noPrefixComments := lintCtx.Settings().Gci.NoPrefixComments
+	sections := lintCtx.Settings().Gci.Sections
+	sectionSeparator := lintCtx.Settings().Gci.SectionSeparator
+	hasOptions := noInLineComments || noPrefixComments || len(sections) > 0 || len(sectionSeparator) > 0
+	if hasOptions {
+		text += " with"
+	}
+	if noInLineComments {
+		text += " -NoInlineComments"
+	}
+	if noPrefixComments {
+		text += " -NoPrefixComments"
+	}
+	if len(sections) > 0 {
+		text += " -s " + strings.Join(sections, ",")
+	}
+	if len(sectionSeparator) > 0 {
+		text += " -x " + strings.Join(sectionSeparator, ",")
+	}
+	return text
+}
+
 func getErrorTextForLinter(lintCtx *linter.Context, linterName string) string {
 	text := "File is not formatted"
 	switch linterName {
+	case gciName:
+		text = getErrorTextForGci(lintCtx)
 	case gofumptName:
 		text = "File is not `gofumpt`-ed"
 		if lintCtx.Settings().Gofumpt.ExtraRules {

test/linters_test.go

@@ -105,7 +105,7 @@ func TestGciLocal(t *testing.T) {
 	require.NoError(t, err)
 
 	testshared.NewLintRunner(t).RunWithYamlConfig(string(cfg), args...).
-		ExpectHasIssue("testdata/gci/gci.go:9:1: Expected '\\n', Found '\\t'")
+		ExpectHasIssue("testdata/gci/gci.go:8: File is not `gci`-ed")
 }
 
 func TestMultipleOutputs(t *testing.T) {
@@ -124,7 +124,7 @@ func TestMultipleOutputs(t *testing.T) {
 	require.NoError(t, err)
 
 	testshared.NewLintRunner(t).RunWithYamlConfig(string(cfg), args...).
-		ExpectHasIssue("testdata/gci/gci.go:9:1: Expected '\\n', Found '\\t'").
+		ExpectHasIssue("testdata/gci/gci.go:8: File is not `gci`-ed").
 		ExpectOutputContains(`"Issues":[`)
 }
 
@@ -144,7 +144,7 @@ func TestStderrOutput(t *testing.T) {
 	require.NoError(t, err)
 
 	testshared.NewLintRunner(t).RunWithYamlConfig(string(cfg), args...).
-		ExpectHasIssue("testdata/gci/gci.go:9:1: Expected '\\n', Found '\\t'").
+		ExpectHasIssue("testdata/gci/gci.go:8: File is not `gci`-ed").
 		ExpectOutputContains(`"Issues":[`)
 }
 
@@ -167,7 +167,7 @@ func TestFileOutput(t *testing.T) {
 	require.NoError(t, err)
 
 	testshared.NewLintRunner(t).RunWithYamlConfig(string(cfg), args...).
-		ExpectHasIssue("testdata/gci/gci.go:9:1: Expected '\\n', Found '\\t'").
+		ExpectHasIssue("testdata/gci/gci.go:8: File is not `gci`-ed").
 		ExpectOutputNotContains(`"Issues":[`)
 
 	b, err := os.ReadFile(resultPath)

test/testdata/fix/in/gci.go

@@ -0,0 +1,15 @@
+//args: -Egci
+//config_path: testdata/configs/gci.yml
+package gci
+
+import (
+	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/pkg/errors"
+	"fmt"
+)
+
+func GoimportsLocalTest() {
+	fmt.Print("x")
+	_ = config.Config{}
+	_ = errors.New("")
+}

test/testdata/fix/out/gci.go

@@ -0,0 +1,17 @@
+//args: -Egci
+//config_path: testdata/configs/gci.yml
+package gci
+
+import (
+	"fmt"
+
+	"github.com/golangci/golangci-lint/pkg/config"
+
+	"github.com/pkg/errors"
+)
+
+func GoimportsLocalTest() {
+	fmt.Print("x")
+	_ = config.Config{}
+	_ = errors.New("")
+}