bundle/artifacts/expand_globs.go

@@ -33,7 +33,12 @@ func createGlobError(v dyn.Value, p dyn.Path, message string) diag.Diagnostic {
 		Severity:  diag.Error,
 		Summary:   fmt.Sprintf("%s: %s", source, message),
 		Locations: []dyn.Location{v.Location()},
-		Paths:     []dyn.Path{p},
+
+		Paths: []dyn.Path{
+			// Hack to clone the path. This path copy is mutable.
+			// To be addressed in a later PR.
+			p.Append(),
+		},
 	}
 }
 

bundle/libraries/expand_glob_references.go

@@ -16,10 +16,12 @@ type expand struct {
 
 func matchError(p dyn.Path, l []dyn.Location, message string) diag.Diagnostic {
 	return diag.Diagnostic{
-		Severity:  diag.Error,
-		Summary:   message,
+		Severity: diag.Error,
+		Summary:  message,
+		Paths: []dyn.Path{
+			p.Append(),
+		},
 		Locations: l,
-		Paths:     []dyn.Path{p},
 	}
 }
 

bundle/libraries/upload.go

@@ -76,7 +76,7 @@ func collectLocalLibraries(b *bundle.Bundle) (map[string][]configLocation, error
 
 				source = filepath.Join(b.RootPath, source)
 				libs[source] = append(libs[source], configLocation{
-					configPath: p,
+					configPath: p.Append(), // Hack to get the copy of path
 					location:   v.Location(),
 				})
 

libs/dyn/visit_test.go

@@ -1,27 +1,26 @@
-package dyn_test
+package dyn
 
 import (
 	"testing"
 
-	"github.com/databricks/cli/libs/dyn"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestVisitCallbackPathCopy(t *testing.T) {
-	vin := dyn.V(map[string]dyn.Value{
-		"foo": dyn.V(42),
-		"bar": dyn.V(43),
+	vin := V(map[string]Value{
+		"foo": V(42),
+		"bar": V(43),
 	})
 
-	var paths []dyn.Path
+	var paths []Path
 
 	// The callback should receive a copy of the path.
 	// If the same underlying value is used, all collected paths will be the same.
-	// This test uses `MapByPattern` to collect all paths in the map.
-	// Visit itself doesn't have public functions and we exclusively use black-box testing for this package.
-	_, _ = dyn.MapByPattern(vin, dyn.NewPattern(dyn.AnyKey()), func(p dyn.Path, v dyn.Value) (dyn.Value, error) {
-		paths = append(paths, p)
-		return v, nil
+	_, _ = visit(vin, EmptyPath, NewPattern(AnyKey()), visitOptions{
+		fn: func(p Path, v Value) (Value, error) {
+			paths = append(paths, p)
+			return v, nil
+		},
 	})
 
 	// Verify that the paths retained their original values.