Correctly render variables panel with overridden lookup vars (#1412)

## Changes Correctly render variables panel with overridden lookup vars Fixes #1410 ## Tests Manually
databricks · Oct 29, 2024 · 7b88151 · 7b88151
1 parent d1b8508
commit 7b88151
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/packages/databricks-vscode/src/bundle/models/BundleVariableModel.ts b/packages/databricks-vscode/src/bundle/models/BundleVariableModel.ts
@@ -13,7 +13,6 @@ import {WorkspaceFolderManager} from "../../vscode-objs/WorkspaceFolderManager";
 export type BundleVariable = Required<BundleSchema>["variables"][string] & {
     valueInTarget?: string;
     vscodeOverrideValue?: string;
-    required: boolean;
 };
 
 export type BundleVariableModelState = {
@@ -180,8 +179,8 @@ export class BundleVariableModel extends BaseModelWithStateCache<BundleVariableM
             variables[key] = {
                 ...definition,
                 lookup:
-                    typeof inTargetVariable === "object"
-                        ? inTargetVariable
+                    typeof inTargetVariable?.lookup === "object"
+                        ? inTargetVariable.lookup
                         : definition.lookup,
 
                 // If the value is not required based on our heuristic check, we use the value from the validate
@@ -194,7 +193,6 @@ export class BundleVariableModel extends BaseModelWithStateCache<BundleVariableM
                           : inTargetPostValidateVariables[key]?.value ??
                             definition.default,
                 vscodeOverrideValue: overrides[key],
-                required: isRequired,
             };
         }