@hubspot/cli 7.7.0-experimental.3 → 7.7.0

package/commands/sandbox/delete.js

@@ -1,203 +1,201 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-const logger_1 = require("@hubspot/local-dev-lib/logger");
-const index_1 = require("@hubspot/local-dev-lib/errors/index");
-const sandboxHubs_1 = require("@hubspot/local-dev-lib/api/sandboxHubs");
-const config_1 = require("@hubspot/local-dev-lib/config");
-const getAccountIdentifier_1 = require("@hubspot/local-dev-lib/config/getAccountIdentifier");
-const urls_1 = require("@hubspot/local-dev-lib/urls");
-const environment_1 = require("@hubspot/local-dev-lib/environment");
-const usageTracking_1 = require("../../lib/usageTracking");
-const index_2 = require("../../lib/errorHandlers/index");
-const lang_1 = require("../../lib/lang");
-const sandboxesPrompt_1 = require("../../lib/prompts/sandboxesPrompt");
-const accountsPrompt_1 = require("../../lib/prompts/accountsPrompt");
-const exitCodes_1 = require("../../lib/enums/exitCodes");
-const promptUtils_1 = require("../../lib/prompts/promptUtils");
-const ui_1 = require("../../lib/ui");
-const yargsUtils_1 = require("../../lib/yargsUtils");
+import { logger } from '@hubspot/local-dev-lib/logger';
+import { isSpecifiedError } from '@hubspot/local-dev-lib/errors/index';
+import { deleteSandbox } from '@hubspot/local-dev-lib/api/sandboxHubs';
+import { getEnv, removeSandboxAccountFromConfig, updateDefaultAccount, getAccountId, getConfigAccounts, } from '@hubspot/local-dev-lib/config';
+import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
+import { getHubSpotWebsiteOrigin } from '@hubspot/local-dev-lib/urls';
+import { getValidEnv } from '@hubspot/local-dev-lib/environment';
+import { trackCommandUsage } from '../../lib/usageTracking.js';
+import { logError, debugError } from '../../lib/errorHandlers/index.js';
+import { i18n } from '../../lib/lang.js';
+import { deleteSandboxPrompt } from '../../lib/prompts/sandboxesPrompt.js';
+import { selectAccountFromConfig } from '../../lib/prompts/accountsPrompt.js';
+import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import { promptUser } from '../../lib/prompts/promptUtils.js';
+import { uiAccountDescription, uiBetaTag, uiCommandReference, } from '../../lib/ui/index.js';
+import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'delete';
-const describe = (0, ui_1.uiBetaTag)((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.describe`), false);
+const describe = uiBetaTag(i18n(`commands.sandbox.subcommands.delete.describe`), false);
 async function handler(args) {
-    const { providedAccountId, force } = args;
-    (0, usageTracking_1.trackCommandUsage)('sandbox-delete', {}, providedAccountId);
+    const { userProvidedAccount, derivedAccountId, force } = args;
+    trackCommandUsage('sandbox-delete', {}, derivedAccountId);
     let accountPrompt;
-    if (!providedAccountId) {
+    if (!userProvidedAccount) {
         if (!force) {
-            accountPrompt = await (0, sandboxesPrompt_1.deleteSandboxPrompt)();
+            accountPrompt = await deleteSandboxPrompt();
         }
         else {
             // Account is required, throw error if force flag is present and no account is specified
-            logger_1.logger.log('');
-            logger_1.logger.error((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.noAccount`));
-            process.exit(exitCodes_1.EXIT_CODES.ERROR);
+            logger.log('');
+            logger.error(i18n(`commands.sandbox.subcommands.delete.failure.noAccount`));
+            process.exit(EXIT_CODES.ERROR);
         }
         if (!accountPrompt) {
-            logger_1.logger.log('');
-            logger_1.logger.error((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.noSandboxAccounts`, {
-                authCommand: (0, ui_1.uiCommandReference)('hs auth'),
+            logger.log('');
+            logger.error(i18n(`commands.sandbox.subcommands.delete.failure.noSandboxAccounts`, {
+                authCommand: uiCommandReference('hs auth'),
             }));
-            process.exit(exitCodes_1.EXIT_CODES.ERROR);
+            process.exit(EXIT_CODES.ERROR);
         }
     }
-    const sandboxAccountId = (0, config_1.getAccountId)(providedAccountId || accountPrompt.account);
+    const sandboxAccountId = getAccountId(userProvidedAccount || accountPrompt.account);
     if (!sandboxAccountId) {
-        logger_1.logger.error((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.noSandboxAccountId`));
-        process.exit(exitCodes_1.EXIT_CODES.ERROR);
+        logger.error(i18n(`commands.sandbox.subcommands.delete.failure.noSandboxAccountId`));
+        process.exit(EXIT_CODES.ERROR);
     }
