cmd: include worker dir for no-range recursive issues

cmd/inspect_parallel.go

@@ -72,6 +72,7 @@ func (cli *CLI) inspectParallel(opts Options) int {
 		if err := json.Unmarshal(stdout, &workerIssues); err != nil {
 			panic(fmt.Errorf("failed to parse issues in %s; %s; stdout=%s; stderr=%s", worker.dir, err, stdout, stderr))
 		}
+		fillNoRangeIssueFilenames(worker.dir, workerIssues)
 		issues = append(issues, workerIssues...)
 
 		if len(stderr) > 0 {
@@ -103,6 +104,19 @@ func (cli *CLI) inspectParallel(opts Options) int {
 	return ExitCodeOK
 }
 
+func fillNoRangeIssueFilenames(dir string, issues tflint.Issues) {
+	for _, issue := range issues {
+		if issue.Range.Filename == "" {
+			issue.Range.Filename = dir
+		}
+		for idx := range issue.Callers {
+			if issue.Callers[idx].Filename == "" {
+				issue.Callers[idx].Filename = dir
+			}
+		}
+	}
+}
+
 // Spawn workers to run in parallel for each directory.
 // A worker is a process that runs itself as a child process.
 // The number of parallelism is controlled by --max-workers flag. The default is the number of CPUs.

cmd/inspect_parallel_test.go

@@ -0,0 +1,43 @@
+package cmd
+
+import (
+	"testing"
+
+	"github.com/google/go-cmp/cmp"
+	hcl "github.com/hashicorp/hcl/v2"
+	"github.com/terraform-linters/tflint/tflint"
+)
+
+func TestFillNoRangeIssueFilenames(t *testing.T) {
+	issues := tflint.Issues{
+		{
+			Range: hcl.Range{},
+			Callers: []hcl.Range{
+				{},
+				{Filename: "main.tf"},
+			},
+		},
+		{
+			Range: hcl.Range{Filename: "subdir/main.tf"},
+		},
+	}
+
+	fillNoRangeIssueFilenames("subdir", issues)
+
+	expected := tflint.Issues{
+		{
+			Range: hcl.Range{Filename: "subdir"},
+			Callers: []hcl.Range{
+				{Filename: "subdir"},
+				{Filename: "main.tf"},
+			},
+		},
+		{
+			Range: hcl.Range{Filename: "subdir/main.tf"},
+		},
+	}
+
+	if diff := cmp.Diff(expected, issues); diff != "" {
+		t.Fatal(diff)
+	}
+}