@mablhq/mabl-cli 1.17.6 → 1.20.1

package/api/basicApiClient.js

@@ -26,7 +26,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.BasicApiClient = exports.AuthType = void 0;
+exports.BasicApiClient = void 0;
 const ApiError_1 = require("./ApiError");
 const httpUtil_1 = require("../util/httpUtil");
 const axios_1 = __importDefault(require("axios"));
@@ -34,6 +34,7 @@ const os = __importStar(require("os"));
 const async_retry_1 = __importDefault(require("async-retry"));
 const logUtils_1 = require("../util/logUtils");
 const asyncUtil_1 = require("../util/asyncUtil");
+const types_1 = require("./types");
 const MABL_ENTITY_VERSION_HEADER = 'x-mabl-entity-version';
 const DEFAULT_RETRYABLE_REQUEST_TIMEOUT_MILLISECONDS = 60000;
 const DEFAULT_RETRIES = 10;
@@ -49,11 +50,6 @@ const RETRYABLE_NODEJS_ERRORS = [
     'ETIMEOUT',
     'ECONNABORTED',
 ];
-var AuthType;
-(function (AuthType) {
-    AuthType[AuthType["Bearer"] = 0] = "Bearer";
-    AuthType[AuthType["ApiKey"] = 1] = "ApiKey";
-})(AuthType = exports.AuthType || (exports.AuthType = {}));
 class BasicApiClient {
     constructor(options) {
         var _a, _b, _c, _d;
@@ -64,13 +60,13 @@ class BasicApiClient {
         config.timeout =
             (_c = (_a = options.requestTimeoutMillis) !== null && _a !== void 0 ? _a : (_b = options.retryConfig) === null || _b === void 0 ? void 0 : _b.requestTimeoutMillis) !== null && _c !== void 0 ? _c : DEFAULT_RETRYABLE_REQUEST_TIMEOUT_MILLISECONDS;
         switch (options.authType) {
-            case AuthType.ApiKey:
+            case types_1.AuthType.ApiKey:
                 config.auth = {
                     username: 'key',
                     password: options.token,
                 };
                 break;
-            case AuthType.Bearer:
+            case types_1.AuthType.Bearer:
                 config.headers.authorization = `Bearer ${options.token}`;
                 break;
             default:
@@ -130,7 +126,13 @@ class BasicApiClient {
         return { ...result, entity_version: versionHeader };
     }
     async makePostRequest(path, requestBody, requestConfig) {
-        const response = await this.debugRequest('POST', path, () => this.httpClient.post(path, requestBody, this.getNonRetryableRequestConfig(requestConfig)));
+        return this.postRequest(path, requestBody, this.getNonRetryableRequestConfig(requestConfig));
+    }
+    async makePostRequestWithRetries(path, requestBody, config) {
+        return this.retryWrappedRequest(`makePostRequestWithRetries('${path}')`, () => this.postRequest(path, requestBody, this.getRetryableRequestConfig(config)), config);
+    }
+    async postRequest(path, requestBody, config) {
+        const response = await this.debugRequest('POST', path, () => this.httpClient.post(path, requestBody, config));
         BasicApiClient.checkResponseStatusCode(response);
         return response.data;
     }

package/api/featureSet.js

@@ -2,7 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.FeatureSet = exports.FindImplementationVersion = void 0;
 const runnerType_1 = require("../browserLauncher/runnerType");
-const PLAYWRIGHT_FEATURE_FLAG = 'nodejs_execution_engine_playwright';
 const SMARTER_WAIT_FEATURE_FLAG = 'smarter_wait';
 var FindImplementationVersion;
 (function (FindImplementationVersion) {
@@ -14,9 +13,7 @@ class FeatureSet {
         this.featureFlags = featureFlags;
     }
     getRunnerType() {
-        return this.featureFlags.has(PLAYWRIGHT_FEATURE_FLAG)
-            ? runnerType_1.RunnerType.Playwright
-            : runnerType_1.RunnerType.Puppeteer;
+        return runnerType_1.RunnerType.Playwright;
     }
     getFindImplementationVersion() {
         return this.featureFlags.has(SMARTER_WAIT_FEATURE_FLAG)

package/api/mablApiClient.js

@@ -303,7 +303,7 @@ class MablApiClient extends basicApiClient_1.BasicApiClient {
             return await this.makeGetRequest(`${env_1.BASE_API_URL}/accounts/${accountId}`);
         }
         catch (error) {
-            throw new ApiError_1.ApiError(`Failed to get account`, error, error.status);
+            throw toApiError(`Failed to get account`, error);
         }
     }
     async getApiKeyDetails() {
@@ -319,13 +319,13 @@ class MablApiClient extends basicApiClient_1.BasicApiClient {
         try {
             const selfInfo = await this.makeGetRequest(`${env_1.BASE_API_URL}/self`);
             if (((_a = selfInfo.preferences) === null || _a === void 0 ? void 0 : _a.default_workspace_id) &&
-                !cliConfigProvider_1.CliConfigProvider.getWorkspace()) {
+                !(await cliConfigProvider_1.CliConfigProvider.getWorkspace())) {
                 const workspace = await this.getWorkspace(selfInfo.preferences.default_workspace_id);
-                cliConfigProvider_1.CliConfigProvider.setWorkspace(workspace);
+                await cliConfigProvider_1.CliConfigProvider.setWorkspace(workspace);
             }
             else if (((_b = selfInfo.roles) === null || _b === void 0 ? void 0 : _b.length) === 1) {
                 const workspace = await this.getWorkspace(selfInfo.roles[0].organization_id);
-                cliConfigProvider_1.CliConfigProvider.setWorkspace(workspace);
+                await cliConfigProvider_1.CliConfigProvider.setWorkspace(workspace);
             }
             return selfInfo;
         }
@@ -454,10 +454,10 @@ class MablApiClient extends basicApiClient_1.BasicApiClient {
                 context_id: contextId,
                 workspace_id: workspaceId,
             };
-            return await this.makePostRequest(`${env_1.BASE_API_URL}/mailbox/address`, body);
+            return await this.makePostRequestWithRetries(`${env_1.BASE_API_URL}/mailbox/address`, body);
         }
         catch (error) {
-            throw new ApiError_1.ApiError('Failed to create mailbox email address', error);
+            throw toApiError('Failed to create mailbox email address', error);
         }
     }
     async createBranch(workspaceId, branchName) {

package/api/mablApiClientFactory.js

@@ -7,13 +7,13 @@ exports.MablApiClientFactory = void 0;
 const mablApiClient_1 = require("./mablApiClient");
 const authenticationProvider_1 = require("../providers/authenticationProvider");
 const cliConfigProvider_1 = require("../providers/cliConfigProvider");
-const basicApiClient_1 = require("./basicApiClient");
+const types_1 = require("./types");
 const chalk_1 = __importDefault(require("chalk"));
 class MablApiClientFactory {
     static createApiClientFromOptionalApiKey(apiKey) {
         if (apiKey) {
             return MablApiClientFactory.createApiClient({
-                authType: basicApiClient_1.AuthType.ApiKey,
+                authType: types_1.AuthType.ApiKey,
                 authToken: apiKey,
             });
         }
@@ -21,18 +21,18 @@ class MablApiClientFactory {
     }
     static createApiClientFromApiKey(apiKey) {
         return MablApiClientFactory.createApiClient({
-            authType: basicApiClient_1.AuthType.ApiKey,
+            authType: types_1.AuthType.ApiKey,
             authToken: apiKey,
         });
     }
     static createApiClientFromBearerToken(bearerToken) {
         return MablApiClientFactory.createApiClient({
-            authType: basicApiClient_1.AuthType.Bearer,
+            authType: types_1.AuthType.Bearer,
             authToken: bearerToken,
         });
     }
     static async createApiClient(opts) {
-        const httpConfig = cliConfigProvider_1.CliConfigProvider.getCliConfig().http;
+        const httpConfig = (await cliConfigProvider_1.CliConfigProvider.getCliConfig()).http;
         if (opts) {
             return new mablApiClient_1.MablApiClient({
                 authType: opts.authType,
@@ -55,15 +55,15 @@ class MablApiClientFactory {
         throw new Error('Please supply an API key or authenticate the mabl CLI tool');
     }
     static async createUserApiClient() {
-        if (authenticationProvider_1.AuthenticationProvider.getAuthType() !== basicApiClient_1.AuthType.Bearer) {
+        if ((await authenticationProvider_1.AuthenticationProvider.getAuthType()) !== types_1.AuthType.Bearer) {
             MablApiClientFactory.throwUserAuthTypeError();
         }
         return MablApiClientFactory.createApiClient();
     }
-    static createApiClientForAccessToken(accessToken) {
-        const httpConfig = cliConfigProvider_1.CliConfigProvider.getCliConfig().http;
+    static async createApiClientForAccessToken(accessToken) {
+        const httpConfig = (await cliConfigProvider_1.CliConfigProvider.getCliConfig()).http;
         return new mablApiClient_1.MablApiClient({
-            authType: basicApiClient_1.AuthType.Bearer,
+            authType: types_1.AuthType.Bearer,
             token: accessToken,
             proxyUrl: httpConfig.proxyHost,
             sslVerify: httpConfig.sslVerify,

package/api/types.js

@@ -0,0 +1,8 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.AuthType = void 0;
+var AuthType;
+(function (AuthType) {
+    AuthType[AuthType["Bearer"] = 0] = "Bearer";
+    AuthType[AuthType["ApiKey"] = 1] = "ApiKey";
+})(AuthType = exports.AuthType || (exports.AuthType = {}));

package/auth/AuthClient.js

@@ -28,6 +28,7 @@ class AuthClient {
         const loginQueryString = query_string_1.default.stringify({
             codeChallenge,
             redirectUri,
+            includeRefreshToken: true,
         });
         return idpType
             ? `${env_1.BASE_APP_URL}/authorize/social/${idpType.toLowerCase()}?${loginQueryString}`

package/auth/OktaClient.js

@@ -1,10 +1,16 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.OktaClient = void 0;
 const env_1 = require("../env/env");
+const axios_1 = __importDefault(require("axios"));
 const authenticationProvider_1 = require("../providers/authenticationProvider");
-const basicApiClient_1 = require("../api/basicApiClient");
+const types_1 = require("../providers/types");
+const types_2 = require("../api/types");
 const AuthClient_1 = require("./AuthClient");
+const loggingProvider_1 = require("../providers/logging/loggingProvider");
 const queryString = require('query-string');
 class OktaClient extends AuthClient_1.AuthClient {
     constructor(httpClient) {
@@ -12,41 +18,80 @@ class OktaClient extends AuthClient_1.AuthClient {
         this.httpClient = httpClient;
     }
     async validateAuthCode(code, codeVerifier, redirectUri) {
-        const url = `${env_1.OKTA_URL}/oauth2/default/v1/token`;
-        const requestOptions = {
-            method: 'POST',
-            headers: { 'content-type': 'application/x-www-form-urlencoded' },
-        };
-        const data = queryString.stringify({
+        return this.getBearerAuthInfo(queryString.stringify({
             grant_type: 'authorization_code',
             client_id: env_1.OKTA_CLIENT_ID,
             code_verifier: codeVerifier,
             code,
             redirect_uri: redirectUri,
-        });
-        const response = await this.httpClient.post(url, data, requestOptions);
-        const authInfo = response.data;
-        authInfo.auth_provider = authenticationProvider_1.AuthProviderType.Okta;
-        authInfo.auth_type = basicApiClient_1.AuthType.Bearer;
-        return authInfo;
+        }));
     }
-    async exchangeRefreshTokenForAccessToken(refreshToken) {
-        const url = `${env_1.OKTA_URL}/oauth2/default/v1/token`;
+    async getRefreshTokenExpiration(refreshToken) {
+        if (!refreshToken) {
+            return;
+        }
+        const url = `${env_1.OKTA_URL}/oauth2/v1/introspect`;
         const requestOptions = {
             method: 'POST',
             headers: { 'content-type': 'application/x-www-form-urlencoded' },
         };
         const data = queryString.stringify({
-            grant_type: 'refresh_token',
             client_id: env_1.OKTA_CLIENT_ID,
-            refresh_token: refreshToken,
-            scope: 'openid profile email offline_access',
+            token_type_hint: 'refresh_token',
+            token: refreshToken,
         });
-        const response = await this.httpClient.post(url, data, requestOptions);
+        try {
+            const response = await this.httpClient.post(url, data, requestOptions);
+            return response.data.exp;
+        }
+        catch (error) {
+            loggingProvider_1.logger.info(`Couldn't get refresh token expiration. Will re-use previous lifetime.`);
+            return;
+        }
+    }
+    async getBearerAuthInfo(queryStringData) {
+        const url = `${env_1.OKTA_URL}/oauth2/default/v1/token`;
+        const requestOptions = {
+            method: 'POST',
+            headers: { 'content-type': 'application/x-www-form-urlencoded' },
+        };
+        const response = await this.httpClient.post(url, queryStringData, requestOptions);
         const authInfo = response.data;
-        authInfo.auth_provider = authenticationProvider_1.AuthProviderType.Okta;
-        authInfo.auth_type = basicApiClient_1.AuthType.Bearer;
+        const refreshTokenExpiresAt = await this.getRefreshTokenExpiration(authInfo.refresh_token);
+        authInfo.auth_provider = types_1.AuthProviderType.Okta;
+        authInfo.auth_type = types_2.AuthType.Bearer;
+        authInfo.refresh_token_expires_at = refreshTokenExpiresAt;
         return authInfo;
     }
+    exchangeRefreshTokenForAccessToken(refreshToken) {
+        var _a;
+        try {
+            return this.getBearerAuthInfo(queryString.stringify({
+                grant_type: 'refresh_token',
+                client_id: env_1.OKTA_CLIENT_ID,
+                refresh_token: refreshToken,
+                scope: 'openid profile email offline_access',
+            }));
+        }
+        catch (error) {
+            if (!axios_1.default.isAxiosError(error)) {
+                throw error;
+            }
+            const data = (_a = error.response) === null || _a === void 0 ? void 0 : _a.data;
+            if (!(0, authenticationProvider_1.isOidcError)(data)) {
+                throw error;
+            }
+            return Promise.resolve({
+                auth_provider: types_1.AuthProviderType.Okta,
+                auth_type: types_2.AuthType.Bearer,
+                access_token: '',
+                id_token: '',
+                token_type: '',
+                expires_in: 0,
+                error: data.error,
+                error_description: data.error_description,
+            });
+        }
+    }
 }
 exports.OktaClient = OktaClient;

package/browserLauncher/browserLauncherFactory.js

@@ -4,7 +4,6 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.BrowserLauncherFactory = void 0;
-const puppeteerBrowserLauncher_1 = require("./puppeteerBrowserLauncher/puppeteerBrowserLauncher");
 const playwrightBrowserLauncher_1 = require("./playwrightBrowserLauncher/playwrightBrowserLauncher");
 const runnerType_1 = require("./runnerType");
 const logUtils_1 = require("../util/logUtils");
@@ -24,8 +23,8 @@ class BrowserLauncherFactory {
             loggerFunc(chalk_1.default.cyan(`Browser launcher:`, chalk_1.default.magenta('Playwright')));
             return new playwrightBrowserLauncher_1.PlaywrightBrowserLauncher();
         }
-        loggerFunc(chalk_1.default.cyan(`Browser launcher:`, chalk_1.default.magenta('Puppeteer')));
-        return new puppeteerBrowserLauncher_1.PuppeteerBrowserLauncher();
+        loggerFunc(chalk_1.default.red.bold(`Unsupported runner type:`, runner));
+        throw new Error(`Unsupported runner type:[${runner}]`);
     }
 }
 exports.BrowserLauncherFactory = BrowserLauncherFactory;

package/browserLauncher/playwrightBrowserLauncher/playwrightBrowser.js

@@ -128,5 +128,8 @@ class PlaywrightBrowser extends events_1.default {
         await playwrightPage.waitForInitialization();
         return playwrightPage;
     }
+    exposeFunction(name, callback) {
+        return this.defaultContext.exposeFunction(name, callback);
+    }
 }
 exports.PlaywrightBrowser = PlaywrightBrowser;

package/browserLauncher/playwrightBrowserLauncher/playwrightDom.js

@@ -132,13 +132,13 @@ class PlaywrightElementHandle extends PlaywrightJsHandle {
         return (0, utils_1.mapIfNotNull)(await this.element.boundingBox(), (boundingBox) => boundingBox);
     }
     async click(options) {
-        var _a, _b, _c, _d;
+        var _a, _b, _c;
         const trial = (_a = options === null || options === void 0 ? void 0 : options.trial) !== null && _a !== void 0 ? _a : false;
         try {
             await this.element.click({
                 clickCount: (_b = options === null || options === void 0 ? void 0 : options.clickCount) !== null && _b !== void 0 ? _b : 1,
                 force: (_c = options === null || options === void 0 ? void 0 : options.force) !== null && _c !== void 0 ? _c : !trial,
-                timeout: (_d = options === null || options === void 0 ? void 0 : options.timeout) !== null && _d !== void 0 ? _d : types_1.DefaultTimeouts.quickActionTimeoutMs,
+                timeout: this.getActionTimeout(options),
                 trial,
             });
         }
@@ -151,12 +151,12 @@ class PlaywrightElementHandle extends PlaywrightJsHandle {
         }
     }
     async doubleClick(options) {
-        var _a, _b, _c;
+        var _a, _b;
         try {
             const trial = (_a = options === null || options === void 0 ? void 0 : options.trial) !== null && _a !== void 0 ? _a : false;
             return this.element.dblclick({
                 force: (_b = options === null || options === void 0 ? void 0 : options.force) !== null && _b !== void 0 ? _b : !trial,
-                timeout: (_c = options === null || options === void 0 ? void 0 : options.timeout) !== null && _c !== void 0 ? _c : types_1.DefaultTimeouts.quickActionTimeoutMs,
+                timeout: this.getActionTimeout(options),
                 trial,
             });
         }
@@ -194,23 +194,13 @@ class PlaywrightElementHandle extends PlaywrightJsHandle {
         return this.element.selectOption(value);
     }
     async type(text, options) {
-        const timeout = PlaywrightElementHandle.getTypeTimeout(text, options);
         await this.element.focus();
         await this.element.selectText({ force: true });
         return this.element.type(text, {
             delay: options === null || options === void 0 ? void 0 : options.delay,
-            timeout,
+            timeout: 0,
         });
     }
-    static getTypeTimeout(text, options) {
-        var _a;
-        let timeout;
-        const newTimeout = text.length * ((_a = options === null || options === void 0 ? void 0 : options.delay) !== null && _a !== void 0 ? _a : 1) * 2;
-        if (newTimeout > types_1.DefaultTimeouts.defaultTextInputTimeoutMs) {
-            timeout = newTimeout;
-        }
-        return timeout;
-    }
     press(key) {
         return this.element.press(key);
     }
@@ -283,5 +273,12 @@ class PlaywrightElementHandle extends PlaywrightJsHandle {
         const result = await this.element.evaluate((el) => el.innerText);
         return (0, pureUtil_1.isNullish)(result) ? undefined : result;
     }
+    getActionTimeout(options) {
+        var _a;
+        const defaultTimeout = (options === null || options === void 0 ? void 0 : options.trial)
+            ? types_1.DefaultTimeouts.defaultTrialTimeoutMs
+            : types_1.DefaultTimeouts.defaultActionTimeoutMs;
+        return (_a = options === null || options === void 0 ? void 0 : options.timeout) !== null && _a !== void 0 ? _a : defaultTimeout;
+    }
 }
 exports.PlaywrightElementHandle = PlaywrightElementHandle;

package/browserLauncher/playwrightBrowserLauncher/playwrightFrame.js

@@ -129,7 +129,7 @@ class PlaywrightFrame extends browserLauncher_1.Frame {
             const response = await this.frame.goto(url, options);
             if (waitForNetworkIdle) {
                 await this.frame.waitForLoadState('networkidle', {
-                    timeout: types_1.DefaultTimeouts.quickActionTimeoutMs,
+                    timeout: types_1.DefaultTimeouts.defaultActionTimeoutMs,
                 });
             }
             return (0, utils_1.mapIfNotNull)(response, (response) => new playwrightHttpResponse_1.PlaywrightHttpResponse(this.parentPage, response));
@@ -178,5 +178,23 @@ class PlaywrightFrame extends browserLauncher_1.Frame {
         }
         return this.frameImpl;
     }
+    async initializeExposedFunctionInSecondaryWorld(functionName) {
+        var _a;
+        try {
+            const pageImpl = this.parentPage.getPageImpl();
+            if (!pageImpl) {
+                return false;
+            }
+            const binding = (_a = pageImpl._pageBindings.get(functionName)) !== null && _a !== void 0 ? _a : pageImpl._browserContext._pageBindings.get(functionName);
+            if (!binding) {
+                return false;
+            }
+            await this.evaluateCodeInSecondaryWorld(binding.source, {});
+        }
+        catch (e) {
+            return false;
+        }
+        return true;
+    }
 }
 exports.PlaywrightFrame = PlaywrightFrame;

package/browserLauncher/runnerType.js

@@ -4,5 +4,4 @@ exports.RunnerType = void 0;
 var RunnerType;
 (function (RunnerType) {
     RunnerType["Playwright"] = "playwright";
-    RunnerType["Puppeteer"] = "puppeteer";
 })(RunnerType = exports.RunnerType || (exports.RunnerType = {}));

package/browserLauncher/types.js

@@ -4,8 +4,8 @@ exports.WaitForOptions = exports.LifecycleEvent = exports.DefaultTimeouts = void
 class DefaultTimeouts {
 }
 exports.DefaultTimeouts = DefaultTimeouts;
-DefaultTimeouts.quickActionTimeoutMs = 5000;
-DefaultTimeouts.defaultTextInputTimeoutMs = 30000;
+DefaultTimeouts.defaultTrialTimeoutMs = 5000;
+DefaultTimeouts.defaultActionTimeoutMs = 0;
 var LifecycleEvent;
 (function (LifecycleEvent) {
     LifecycleEvent["DomContentLoaded"] = "domcontentloaded";

package/cli.js

@@ -85,9 +85,7 @@ yargs
         catch (_) {
         }
     },
-    (argv) => {
-        (0, analytics_1.trackCliEvent)(argv._, (0, pureUtil_1.extractKeyCountsFromArgs)(argv));
-    },
+    async (argv) => (0, analytics_1.trackCliEvent)(argv._, (0, pureUtil_1.extractKeyCountsFromArgs)(argv)),
     () => {
         updateNotifier({
             pkg: cliPackage,

package/commands/applications/applications_cmds/list.js

@@ -18,7 +18,7 @@ exports.handler = (0, util_1.failWrapper)(listApplications);
 async function listApplications(parsed) {
     const output = parsed.output;
     const limit = parsed.limit;
-    const workspaceId = (0, util_1.getWorkspaceId)(parsed);
+    const workspaceId = await (0, util_1.getWorkspaceId)(parsed);
     const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
     const applications = await apiClient.getApplications(workspaceId, limit);
     printApplications(applications, output);

package/commands/auth/auth_cmds/activate-key.js

@@ -24,6 +24,6 @@ async function activateApiKey(parsed) {
     const mablApi = await mablApiClientFactory_1.MablApiClientFactory.createApiClientFromApiKey(cleanApiKey);
     const apiKeyDetails = await mablApi.getApiKeyDetails();
     const apiKeyWorkspace = await mablApi.getWorkspace(apiKeyDetails.workspace_id);
-    cliConfigProvider_1.CliConfigProvider.setWorkspace(apiKeyWorkspace);
-    auth.activateApiKey(cleanApiKey);
+    await cliConfigProvider_1.CliConfigProvider.setWorkspace(apiKeyWorkspace);
+    return auth.activateApiKey(cleanApiKey);
 }

package/commands/auth/auth_cmds/clear.js

@@ -6,5 +6,5 @@ exports.describe = 'Clear logged in user';
 exports.builder = {};
 exports.handler = exports.handler = () => {
     const auth = new authenticationProvider_1.AuthenticationProvider();
-    auth.clearAuthentication();
+    return auth.clearAuthentication();
 };

package/commands/auth/auth_cmds/info.js

@@ -6,5 +6,5 @@ exports.describe = 'Display logged in user';
 exports.builder = {};
 exports.handler = () => {
     const auth = new authenticationProvider_1.AuthenticationProvider();
-    auth.displayInfo();
+    return auth.displayInfo();
 };

package/commands/branches/branches_cmds/create.js

@@ -39,7 +39,7 @@ exports.builder = (yargs) => {
 exports.handler = (0, util_1.failWrapper)(createBranch);
 async function createBranch(parsed) {
     const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
-    const workspaceId = (0, util_1.getWorkspaceId)(parsed);
+    const workspaceId = await (0, util_1.getWorkspaceId)(parsed);
     const branchName = parsed.name;
     loggingProvider_1.logger.info(`Creating Branch [${branchName}]`);
     try {

package/commands/branches/branches_cmds/describe.js

@@ -39,7 +39,7 @@ async function getBranch(parsed) {
     const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
     let branch;
     if (lookupByName) {
-        const workspaceId = (0, util_1.getWorkspaceId)(parsed);
+        const workspaceId = await (0, util_1.getWorkspaceId)(parsed);
         branch = await apiClient.getBranchByName(workspaceId, branchIdentifier);
     }
     else {

package/commands/branches/branches_cmds/list.js

@@ -43,7 +43,7 @@ async function listBranches(parsed) {
     const output = parsed.output;
     const limit = parsed.limit;
     const statusFilter = parsed.status;
-    const workspaceId = (0, util_1.getWorkspaceId)(parsed);
+    const workspaceId = await (0, util_1.getWorkspaceId)(parsed);
     const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
     const branches = await apiClient.getBranches(workspaceId, limit, statusFilter);
     printBranches(branches, output);

package/commands/branches/branches_cmds/merge.js

@@ -47,7 +47,7 @@ exports.builder = (yargs) => {
 exports.handler = (0, util_1.failWrapper)(mergeBranch);
 async function mergeBranch(parsed) {
     const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
-    const workspaceId = (0, util_1.getWorkspaceId)(parsed);
+    const workspaceId = await (0, util_1.getWorkspaceId)(parsed);
     const fromBranchName = parsed.from;
     const toBranchName = parsed.to;
     const branch = await apiClient.mergeBranch(workspaceId, fromBranchName, toBranchName);

package/commands/commandUtil/util.js

@@ -24,12 +24,12 @@ function failWrapper(func, exitCodeOnError = 1) {
     });
 }
 exports.failWrapper = failWrapper;
-function getWorkspaceId(parsed) {
-    const workspaceId = parsed['workspace-id'];
+async function getWorkspaceId(parsed) {
+    const workspaceId = parsed[constants_1.CommandArgWorkspaceId];
     if (workspaceId) {
         return workspaceId;
     }
-    const configuredWorkspace = cliConfigProvider_1.CliConfigProvider.getWorkspace();
+    const configuredWorkspace = await cliConfigProvider_1.CliConfigProvider.getWorkspace();
     if (configuredWorkspace === null || configuredWorkspace === void 0 ? void 0 : configuredWorkspace.id) {
         return configuredWorkspace.id;
     }

package/commands/config/config_cmds/configKeys.js

@@ -0,0 +1,11 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.validConfigKeyChoices = exports.configKeys = void 0;
+exports.configKeys = Object.freeze({
+    browserPath: 'browser.path',
+    enableSourceControlMetadataCollection: 'alpha.scm_metadata.enable',
+    defaultWorkspaceId: 'workspace',
+    proxy: 'http.proxy',
+    sslVerify: 'http.sslVerify',
+});
+exports.validConfigKeyChoices = Object.values(exports.configKeys);

package/commands/config/config_cmds/delete.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const cliConfigProvider_1 = require("../../../providers/cliConfigProvider");
 const set_1 = require("./set");
+const configKeys_1 = require("./configKeys");
 const list_1 = require("./list");
 const loggingProvider_1 = require("../../../providers/logging/loggingProvider");
 exports.command = `delete <${set_1.configKeyCommandArg}>`;
@@ -10,18 +11,18 @@ exports.builder = (yargs) => {
     yargs.positional(set_1.configKeyCommandArg, {
         describe: 'configuration key to delete',
         type: 'string',
-        choices: set_1.validConfigKeyChoices,
+        choices: configKeys_1.validConfigKeyChoices,
     });
 };
 exports.handler = deleteConfig;
-function deleteConfig(parsed) {
+async function deleteConfig(parsed) {
     const key = parsed['config-key'];
     switch (key) {
-        case set_1.configKeys.defaultWorkspaceId:
-            cliConfigProvider_1.CliConfigProvider.clearWorkspace();
+        case configKeys_1.configKeys.defaultWorkspaceId:
+            await cliConfigProvider_1.CliConfigProvider.clearWorkspace();
             break;
         default:
-            cliConfigProvider_1.CliConfigProvider.clearConfigProperty(key);
+            await cliConfigProvider_1.CliConfigProvider.clearConfigProperty(key);
     }
     loggingProvider_1.logger.info(`Deleted config for [${key}]`);
     return (0, list_1.listConfig)();