@mablhq/mabl-cli 1.19.3 → 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:

package/api/mablApiClient.js

@@ -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;
         }

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/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,
             });
         }
@@ -273,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));

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)();

package/commands/config/config_cmds/get.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 const cliConfigProvider_1 = require("../../../providers/cliConfigProvider");
 const set_1 = require("./set");
+const configKeys_1 = require("./configKeys");
 const cli_table3_1 = __importDefault(require("cli-table3"));
 const list_1 = require("./list");
 const loggingProvider_1 = require("../../../providers/logging/loggingProvider");
@@ -17,15 +18,15 @@ exports.builder = (yargs) => {
     });
 };
 exports.handler = getConfig;
-function getConfig(parsed) {
+async function getConfig(parsed) {
     const key = parsed['config-key'];
     const table = new cli_table3_1.default({
         head: ['Config', 'Value', 'Details'],
     });
     let value;
     switch (key) {
-        case set_1.configKeys.defaultWorkspaceId:
-            const workspace = cliConfigProvider_1.CliConfigProvider.getWorkspace();
+        case configKeys_1.configKeys.defaultWorkspaceId:
+            const workspace = await cliConfigProvider_1.CliConfigProvider.getWorkspace();
             if (workspace) {
                 table.push([key, workspace.id, workspace.name]);
                 value = workspace.id;
@@ -35,10 +36,10 @@ function getConfig(parsed) {
             }
             break;
         default:
-            if (!Object.values(set_1.configKeys).includes(key)) {
+            if (!Object.values(configKeys_1.configKeys).includes(key)) {
                 throw new Error(`Unknown key [${key}]`);
             }
-            const propertyValue = cliConfigProvider_1.CliConfigProvider.getConfigProperty(key);
+            const propertyValue = await cliConfigProvider_1.CliConfigProvider.getConfigProperty(key);
             if (propertyValue !== undefined) {
                 table.push([key, propertyValue, list_1.defaultTupleValue]);
             }

package/commands/config/config_cmds/list.js

@@ -6,33 +6,33 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.listConfig = exports.defaultTupleValue = void 0;
 const cliConfigProvider_1 = require("../../../providers/cliConfigProvider");
 const cli_table3_1 = __importDefault(require("cli-table3"));
-const set_1 = require("./set");
+const configKeys_1 = require("./configKeys");
 const loggingProvider_1 = require("../../../providers/logging/loggingProvider");
 exports.command = `list`;
 exports.describe = 'List all user config values';
 exports.handler = listConfig;
 exports.defaultTupleValue = '---';
-function listConfig() {
+async function listConfig() {
     const table = new cli_table3_1.default({
         head: ['Config', 'Value', 'Details'],
         colWidths: [null, 60],
     });
-    Object.values(set_1.configKeys).forEach((propertyKey) => {
+    for (const propertyKey of configKeys_1.validConfigKeyChoices) {
         let value;
         let details;
         switch (propertyKey) {
-            case set_1.configKeys.defaultWorkspaceId:
-                const maybeWorkspace = cliConfigProvider_1.CliConfigProvider.getWorkspace();
+            case configKeys_1.configKeys.defaultWorkspaceId:
+                const maybeWorkspace = await cliConfigProvider_1.CliConfigProvider.getWorkspace();
                 value = maybeWorkspace === null || maybeWorkspace === void 0 ? void 0 : maybeWorkspace.id;
                 details = maybeWorkspace === null || maybeWorkspace === void 0 ? void 0 : maybeWorkspace.name;
                 break;
             default:
-                value = cliConfigProvider_1.CliConfigProvider.getConfigProperty(propertyKey);
+                value = await cliConfigProvider_1.CliConfigProvider.getConfigProperty(propertyKey);
         }
         const displayValue = value !== undefined ? value : exports.defaultTupleValue;
         const displayDetails = details !== undefined ? details : exports.defaultTupleValue;
         table.push([propertyKey, displayValue, displayDetails]);
-    });
+    }
     loggingProvider_1.logger.info(table.toString());
     return table;
 }

package/commands/config/config_cmds/set.js

@@ -1,20 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.validConfigKeyChoices = exports.configKeys = exports.configKeyCommandArg = void 0;
+exports.configKeyCommandArg = void 0;
 const cliConfigProvider_1 = require("../../../providers/cliConfigProvider");
 const mablApiClientFactory_1 = require("../../../api/mablApiClientFactory");
 const list_1 = require("./list");
 const loggingProvider_1 = require("../../../providers/logging/loggingProvider");
+const configKeys_1 = require("./configKeys");
 exports.configKeyCommandArg = 'config-key';
 const configValue = 'config-value';
-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);
 exports.command = `set <${exports.configKeyCommandArg}> <${configValue}>`;
 exports.describe = 'Set a default configuration key';
 exports.builder = (yargs) => {
@@ -22,7 +15,7 @@ exports.builder = (yargs) => {
         .positional(exports.configKeyCommandArg, {
         describe: 'The configuration key to set',
         type: 'string',
-        choices: exports.validConfigKeyChoices,
+        choices: configKeys_1.validConfigKeyChoices,
     })
         .positional(configValue, {
         describe: 'The value to configure for the config',
@@ -34,33 +27,33 @@ async function setConfig(parsed) {
     const key = parsed['config-key'];
     const value = parsed['config-value'];
     switch (key) {
-        case exports.configKeys.browserPath:
-            cliConfigProvider_1.CliConfigProvider.setConfigProperty(exports.configKeys.browserPath, value);
+        case configKeys_1.configKeys.browserPath:
+            await cliConfigProvider_1.CliConfigProvider.setConfigProperty(configKeys_1.configKeys.browserPath, value);
             break;
-        case exports.configKeys.defaultWorkspaceId:
+        case configKeys_1.configKeys.defaultWorkspaceId:
             const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
             const workspace = await apiClient.getWorkspace(value);
-            cliConfigProvider_1.CliConfigProvider.setWorkspace(workspace);
+            await cliConfigProvider_1.CliConfigProvider.setWorkspace(workspace);
             break;
-        case exports.configKeys.enableSourceControlMetadataCollection:
-        case exports.configKeys.sslVerify:
+        case configKeys_1.configKeys.enableSourceControlMetadataCollection:
+        case configKeys_1.configKeys.sslVerify:
             if (!['true', 'false'].includes(value)) {
                 throw new Error("Valid values are 'true' and 'false'");
             }
             const booleanValue = value === 'true';
-            cliConfigProvider_1.CliConfigProvider.setConfigProperty(key, booleanValue);
+            await cliConfigProvider_1.CliConfigProvider.setConfigProperty(key, booleanValue);
             break;
-        case exports.configKeys.proxy:
+        case configKeys_1.configKeys.proxy:
             if (!isValidUrl(value)) {
                 throw new Error(`Invalid URL specified. The value must be a fully specified URL with a valid protocol and domain: ${value}`);
             }
-            cliConfigProvider_1.CliConfigProvider.setConfigProperty(key, value);
+            await cliConfigProvider_1.CliConfigProvider.setConfigProperty(key, value);
             break;
         default:
-            throw new Error(`<${exports.configKeyCommandArg}> value not one of supported types: ${JSON.stringify(exports.validConfigKeyChoices)}`);
+            throw new Error(`<${exports.configKeyCommandArg}> value not one of supported types: ${JSON.stringify(configKeys_1.validConfigKeyChoices)}`);
     }
     loggingProvider_1.logger.info(`Configured [${key}] = [${value}]`);
-    (0, list_1.listConfig)();
+    await (0, list_1.listConfig)();
     return key;
 }
 function isValidUrl(value) {