Test mergeReplaceDirectives with nested replace directives

Author: sublee

pkg/commands/internal/builder_test.go

@@ -63,7 +63,7 @@ func Test_sanitizeVersion(t *testing.T) {
 func TestMergeReplaceDirectives(t *testing.T) {
 	t.Parallel()
 
-	// Create a temporary module structure:
+	// Create a temporary module with the following structure:
 	// tmp/
 	//   go.mod
 	//   golangci-lint/
@@ -78,6 +78,9 @@ go 1.24.0
 		{Module: "example.com/plugin", Path: "testdata/plugin"},
 	}}, tmp)
 
+	// Merge replace directives from the plugin's go.mod into the temporary
+	// repo. Only the plugin's own replace rules are applied; transitive
+	// replaces from its dependencies are not automatically merged.
 	err := b.mergeReplaceDirectives(t.Context(), filepath.Join("testdata", "plugin"))
 	require.NoError(t, err)
 
@@ -92,6 +95,12 @@ go 1.24.0
 	err = json.Unmarshal(output, &goMod)
 	require.NoError(t, err)
 
+	// The go.mod file should include a replace directive for
+	// example.com/target, pointing to the local path, because
+	// example.com/plugin's go.mod defines it. However, it should not include a
+	// replace directive for example.com/other, since example.com/plugin does
+	// not directly depend on it, and go mod ignores such transitive
+	// replacements.
 	require.Len(t, goMod.Replace, 1)
 	assert.Contains(t, goMod.Replace[0].New.Path, "testdata/plugin/target")
 }

pkg/commands/internal/testdata/plugin/target/go.mod

@@ -1,3 +1,7 @@
 module example.com/target
 
 go 1.24.0
+
+require example.com/other v0.0.0
+
+replace example.com/other => ./other

pkg/commands/internal/testdata/plugin/target/other/go.mod

@@ -0,0 +1,3 @@
+module example.com/other
+
+go 1.24.0

pkg/commands/internal/testdata/plugin/target/other/other.go

@@ -0,0 +1 @@
+package other

pkg/commands/internal/testdata/plugin/target/target.go

@@ -1 +1,3 @@
 package target
+
+import _ "example.com/other"