-    const isDefaultAccount = sandboxAccountId === (0, config_1.getAccountId)();
-    const baseUrl = (0, urls_1.getHubSpotWebsiteOrigin)((0, environment_1.getValidEnv)((0, config_1.getEnv)(sandboxAccountId)));
+    const isDefaultAccount = sandboxAccountId === getAccountId();
+    const baseUrl = getHubSpotWebsiteOrigin(getValidEnv(getEnv(sandboxAccountId)));
     let parentAccountId;
-    const accountsList = (0, config_1.getConfigAccounts)() || [];
+    const accountsList = getConfigAccounts() || [];
     for (const portal of accountsList) {
-        if ((0, getAccountIdentifier_1.getAccountIdentifier)(portal) === sandboxAccountId) {
+        if (getAccountIdentifier(portal) === sandboxAccountId) {
             if (portal.parentAccountId) {
                 parentAccountId = portal.parentAccountId;
             }
             else if (!force) {
-                const parentAccountPrompt = await (0, sandboxesPrompt_1.deleteSandboxPrompt)(true);
+                const parentAccountPrompt = await deleteSandboxPrompt(true);
                 if (!parentAccountPrompt) {
-                    logger_1.logger.error((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.noParentAccount`, {
-                        authCommand: (0, ui_1.uiCommandReference)('hs auth'),
+                    logger.error(i18n(`commands.sandbox.subcommands.delete.failure.noParentAccount`, {
+                        authCommand: uiCommandReference('hs auth'),
                     }));
-                    process.exit(exitCodes_1.EXIT_CODES.ERROR);
+                    process.exit(EXIT_CODES.ERROR);
                 }
-                parentAccountId = (0, config_1.getAccountId)(parentAccountPrompt.account);
+                parentAccountId = getAccountId(parentAccountPrompt.account);
             }
             else {
-                logger_1.logger.error((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.noParentAccount`, {
-                    authCommand: (0, ui_1.uiCommandReference)('hs auth'),
+                logger.error(i18n(`commands.sandbox.subcommands.delete.failure.noParentAccount`, {
+                    authCommand: uiCommandReference('hs auth'),
                 }));
-                process.exit(exitCodes_1.EXIT_CODES.ERROR);
+                process.exit(EXIT_CODES.ERROR);
             }
         }
     }
     const url = `${baseUrl}/sandboxes/${parentAccountId}`;
-    const command = `hs auth ${(0, config_1.getEnv)(sandboxAccountId) === 'qa' ? '--qa' : ''} --account=${parentAccountId}`;
-    if (parentAccountId && !(0, config_1.getAccountId)(parentAccountId)) {
-        logger_1.logger.log('');
-        logger_1.logger.error((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.noParentPortalAvailable`, {
+    const command = `hs auth ${getEnv(sandboxAccountId) === 'qa' ? '--qa' : ''} --account=${parentAccountId}`;
+    if (parentAccountId && !getAccountId(parentAccountId)) {
+        logger.log('');
+        logger.error(i18n(`commands.sandbox.subcommands.delete.failure.noParentPortalAvailable`, {
             parentAccountId,
             url,
             command,
         }));
-        logger_1.logger.log('');
-        process.exit(exitCodes_1.EXIT_CODES.ERROR);
+        logger.log('');
+        process.exit(EXIT_CODES.ERROR);
     }
-    logger_1.logger.debug((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.debug.deleting`, {
-        account: (0, ui_1.uiAccountDescription)(sandboxAccountId),
+    logger.debug(i18n(`commands.sandbox.subcommands.delete.debug.deleting`, {
+        account: uiAccountDescription(sandboxAccountId),
     }));
     if (isDefaultAccount) {
-        logger_1.logger.info((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.defaultAccountWarning`, {
-            account: (0, ui_1.uiAccountDescription)(sandboxAccountId),
+        logger.info(i18n(`commands.sandbox.subcommands.delete.defaultAccountWarning`, {
+            account: uiAccountDescription(sandboxAccountId),
         }));
-        logger_1.logger.log('');
+        logger.log('');
     }
     try {
         if (!force) {
-            const { confirmSandboxDeletePrompt: confirmed } = await (0, promptUtils_1.promptUser)([
+            const { confirmSandboxDeletePrompt: confirmed } = await promptUser([
                 {
                     name: 'confirmSandboxDeletePrompt',
                     type: 'confirm',
-                    message: (0, lang_1.i18n)(`commands.sandbox.subcommands.delete.confirm`, {
-                        account: (0, ui_1.uiAccountDescription)(sandboxAccountId),
+                    message: i18n(`commands.sandbox.subcommands.delete.confirm`, {
+                        account: uiAccountDescription(sandboxAccountId),
                     }),
                 },
             ]);
             if (!confirmed) {
-                process.exit(exitCodes_1.EXIT_CODES.SUCCESS);
+                process.exit(EXIT_CODES.SUCCESS);
             }
         }
-        await (0, sandboxHubs_1.deleteSandbox)(parentAccountId, sandboxAccountId);
+        await deleteSandbox(parentAccountId, sandboxAccountId);
         const deleteKey = isDefaultAccount
             ? `commands.sandbox.subcommands.delete.success.deleteDefault`
             : `commands.sandbox.subcommands.delete.success.delete`;
-        logger_1.logger.log('');
-        logger_1.logger.success((0, lang_1.i18n)(deleteKey, {
-            account: providedAccountId || accountPrompt.account,
+        logger.log('');
+        logger.success(i18n(deleteKey, {
+            account: userProvidedAccount || accountPrompt.account,
             sandboxHubId: sandboxAccountId || '',
         }));
-        logger_1.logger.log('');
-        const promptDefaultAccount = (0, config_1.removeSandboxAccountFromConfig)(sandboxAccountId);
+        logger.log('');
+        const promptDefaultAccount = removeSandboxAccountFromConfig(sandboxAccountId);
         if (promptDefaultAccount && !force) {
-            const newDefaultAccount = await (0, accountsPrompt_1.selectAccountFromConfig)();
-            (0, config_1.updateDefaultAccount)(newDefaultAccount);
+            const newDefaultAccount = await selectAccountFromConfig();
+            updateDefaultAccount(newDefaultAccount);
         }
-        else {
+        else if (isDefaultAccount && force) {
             // If force is specified, skip prompt and set the parent account id as the default account
-            (0, config_1.updateDefaultAccount)(parentAccountId);
+            updateDefaultAccount(parentAccountId);
         }
-        process.exit(exitCodes_1.EXIT_CODES.SUCCESS);
+        process.exit(EXIT_CODES.SUCCESS);
     }
     catch (err) {
-        (0, index_2.debugError)(err);
-        if ((0, index_1.isSpecifiedError)(err, { statusCode: 401 })) {
+        debugError(err);
+        if (isSpecifiedError(err, { statusCode: 401 })) {
             // Intercept invalid key error
             // This command uses the parent portal PAK to delete a sandbox, so we must specify which account needs a new key
-            logger_1.logger.log('');
-            logger_1.logger.error((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.invalidKey`, {
-                account: (0, ui_1.uiAccountDescription)(parentAccountId),
-                authCommand: (0, ui_1.uiCommandReference)('hs auth'),
+            logger.log('');
+            logger.error(i18n(`commands.sandbox.subcommands.delete.failure.invalidKey`, {
+                account: uiAccountDescription(parentAccountId),
+                authCommand: uiCommandReference('hs auth'),
             }));
         }
-        else if ((0, index_1.isSpecifiedError)(err, {
+        else if (isSpecifiedError(err, {
             statusCode: 403,
             category: 'BANNED',
             subCategory: 'SandboxErrors.USER_ACCESS_NOT_ALLOWED',
         })) {
-            logger_1.logger.log('');
-            logger_1.logger.error((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.invalidUser`, {
-                accountName: (0, ui_1.uiAccountDescription)(sandboxAccountId),
-                parentAccountName: (0, ui_1.uiAccountDescription)(parentAccountId),
+            logger.log('');
+            logger.error(i18n(`commands.sandbox.subcommands.delete.failure.invalidUser`, {
+                accountName: uiAccountDescription(sandboxAccountId),
+                parentAccountName: uiAccountDescription(parentAccountId),
             }));
-            logger_1.logger.log('');
+            logger.log('');
         }
-        else if ((0, index_1.isSpecifiedError)(err, {
+        else if (isSpecifiedError(err, {
             statusCode: 404,
             category: 'OBJECT_NOT_FOUND',
             subCategory: 'SandboxErrors.SANDBOX_NOT_FOUND',
         })) {
-            logger_1.logger.log('');
-            logger_1.logger.warn((0, lang_1.i18n)(`commands.sandbox.subcommands.delete.failure.objectNotFound`, {
-                account: (0, ui_1.uiAccountDescription)(sandboxAccountId),
+            logger.log('');
+            logger.warn(i18n(`commands.sandbox.subcommands.delete.failure.objectNotFound`, {
+                account: uiAccountDescription(sandboxAccountId),
             }));
-            logger_1.logger.log('');
-            const promptDefaultAccount = (0, config_1.removeSandboxAccountFromConfig)(sandboxAccountId);
+            logger.log('');
+            const promptDefaultAccount = removeSandboxAccountFromConfig(sandboxAccountId);
             if (promptDefaultAccount && !force) {
-                const newDefaultAccount = await (0, accountsPrompt_1.selectAccountFromConfig)();
-                (0, config_1.updateDefaultAccount)(newDefaultAccount);
+                const newDefaultAccount = await selectAccountFromConfig();
+                updateDefaultAccount(newDefaultAccount);
             }
             else {
                 // If force is specified, skip prompt and set the parent account id as the default account
-                (0, config_1.updateDefaultAccount)(parentAccountId);
+                updateDefaultAccount(parentAccountId);
             }
-            process.exit(exitCodes_1.EXIT_CODES.SUCCESS);
+            process.exit(EXIT_CODES.SUCCESS);
         }
         else {
-            (0, index_2.logError)(err);
+            logError(err);
         }
-        process.exit(exitCodes_1.EXIT_CODES.ERROR);
+        process.exit(EXIT_CODES.ERROR);
     }
 }
 function sandboxDeleteBuilder(yargs) {
     yargs.option('account', {
-        describe: (0, lang_1.i18n)(`commands.sandbox.subcommands.delete.options.account.describe`),
+        describe: i18n(`commands.sandbox.subcommands.delete.options.account.describe`),
         type: 'string',
     });
     yargs.option('force', {
         type: 'boolean',
         alias: 'f',
-        describe: (0, lang_1.i18n)(`commands.sandbox.subcommands.delete.options.force.describe`),
+        describe: i18n(`commands.sandbox.subcommands.delete.options.force.describe`),
     });
     yargs.example([
         [
             '$0 sandbox delete --account=MySandboxAccount',
-            (0, lang_1.i18n)(`commands.sandbox.subcommands.delete.examples.default`),
+            i18n(`commands.sandbox.subcommands.delete.examples.default`),
         ],
     ]);
     return yargs;
 }
-const builder = (0, yargsUtils_1.makeYargsBuilder)(sandboxDeleteBuilder, command, describe, {
+const builder = makeYargsBuilder(sandboxDeleteBuilder, command, describe, {
     useGlobalOptions: true,
     useConfigOptions: true,
     useAccountOptions: true,
@@ -210,4 +208,4 @@ const sandboxDeleteCommand = {
     handler,
     builder,
 };
-exports.default = sandboxDeleteCommand;
+export default sandboxDeleteCommand;

package/commands/sandbox.d.ts

@@ -1,3 +1,3 @@
-import { YargsCommandModuleBucket } from '../types/Yargs';
+import { YargsCommandModuleBucket } from '../types/Yargs.js';
 declare const sandboxCommand: YargsCommandModuleBucket;
 export default sandboxCommand;

package/commands/sandbox.js

@@ -1,26 +1,19 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-const lang_1 = require("../lib/lang");
-const ui_1 = require("../lib/ui");
-const create_1 = __importDefault(require("./sandbox/create"));
-const delete_1 = __importDefault(require("./sandbox/delete"));
-const yargsUtils_1 = require("../lib/yargsUtils");
+import { i18n } from '../lib/lang.js';
+import { uiBetaTag } from '../lib/ui/index.js';
+import create from './sandbox/create.js';
+import del from './sandbox/delete.js';
+import { makeYargsBuilder } from '../lib/yargsUtils.js';
 const command = ['sandbox', 'sandboxes'];
-const describe = (0, ui_1.uiBetaTag)((0, lang_1.i18n)(`commands.sandbox.describe`), false);
+const describe = uiBetaTag(i18n(`commands.sandbox.describe`), false);
 function sandboxBuilder(yargs) {
-    yargs.command(create_1.default).command(delete_1.default).demandCommand(1, '');
+    yargs.command(create).command(del).demandCommand(1, '');
     return yargs;
 }
-const builder = (0, yargsUtils_1.makeYargsBuilder)(sandboxBuilder, command, describe);
+const builder = makeYargsBuilder(sandboxBuilder, command, describe);
 const sandboxCommand = {
     command,
     describe,
     builder,
     handler: () => { },
 };
-exports.default = sandboxCommand;
-// TODO Remove this legacy export once we've migrated all commands to TS
-module.exports = sandboxCommand;
+export default sandboxCommand;

package/commands/secret/__tests__/addSecret.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/secret/__tests__/addSecret.test.js

@@ -0,0 +1,29 @@
+import yargs from 'yargs';
+import { addConfigOptions, addAccountOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
+import addSecretCommand from '../addSecret.js';
+vi.mock('../../../lib/commonOpts');
+describe('commands/secret/addSecret', () => {
+    const yargsMock = yargs;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(addSecretCommand.command).toEqual('add [name]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(addSecretCommand.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            addSecretCommand.builder(yargsMock);
+            expect(yargsMock.positional).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/secret/__tests__/deleteSecret.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/secret/__tests__/deleteSecret.test.js

@@ -0,0 +1,41 @@
+import yargs from 'yargs';
+import { addConfigOptions, addAccountOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
+import deleteSecretCommand from '../deleteSecret.js';
+vi.mock('yargs');
+vi.mock('../../../lib/commonOpts');
+describe('commands/secret/deleteSecret', () => {
+    let yargsMock = yargs;
+    beforeEach(() => {
+        yargsMock = {
+            positional: vi.fn().mockReturnThis(),
+            options: vi.fn().mockReturnThis(),
+            command: vi.fn().mockReturnThis(),
+            demandCommand: vi.fn().mockReturnThis(),
+            help: vi.fn().mockReturnThis(),
+            alias: vi.fn().mockReturnThis(),
+            argv: {},
+        };
+    });
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(deleteSecretCommand.command).toEqual('delete [name]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(deleteSecretCommand.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            deleteSecretCommand.builder(yargsMock);
+            expect(yargsMock.positional).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/secret/__tests__/listSecret.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/secret/__tests__/listSecret.test.js

@@ -0,0 +1,29 @@
+import yargs from 'yargs';
+import { addConfigOptions, addAccountOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
+import listSecretCommand from '../listSecret.js';
+vi.mock('yargs');
+vi.mock('../../../lib/commonOpts');
+describe('commands/secret/listSecret', () => {
+    const yargsMock = yargs;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(listSecretCommand.command).toEqual('list');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(listSecretCommand.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            listSecretCommand.builder(yargsMock);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/secret/__tests__/updateSecret.test.d.ts

@@ -0,0 +1 @@
+export {};

package/commands/secret/__tests__/updateSecret.test.js

@@ -0,0 +1,29 @@
+import yargs from 'yargs';
+import { addConfigOptions, addAccountOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
+import updateSecretCommand from '../updateSecret.js';
+vi.mock('../../../lib/commonOpts');
+describe('commands/secret/updateSecret', () => {
+    const yargsMock = yargs;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(updateSecretCommand.command).toEqual('update [name]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(updateSecretCommand.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            updateSecretCommand.builder(yargsMock);
+            expect(yargsMock.positional).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/secret/addSecret.d.ts

@@ -1,4 +1,4 @@
-import { CommonArgs, ConfigArgs, AccountArgs, EnvironmentArgs, YargsCommandModule } from '../../types/Yargs';
+import { CommonArgs, ConfigArgs, AccountArgs, EnvironmentArgs, YargsCommandModule } from '../../types/Yargs.js';
 type AddSecretArgs = CommonArgs & ConfigArgs & AccountArgs & EnvironmentArgs & {
     name?: string;
 };

package/commands/secret/addSecret.js

@@ -1,46 +1,44 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-const logger_1 = require("@hubspot/local-dev-lib/logger");
-const secrets_1 = require("@hubspot/local-dev-lib/api/secrets");
-const index_1 = require("../../lib/errorHandlers/index");
-const usageTracking_1 = require("../../lib/usageTracking");
-const ui_1 = require("../../lib/ui");
-const secretPrompt_1 = require("../../lib/prompts/secretPrompt");
-const lang_1 = require("../../lib/lang");
-const exitCodes_1 = require("../../lib/enums/exitCodes");
-const ui_2 = require("../../lib/ui");
-const yargsUtils_1 = require("../../lib/yargsUtils");
+import { logger } from '@hubspot/local-dev-lib/logger';
+import { addSecret, fetchSecrets } from '@hubspot/local-dev-lib/api/secrets';
+import { logError, ApiErrorContext } from '../../lib/errorHandlers/index.js';
+import { trackCommandUsage } from '../../lib/usageTracking.js';
+import { uiAccountDescription } from '../../lib/ui/index.js';
+import { secretValuePrompt, secretNamePrompt, } from '../../lib/prompts/secretPrompt.js';
+import { i18n } from '../../lib/lang.js';
+import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import { uiCommandReference } from '../../lib/ui/index.js';
+import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'add [name]';
-const describe = (0, lang_1.i18n)(`commands.secret.subcommands.add.describe`);
+const describe = i18n(`commands.secret.subcommands.add.describe`);
 async function handler(args) {
     const { name, derivedAccountId } = args;
     let secretName = name;
-    (0, usageTracking_1.trackCommandUsage)('secrets-add', {}, derivedAccountId);
+    trackCommandUsage('secrets-add', {}, derivedAccountId);
     try {
         if (!secretName) {
-            const { secretName: name } = await (0, secretPrompt_1.secretNamePrompt)();
+            const { secretName: name } = await secretNamePrompt();
             secretName = name;
         }
-        const { data: { results: secrets }, } = await (0, secrets_1.fetchSecrets)(derivedAccountId);
+        const { data: { results: secrets }, } = await fetchSecrets(derivedAccountId);
         if (secrets.includes(secretName)) {
-            logger_1.logger.error((0, lang_1.i18n)(`commands.secret.subcommands.add.errors.alreadyExists`, {
+            logger.error(i18n(`commands.secret.subcommands.add.errors.alreadyExists`, {
                 secretName,
-                command: (0, ui_2.uiCommandReference)('hs secret update'),
+                command: uiCommandReference('hs secret update'),
             }));
-            process.exit(exitCodes_1.EXIT_CODES.ERROR);
+            process.exit(EXIT_CODES.ERROR);
         }
-        const { secretValue } = await (0, secretPrompt_1.secretValuePrompt)();
-        await (0, secrets_1.addSecret)(derivedAccountId, secretName, secretValue);
-        logger_1.logger.success((0, lang_1.i18n)(`commands.secret.subcommands.add.success.add`, {
-            accountIdentifier: (0, ui_1.uiAccountDescription)(derivedAccountId),
+        const { secretValue } = await secretValuePrompt();
+        await addSecret(derivedAccountId, secretName, secretValue);
+        logger.success(i18n(`commands.secret.subcommands.add.success.add`, {
+            accountIdentifier: uiAccountDescription(derivedAccountId),
             secretName,
         }));
     }
     catch (err) {
-        logger_1.logger.error((0, lang_1.i18n)(`commands.secret.subcommands.add.errors.add`, {
+        logger.error(i18n(`commands.secret.subcommands.add.errors.add`, {
             secretName: secretName || '',
         }));
-        (0, index_1.logError)(err, new index_1.ApiErrorContext({
+        logError(err, new ApiErrorContext({
             request: 'add secret',
             accountId: derivedAccountId,
         }));
@@ -48,12 +46,12 @@ async function handler(args) {
 }
 function addSecretBuilder(yargs) {
     yargs.positional('name', {
-        describe: (0, lang_1.i18n)(`commands.secret.subcommands.add.positionals.name.describe`),
+        describe: i18n(`commands.secret.subcommands.add.positionals.name.describe`),
         type: 'string',
     });
     return yargs;
 }
-const builder = (0, yargsUtils_1.makeYargsBuilder)(addSecretBuilder, command, describe, {
+const builder = makeYargsBuilder(addSecretBuilder, command, describe, {
     useGlobalOptions: true,
     useConfigOptions: true,
     useAccountOptions: true,
@@ -65,4 +63,4 @@ const addSecretCommand = {
     handler,
     builder,
 };
-exports.default = addSecretCommand;
+export default addSecretCommand;

package/commands/secret/deleteSecret.d.ts

@@ -1,4 +1,4 @@
-import { CommonArgs, ConfigArgs, AccountArgs, EnvironmentArgs, YargsCommandModule } from '../../types/Yargs';
+import { CommonArgs, ConfigArgs, AccountArgs, EnvironmentArgs, YargsCommandModule } from '../../types/Yargs.js';
 type DeleteSecretArgs = CommonArgs & ConfigArgs & AccountArgs & EnvironmentArgs & {
     name?: string;
     force?: boolean;