@hubspot/cli 7.8.11-experimental.0 → 7.8.12-experimental.1

package/commands/list.js

@@ -1,13 +1,13 @@
 import chalk from 'chalk';
 import { trackCommandUsage } from '../lib/usageTracking.js';
 import { isPathFolder } from '../lib/filesystem.js';
-import { logger } from '@hubspot/local-dev-lib/logger';
 import { logError } from '../lib/errorHandlers/index.js';
 import { getDirectoryContentsByPath } from '@hubspot/local-dev-lib/api/fileMapper';
 import { HUBSPOT_FOLDER, MARKETPLACE_FOLDER } from '../lib/constants.js';
-import { i18n } from '../lib/lang.js';
 import { EXIT_CODES } from '../lib/enums/exitCodes.js';
 import { makeYargsBuilder } from '../lib/yargsUtils.js';
+import { uiLogger } from '../lib/ui/logger.js';
+import { commands } from '../lang/en.js';
 function addColorToContents(fileOrFolder) {
     if (!isPathFolder(fileOrFolder)) {
         return chalk.reset.cyan(fileOrFolder);
@@ -35,15 +35,13 @@ function sortContents(a, b) {
     return a.localeCompare(b);
 }
 const command = ['list [path]', 'ls [path]'];
-const describe = i18n(`commands.list.describe`);
+const describe = commands.list.describe;
 async function handler(args) {
     const { path, derivedAccountId } = args;
     const directoryPath = path || '/';
     let contentsResp;
     trackCommandUsage('list', undefined, derivedAccountId);
-    logger.debug(i18n(`commands.list.gettingPathContents`, {
-        path: directoryPath,
-    }));
+    uiLogger.debug(commands.list.gettingPathContents(directoryPath));
     try {
         const { data } = await getDirectoryContentsByPath(derivedAccountId, directoryPath);
         contentsResp = data;
@@ -53,9 +51,7 @@ async function handler(args) {
         process.exit(EXIT_CODES.ERROR);
     }
     if (!contentsResp.folder) {
-        logger.info(i18n(`commands.list.noFilesFoundAtPath`, {
-            path: directoryPath,
-        }));
+        uiLogger.info(commands.list.noFilesFoundAtPath(directoryPath));
         return;
     }
     // getDirectoryContentsByPath omits @hubspot
@@ -63,21 +59,19 @@ async function handler(args) {
         ? ['@hubspot', ...contentsResp.children]
         : contentsResp.children;
     if (contents.length === 0) {
-        logger.info(i18n(`commands.list.noFilesFoundAtPath`, {
-            path: directoryPath,
-        }));
+        uiLogger.info(commands.list.noFilesFoundAtPath(directoryPath));
         return;
     }
     const folderContentsOutput = contents
         .map(addColorToContents)
         .sort(sortContents)
         .join('\n');
-    logger.log(folderContentsOutput);
+    uiLogger.log(folderContentsOutput);
     process.exit(EXIT_CODES.SUCCESS);
 }
 function cmsListBuilder(yargs) {
     yargs.positional('path', {
-        describe: i18n(`commands.list.positionals.path.describe`),
+        describe: commands.list.positionals.path.describe,
         type: 'string',
     });
     yargs.example([['$0 list'], ['$0 list /'], ['$0 list my-modules']]);

package/commands/logs.js

@@ -1,19 +1,16 @@
 import { trackCommandUsage } from '../lib/usageTracking.js';
-import { logger } from '@hubspot/local-dev-lib/logger';
 import { outputLogs } from '../lib/ui/serverlessFunctionLogs.js';
 import { getFunctionLogs, getLatestFunctionLog, } from '@hubspot/local-dev-lib/api/functions';
 import { tailLogs } from '../lib/serverlessLogs.js';
-import { i18n } from '../lib/lang.js';
 import { promptUser } from '../lib/prompts/promptUtils.js';
 import { EXIT_CODES } from '../lib/enums/exitCodes.js';
 import { isHubSpotHttpError } from '@hubspot/local-dev-lib/errors/index';
 import { makeYargsBuilder } from '../lib/yargsUtils.js';
+import { uiLogger } from '../lib/ui/logger.js';
+import { commands } from '../lang/en.js';
 const handleLogsError = (e, accountId, functionPath) => {
     if (isHubSpotHttpError(e) && (e.status === 404 || e.status == 400)) {
-        logger.error(i18n(`commands.logs.errors.noLogsFound`, {
-            accountId,
-            functionPath,
-        }));
+        uiLogger.error(commands.logs.errors.noLogsFound(functionPath, accountId));
     }
 };
 const endpointLog = async (accountId, functionPath, options) => {
@@ -24,10 +21,7 @@ const endpointLog = async (accountId, functionPath, options) => {
         follow,
         endpoint: functionPath,
     };
-    logger.debug(i18n(`commands.logs.gettingLogs`, {
-        latest,
-        functionPath,
-    }));
+    uiLogger.debug(commands.logs.gettingLogs(latest, functionPath));
     let logsResp;
     if (follow) {
         const tailCall = (after) => getFunctionLogs(accountId, functionPath, { after });
@@ -67,48 +61,48 @@ const endpointLog = async (accountId, functionPath, options) => {
     }
 };
 const command = 'logs [endpoint]';
-const describe = i18n(`commands.logs.describe`);
+const describe = commands.logs.describe;
 const handler = async (options) => {
     const { endpoint: endpointArgValue, latest, derivedAccountId } = options;
     trackCommandUsage('logs', latest ? { action: 'latest' } : {}, derivedAccountId);
     const { endpointPromptValue } = await promptUser({
         name: 'endpointPromptValue',
-        message: i18n(`commands.logs.endpointPrompt`),
+        message: commands.logs.endpointPrompt,
         when: !endpointArgValue,
     });
     await endpointLog(derivedAccountId, endpointArgValue || endpointPromptValue, options);
 };
 function logsBuilder(yargs) {
     yargs.positional('endpoint', {
-        describe: i18n(`commands.logs.positionals.endpoint.describe`),
+        describe: commands.logs.positionals.endpoint.describe,
         type: 'string',
     });
     yargs
         .options({
         latest: {
             alias: 'l',
-            describe: i18n(`commands.logs.options.latest.describe`),
+            describe: commands.logs.options.latest.describe,
             type: 'boolean',
         },
         compact: {
-            describe: i18n(`commands.logs.options.compact.describe`),
+            describe: commands.logs.options.compact.describe,
             type: 'boolean',
         },
         follow: {
             alias: ['f'],
-            describe: i18n(`commands.logs.options.follow.describe`),
+            describe: commands.logs.options.follow.describe,
             type: 'boolean',
         },
         limit: {
-            describe: i18n(`commands.logs.options.limit.describe`),
+            describe: commands.logs.options.limit.describe,
             type: 'number',
         },
     })
         .conflicts('follow', 'limit');
     yargs.example([
-        ['$0 logs my-endpoint', i18n(`commands.logs.examples.default`)],
-        ['$0 logs my-endpoint --limit=10', i18n(`commands.logs.examples.limit`)],
-        ['$0 logs my-endpoint --follow', i18n(`commands.logs.examples.follow`)],
+        ['$0 logs my-endpoint', commands.logs.examples.default],
+        ['$0 logs my-endpoint --limit=10', commands.logs.examples.limit],
+        ['$0 logs my-endpoint --follow', commands.logs.examples.follow],
     ]);
     return yargs;
 }

package/commands/mv.js

@@ -1,12 +1,12 @@
-import { logger } from '@hubspot/local-dev-lib/logger';
 import { moveFile } from '@hubspot/local-dev-lib/api/fileMapper';
 import { isSpecifiedError } from '@hubspot/local-dev-lib/errors/index';
 import { logError, ApiErrorContext } from '../lib/errorHandlers/index.js';
 import { trackCommandUsage } from '../lib/usageTracking.js';
 import { isPathFolder } from '../lib/filesystem.js';
-import { i18n } from '../lib/lang.js';
 import { uiBetaTag } from '../lib/ui/index.js';
 import { makeYargsBuilder } from '../lib/yargsUtils.js';
+import { uiLogger } from '../lib/ui/logger.js';
+import { commands } from '../lang/en.js';
 function getCorrectedDestPath(srcPath, destPath) {
     if (!isPathFolder(srcPath)) {
         return destPath;
@@ -15,29 +15,18 @@ function getCorrectedDestPath(srcPath, destPath) {
     return `${destPath}/${srcPath.split('/').pop()}`;
 }
 const command = 'mv <srcPath> <destPath>';
-const describe = uiBetaTag(i18n(`commands.mv.describe`), false);
+const describe = uiBetaTag(commands.mv.describe, false);
 async function handler(args) {
     const { srcPath, destPath, derivedAccountId } = args;
     trackCommandUsage('mv', undefined, derivedAccountId);
     try {
         await moveFile(derivedAccountId, srcPath, getCorrectedDestPath(srcPath, destPath));
-        logger.success(i18n(`commands.mv.move`, {
-            accountId: derivedAccountId,
-            destPath,
-            srcPath,
-        }));
+        uiLogger.success(commands.mv.move(srcPath, destPath, derivedAccountId));
     }
     catch (error) {
-        logger.error(i18n(`commands.mv.errors.moveFailed`, {
-            accountId: derivedAccountId,
-            destPath,
-            srcPath,
-        }));
+        uiLogger.error(commands.mv.errors.moveFailed(srcPath, destPath, derivedAccountId));
         if (isSpecifiedError(error, { statusCode: 409 })) {
-            logger.error(i18n(`commands.mv.errors.sourcePathExists`, {
-                destPath,
-                srcPath,
-            }));
+            uiLogger.error(commands.mv.errors.sourcePathExists(srcPath, destPath));
         }
         else {
             logError(error, new ApiErrorContext({

package/commands/open.js

@@ -1,7 +1,7 @@
 import { trackCommandUsage } from '../lib/usageTracking.js';
 import { logSiteLinks, getSiteLinksAsArray, openLink } from '../lib/links.js';
 import { promptUser } from '../lib/prompts/promptUtils.js';
-import { i18n } from '../lib/lang.js';
+import { commands } from '../lang/en.js';
 import { EXIT_CODES } from '../lib/enums/exitCodes.js';
 import { makeYargsBuilder } from '../lib/yargsUtils.js';
 const separator = ' => ';
@@ -10,7 +10,7 @@ async function createListPrompt(accountId) {
         name: 'open',
         type: 'rawlist',
         pageSize: 20,
-        message: i18n('commands.open.selectLink'),
+        message: commands.open.selectLink,
         choices: getSiteLinksAsArray(accountId)
             .filter(l => !!l.url)
             .map(l => ({
@@ -20,7 +20,7 @@ async function createListPrompt(accountId) {
     });
 }
 const command = 'open [shortcut]';
-const describe = i18n('commands.open.describe');
+const describe = commands.open.describe;
 async function handler(args) {
     const { shortcut, list, derivedAccountId } = args;
     trackCommandUsage('open', undefined, derivedAccountId);
@@ -38,12 +38,12 @@ async function handler(args) {
 }
 function openBuilder(yargs) {
     yargs.positional('[shortcut]', {
-        describe: i18n('commands.open.positionals.shortcut.describe'),
+        describe: commands.open.positionals.shortcut.describe,
         type: 'string',
     });
     yargs.option('list', {
         alias: 'l',
-        describe: i18n('commands.open.options.list.describe'),
+        describe: commands.open.options.list.describe,
         type: 'boolean',
     });
     yargs.example([

package/commands/project/__tests__/add.test.js

@@ -4,9 +4,11 @@ import { marketplaceDistribution, oAuth, privateDistribution, staticAuth, } from
 import { v3AddComponent } from '../../../lib/projects/add/v3AddComponent.js';
 import { legacyAddComponent } from '../../../lib/projects/add/legacyAddComponent.js';
 import { getProjectConfig } from '../../../lib/projects/config.js';
-import { useV3Api } from '../../../lib/projects/platformVersion.js';
+import { isV2Project } from '../../../lib/projects/platformVersion.js';
 import { trackCommandUsage } from '../../../lib/usageTracking.js';
 vi.mock('../../../lib/commonOpts');
+vi.mock('../../../lib/ui/logger.js');
+vi.mock('../../../lib/errorHandlers/index.js');
 vi.mock('../../../lib/projects/add/v3AddComponent');
 vi.mock('../../../lib/projects/add/legacyAddComponent');
 vi.mock('../../../lib/projects/config');
@@ -15,7 +17,7 @@ vi.mock('../../../lib/usageTracking');
 const mockedV3AddComponent = vi.mocked(v3AddComponent);
 const mockedLegacyAddComponent = vi.mocked(legacyAddComponent);
 const mockedGetProjectConfig = vi.mocked(getProjectConfig);
-const mockedUseV3Api = vi.mocked(useV3Api);
+const mockedUseV3Api = vi.mocked(isV2Project);
 const mockedTrackCommandUsage = vi.mocked(trackCommandUsage);
 describe('commands/project/add', () => {
     const yargsMock = yargs;

package/commands/project/__tests__/deploy.test.js

@@ -1,7 +1,6 @@
 import { HttpStatusCode } from 'axios';
 import yargs from 'yargs';
 import chalk from 'chalk';
-import { logger } from '@hubspot/local-dev-lib/logger';
 import * as configUtils from '@hubspot/local-dev-lib/config';
 import * as projectApiUtils from '@hubspot/local-dev-lib/api/projects';
 import * as ui from '../../../lib/ui/index.js';
@@ -29,7 +28,7 @@ vi.mock('../../../lib/prompts/projectNamePrompt');
 vi.mock('../../../lib/prompts/promptUtils');
 vi.mock('../../../lib/usageTracking');
 vi.mock('../../../lib/ui/logger');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../ui/logger.js');
 vi.spyOn(ui, 'uiLine');
 const uiCommandReferenceSpy = vi.spyOn(ui, 'uiCommandReference');
 const uiAccountDescriptionSpy = vi.spyOn(ui, 'uiAccountDescription');
@@ -297,8 +296,8 @@ describe('commands/project/deploy', () => {
                 });
             });
             await projectDeployCommand.handler(args);
-            expect(logger.error).toHaveBeenCalledTimes(1);
-            expect(logger.error).toHaveBeenCalledWith(`The request for 'project deploy' in account ${args.derivedAccountId} failed due to a client error.`);
+            expect(uiLogger.error).toHaveBeenCalledTimes(1);
+            expect(uiLogger.error).toHaveBeenCalledWith(`The request for 'project deploy' in account ${args.derivedAccountId} failed due to a client error.`);
             expect(processExitSpy).toHaveBeenCalledTimes(1);
             expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
         });

package/commands/project/__tests__/installDeps.test.js

@@ -1,13 +1,13 @@
 import yargs from 'yargs';
 import path from 'path';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../../../lib/ui/logger.js';
 import * as projectUtils from '../../../lib/projects/config.js';
 import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
 import { trackCommandUsage } from '../../../lib/usageTracking.js';
 import * as dependencyManagement from '../../../lib/dependencyManagement.js';
 import * as promptUtils from '../../../lib/prompts/promptUtils.js';
 import projectInstallDepsCommand from '../installDeps.js';
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../../lib/ui/logger.js');
 vi.mock('../../../lib/projects/config');
 vi.mock('../../../lib/dependencyManagement');
 vi.mock('../../../lib/prompts/promptUtils');
@@ -56,8 +56,8 @@ describe('commands/project/installDeps', () => {
                 throw error;
             });
             await projectInstallDepsCommand.handler(args);
-            expect(logger.error).toHaveBeenCalledTimes(1);
-            expect(logger.error).toHaveBeenCalledWith(error.message);
+            expect(uiLogger.error).toHaveBeenCalledTimes(1);
+            expect(uiLogger.error).toHaveBeenCalledWith(error.message);
             expect(processExitSpy).toHaveBeenCalledTimes(1);
             expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
         });
@@ -67,8 +67,8 @@ describe('commands/project/installDeps', () => {
                 projectConfig: null,
             });
             await projectInstallDepsCommand.handler(args);
-            expect(logger.error).toHaveBeenCalledTimes(1);
-            expect(logger.error).toHaveBeenCalledWith('No project detected. Run this command from a project directory.');
+            expect(uiLogger.error).toHaveBeenCalledTimes(1);
+            expect(uiLogger.error).toHaveBeenCalledWith('No project detected. Run this command from a project directory.');
             expect(processExitSpy).toHaveBeenCalledTimes(1);
             expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
         });
@@ -78,8 +78,8 @@ describe('commands/project/installDeps', () => {
                 projectConfig: null,
             });
             await projectInstallDepsCommand.handler(args);
-            expect(logger.error).toHaveBeenCalledTimes(1);
-            expect(logger.error).toHaveBeenCalledWith('No project detected. Run this command from a project directory.');
+            expect(uiLogger.error).toHaveBeenCalledTimes(1);
+            expect(uiLogger.error).toHaveBeenCalledWith('No project detected. Run this command from a project directory.');
             expect(processExitSpy).toHaveBeenCalledTimes(1);
             expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
         });

package/commands/project/__tests__/logs.test.js

@@ -8,7 +8,7 @@ import * as ui from '../../../lib/ui/index.js';
 import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
 import { logError } from '../../../lib/errorHandlers/index.js';
 import projectLogsCommand from '../logs.js';
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../ui/logger.js');
 vi.mock('../../../lib/commonOpts');
 vi.mock('../../../lib/usageTracking');
 vi.mock('../../../lib/validation');

package/commands/project/__tests__/migrate.test.js

@@ -1,12 +1,12 @@
 import yargs from 'yargs';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../../../lib/ui/logger.js';
 import { PLATFORM_VERSIONS } from '@hubspot/local-dev-lib/constants/projects';
 import migrateCommand from '../migrate.js';
 import { migrateApp2025_2 } from '../../../lib/app/migrate.js';
 import { getProjectConfig } from '../../../lib/projects/config.js';
 import { commands } from '../../../lang/en.js';
 import { uiBetaTag, uiCommandReference } from '../../../lib/ui/index.js';
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../../lib/ui/logger.js');
 vi.mock('../../../lib/app/migrate');
 vi.mock('../../../lib/projects/config');
 vi.mock('../../../lib/ui');
@@ -24,8 +24,8 @@ describe('commands/project/migrate', () => {
         .mockImplementation(() => undefined);
     beforeEach(() => {
         // Mock logger methods
-        vi.spyOn(logger, 'log').mockImplementation(() => { });
-        vi.spyOn(logger, 'error').mockImplementation(() => { });
+        vi.spyOn(uiLogger, 'log').mockImplementation(() => { });
+        vi.spyOn(uiLogger, 'error').mockImplementation(() => { });
         migrateApp2025_2Mock.mockResolvedValue(undefined);
         getProjectConfigMock.mockResolvedValue({
             projectConfig: { name: 'test-project' },
@@ -74,7 +74,7 @@ describe('commands/project/migrate', () => {
         it('should exit with error if no project config exists', async () => {
             getProjectConfigMock.mockResolvedValue({ projectConfig: null });
             await migrateCommand.handler(options);
-            expect(logger.error).toHaveBeenCalledWith(commands.project.migrate.errors.noProjectConfig('command reference'));
+            expect(uiLogger.error).toHaveBeenCalledWith(commands.project.migrate.errors.noProjectConfig('command reference'));
             expect(mockExit).toHaveBeenCalledWith(1);
             expect(mockExit).toHaveBeenCalledTimes(1);
         });

package/commands/project/__tests__/migrateApp.test.js

@@ -1,11 +1,10 @@
 import yargs from 'yargs';
-import { i18n } from '../../../lib/lang.js';
 import { uiCommandReference, uiDeprecatedTag } from '../../../lib/ui/index.js';
 import { handlerGenerator } from '../../app/migrate.js';
 import { PLATFORM_VERSIONS } from '@hubspot/local-dev-lib/constants/projects';
 import migrateAppCommand from '../migrateApp.js';
-vi.mock('@hubspot/local-dev-lib/logger');
-vi.mock('../../../lib/lang');
+vi.mock('../../ui/logger.js');
+vi.mock('../../../lang/en.js');
 vi.mock('../../../lib/ui');
 vi.mock('../../app/migrate');
 const { v2023_2, v2025_2 } = PLATFORM_VERSIONS;
@@ -14,13 +13,11 @@ describe('commands/project/migrateApp', () => {
     const optionsSpy = vi.spyOn(yargsMock, 'options').mockReturnValue(yargsMock);
     const exampleSpy = vi.spyOn(yargsMock, 'example').mockReturnValue(yargsMock);
     // Mock the imported functions
-    const i18nMock = i18n;
     const uiDeprecatedTagMock = uiDeprecatedTag;
     const uiCommandReferenceMock = uiCommandReference;
     const handlerGeneratorMock = handlerGenerator;
     beforeEach(() => {
         vi.clearAllMocks();
-        i18nMock.mockReturnValue('test description');
         uiDeprecatedTagMock.mockReturnValue('deprecated test description');
         uiCommandReferenceMock.mockReturnValue('command reference');
         handlerGeneratorMock.mockReturnValue(vi.fn().mockResolvedValue(undefined));

package/commands/project/__tests__/validate.test.js

@@ -0,0 +1,98 @@
+import path from 'path';
+import { vi } from 'vitest';
+import { validateSourceDirectory } from '../../../lib/projects/upload.js';
+import { getProjectConfig, validateProjectConfig, } from '../../../lib/projects/config.js';
+import { uiLogger } from '../../../lib/ui/logger.js';
+import { commands } from '../../../lang/en.js';
+import { isV2Project } from '../../../lib/projects/platformVersion.js';
+import { loadAndValidateProfile } from '../../../lib/projectProfiles.js';
+import { trackCommandUsage } from '../../../lib/usageTracking.js';
+import { getAccountConfig } from '@hubspot/local-dev-lib/config';
+import { handleTranslate } from '../../../lib/projects/upload.js';
+import projectValidateCommand from '../validate.js';
+// Mock dependencies
+vi.mock('../../../lib/projects/upload.js');
+vi.mock('../../../lib/projects/config.js');
+vi.mock('../../../lib/ui/logger.js');
+vi.mock('../../../lib/usageTracking.js');
+vi.mock('../../../lib/projectProfiles.js');
+vi.mock('../../../lib/errorHandlers/index.js');
+vi.mock('@hubspot/local-dev-lib/config');
+vi.mock('../../../lib/projects/platformVersion.js');
+describe('commands/project/validate', () => {
+    const projectDir = '/test/project';
+    let exitSpy;
+    beforeEach(() => {
+        // Mock process.exit to throw to stop execution
+        exitSpy = vi.spyOn(process, 'exit').mockImplementation(code => {
+            throw new Error(`Process exited with code ${code}`);
+        });
+        vi.clearAllMocks();
+    });
+    afterEach(() => {
+        exitSpy.mockRestore();
+    });
+    describe('project configuration validation', () => {
+        it('should exit with error when project config is null', async () => {
+            vi.mocked(getProjectConfig).mockResolvedValue({
+                projectConfig: null,
+                projectDir: null,
+            });
+            await expect(
+            // @ts-expect-error partial mock
+            projectValidateCommand.handler({
+                derivedAccountId: 123,
+                d: false,
+                debug: false,
+            })).rejects.toThrow('Process exited with code 1');
+            expect(uiLogger.error).toHaveBeenCalledWith(commands.project.validate.mustBeRanWithinAProject);
+        });
+        it('should exit with error when project directory is null', async () => {
+            vi.mocked(getProjectConfig).mockResolvedValue({
+                projectConfig: {
+                    name: 'test',
+                    srcDir: 'src',
+                    platformVersion: '2025.2',
+                },
+                projectDir: null,
+            });
+            await expect(
+            // @ts-expect-error partial mock
+            projectValidateCommand.handler({
+                derivedAccountId: 123,
+                d: false,
+                debug: false,
+            })).rejects.toThrow('Process exited with code 1');
+            expect(uiLogger.error).toHaveBeenCalledWith(commands.project.validate.mustBeRanWithinAProject);
+        });
+    });
+    it('should call validateSourceDirectory with correct parameters', async () => {
+        const mockProjectConfig = {
+            name: 'test-project',
+            srcDir: 'src',
+            platformVersion: '2025.2',
+        };
+        vi.mocked(getProjectConfig).mockResolvedValue({
+            projectConfig: mockProjectConfig,
+            projectDir,
+        });
+        vi.mocked(isV2Project).mockReturnValue(true);
+        vi.mocked(validateProjectConfig).mockReturnValue(undefined);
+        vi.mocked(loadAndValidateProfile).mockResolvedValue(123);
+        vi.mocked(getAccountConfig).mockReturnValue({
+            accountType: 'STANDARD',
+            accountId: 123,
+            env: 'prod',
+        });
+        vi.mocked(trackCommandUsage);
+        vi.mocked(validateSourceDirectory).mockResolvedValue(undefined);
+        vi.mocked(handleTranslate).mockResolvedValue(undefined);
+        await expect(projectValidateCommand.handler({
+            derivedAccountId: 123,
+            d: false,
+            debug: false,
+        })).rejects.toThrow('Process exited with code 0');
+        const expectedSrcDir = path.resolve(projectDir, mockProjectConfig.srcDir);
+        expect(validateSourceDirectory).toHaveBeenCalledWith(expectedSrcDir, mockProjectConfig, projectDir);
+    });
+});

package/commands/project/add.js

@@ -3,7 +3,7 @@ import { getProjectConfig } from '../../lib/projects/config.js';
 import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { commands } from '../../lang/en.js';
-import { useV3Api } from '../../lib/projects/platformVersion.js';
+import { isV2Project } from '../../lib/projects/platformVersion.js';
 import { legacyAddComponent } from '../../lib/projects/add/legacyAddComponent.js';
 import { v3AddComponent } from '../../lib/projects/add/v3AddComponent.js';
 import { marketplaceDistribution, oAuth, privateDistribution, staticAuth, } from '../../lib/constants.js';
@@ -18,8 +18,8 @@ async function handler(args) {
             uiLogger.error(commands.project.add.error.locationInProject);
             process.exit(EXIT_CODES.ERROR);
         }
-        const isV3ProjectCreate = useV3Api(projectConfig.platformVersion);
-        if (isV3ProjectCreate) {
+        const isV2ProjectCreate = isV2Project(projectConfig.platformVersion);
+        if (isV2ProjectCreate) {
             await v3AddComponent(args, projectDir, projectConfig, derivedAccountId);
         }
         else {

package/commands/project/cloneApp.js

@@ -1,7 +1,7 @@
 import path from 'path';
 import fs from 'fs';
 import { trackCommandUsage, trackCommandMetadataUsage, } from '../../lib/usageTracking.js';
-import { i18n } from '../../lib/lang.js';
+import { commands } from '../../lang/en.js';
 import { selectPublicAppForMigrationPrompt } from '../../lib/prompts/selectPublicAppForMigrationPrompt.js';
 import { projectNameAndDestPrompt } from '../../lib/prompts/projectNameAndDestPrompt.js';
 import { poll } from '../../lib/polling.js';
@@ -12,7 +12,7 @@ import { writeProjectConfig } from '../../lib/projects/config.js';
 import { PROJECT_CONFIG_FILE } from '../../lib/constants.js';
 import { cloneApp, checkCloneStatus, downloadClonedProject, } from '@hubspot/local-dev-lib/api/projects';
 import { getCwd, sanitizeFileName } from '@hubspot/local-dev-lib/path';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../../lib/ui/logger.js';
 import { extractZipArchive } from '@hubspot/local-dev-lib/archive';
 import { getAccountConfig } from '@hubspot/local-dev-lib/config';
 import SpinniesManager from '../../lib/ui/SpinniesManager.js';
@@ -20,7 +20,7 @@ import { logInvalidAccountError } from '../../lib/app/migrate.js';
 import { makeYargsBuilder } from '../../lib/yargsUtils.js';
 import { uiDeprecatedTag, uiLine, uiAccountDescription, } from '../../lib/ui/index.js';
 const command = 'clone-app';
-const describe = uiDeprecatedTag(i18n(`commands.project.subcommands.cloneApp.describe`), false);
+const describe = uiDeprecatedTag(commands.project.cloneApp.describe, false);
 const deprecated = true;
 async function handler(args) {
     const { derivedAccountId } = args;
@@ -28,7 +28,7 @@ async function handler(args) {
     const accountConfig = getAccountConfig(derivedAccountId);
     const accountName = uiAccountDescription(derivedAccountId);
     if (!accountConfig) {
-        throw new Error(i18n(`commands.projects.subcommands.cloneApp.errors.noAccountConfig`));
+        throw new Error(commands.project.cloneApp.errors.noAccountConfig(derivedAccountId));
     }
     const defaultAccountIsUnified = await isUnifiedAccount(accountConfig);
     if (!isAppDeveloperAccount(accountConfig) && !defaultAccountIsUnified) {
@@ -60,7 +60,7 @@ async function handler(args) {
     try {
         SpinniesManager.init();
         SpinniesManager.add('cloneApp', {
-            text: i18n(`commands.project.subcommands.cloneApp.cloneStatus.inProgress`),
+            text: commands.project.cloneApp.cloneStatus.inProgress,
         });
         const { data: { exportId }, } = await cloneApp(derivedAccountId, appId);
         const { status } = await poll(() => checkCloneStatus(derivedAccountId, exportId));
@@ -84,25 +84,23 @@ async function handler(args) {
             };
             const success = writeProjectConfig(configPath, configContent);
             SpinniesManager.succeed('cloneApp', {
-                text: i18n(`commands.project.subcommands.cloneApp.cloneStatus.done`),
+                text: commands.project.cloneApp.cloneStatus.done,
                 succeedColor: 'white',
             });
             if (!success) {
-                logger.error(i18n(`commands.project.subcommands.cloneApp.errors.couldNotWriteConfigPath`), configPath);
+                uiLogger.error(commands.project.cloneApp.errors.couldNotWriteConfigPath(configPath));
             }
-            logger.log('');
+            uiLogger.log('');
             uiLine();
-            logger.success(i18n(`commands.project.subcommands.cloneApp.cloneStatus.success`, {
-                dest: projectDest,
-            }));
-            logger.log('');
+            uiLogger.success(commands.project.cloneApp.cloneStatus.success(projectDest));
+            uiLogger.log('');
             process.exit(EXIT_CODES.SUCCESS);
         }
     }
     catch (error) {
         await trackCommandMetadataUsage('clone-app', { successful: false }, derivedAccountId);
         SpinniesManager.fail('cloneApp', {
-            text: i18n(`commands.project.subcommands.cloneApp.cloneStatus.failure`),
+            text: commands.project.cloneApp.cloneStatus.failure,
             failColor: 'white',
         });
         // Migrations endpoints return a response object with an errors property. The errors property contains an array of errors.
@@ -122,19 +120,16 @@ async function handler(args) {
 function cloneAppBuilder(yargs) {
     yargs.options({
         dest: {
-            describe: i18n(`commands.project.subcommands.cloneApp.options.dest.describe`),
+            describe: commands.project.cloneApp.options.dest.describe,
             type: 'string',
         },
         'app-id': {
-            describe: i18n(`commands.project.subcommands.cloneApp.options.appId.describe`),
+            describe: commands.project.cloneApp.options.appId.describe,
             type: 'number',
         },
     });
     yargs.example([
-        [
-            '$0 project clone-app',
-            i18n(`commands.project.subcommands.cloneApp.examples.default`),
-        ],
+        ['$0 project clone-app', commands.project.cloneApp.examples.default],
     ]);
     return yargs;
 }