@hubspot/cli 7.7.34-experimental.0 → 7.8.0-beta.0

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/mcp/__tests__/setup.test.js

@@ -3,12 +3,12 @@ vi.mock('../../../lib/commonOpts');
 describe('commands/mcp/setup', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(setupCommand.command).toEqual(['setup', 'update']);
+            expect(setupCommand.command).toEqual(['setup']);
         });
     });
     describe('describe', () => {
         it('should be undefined to keep the command hidden', () => {
-            expect(setupCommand.describe).toBeUndefined();
+            expect(setupCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {

package/commands/mcp/setup.js

@@ -4,9 +4,18 @@ import { commands } from '../../lang/en.js';
 import { uiLogger } from '../../lib/ui/logger.js';
 import { addMcpServerToConfig, supportedTools } from '../../lib/mcp/setup.js';
 import { trackCommandUsage } from '../../lib/usageTracking.js';
-const command = ['setup', 'update'];
-const describe = undefined; // Leave hidden for now
+import { hasFeature } from '../../lib/hasFeature.js';
+import { FEATURES } from '../../lib/constants.js';
+import { uiBetaTag } from '../../lib/ui/index.js';
+const command = ['setup'];
+const describe = uiBetaTag(commands.mcp.setup.describe, false);
 async function handler(args) {
+    const { derivedAccountId } = args;
+    const hasMcpAccess = await hasFeature(derivedAccountId, FEATURES.MCP_ACCESS);
+    if (!hasMcpAccess) {
+        uiLogger.error(commands.mcp.setup.errors.needsMcpAccess(derivedAccountId));
+        process.exit(EXIT_CODES.ERROR);
+    }
     try {
         await import('@modelcontextprotocol/sdk/server/mcp.js');
     }

package/commands/mcp.js

@@ -2,14 +2,14 @@ import startCommand from './mcp/start.js';
 import setupCommand from './mcp/setup.js';
 import { makeYargsBuilder } from '../lib/yargsUtils.js';
 import { commands } from '../lang/en.js';
+import { uiBetaTag } from '../lib/ui/index.js';
 const command = 'mcp';
-// Leave this as undefined to hide the command
-const describe = undefined;
+const describe = uiBetaTag(commands.mcp.describe, false);
 function mcpBuilder(yargs) {
     yargs.command(startCommand).command(setupCommand).demandCommand(1, '');
     return yargs;
 }
-const builder = makeYargsBuilder(mcpBuilder, command, commands.mcp.describe, {
+const builder = makeYargsBuilder(mcpBuilder, command, describe, {
     useGlobalOptions: true,
 });
 const mcpCommand = {

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__/create.test.js

@@ -48,10 +48,10 @@ describe('commands/project/create', () => {
             projectCreateCommand.builder(yargsMock);
             const optionsCall = optionsSpy.mock.calls[0][0];
             expect(optionsCall['platform-version']).toEqual(expect.objectContaining({
-                hidden: true,
+                describe: 'The target platform version for the new project.',
                 type: 'string',
                 choices: ['2023.2', '2025.1', '2025.2'],
-                default: '2023.2',
+                default: '2025.2',
             }));
         });
         it('should define project base option with correct choices', () => {
@@ -59,7 +59,7 @@ describe('commands/project/create', () => {
             projectCreateCommand.builder(yargsMock);
             const optionsCall = optionsSpy.mock.calls[0][0];
             expect(optionsCall['project-base']).toEqual(expect.objectContaining({
-                hidden: true,
+                describe: 'The top level component to include in the project.',
                 type: 'string',
                 choices: ['empty', 'app'],
             }));
@@ -69,7 +69,7 @@ describe('commands/project/create', () => {
             projectCreateCommand.builder(yargsMock);
             const optionsCall = optionsSpy.mock.calls[0][0];
             expect(optionsCall.distribution).toEqual(expect.objectContaining({
-                hidden: true,
+                describe: 'How the app will be distributed.',
                 type: 'string',
                 choices: ['private', 'marketplace'],
             }));
@@ -79,7 +79,7 @@ describe('commands/project/create', () => {
             projectCreateCommand.builder(yargsMock);
             const optionsCall = optionsSpy.mock.calls[0][0];
             expect(optionsCall.auth).toEqual(expect.objectContaining({
-                hidden: true,
+                describe: 'Authentication model for the application.',
                 type: 'string',
                 choices: ['oauth', 'static'],
             }));
@@ -89,7 +89,7 @@ describe('commands/project/create', () => {
             projectCreateCommand.builder(yargsMock);
             const optionsCall = optionsSpy.mock.calls[0][0];
             expect(optionsCall.features).toEqual(expect.objectContaining({
-                hidden: true,
+                describe: 'Features to include in the project. Only valid if project-base is app',
                 type: 'array',
             }));
         });

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');
@@ -62,9 +61,6 @@ describe('commands/project/deploy', () => {
         });
     });
     describe('describe', () => {
-        it('should contain the beta tag', () => {
-            expect(projectDeployCommand.describe).toContain('[BETA]');
-        });
         it('should provide a description', () => {
             expect(projectDeployCommand.describe).toBeDefined();
         });
@@ -300,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__/devUnifiedFlow.test.js

@@ -324,8 +324,7 @@ describe('unifiedProjectDevFlow', () => {
             });
             expect(createNewProjectForLocalDev).not.toHaveBeenCalled();
             expect(LocalDevProcess).toHaveBeenCalledWith(expect.objectContaining({
-                projectId: mockProject.id,
-                projectName: mockProject.name,
+                projectData: mockProject,
             }));
         });
     });
@@ -346,8 +345,7 @@ describe('unifiedProjectDevFlow', () => {
                 targetTestingAccountId: mockProvidedTargetTestingAccountId,
                 projectConfig: mockProjectConfig,
                 projectDir: mockProjectDir,
-                projectName: mockProject.name,
-                projectId: mockProject.id,
+                projectData: mockProject,
                 env: ENVIRONMENTS.PROD,
             });
         });

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');
@@ -45,9 +45,6 @@ describe('commands/project/logs', () => {
         });
     });
     describe('describe', () => {
-        it('should contain the beta tag', () => {
-            expect(projectLogsCommand.describe).toContain('[BETA]');
-        });
         it('should provide a description', () => {
             expect(projectLogsCommand.describe).toBeDefined();
         });

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' },
@@ -44,7 +44,7 @@ describe('commands/project/migrate', () => {
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(migrateCommand.describe).toBe(undefined);
+            expect(migrateCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
@@ -54,7 +54,6 @@ describe('commands/project/migrate', () => {
                 type: 'string',
                 choices: [v2025_2],
                 default: v2025_2,
-                hidden: true,
             });
             expect(optionsSpy).toHaveBeenCalledWith('unstable', {
                 type: 'boolean',
@@ -75,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));
@@ -57,8 +54,7 @@ describe('commands/project/migrateApp', () => {
                 'platform-version': expect.objectContaining({
                     type: 'string',
                     choices: [v2023_2, v2025_2],
-                    hidden: true,
-                    default: v2023_2,
+                    default: v2025_2,
                 }),
             });
             expect(exampleSpy).toHaveBeenCalled();

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

@@ -19,7 +19,7 @@ describe('commands/project', () => {
     });
     describe('describe', () => {
         it('should not provide a description', () => {
-            expect(profileCommand.describe).not.toBeDefined();
+            expect(profileCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {