@lightdash/cli 0.2100.0 → 0.2101.0

package/dist/dbt/targets/snowflake.js

@@ -23,6 +23,10 @@ const snowflakeSchema = {
             type: 'string',
             nullable: true,
         },
+        authenticator: {
+            type: 'string',
+            nullable: true,
+        },
         private_key_path: {
             type: 'string',
             nullable: true,
@@ -96,6 +100,18 @@ const convertSnowflakeSchema = async (target) => {
         if (target.private_key) {
             privateKey = target.private_key;
         }
+        // Determine authentication type based on authenticator field or credentials present
+        let authenticationType;
+        if (target.authenticator &&
+            target.authenticator.toLowerCase() === 'externalbrowser') {
+            authenticationType = common_1.SnowflakeAuthenticationType.EXTERNAL_BROWSER;
+        }
+        else if (privateKey) {
+            authenticationType = common_1.SnowflakeAuthenticationType.PRIVATE_KEY;
+        }
+        else {
+            authenticationType = common_1.SnowflakeAuthenticationType.PASSWORD;
+        }
         return {
             type: common_1.WarehouseTypes.SNOWFLAKE,
             account: target.account,
@@ -103,9 +119,7 @@ const convertSnowflakeSchema = async (target) => {
             password: target.password,
             privateKey,
             privateKeyPass: target.private_key_passphrase,
-            authenticationType: privateKey
-                ? common_1.SnowflakeAuthenticationType.PRIVATE_KEY
-                : common_1.SnowflakeAuthenticationType.PASSWORD,
+            authenticationType,
             role: target.role,
             warehouse: target.warehouse,
             database: target.database,

package/dist/handlers/dbt/getWarehouseClient.js

@@ -5,12 +5,32 @@ exports.default = getWarehouseClient;
 const tslib_1 = require("tslib");
 const common_1 = require("@lightdash/common");
 const warehouses_1 = require("@lightdash/warehouses");
+const crypto_1 = tslib_1.__importDefault(require("crypto"));
 const execa_1 = tslib_1.__importDefault(require("execa"));
 const path_1 = tslib_1.__importDefault(require("path"));
 const config_1 = require("../../config");
 const profile_1 = require("../../dbt/profile");
 const globalState_1 = tslib_1.__importDefault(require("../../globalState"));
 const apiClient_1 = require("./apiClient");
+/**
+ * Cache warehouse clients to avoid repeated authentication prompts
+ * Currently used for:
+ * - Snowflake external browser auth (avoids opening multiple browser tabs)
+ */
+const warehouseClientCache = new Map();
+/**
+ * Generates a unique cache key for warehouse credentials by hashing the credentials
+ */
+function getWarehouseClientCacheKey(credentials) {
+    // Create a hash of the stringified credentials
+    // This provides a unique key regardless of warehouse type
+    const credentialsString = JSON.stringify(credentials);
+    const hash = crypto_1.default
+        .createHash('sha256')
+        .update(credentialsString)
+        .digest('hex');
+    return hash;
+}
 const getTableSchema = async ({ projectUuid, tableName, schemaName, databaseName, }) => (0, apiClient_1.lightdashApi)({
     method: 'GET',
     url: `/api/v1/projects/${projectUuid}/sqlRunner/fields?tableName=${tableName}&schemaName=${schemaName}&databaseName=${databaseName}`,
@@ -197,12 +217,22 @@ async function getWarehouseClient(options) {
         });
         globalState_1.default.debug(`> Using target ${target.type}`);
         credentials = await (0, profile_1.warehouseCredentialsFromDbtTarget)(target);
-        warehouseClient = (0, warehouses_1.warehouseClientFromCredentials)({
-            ...credentials,
-            startOfWeek: (0, common_1.isWeekDay)(options.startOfWeek)
-                ? options.startOfWeek
-                : undefined,
-        });
+        // Check if we should use cached client (e.g., for auth methods requiring user interaction)
+        const cacheKey = getWarehouseClientCacheKey(credentials);
+        if (warehouseClientCache.has(cacheKey)) {
+            globalState_1.default.debug(`> Reusing cached warehouse client (${credentials.type})`);
+            warehouseClient = warehouseClientCache.get(cacheKey);
+        }
+        else {
+            globalState_1.default.debug(`> Creating new warehouse client to cache (${credentials.type})`);
+            warehouseClient = (0, warehouses_1.warehouseClientFromCredentials)({
+                ...credentials,
+                startOfWeek: (0, common_1.isWeekDay)(options.startOfWeek)
+                    ? options.startOfWeek
+                    : undefined,
+            });
+            warehouseClientCache.set(cacheKey, warehouseClient);
+        }
     }
     return {
         warehouseClient,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightdash/cli",
-  "version": "0.2100.0",
+  "version": "0.2101.0",
   "license": "MIT",
   "bin": {
     "lightdash": "dist/index.js"
@@ -33,8 +33,8 @@
     "parse-node-version": "^2.0.0",
     "unique-names-generator": "^4.7.1",
     "uuid": "^11.0.3",
-    "@lightdash/common": "0.2100.0",
-    "@lightdash/warehouses": "0.2100.0"
+    "@lightdash/warehouses": "0.2101.0",
+    "@lightdash/common": "0.2101.0"
   },
   "description": "Lightdash CLI tool",
   "devDependencies": {