cmd/fix: Filter files loaded for fixing

cmd/fix.go

@@ -237,7 +237,13 @@ func fix(args []string, params *fixCommandParams) error {
 	f := fixer.NewFixer()
 	f.RegisterFixes(fixes.NewDefaultFixes()...)
 
-	fileProvider := fileprovider.NewFSFileProvider(args...)
+	ignore := userConfig.Ignore.Files
+
+	if len(params.ignoreFiles.v) > 0 {
+		ignore = params.ignoreFiles.v
+	}
+
+	fileProvider := fileprovider.NewFSFileProvider(ignore, args...)
 
 	fixReport, err := f.Fix(ctx, &l, fileProvider)
 	if err != nil {

e2e/cli_test.go

@@ -751,6 +751,12 @@ allow if {
 		t.Fatalf("failed to write main.rego: %v", err)
 	}
 
+	unrelatedFileContents := []byte(`foobar`)
+	err = os.WriteFile(filepath.Join(td, "unrelated.txt"), unrelatedFileContents, 0o644)
+	if err != nil {
+		t.Fatalf("failed to write unrelated.txt: %v", err)
+	}
+
 	err = regal(&stdout, &stderr)("fix", td)
 
 	// 0 exit status is expected as all violations should have been fixed

pkg/fixer/fileprovider/fs.go

@@ -3,45 +3,33 @@ package fileprovider
 import (
 	"fmt"
 	"os"
-	"path/filepath"
 
 	"github.com/open-policy-agent/opa/ast"
 
 	"github.com/styrainc/regal/internal/parse"
+	"github.com/styrainc/regal/pkg/config"
 	"github.com/styrainc/regal/pkg/rules"
 )
 
 type FSFileProvider struct {
-	roots []string
+	roots  []string
+	ignore []string
 }
 
-func NewFSFileProvider(roots ...string) *FSFileProvider {
+func NewFSFileProvider(ignore []string, roots ...string) *FSFileProvider {
 	return &FSFileProvider{
-		roots: roots,
+		roots:  roots,
+		ignore: ignore,
 	}
 }
 
 func (p *FSFileProvider) ListFiles() ([]string, error) {
-	var files []string
-
-	for _, root := range p.roots {
-		err := filepath.Walk(root, func(path string, info os.FileInfo, err error) error {
-			if err != nil {
-				return err
-			}
-
-			if !info.IsDir() {
-				files = append(files, path)
-			}
-
-			return nil
-		})
-		if err != nil {
-			return nil, fmt.Errorf("failed to list files: %w", err)
-		}
+	filtered, err := config.FilterIgnoredPaths(p.roots, p.ignore, true, "")
+	if err != nil {
+		return nil, fmt.Errorf("failed to filter ignored paths: %w", err)
 	}
 
-	return files, nil
+	return filtered, nil
 }
 
 func (*FSFileProvider) GetFile(file string) ([]byte, error) {

pkg/fixer/fixes/useassignmentoperator.go

@@ -28,6 +28,10 @@ func (*UseAssignmentOperator) Fix(fc *FixCandidate, opts *RuntimeOptions) ([]Fix
 
 		line := lines[loc.Row-1]
 
+		if loc.Col-1 < 0 || loc.Col-1 >= len(line) {
+			continue
+		}
+
 		// unexpected character at location column, skipping
 		if line[loc.Col-1] != byte('=') {
 			continue