bundle migrate: remove new top-level command, keep deployment migrate

The new approach (BuildStateFromTF, no API calls) has been folded into the
existing 'bundle deployment migrate' command. The separate 'bundle migrate'
command is removed — it was redundant.

Also add resolve_test.go: verify that int and bool cross-resource references
work correctly when Method B returns a string value. structaccess.Set already
handles string→int and string→bool conversion via strconv, so no bug exists.

Co-authored-by: Isaac

Author: denik

bundle/migrate/resolve_test.go

@@ -0,0 +1,78 @@
+package migrate_test
+
+import (
+	"encoding/json"
+	"testing"
+
+	"github.com/databricks/cli/bundle/migrate"
+	"github.com/databricks/cli/libs/structs/structaccess"
+	"github.com/databricks/cli/libs/structs/structpath"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+// state with src job having int and bool fields set.
+func testState() migrate.TFStateAttrs {
+	return migrate.TFStateAttrs{
+		"databricks_job": {
+			"src": json.RawMessage(`{
+				"id": "111",
+				"max_concurrent_runs": 4,
+				"always_running": true
+			}`),
+			"dst": json.RawMessage(`{
+				"id": "222",
+				"max_concurrent_runs": 4,
+				"always_running": true
+			}`),
+		},
+	}
+}
+
+// TestResolveFieldRefInt proves that when Method B (template evaluation) wins for
+// an int field, the returned string value is still usable: structaccess.Set must
+// parse it back to int and not error.
+func TestResolveFieldRefInt(t *testing.T) {
+	state := testState()
+	// Remove dst from state so Method A fails and Method B must be used.
+	delete(state["databricks_job"], "dst")
+
+	ctx := t.Context()
+	fieldPath, err := structpath.ParsePath("max_concurrent_runs")
+	require.NoError(t, err)
+
+	value, err := migrate.ResolveFieldRef(ctx, state, "jobs", "dst", fieldPath,
+		"${resources.jobs.src.max_concurrent_runs}")
+	require.NoError(t, err)
+
+	// Method B succeeds: returns string "4". Verify Set converts it to int.
+	type target struct {
+		MaxConcurrentRuns int `json:"max_concurrent_runs"`
+	}
+	s := &target{}
+	err = structaccess.Set(s, fieldPath, value)
+	assert.NoError(t, err, "Set should parse string %q into int field", value)
+	assert.Equal(t, 4, s.MaxConcurrentRuns)
+}
+
+// TestResolveFieldRefBool is the same for a bool field.
+func TestResolveFieldRefBool(t *testing.T) {
+	state := testState()
+	delete(state["databricks_job"], "dst")
+
+	ctx := t.Context()
+	fieldPath, err := structpath.ParsePath("always_running")
+	require.NoError(t, err)
+
+	value, err := migrate.ResolveFieldRef(ctx, state, "jobs", "dst", fieldPath,
+		"${resources.jobs.src.always_running}")
+	require.NoError(t, err)
+
+	type target struct {
+		AlwaysRunning bool `json:"always_running"`
+	}
+	s := &target{}
+	err = structaccess.Set(s, fieldPath, value)
+	assert.NoError(t, err, "Set should parse string %q into bool field", value)
+	assert.Equal(t, true, s.AlwaysRunning)
+}

cmd/bundle/bundle.go

@@ -38,7 +38,6 @@ Online documentation: https://docs.databricks.com/en/dev-tools/bundles/index.htm
 	cmd.AddCommand(newDebugCommand())
 	cmd.AddCommand(newOpenCommand())
 	cmd.AddCommand(newPlanCommand())
-	cmd.AddCommand(newMigrateCommand())
 	cmd.AddCommand(newConfigRemoteSyncCommand())
 
 	// Bundle Metadata Service (DMS) read-only command groups. Only `get`