@hubspot/cli 8.3.0 → 8.4.0-beta.0

package/commands/sandbox/create.js

@@ -7,30 +7,29 @@ import { commands, lib } from '../../lang/en.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { uiFeatureHighlight, uiBetaTag } from '../../lib/ui/index.js';
 import { SANDBOX_TYPE_MAP, validateSandboxUsageLimits, } from '../../lib/sandboxes.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { sandboxTypePrompt } from '../../lib/prompts/sandboxesPrompt.js';
 import { promptUser } from '../../lib/prompts/promptUtils.js';
 import { logError } from '../../lib/errorHandlers/index.js';
 import { buildV2Sandbox } from '../../lib/buildAccount.js';
 import { hubspotAccountNamePrompt } from '../../lib/prompts/accountNamePrompt.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'create';
 const describe = uiBetaTag(commands.sandbox.subcommands.create.describe, false);
 async function handler(args) {
-    const { name, type, force, derivedAccountId } = args;
+    const { name, type, force, derivedAccountId, exit } = args;
     const accountConfig = getConfigAccountById(derivedAccountId);
     const env = getConfigAccountEnvironment(derivedAccountId);
-    trackCommandUsage('sandbox-create', {}, derivedAccountId);
     // Check if account config exists
     if (!accountConfig) {
         uiLogger.error(commands.sandbox.subcommands.create.failure.noAccountConfig(derivedAccountId));
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     // Default account is not a production portal
     if (accountConfig.accountType &&
         accountConfig.accountType !== HUBSPOT_ACCOUNT_TYPES.STANDARD) {
         uiLogger.error(commands.sandbox.subcommands.create.failure.invalidAccountType(HUBSPOT_ACCOUNT_TYPE_STRINGS[HUBSPOT_ACCOUNT_TYPES[accountConfig.accountType]], accountConfig.name || ''));
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     let typePrompt;
     let namePrompt;
@@ -40,7 +39,7 @@ async function handler(args) {
         }
         else {
             uiLogger.error(commands.sandbox.subcommands.create.failure.optionMissing.type);
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
     }
     const sandboxType = type
@@ -64,7 +63,7 @@ async function handler(args) {
         else {
             logError(err);
         }
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     if (!name) {
         if (!force) {
@@ -72,14 +71,15 @@ async function handler(args) {
         }
         else {
             uiLogger.error(commands.sandbox.subcommands.create.failure.optionMissing.name);
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
     }
-    const sandboxName = name || (namePrompt && namePrompt.name);
-    if (!sandboxName) {
+    const maybeSandboxName = name || (namePrompt && namePrompt.name);
+    if (!maybeSandboxName) {
         uiLogger.error(commands.sandbox.subcommands.create.failure.optionMissing.name);
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
+    const sandboxName = maybeSandboxName;
     let contactRecordsSyncPromptResult = false;
     if (!force) {
         const isStandardSandbox = sandboxType === HUBSPOT_ACCOUNT_TYPES.STANDARD_SANDBOX;
@@ -101,7 +101,7 @@ async function handler(args) {
         // Check if sandbox account config exists
         if (!sandboxAccountConfig) {
             uiLogger.error(commands.sandbox.subcommands.create.failure.noSandboxAccountConfig(result.sandbox.sandboxHubId));
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
         const highlightItems = ['accountsUseCommand', 'projectCreateCommand'];
         if (sandboxType === HUBSPOT_ACCOUNT_TYPES.DEVELOPMENT_SANDBOX) {
@@ -111,11 +111,11 @@ async function handler(args) {
             highlightItems.push('projectUploadCommand');
         }
         uiFeatureHighlight(highlightItems);
-        process.exit(EXIT_CODES.SUCCESS);
+        return exit(EXIT_CODES.SUCCESS);
     }
     catch (error) {
         // Errors are logged in util functions
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
 }
 function sandboxCreateBuilder(yargs) {
@@ -150,7 +150,7 @@ const builder = makeYargsBuilder(sandboxCreateBuilder, command, describe, {
 const sandboxCreateCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('sandbox-create', handler),
     builder,
 };
 export default sandboxCreateCommand;

package/commands/sandbox/delete.js

@@ -4,7 +4,6 @@ import { deleteSandbox } from '@hubspot/local-dev-lib/api/sandboxHubs';
 import { getConfigAccountEnvironment, removeAccountFromConfig, setConfigAccountAsDefault, getAllConfigAccounts, getConfigAccountIfExists, getConfigDefaultAccountIfExists, } from '@hubspot/local-dev-lib/config';
 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 { commands } from '../../lang/en.js';
 import { deleteSandboxPrompt } from '../../lib/prompts/sandboxesPrompt.js';
@@ -12,12 +11,12 @@ import { selectAccountFromConfig } from '../../lib/prompts/accountsPrompt.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { promptUser } from '../../lib/prompts/promptUtils.js';
 import { uiAuthCommandReference, uiBetaTag } from '../../lib/ui/index.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'delete';
 const describe = uiBetaTag(commands.sandbox.subcommands.delete.describe, false);
 async function handler(args) {
-    const { userProvidedAccount, derivedAccountId, force } = args;
-    trackCommandUsage('sandbox-delete', {}, derivedAccountId);
+    const { userProvidedAccount, force, exit } = args;
     let accountPrompt;
     if (!userProvidedAccount) {
         if (!force) {
@@ -27,19 +26,19 @@ async function handler(args) {
             // Account is required, throw error if force flag is present and no account is specified
             uiLogger.log('');
             uiLogger.error(commands.sandbox.subcommands.delete.failure.noAccount);
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
         if (!accountPrompt) {
             uiLogger.log('');
             uiLogger.error(commands.sandbox.subcommands.delete.failure.noSandboxAccounts);
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
     }
     const accountIdentifier = userProvidedAccount || accountPrompt.account;
     const sandboxAccount = getConfigAccountIfExists(accountIdentifier);
     if (!sandboxAccount) {
         uiLogger.error(commands.sandbox.subcommands.delete.failure.noSandboxAccountId);
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     const sandboxAccountId = sandboxAccount.accountId;
     const defaultAccount = getConfigDefaultAccountIfExists();
@@ -56,14 +55,14 @@ async function handler(args) {
                 const parentAccountPrompt = await deleteSandboxPrompt(true);
                 if (!parentAccountPrompt) {
                     uiLogger.error(commands.sandbox.subcommands.delete.failure.noParentAccount);
-                    process.exit(EXIT_CODES.ERROR);
+                    return exit(EXIT_CODES.ERROR);
                 }
                 const parentAccount = getConfigAccountIfExists(parentAccountPrompt.account);
                 parentAccountId = parentAccount?.accountId;
             }
             else {
                 uiLogger.error(commands.sandbox.subcommands.delete.failure.noParentAccount);
-                process.exit(EXIT_CODES.ERROR);
+                return exit(EXIT_CODES.ERROR);
             }
         }
     }
@@ -76,7 +75,7 @@ async function handler(args) {
         uiLogger.log('');
         uiLogger.error(commands.sandbox.subcommands.delete.failure.noParentPortalAvailable(command, url));
         uiLogger.log('');
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     uiLogger.debug(commands.sandbox.subcommands.delete.debug.deleting(sandboxAccountId));
     if (isDefaultAccount) {
@@ -93,7 +92,7 @@ async function handler(args) {
                 },
             ]);
             if (!confirmed) {
-                process.exit(EXIT_CODES.SUCCESS);
+                return exit(EXIT_CODES.SUCCESS);
             }
         }
         await deleteSandbox(parentAccountId, sandboxAccountId);
@@ -111,7 +110,7 @@ async function handler(args) {
             // If force is specified, skip prompt and set the parent account id as the default account
             setConfigAccountAsDefault(parentAccountId);
         }
-        process.exit(EXIT_CODES.SUCCESS);
+        return exit(EXIT_CODES.SUCCESS);
     }
     catch (err) {
         debugError(err);
@@ -147,12 +146,12 @@ async function handler(args) {
                 // If force is specified, skip prompt and set the parent account id as the default account
                 setConfigAccountAsDefault(parentAccountId);
             }
-            process.exit(EXIT_CODES.SUCCESS);
+            return exit(EXIT_CODES.SUCCESS);
         }
         else {
             logError(err);
         }
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
 }
 function sandboxDeleteBuilder(yargs) {
@@ -183,7 +182,7 @@ const builder = makeYargsBuilder(sandboxDeleteBuilder, command, describe, {
 const sandboxDeleteCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('sandbox-delete', handler),
     builder,
 };
 export default sandboxDeleteCommand;

package/commands/secret/addSecret.js

@@ -1,17 +1,16 @@
 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 { secretValuePrompt, secretNamePrompt, } from '../../lib/prompts/secretPrompt.js';
 import { commands } from '../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'add [name]';
 const describe = commands.secret.subcommands.add.describe;
 async function handler(args) {
-    const { name, derivedAccountId } = args;
+    const { name, derivedAccountId, exit } = args;
     let secretName = name;
-    trackCommandUsage('secrets-add', {}, derivedAccountId);
     try {
         if (!secretName) {
             const { secretName: name } = await secretNamePrompt();
@@ -20,7 +19,7 @@ async function handler(args) {
         const { data: { results: secrets }, } = await fetchSecrets(derivedAccountId);
         if (secrets.includes(secretName)) {
             uiLogger.error(commands.secret.subcommands.add.errors.alreadyExists(secretName));
-            return process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
         const { secretValue } = await secretValuePrompt();
         await addSecret(derivedAccountId, secretName, secretValue);
@@ -32,9 +31,9 @@ async function handler(args) {
             request: 'add secret',
             accountId: derivedAccountId,
         }));
-        return process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
-    process.exit(EXIT_CODES.SUCCESS);
+    return exit(EXIT_CODES.SUCCESS);
 }
 function addSecretBuilder(yargs) {
     yargs.positional('name', {
@@ -52,7 +51,7 @@ const builder = makeYargsBuilder(addSecretBuilder, command, describe, {
 const addSecretCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('secrets-add', handler),
     builder,
 };
 export default addSecretCommand;

package/commands/secret/deleteSecret.js

@@ -3,21 +3,20 @@ import { secretListPrompt } from '../../lib/prompts/secretPrompt.js';
 import { confirmPrompt } from '../../lib/prompts/promptUtils.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { ApiErrorContext, logError } from '../../lib/errorHandlers/index.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { commands } from './../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'delete [name]';
 const describe = commands.secret.subcommands.delete.describe;
 async function handler(args) {
-    const { name, derivedAccountId, force } = args;
+    const { name, derivedAccountId, force, exit } = args;
     let secretName = name;
-    trackCommandUsage('secrets-delete', {}, derivedAccountId);
     try {
         const { data: { results: secrets }, } = await fetchSecrets(derivedAccountId);
         if (secretName && !secrets.includes(secretName)) {
             uiLogger.error(commands.secret.subcommands.delete.errors.noSecret(secretName));
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
         if (!secretName) {
             const { secretToModify } = await secretListPrompt(secrets, commands.secret.subcommands.delete.selectSecret);
@@ -29,7 +28,7 @@ async function handler(args) {
             }));
         if (!confirmDelete) {
             uiLogger.success(commands.secret.subcommands.delete.deleteCanceled);
-            process.exit(EXIT_CODES.SUCCESS);
+            return exit(EXIT_CODES.SUCCESS);
         }
         await deleteSecret(derivedAccountId, secretName);
         uiLogger.success(commands.secret.subcommands.delete.success.delete(secretName, derivedAccountId));
@@ -65,7 +64,7 @@ const builder = makeYargsBuilder(deleteSecretBuilder, command, describe, {
 const deleteSecretCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('secrets-delete', handler),
     builder,
 };
 export default deleteSecretCommand;

package/commands/secret/listSecret.js

@@ -1,15 +1,14 @@
 import { 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 { commands } from '../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'list';
 const describe = commands.secret.subcommands.list.describe;
 async function handler(args) {
     const { derivedAccountId } = args;
-    trackCommandUsage('secrets-list', {}, derivedAccountId);
     try {
         const { data: { results }, } = await fetchSecrets(derivedAccountId);
         const groupLabel = commands.secret.subcommands.list.groupLabel(uiAccountDescription(derivedAccountId));
@@ -37,7 +36,7 @@ const builder = makeYargsBuilder(listSecretBuilder, command, describe, {
 const listSecretCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('secrets-list', handler),
     builder,
 };
 export default listSecretCommand;

package/commands/secret/updateSecret.js

@@ -1,22 +1,21 @@
 import { updateSecret, fetchSecrets } from '@hubspot/local-dev-lib/api/secrets';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { ApiErrorContext, logError } from '../../lib/errorHandlers/index.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { secretValuePrompt, secretListPrompt, } from '../../lib/prompts/secretPrompt.js';
 import { commands } from '../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'update [name]';
 const describe = commands.secret.subcommands.update.describe;
 async function handler(args) {
-    const { name, derivedAccountId } = args;
+    const { name, derivedAccountId, exit } = args;
     let secretName = name;
-    trackCommandUsage('secrets-update', {}, derivedAccountId);
     try {
         const { data: { results: secrets }, } = await fetchSecrets(derivedAccountId);
         if (secretName && !secrets.includes(secretName)) {
             uiLogger.error(commands.secret.subcommands.update.errors.noSecret(secretName));
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
         if (!secretName) {
             const { secretToModify } = await secretListPrompt(secrets, commands.secret.subcommands.update.selectSecret);
@@ -51,7 +50,7 @@ const builder = makeYargsBuilder(updateSecretBuilder, command, describe, {
 const updateSecretCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('secrets-update', handler),
     builder,
 };
 export default updateSecretCommand;

package/commands/testAccount/create.d.ts

@@ -1,5 +1,5 @@
 import { AccountLevel } from '@hubspot/local-dev-lib/types/developerTestAccounts.js';
-import { CommonArgs, ConfigArgs, AccountArgs, TestingArgs, YargsCommandModule, EnvironmentArgs, JSONOutputArgs } from '../../types/Yargs.js';
+import { CommonArgs, ConfigArgs, AccountArgs, TestingArgs, EnvironmentArgs, JSONOutputArgs, YargsCommandModule } from '../../types/Yargs.js';
 type CreateTestAccountArgs = CommonArgs & AccountArgs & ConfigArgs & TestingArgs & EnvironmentArgs & JSONOutputArgs & {
     configPath?: string;
     name?: string;

package/commands/testAccount/create.js

@@ -3,16 +3,16 @@ import path from 'path';
 import { getValidEnv } from '@hubspot/local-dev-lib/environment';
 import { getConfigAccountEnvironment } from '@hubspot/local-dev-lib/config';
 import { getCwd } from '@hubspot/local-dev-lib/path';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { promptUser, listPrompt } from '../../lib/prompts/promptUtils.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { uiLogger } from '../../lib/ui/logger.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { fileExists } from '../../lib/validation.js';
 import { commands } from '../../lang/en.js';
 import { createDeveloperTestAccountConfigPrompt } from '../../lib/prompts/createDeveloperTestAccountConfigPrompt.js';
 import { debugError, logError } from '../../lib/errorHandlers/index.js';
-import { PromptExitError } from '../../lib/errors/PromptExitError.js';
+import { isPromptExitError } from '../../lib/errors/PromptExitError.js';
 import SpinniesManager from '../../lib/ui/SpinniesManager.js';
 import { createDeveloperTestAccountV2, saveAccountToConfig, } from '../../lib/buildAccount.js';
 import { ACCOUNT_LEVEL_CHOICES, ACCOUNT_LEVELS } from '../../lib/constants.js';
@@ -29,18 +29,16 @@ function hasAnyFlags(args) {
         contentLevel ||
         commerceLevel);
 }
-async function readConfigFile(configPath) {
+function readConfigFile(configPath) {
     const absoluteConfigPath = path.resolve(getCwd(), configPath);
     if (!fileExists(absoluteConfigPath)) {
-        uiLogger.error(commands.testAccount.create.errors.configFileNotFound(absoluteConfigPath));
-        process.exit(EXIT_CODES.ERROR);
+        throw new Error(commands.testAccount.create.errors.configFileNotFound(absoluteConfigPath));
     }
     try {
         return JSON.parse(fs.readFileSync(absoluteConfigPath, 'utf8'));
     }
     catch (err) {
-        uiLogger.error(commands.testAccount.create.errors.configFileParseFailed(absoluteConfigPath));
-        process.exit(EXIT_CODES.ERROR);
+        throw new Error(commands.testAccount.create.errors.configFileParseFailed(absoluteConfigPath));
     }
 }
 async function promptForConfigPath() {
@@ -90,10 +88,16 @@ async function buildTestAccountConfig(args) {
     });
 }
 async function handler(args) {
-    const { derivedAccountId, formatOutputAsJson } = args;
-    trackCommandUsage('test-account-create', {}, derivedAccountId);
+    const { derivedAccountId, formatOutputAsJson, exit } = args;
     const env = getValidEnv(getConfigAccountEnvironment(derivedAccountId));
-    const testAccountConfig = await buildTestAccountConfig(args);
+    let testAccountConfig;
+    try {
+        testAccountConfig = await buildTestAccountConfig(args);
+    }
+    catch (error) {
+        logError(error);
+        return exit(EXIT_CODES.ERROR);
+    }
     const resultJson = {};
     SpinniesManager.init({
         succeedColor: 'white',
@@ -112,7 +116,7 @@ async function handler(args) {
         SpinniesManager.fail('createTestAccount', {
             text: commands.testAccount.create.polling.createFailure,
         });
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     SpinniesManager.succeed('createTestAccount', {
         text: commands.testAccount.create.polling.success(testAccountConfig.accountName, resultJson.accountId, derivedAccountId),
@@ -131,15 +135,15 @@ async function handler(args) {
             }
         }
         catch (e) {
-            if (e instanceof PromptExitError) {
-                process.exit(e.exitCode);
+            if (isPromptExitError(e)) {
+                throw e;
             }
             debugError(e);
             uiLogger.error(commands.testAccount.create.errors.saveAccountToConfigFailure(testAccountConfig.accountName));
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
     }
-    process.exit(EXIT_CODES.SUCCESS);
+    return exit(EXIT_CODES.SUCCESS);
 }
 function createTestAccountBuilder(yargs) {
     yargs.option('config-path', {
@@ -226,7 +230,7 @@ const builder = makeYargsBuilder(createTestAccountBuilder, command, describe, {
 const createTestAccountCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('test-account-create', handler),
     builder,
 };
 export default createTestAccountCommand;

package/commands/testAccount/createConfig.js

@@ -1,19 +1,18 @@
 import fs from 'fs-extra';
 import path from 'path';
 import { getCwd } from '@hubspot/local-dev-lib/path';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { promptUser } from '../../lib/prompts/promptUtils.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { uiLogger } from '../../lib/ui/logger.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { commands } from '../../lang/en.js';
 import { createDeveloperTestAccountConfigPrompt } from '../../lib/prompts/createDeveloperTestAccountConfigPrompt.js';
 import { fileExists } from '../../lib/validation.js';
 const command = 'create-config';
 const describe = commands.testAccount.createConfig.describe;
 async function handler(args) {
-    const { derivedAccountId, name, description, path: configPath } = args;
-    trackCommandUsage('test-account-create-config', {}, derivedAccountId);
+    const { name, description, path: configPath, exit } = args;
     let accountConfigPath = configPath;
     const testAccountConfig = await createDeveloperTestAccountConfigPrompt({
         name,
@@ -43,21 +42,21 @@ async function handler(args) {
     }
     if (!accountConfigPath) {
         uiLogger.error(commands.testAccount.createConfig.errors.pathError);
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     if (fileExists(accountConfigPath)) {
         uiLogger.error(commands.testAccount.createConfig.errors.pathExistsError);
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     try {
         fs.writeFileSync(path.resolve(getCwd(), accountConfigPath), JSON.stringify(testAccountConfig, null, 2), 'utf8');
     }
     catch (err) {
         uiLogger.error(commands.testAccount.createConfig.errors.failedToCreate);
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     uiLogger.success(commands.testAccount.createConfig.success.configFileCreated(accountConfigPath));
-    process.exit(EXIT_CODES.SUCCESS);
+    return exit(EXIT_CODES.SUCCESS);
 }
 function createTestAccountConfigBuilder(yargs) {
     yargs.option('name', {
@@ -86,7 +85,7 @@ const builder = makeYargsBuilder(createTestAccountConfigBuilder, command, descri
 const createTestAccountConfigCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('test-account-create-config', handler),
     builder,
 };
 export default createTestAccountConfigCommand;

package/commands/testAccount/delete.js

@@ -1,12 +1,12 @@
 import { fetchDeveloperTestAccounts, deleteDeveloperTestAccount, } from '@hubspot/local-dev-lib/api/developerTestAccounts';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { uiLogger } from '../../lib/ui/logger.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { commands } from '../../lang/en.js';
 import { removeAccountFromConfig, getConfigAccountById, getConfigAccountIfExists, setConfigAccountAsDefault, getConfigDefaultAccountIfExists, } from '@hubspot/local-dev-lib/config';
 import { promptUser } from '../../lib/prompts/promptUtils.js';
-import { debugError } from '../../lib/errorHandlers/index.js';
+import { debugError, logError } from '../../lib/errorHandlers/index.js';
 const command = 'delete [test-account]';
 const describe = commands.testAccount.delete.describe;
 async function getAccountPromptOptions(derivedAccountId) {
@@ -25,8 +25,7 @@ async function getAccountPromptOptions(derivedAccountId) {
 async function accountToDeleteSelectionPrompt(derivedAccountId) {
     const accountData = await getAccountPromptOptions(derivedAccountId);
     if (accountData.length === 0) {
-        uiLogger.error(commands.testAccount.delete.errors.noAccountsToDelete(derivedAccountId));
-        process.exit(EXIT_CODES.ERROR);
+        throw new Error(commands.testAccount.delete.errors.noAccountsToDelete(derivedAccountId));
     }
     const { testAccountToDelete } = await promptUser([
         {
@@ -79,33 +78,35 @@ async function deleteTestAccountFromConfig(testAccountId, parentAccountName, acc
         uiLogger.error(commands.testAccount.delete.errors.failedToDeleteFromConfig(testAccountId));
     }
 }
-async function validateTestAccountConfigs(testAccountId) {
+function validateTestAccountConfigs(testAccountId) {
     if (!testAccountId) {
-        uiLogger.error(commands.testAccount.delete.errors.testAccountNotFound(testAccountId));
-        process.exit(EXIT_CODES.ERROR);
+        throw new Error(commands.testAccount.delete.errors.testAccountNotFound(testAccountId));
     }
     const testAccountConfig = getConfigAccountById(testAccountId);
     if (!testAccountConfig) {
-        uiLogger.error(commands.testAccount.delete.errors.testAccountNotFound(testAccountId));
-        process.exit(EXIT_CODES.ERROR);
+        throw new Error(commands.testAccount.delete.errors.testAccountNotFound(testAccountId));
     }
     const parentAccountConfig = getConfigAccountById(testAccountConfig.parentAccountId);
     if (!parentAccountConfig) {
-        uiLogger.error(commands.testAccount.delete.errors.parentAccountNotFound(testAccountId));
-        process.exit(EXIT_CODES.ERROR);
+        throw new Error(commands.testAccount.delete.errors.parentAccountNotFound(testAccountId));
     }
     const parentAccountName = parentAccountConfig.name;
     return { testAccountConfig, parentAccountName };
 }
 async function handler(args) {
-    const { derivedAccountId, testAccount, force } = args;
-    trackCommandUsage('test-account-delete', {}, derivedAccountId);
+    const { derivedAccountId, testAccount, force, exit } = args;
     let testAccountIdToDelete = 0;
     // See if the account exists
     if (testAccount) {
         const account = getConfigAccountIfExists(testAccount);
         const accountId = account?.accountId || null;
-        await validateTestAccountConfigs(accountId);
+        try {
+            validateTestAccountConfigs(accountId);
+        }
+        catch (error) {
+            logError(error);
+            return exit(EXIT_CODES.ERROR);
+        }
         if (accountId) {
             testAccountIdToDelete = accountId;
         }
@@ -119,10 +120,18 @@ async function handler(args) {
         catch (err) {
             debugError(err);
             uiLogger.error(commands.testAccount.delete.errors.failedToSelectAccount);
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
     }
-    const { testAccountConfig, parentAccountName } = await validateTestAccountConfigs(testAccountIdToDelete);
+    let testAccountConfig;
+    let parentAccountName;
+    try {
+        ({ testAccountConfig, parentAccountName } = validateTestAccountConfigs(testAccountIdToDelete));
+    }
+    catch (error) {
+        logError(error);
+        return exit(EXIT_CODES.ERROR);
+    }
     // If --force, don't prompt user; else confirm deletion
     let shouldDeleteAccount;
     if (force) {
@@ -139,7 +148,7 @@ async function handler(args) {
     else {
         uiLogger.info(commands.testAccount.delete.info.deletionCanceled);
     }
-    process.exit(EXIT_CODES.SUCCESS);
+    return exit(EXIT_CODES.SUCCESS);
 }
 function deleteTestAccountBuilder(yargs) {
     yargs.positional('test-account', {
@@ -182,7 +191,7 @@ const builder = makeYargsBuilder(deleteTestAccountBuilder, command, describe, {
 const deleteTestAccountCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('test-account-delete', handler),
     builder,
 };
 export default deleteTestAccountCommand;