@hubspot/cli 8.1.0-beta.0 → 8.1.0-experimental.0

package/commands/project/dev/unifiedFlow.js

@@ -3,13 +3,13 @@ import util from 'util';
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
 import { startPortManagerServer, stopPortManagerServer, } from '@hubspot/local-dev-lib/portManager';
 import { isTranslationError, translateForLocalDev, } from '@hubspot/project-parsing-lib/translate';
-import { getConfigAccountEnvironment, getAllConfigAccounts, getConfigAccountById, } from '@hubspot/local-dev-lib/config';
+import { getConfigAccountEnvironment, getLinkedOrAllConfigAccounts, getConfigAccountById, getConfigAccountIfExists, } from '@hubspot/local-dev-lib/config';
 import { logError } from '../../../lib/errorHandlers/index.js';
 import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
 import { ensureProjectExists } from '../../../lib/projects/ensureProjectExists.js';
 import { createInitialBuildForNewProject, createNewProjectForLocalDev, compareLocalProjectToDeployed, checkAndInstallDependencies, } from '../../../lib/projects/localDev/helpers/project.js';
 import { useExistingDevTestAccount, createDeveloperTestAccountForLocalDev, selectAccountTypePrompt, createSandboxForLocalDev, } from '../../../lib/projects/localDev/helpers/account.js';
-import { selectDeveloperTestTargetAccountPrompt, selectSandboxTargetAccountPrompt, } from '../../../lib/prompts/projectDevTargetAccountPrompt.js';
+import { selectDeveloperTestTargetAccountPrompt, selectSandboxTargetAccountPrompt, confirmLinkExistingDeveloperTestAccountPrompt, } from '../../../lib/prompts/projectDevTargetAccountPrompt.js';
 import LocalDevProcess from '../../../lib/projects/localDev/LocalDevProcess.js';
 import LocalDevWatcher from '../../../lib/projects/localDev/LocalDevWatcher.js';
 import { handleExit, handleKeypress } from '../../../lib/process.js';
@@ -17,9 +17,15 @@ import { isTestAccountOrSandbox, isUnifiedAccount, } from '../../../lib/accountT
 import { uiLogger } from '../../../lib/ui/logger.js';
 import { commands } from '../../../lang/en.js';
 import LocalDevWebsocketServer from '../../../lib/projects/localDev/LocalDevWebsocketServer.js';
-import { confirmLocalDevIsNotRunning } from '../../../lib/projects/localDev/helpers/process.js';
+import { isLocalDevRunning } from '../../../lib/projects/localDev/helpers/process.js';
+import { getHsSettingsFileIfExists, getHsSettingsFilePath, } from '@hubspot/local-dev-lib/config/hsSettings';
+import { isDirectoryLinked, addAccountToLinkedSettings, } from '../../../lib/link/linkUtils.js';
 export async function unifiedProjectDevFlow({ args, targetProjectAccountId, providedTargetTestingAccountId, projectConfig, projectDir, }) {
-    await confirmLocalDevIsNotRunning();
+    const { exit } = args;
+    if (await isLocalDevRunning()) {
+        uiLogger.error(commands.project.dev.errors.localDevAlreadyRunning);
+        return exit(EXIT_CODES.ERROR);
+    }
     const env = getConfigAccountEnvironment(targetProjectAccountId);
     let projectNodes;
     let projectProfileData;
@@ -41,23 +47,29 @@ export async function unifiedProjectDevFlow({ args, targetProjectAccountId, prov
         else {
             logError(e);
         }
-        return process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     if (!Object.keys(projectNodes).length) {
         uiLogger.error(commands.project.dev.errors.noRunnableComponents);
-        process.exit(EXIT_CODES.SUCCESS);
+        return exit(EXIT_CODES.SUCCESS);
     }
     const targetProjectAccountConfig = getConfigAccountById(targetProjectAccountId);
     if (!targetProjectAccountConfig) {
         uiLogger.error(commands.project.dev.errors.noAccount(targetProjectAccountId));
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
-    const accounts = getAllConfigAccounts();
+    const hsSettings = getHsSettingsFileIfExists();
+    const directoryIsLinked = isDirectoryLinked(hsSettings);
+    const accounts = getLinkedOrAllConfigAccounts();
     const accountIsCombined = await isUnifiedAccount(targetProjectAccountConfig);
     const targetProjectAccountIsTestAccountOrSandbox = isTestAccountOrSandbox(targetProjectAccountConfig);
     if (!accountIsCombined) {
         uiLogger.error(commands.project.dev.errors.accountNotCombined);
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
+    }
+    if (directoryIsLinked && !providedTargetTestingAccountId) {
+        uiLogger.log('');
+        uiLogger.info(commands.account.subcommands.link.shared.usingLinkedAccounts(getHsSettingsFilePath()));
     }
     let targetTestingAccountId = providedTargetTestingAccountId;
     // Temporarily removing logic to use profile account as testing account
@@ -77,16 +89,39 @@ export async function unifiedProjectDevFlow({ args, targetProjectAccountId, prov
         const accountType = await selectAccountTypePrompt(targetProjectAccountConfig);
         if (accountType === HUBSPOT_ACCOUNT_TYPES.DEVELOPER_TEST) {
             const devAccountPromptResponse = await selectDeveloperTestTargetAccountPrompt(accounts, targetProjectAccountConfig);
+            const { notInConfigAccount } = devAccountPromptResponse;
             targetTestingAccountId =
                 devAccountPromptResponse.targetAccountId || undefined;
-            if (!!devAccountPromptResponse.notInConfigAccount) {
-                // When the developer test account isn't configured in the CLI config yet
-                // Walk the user through adding the account's PAK to the config
-                await useExistingDevTestAccount(env, devAccountPromptResponse.notInConfigAccount);
+            if (notInConfigAccount) {
+                const existingGlobalConfig = getConfigAccountIfExists(notInConfigAccount.id);
+                if (directoryIsLinked && existingGlobalConfig) {
+                    const shouldLink = await confirmLinkExistingDeveloperTestAccountPrompt(notInConfigAccount.accountName);
+                    if (!shouldLink) {
+                        return exit(EXIT_CODES.SUCCESS);
+                    }
+                    addAccountToLinkedSettings(notInConfigAccount.id);
+                }
+                else {
+                    const accountAdded = await useExistingDevTestAccount(env, notInConfigAccount);
+                    if (!accountAdded) {
+                        return exit(EXIT_CODES.SUCCESS);
+                    }
+                    if (directoryIsLinked) {
+                        addAccountToLinkedSettings(notInConfigAccount.id);
+                    }
+                }
             }
             else if (devAccountPromptResponse.createNestedAccount) {
                 // Create a new developer test account and automatically add it to the CLI config
-                targetTestingAccountId = await createDeveloperTestAccountForLocalDev(targetProjectAccountId, targetProjectAccountConfig, env, true);
+                try {
+                    targetTestingAccountId = await createDeveloperTestAccountForLocalDev(targetProjectAccountId, targetProjectAccountConfig, env, true);
+                }
+                catch {
+                    return exit(EXIT_CODES.ERROR);
+                }
+                if (directoryIsLinked) {
+                    addAccountToLinkedSettings(targetTestingAccountId);
+                }
             }
         }
         else if (accountType === HUBSPOT_ACCOUNT_TYPES.DEVELOPMENT_SANDBOX) {
@@ -94,7 +129,15 @@ export async function unifiedProjectDevFlow({ args, targetProjectAccountId, prov
             targetTestingAccountId =
                 sandboxAccountPromptResponse.targetAccountId || undefined;
             if (sandboxAccountPromptResponse.createNestedAccount) {
-                targetTestingAccountId = await createSandboxForLocalDev(targetProjectAccountId, targetProjectAccountConfig, env);
+                try {
+                    targetTestingAccountId = await createSandboxForLocalDev(targetProjectAccountId, targetProjectAccountConfig, env);
+                }
+                catch {
+                    return exit(EXIT_CODES.ERROR);
+                }
+                if (directoryIsLinked) {
+                    addAccountToLinkedSettings(targetTestingAccountId);
+                }
             }
         }
         else {
@@ -111,21 +154,26 @@ export async function unifiedProjectDevFlow({ args, targetProjectAccountId, prov
     });
     let project = uploadedProject;
     if (projectExists && project) {
-        await compareLocalProjectToDeployed(projectConfig, targetProjectAccountId, project.deployedBuild?.buildId, projectNodes, args.profile);
+        await compareLocalProjectToDeployed(projectConfig, targetProjectAccountId, project.deployedBuild?.buildId, projectNodes, exit, args.profile);
     }
     else {
-        project = await createNewProjectForLocalDev(projectConfig, targetProjectAccountId, false, false);
-        await createInitialBuildForNewProject(projectConfig, projectDir, targetProjectAccountId, true, args.profile);
+        project = await createNewProjectForLocalDev(projectConfig, targetProjectAccountId, false, false, exit);
+        await createInitialBuildForNewProject(projectConfig, projectDir, targetProjectAccountId, exit, true, args.profile);
     }
     // Check for missing/outdated dependencies
-    await checkAndInstallDependencies();
+    try {
+        await checkAndInstallDependencies();
+    }
+    catch {
+        return exit(EXIT_CODES.ERROR);
+    }
     // End setup, start local dev process
     try {
-        await startPortManagerServer();
+        await startPortManagerServer(args.port);
     }
     catch (e) {
         logError(e);
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     const localDevProcess = new LocalDevProcess({
         initialProjectNodes: projectNodes,
@@ -138,6 +186,7 @@ export async function unifiedProjectDevFlow({ args, targetProjectAccountId, prov
         projectDir,
         projectData: project,
         env,
+        actions: { exit },
     });
     const websocketServer = new LocalDevWebsocketServer(localDevProcess, args.debug);
     const watcher = new LocalDevWatcher(localDevProcess);

package/commands/project/download.js

@@ -3,26 +3,26 @@ import { getCwd, sanitizeFileName } from '@hubspot/local-dev-lib/path';
 import { extractZipArchive } from '@hubspot/local-dev-lib/archive';
 import { downloadProject, fetchProjectBuilds, } from '@hubspot/local-dev-lib/api/projects';
 import { logError, ApiErrorContext } from '../../lib/errorHandlers/index.js';
+import { isPromptExitError } from '../../lib/errors/PromptExitError.js';
 import { getProjectConfig } from '../../lib/projects/config.js';
 import { downloadProjectPrompt } from '../../lib/prompts/downloadProjectPrompt.js';
 import { commands } from '../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'download';
 const describe = commands.project.download.describe;
 async function handler(args) {
+    const { dest, build, derivedAccountId, exit } = args;
     const { projectConfig } = await getProjectConfig();
     if (projectConfig) {
         uiLogger.error(commands.project.download.warnings.cannotDownloadWithinProject);
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
-    const { dest, build, derivedAccountId } = args;
-    const { project: projectName } = await downloadProjectPrompt(args);
     let buildNumberToDownload = build;
-    trackCommandUsage('project-download', undefined, derivedAccountId);
     try {
+        const { project: projectName } = await downloadProjectPrompt(args);
         if (!buildNumberToDownload) {
             const { data: projectBuildsResult } = await fetchProjectBuilds(derivedAccountId, projectName);
             const { results: projectBuilds } = projectBuildsResult;
@@ -33,7 +33,7 @@ async function handler(args) {
         }
         if (!buildNumberToDownload) {
             uiLogger.error(commands.project.download.errors.noBuildIdToDownload);
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
         const sanitizedProjectName = sanitizeFileName(projectName);
         const absoluteDestPath = dest
@@ -42,14 +42,17 @@ async function handler(args) {
         const { data: zippedProject } = await downloadProject(derivedAccountId, projectName, buildNumberToDownload);
         await extractZipArchive(zippedProject, sanitizeFileName(projectName), path.resolve(absoluteDestPath));
         uiLogger.log(commands.project.download.logs.downloadSucceeded(buildNumberToDownload, projectName));
-        process.exit(EXIT_CODES.SUCCESS);
+        return exit(EXIT_CODES.SUCCESS);
     }
     catch (e) {
+        if (isPromptExitError(e)) {
+            throw e;
+        }
         logError(e, new ApiErrorContext({
             accountId: derivedAccountId,
             request: 'project download',
         }));
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
 }
 function projectDownloadBuilder(yargs) {
@@ -85,7 +88,7 @@ const builder = makeYargsBuilder(projectDownloadBuilder, command, describe, {
 const projectDownloadCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('project-download', handler),
     builder,
 };
 export default projectDownloadCommand;

package/commands/project/info.d.ts

@@ -0,0 +1,4 @@
+import { AccountArgs, CommonArgs, ConfigArgs, JSONOutputArgs, YargsCommandModule } from '../../types/Yargs.js';
+type ProjectInfoArgs = CommonArgs & ConfigArgs & AccountArgs & JSONOutputArgs;
+declare const projectInfoCommand: YargsCommandModule<unknown, ProjectInfoArgs>;
+export default projectInfoCommand;

package/commands/project/info.js

@@ -0,0 +1,67 @@
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
+import { makeYargsBuilder } from '../../lib/yargsUtils.js';
+import { fetchProject } from '@hubspot/local-dev-lib/api/projects';
+import { debugError } from '../../lib/errorHandlers/index.js';
+import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import { uiLogger } from '../../lib/ui/logger.js';
+import { commands } from '../../lang/en.js';
+import { getProjectConfig } from '../../lib/projects/config.js';
+import { isLegacyProject } from '@hubspot/project-parsing-lib/projects';
+import { getProjectInfo, logProjectInfo, } from '../../lib/projects/projectInfo.js';
+const command = 'info';
+const describe = commands.project.info.describe;
+const verboseDescribe = commands.project.info.verboseDescribe;
+async function handler(args) {
+    const { derivedAccountId, formatOutputAsJson, exit } = args;
+    const { projectConfig } = await getProjectConfig();
+    if (!projectConfig) {
+        uiLogger.error(commands.project.info.errors.noProjectConfig);
+        return exit(EXIT_CODES.ERROR);
+    }
+    if (isLegacyProject(projectConfig.platformVersion)) {
+        uiLogger.error(commands.project.info.errors.unsupportedPlatformVersion(projectConfig.platformVersion));
+        return exit(EXIT_CODES.ERROR);
+    }
+    const projectName = projectConfig.name;
+    let project;
+    try {
+        const response = await fetchProject(derivedAccountId, projectName);
+        project = response.data;
+    }
+    catch (error) {
+        debugError(error);
+        uiLogger.error(commands.project.info.errors.projectNotFound(projectName, derivedAccountId));
+        return exit(EXIT_CODES.ERROR);
+    }
+    const deployedBuild = project.deployedBuild;
+    if (!deployedBuild) {
+        uiLogger.error(commands.project.info.errors.noDeployedBuild);
+        return exit(EXIT_CODES.ERROR);
+    }
+    const projectInfo = await getProjectInfo(project, projectConfig.platformVersion, derivedAccountId);
+    if (formatOutputAsJson) {
+        uiLogger.json(projectInfo);
+        return;
+    }
+    logProjectInfo(projectInfo);
+}
+function projectInfoBuilder(yargs) {
+    yargs.example([
+        ['$0 project info', commands.project.info.examples.default],
+        ['$0 project info --json', commands.project.info.examples.json],
+    ]);
+    return yargs;
+}
+const builder = makeYargsBuilder(projectInfoBuilder, command, verboseDescribe, {
+    useGlobalOptions: true,
+    useConfigOptions: true,
+    useAccountOptions: true,
+    useJSONOutputOptions: true,
+});
+const projectInfoCommand = {
+    command,
+    describe,
+    handler: makeYargsHandlerWithUsageTracking('project-info', handler),
+    builder,
+};
+export default projectInfoCommand;

package/commands/project/installDeps.js

@@ -1,23 +1,23 @@
 import { installPackages, getProjectPackageJsonLocations, } from '../../lib/dependencyManagement.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import { isPromptExitError } from '../../lib/errors/PromptExitError.js';
 import { getProjectConfig } from '../../lib/projects/config.js';
 import { promptUser } from '../../lib/prompts/promptUtils.js';
 import path from 'path';
 import { commands } from '../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { logError } from '../../lib/errorHandlers/index.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 const command = 'install-deps [packages..]';
 const describe = commands.project.installDeps.help.describe;
 async function handler(args) {
-    const { derivedAccountId, packages } = args;
+    const { packages, exit } = args;
     try {
-        trackCommandUsage('project-install-deps', undefined, derivedAccountId);
         const projectConfig = await getProjectConfig();
         if (!projectConfig || !projectConfig.projectDir) {
             uiLogger.error(commands.project.installDeps.noProjectConfig);
-            return process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
         const { projectDir } = projectConfig;
         let installLocations = await getProjectPackageJsonLocations();
@@ -50,8 +50,11 @@ async function handler(args) {
         });
     }
     catch (e) {
+        if (isPromptExitError(e)) {
+            throw e;
+        }
         logError(e);
-        return process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
 }
 function projectInstallDepsBuilder(yargs) {
@@ -74,7 +77,7 @@ const builder = makeYargsBuilder(projectInstallDepsBuilder, command, describe, {
 const projectInstallDepsCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('project-install-deps', handler),
     builder,
 };
 export default projectInstallDepsCommand;

package/commands/project/lint.js

@@ -1,25 +1,25 @@
 import path from 'path';
 import { getProjectPackageJsonLocations, installPackages, } from '../../lib/dependencyManagement.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import { isPromptExitError } from '../../lib/errors/PromptExitError.js';
 import { getProjectConfig } from '../../lib/projects/config.js';
 import { commands } from '../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { logError } from '../../lib/errorHandlers/index.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { promptUser } from '../../lib/prompts/promptUtils.js';
 import SpinniesManager from '../../lib/ui/SpinniesManager.js';
-import { areAllLintPackagesInstalled, getMissingLintPackages, lintPackages, displayLintResults, hasEslintConfig, hasDeprecatedEslintConfig, getDeprecatedEslintConfigFiles, createEslintConfig, REQUIRED_PACKAGES_AND_MIN_VERSIONS, } from '../../lib/projects/uieLinting.js';
+import { areAllLintPackagesInstalled, getMissingLintPackages, getMissingLintScripts, addLintScriptsToPackageJson, lintPackages, displayLintResults, hasEslintConfig, hasDeprecatedEslintConfig, getDeprecatedEslintConfigFiles, createEslintConfig, REQUIRED_PACKAGES_AND_MIN_VERSIONS, } from '../../lib/projects/uieLinting.js';
 const command = 'lint';
 const describe = commands.project.lint.help.describe;
 async function handler(args) {
-    const { derivedAccountId, installMissingDeps } = args;
+    const { installMissingDeps, exit } = args;
     try {
-        trackCommandUsage('project-lint', undefined, derivedAccountId);
         const projectConfig = await getProjectConfig();
         if (!projectConfig || !projectConfig.projectDir) {
             uiLogger.error(commands.project.lint.noProjectConfig);
-            return process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
         }
         SpinniesManager.init({ succeedColor: 'white' });
         SpinniesManager.add('lintCheck', {
@@ -117,9 +117,10 @@ async function handler(args) {
                     SpinniesManager.add('lintConfigCreate', {
                         text: commands.project.lint.loading.creatingConfig,
                     });
+                    const platformVersion = projectConfig.projectConfig?.platformVersion ?? null;
                     const createdConfigs = [];
                     for (const location of locationsNeedingConfig) {
-                        const configPath = createEslintConfig(location);
+                        const configPath = await createEslintConfig(location, platformVersion);
                         createdConfigs.push(configPath);
                     }
                     SpinniesManager.succeed('lintConfigCreate');
@@ -129,9 +130,26 @@ async function handler(args) {
                 }
                 else {
                     uiLogger.error(commands.project.lint.eslintConfigRequired);
-                    return process.exit(EXIT_CODES.ERROR);
+                    return exit(EXIT_CODES.ERROR);
                 }
             }
+            const locationsNeedingScripts = locationsReadyToLint.filter(location => getMissingLintScripts(location).length > 0);
+            if (locationsNeedingScripts.length > 0) {
+                SpinniesManager.add('lintScriptsAdd', {
+                    text: commands.project.lint.loading.addingLintScripts,
+                });
+                const addedResults = [];
+                for (const location of locationsNeedingScripts) {
+                    const result = addLintScriptsToPackageJson(location);
+                    if (result.added.length > 0) {
+                        addedResults.push(result);
+                    }
+                }
+                SpinniesManager.succeed('lintScriptsAdd');
+                addedResults.forEach(({ added, relativePath }) => {
+                    uiLogger.success(commands.project.lint.lintScriptsAdded(added, relativePath));
+                });
+            }
             SpinniesManager.add('lintRun', {
                 text: commands.project.lint.loading.linting,
             });
@@ -139,13 +157,16 @@ async function handler(args) {
             SpinniesManager.succeed('lintRun');
             displayLintResults(results);
             if (!success) {
-                return process.exit(EXIT_CODES.ERROR);
+                return exit(EXIT_CODES.ERROR);
             }
         }
     }
     catch (e) {
+        if (isPromptExitError(e)) {
+            throw e;
+        }
         logError(e);
-        return process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
 }
 function projectLintBuilder(yargs) {
@@ -172,7 +193,7 @@ const builder = makeYargsBuilder(projectLintBuilder, command, describe, {
 const projectLintCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('project-lint', handler),
     builder,
 };
 export default projectLintCommand;

package/commands/project/list.js

@@ -1,4 +1,4 @@
-import { trackCommandUsage } from '../../lib/usageTracking.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { fetchProjects } from '@hubspot/local-dev-lib/api/projects';
 import { logError } from '../../lib/errorHandlers/index.js';
@@ -9,14 +9,8 @@ import { renderTable } from '../../ui/render.js';
 const command = ['list', 'ls'];
 const describe = commands.project.list.describe;
 async function getProjectData(accountId) {
-    try {
-        const { data: projects } = await fetchProjects(accountId);
-        return projects.results;
-    }
-    catch (e) {
-        logError(e);
-        process.exit(EXIT_CODES.ERROR);
-    }
+    const { data: projects } = await fetchProjects(accountId);
+    return projects.results;
 }
 function formatProjectsAsTableRows(projects) {
     const projectListData = [];
@@ -29,12 +23,18 @@ function formatProjectsAsTableRows(projects) {
     return projectListData;
 }
 async function handler(args) {
-    const { derivedAccountId } = args;
-    trackCommandUsage('projects-list', undefined, derivedAccountId);
-    const projectData = await getProjectData(derivedAccountId);
+    const { derivedAccountId, exit } = args;
+    let projectData;
+    try {
+        projectData = await getProjectData(derivedAccountId);
+    }
+    catch (e) {
+        logError(e);
+        return exit(EXIT_CODES.ERROR);
+    }
     if (projectData.length === 0) {
         uiLogger.error(commands.project.list.errors.noProjectsFound(derivedAccountId));
-        process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
     const projectListData = formatProjectsAsTableRows(projectData);
     const tableHeader = [
@@ -56,7 +56,7 @@ const builder = makeYargsBuilder(projectListBuilder, command, describe, {
 const projectListCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('projects-list', handler),
     builder,
 };
 export default projectListCommand;

package/commands/project/listBuilds.js

@@ -5,9 +5,9 @@ import { getProjectConfig, validateProjectConfig, } from '../../lib/projects/con
 import { getProjectDetailUrl } from '../../lib/projects/urls.js';
 import moment from 'moment';
 import { promptUser } from '../../lib/prompts/promptUtils.js';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { uiLogger } from '../../lib/ui/logger.js';
 import { logError, ApiErrorContext } from '../../lib/errorHandlers/index.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { commands } from '../../lang/en.js';
@@ -58,8 +58,7 @@ async function fetchAndDisplayBuilds(accountId, project, options) {
     }
 }
 async function handler(args) {
-    const { project: projectFlagValue, limit, derivedAccountId } = args;
-    trackCommandUsage('project-list-builds', undefined, derivedAccountId);
+    const { project: projectFlagValue, limit, derivedAccountId, exit } = args;
     let projectName = projectFlagValue;
     if (!projectName) {
         const { projectConfig, projectDir } = await getProjectConfig();
@@ -68,7 +67,10 @@ async function handler(args) {
         }
         catch (error) {
             logError(error);
-            process.exit(EXIT_CODES.ERROR);
+            return exit(EXIT_CODES.ERROR);
+        }
+        if (!projectConfig) {
+            return exit(EXIT_CODES.ERROR);
         }
         projectName = projectConfig.name;
     }
@@ -87,7 +89,7 @@ async function handler(args) {
             }));
         }
     }
-    process.exit(EXIT_CODES.SUCCESS);
+    return exit(EXIT_CODES.SUCCESS);
 }
 function projectListBuildsBuilder(yargs) {
     yargs.options({
@@ -114,7 +116,7 @@ const builder = makeYargsBuilder(projectListBuildsBuilder, command, describe, {
 const projectListBuildsCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('project-list-builds', handler),
     builder,
 };
 export default projectListBuildsCommand;

package/commands/project/logs.js

@@ -1,6 +1,5 @@
 import { getConfigAccountEnvironment } from '@hubspot/local-dev-lib/config';
 import { getHubSpotWebsiteOrigin } from '@hubspot/local-dev-lib/urls';
-import { trackCommandUsage } from '../../lib/usageTracking.js';
 import { logError } from '../../lib/errorHandlers/index.js';
 import { uiLine } from '../../lib/ui/index.js';
 import { projectLogsPrompt } from '../../lib/prompts/projectsLogsPrompt.js';
@@ -8,6 +7,7 @@ import { commands } from '../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { ProjectLogsManager } from '../../lib/projects/ProjectLogsManager.js';
+import { makeYargsHandlerWithUsageTracking } from '../../lib/yargs/makeYargsHandlerWithUsageTracking.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { renderTable } from '../../ui/render.js';
 function getPrivateAppsUrl(accountId) {
@@ -51,8 +51,7 @@ function logPreamble() {
 const command = 'logs';
 const describe = commands.project.logs.describe;
 async function handler(args) {
-    const { derivedAccountId } = args;
-    trackCommandUsage('project-logs', undefined, derivedAccountId);
+    const { derivedAccountId, exit } = args;
     try {
         await ProjectLogsManager.init(derivedAccountId);
         const { functionName } = await projectLogsPrompt({
@@ -68,9 +67,9 @@ async function handler(args) {
             accountId: derivedAccountId,
             projectName: ProjectLogsManager.projectName,
         });
-        return process.exit(EXIT_CODES.ERROR);
+        return exit(EXIT_CODES.ERROR);
     }
-    process.exit(EXIT_CODES.SUCCESS);
+    return exit(EXIT_CODES.SUCCESS);
 }
 function projectLogsBuilder(yargs) {
     yargs.options({
@@ -113,7 +112,7 @@ const builder = makeYargsBuilder(projectLogsBuilder, command, describe, { useGlo
 const projectLogsCommand = {
     command,
     describe,
-    handler,
+    handler: makeYargsHandlerWithUsageTracking('project-logs', handler),
     builder,
 };
 export default projectLogsCommand;