@hubspot/cli 7.7.35-experimental.0 → 7.8.0-beta.1

package/lang/en.lyaml

@@ -2,7 +2,7 @@ en:
   commands:
     generalErrors:
       updateNotify:
-        notifyTitle: "Update available"
+        notifyTitle: "CLI update available"
         cmsUpdateNotification: "{{#bold}}The CMS CLI is now the HubSpot CLI{{/bold}}\n\nTo upgrade, uninstall {{#bold}}{{ packageName }}{{/bold}}\nand then run {{ updateCommand }}"
         cliUpdateNotification: "HubSpot CLI version {{#cyan}}{{#bold}}{currentVersion}{{/bold}}{{/cyan}} is outdated.\nRun {{ updateCommand }} to upgrade to version {{#cyan}}{{#bold}}{latestVersion}{{/bold}}{{/cyan}}"
       srcIsProject: "\"{{ src }}\" is in a project folder. Did you mean \"hs project {{command}}\"?"
@@ -801,7 +801,7 @@ en:
             uploadFailed: "Failed to upload file \"{{ filePath }}\" to \"{{ remotePath }}\""
             deleteFileFailed: "Failed to delete file \"{{ remotePath }}\""
             deleteFolderFailed: "Failed to delete folder \"{{ remotePath }}\""
-            v3ApiError: "{{ command }} is not supported for platform version '{{ platformVersion}}', use {{ newCommand }} instead to develop locally. {{ linkToDocs }}"
+            v2ApiError: "{{ command }} is not supported for platform version '{{ platformVersion}}', use {{ newCommand }} instead to develop locally. {{ linkToDocs }}"
         download:
           describe: "Download your project files from HubSpot."
           examples:
@@ -1162,7 +1162,7 @@ en:
         createNewSandboxOption: "<Test on a new development sandbox>"
         createNewDeveloperTestAccountOption: "<Test on a new developer test account>"
         chooseDefaultAccountOption: "<{{#bold}}\U00002757{{/bold}} Test on this production account {{#bold}}\U00002757{{/bold}}>"
-        promptMessage: "[--account] Choose a {{ accountType }} under {{ accountIdentifier }} to test with:"
+        promptMessage: "Choose a {{ accountType }} under {{ accountIdentifier }} to test with:"
         sandboxLimit: "Your account reached the limit of {{ limit }} development sandboxes"
         sandboxLimitWithSuggestion: "Your account reached the limit of {{ limit }} development sandboxes. Run {{ authCommand }} to add an existing one to the config."
         developerTestAccountLimit: "Your account reached the limit of {{ limit }} developer test accounts."
@@ -1348,7 +1348,7 @@ en:
             succeed: "Created {{#bold}}{{ accountName }} [standard sandbox] ({{ accountId }}){{/bold}}."
         failure:
           invalidUser: "Couldn't create {{#bold}}{{ accountName }}{{/bold}} because your account has been removed from {{#bold}}{{ parentAccountName }}{{/bold}} or your permission set doesn't allow you to create the sandbox. To update your permissions, contact a super admin in {{#bold}}{{ parentAccountName }}{{/bold}}."
-          403Gating: "Couldn't create {{#bold}}{{ accountName }}{{/bold}} because {{#bold}}{{ parentAccountName }}{{/bold}} does not have access to development sandboxes. To opt in to the CRM Development Beta and use development sandboxes, visit https://app.hubspot.com/l/product-updates/in-beta?update=13899236."
+          403Gating: "We couldn't create {{#bold}}{{ accountName }}{{/bold}} because {{#bold}}{{ parentAccountName }}{{/bold}} is a developer account and does not have access to development sandboxes. To gain access to developer sandboxes, you can convert your account: https://developers.hubspot.com/docs/getting-started/convert-your-account-into-a-developer-crm-account"
           usageLimitsFetch: "Unable to fetch sandbox usage limits. Please try again."
           generic: "An error occurred while creating a new sandbox. Please try again."
           limit:

package/lib/__tests__/buildAccount.test.js

@@ -10,7 +10,8 @@ vi.mock('@hubspot/local-dev-lib/personalAccessKey');
 vi.mock('@hubspot/local-dev-lib/config');
 vi.mock('@hubspot/local-dev-lib/api/developerTestAccounts');
 vi.mock('@hubspot/local-dev-lib/api/sandboxHubs');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../ui/logger.js');
+vi.mock('../errorHandlers/index.js');
 vi.mock('../prompts/personalAccessKeyPrompt');
 vi.mock('../prompts/accountNamePrompt');
 vi.mock('../ui/SpinniesManager', () => ({
@@ -101,7 +102,7 @@ describe('lib/buildAccount', () => {
             expect(result).toBe('test-account-with-new-name');
         });
     });
-    describe('createDeveloperTestAccountV3()', () => {
+    describe('createDeveloperTestAccountV2()', () => {
         const parentAccountId = 123456;
         const mockDeveoperTestAccountConfig = {
             accountName: 'Developer Test Account',
@@ -119,7 +120,7 @@ describe('lib/buildAccount', () => {
             });
         });
         it('should create a developer test account successfully', async () => {
-            const result = await buildAccount.createDeveloperTestAccountV3(parentAccountId, mockDeveoperTestAccountConfig);
+            const result = await buildAccount.createDeveloperTestAccountV2(parentAccountId, mockDeveoperTestAccountConfig);
             expect(result).toEqual({
                 accountName: mockDeveoperTestAccountConfig.accountName,
                 accountId: 123456,

package/lib/__tests__/commonOpts.test.js

@@ -12,7 +12,7 @@ function buildArguments(args) {
     };
 }
 vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../ui/logger.js');
 describe('lib/commonOpts', () => {
     describe('getCmsPublishMode()', () => {
         const accounts = {

package/lib/__tests__/dependencyManagement.test.js

@@ -6,7 +6,7 @@ import { getProjectConfig } from '../projects/config.js';
 import SpinniesManager from '../ui/SpinniesManager.js';
 import fs from 'fs';
 vi.mock('../projects/config');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../ui/logger.js');
 vi.mock('@hubspot/local-dev-lib/fs');
 vi.mock('fs');
 vi.mock('../ui/SpinniesManager', () => ({

package/lib/__tests__/developerTestAccounts.test.js

@@ -1,12 +1,12 @@
 import { getAccountId, getConfigAccounts } from '@hubspot/local-dev-lib/config';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
 import { fetchDeveloperTestAccounts } from '@hubspot/local-dev-lib/api/developerTestAccounts';
 import { mockHubSpotHttpError } from '../testUtils.js';
 import * as errorHandlers from '../errorHandlers/index.js';
 import { getHasDevTestAccounts, handleDeveloperTestAccountCreateError, validateDevTestAccountUsageLimits, } from '../developerTestAccounts.js';
 vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../ui/logger.js');
 vi.mock('@hubspot/local-dev-lib/api/developerTestAccounts');
 vi.mock('../errorHandlers');
 const mockedGetAccountId = getAccountId;
@@ -101,7 +101,7 @@ describe('lib/developerTestAccounts', () => {
         let loggerErrorSpy;
         let logErrorSpy;
         beforeEach(() => {
-            loggerErrorSpy = vi.spyOn(logger, 'error');
+            loggerErrorSpy = vi.spyOn(uiLogger, 'error');
             logErrorSpy = vi.spyOn(errorHandlers, 'logError');
         });
         afterEach(() => {

package/lib/__tests__/npm.test.js

@@ -1,7 +1,7 @@
 import util from 'util';
 import { isGloballyInstalled, getLatestCliVersion, DEFAULT_PACKAGE_MANAGER, } from '../npm.js';
 import pkg from '../../package.json' with { type: 'json' };
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../ui/logger.js');
 vi.mock('../ui/SpinniesManager');
 describe('lib/npm', () => {
     let execMock;

package/lib/__tests__/oauth.test.js

@@ -3,7 +3,7 @@ import open from 'open';
 import { OAuth2Manager } from '@hubspot/local-dev-lib/models/OAuth2Manager';
 import { getAccountConfig } from '@hubspot/local-dev-lib/config';
 import { addOauthToAccountConfig } from '@hubspot/local-dev-lib/oauth';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { ENVIRONMENTS } from '@hubspot/local-dev-lib/constants/environments';
 import { DEFAULT_OAUTH_SCOPES } from '@hubspot/local-dev-lib/constants/auth';
 import { authenticateWithOauth } from '../oauth.js';
@@ -12,7 +12,7 @@ vi.mock('open');
 vi.mock('@hubspot/local-dev-lib/models/OAuth2Manager');
 vi.mock('@hubspot/local-dev-lib/config');
 vi.mock('@hubspot/local-dev-lib/oauth');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../ui/logger.js');
 const mockedExpress = express;
 const mockedOAuth2Manager = OAuth2Manager;
 const mockedGetAccountConfig = getAccountConfig;
@@ -59,7 +59,7 @@ describe('lib/oauth', () => {
                 env: ENVIRONMENTS.PROD,
             });
             // Verify logger was called
-            expect(logger.log).toHaveBeenCalledWith('Authorizing');
+            expect(uiLogger.log).toHaveBeenCalledWith('Authorizing');
             // Verify OAuth tokens were added to config
             expect(addOauthToAccountConfig).toHaveBeenCalledWith(mockOAuth2Manager);
         });
@@ -75,7 +75,7 @@ describe('lib/oauth', () => {
                 throw new Error('exit');
             });
             await expect(authenticateWithOauth(invalidConfig)).rejects.toThrow('exit');
-            expect(logger.error).toHaveBeenCalled();
+            expect(uiLogger.error).toHaveBeenCalled();
             expect(exitSpy).toHaveBeenCalled();
             exitSpy.mockRestore();
         });

package/lib/__tests__/process.test.js

@@ -1,10 +1,15 @@
 import readline from 'readline';
-import { logger, setLogLevel, LOG_LEVEL } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
+import { setLogLevel, LOG_LEVEL } from '@hubspot/local-dev-lib/logger';
 import { handleExit, handleKeypress, TERMINATION_SIGNALS } from '../process.js';
 vi.mock('readline');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../ui/logger.js');
+vi.mock('@hubspot/local-dev-lib/logger', async () => {
+    const logger = await vi.importActual('@hubspot/local-dev-lib/logger');
+    return { ...logger, setLogLevel: vi.fn() };
+});
 const mockedReadline = readline;
-const mockedLogger = logger;
+const mockedUiLogger = uiLogger;
 const mockedSetLogLevel = setLogLevel;
 const processRemoveListenerSpy = vi.spyOn(process, 'removeAllListeners');
 const processOnSpy = vi.spyOn(process, 'on');
@@ -26,7 +31,7 @@ describe('lib/process', () => {
             sighupCallback();
             expect(mockedSetLogLevel).toHaveBeenCalledWith(LOG_LEVEL.NONE);
             expect(mockCallback).toHaveBeenCalledWith({ isSIGHUP: true });
-            expect(mockedLogger.debug).toHaveBeenCalledWith(expect.stringContaining('SIGHUP'));
+            expect(mockedUiLogger.debug).toHaveBeenCalledWith(expect.stringContaining('SIGHUP'));
         });
         it('should handle non-SIGHUP signals correctly', async () => {
             handleExit(mockCallback);
@@ -36,7 +41,7 @@ describe('lib/process', () => {
             sigintCallback();
             expect(mockedSetLogLevel).not.toHaveBeenCalled();
             expect(mockCallback).toHaveBeenCalledWith({ isSIGHUP: false });
-            expect(mockedLogger.debug).toHaveBeenCalledWith(expect.stringContaining('SIGINT'));
+            expect(mockedUiLogger.debug).toHaveBeenCalledWith(expect.stringContaining('SIGINT'));
         });
         it('should prevent duplicate exit handling', async () => {
             handleExit(mockCallback);

package/lib/__tests__/sandboxSync.test.js

@@ -1,4 +1,4 @@
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { initiateSync } from '@hubspot/local-dev-lib/api/sandboxSync';
 import { getAccountId } from '@hubspot/local-dev-lib/config';
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
@@ -6,12 +6,12 @@ import { mockHubSpotHttpError } from '../testUtils.js';
 import { getAvailableSyncTypes } from '../sandboxes.js';
 import { syncSandbox } from '../sandboxSync.js';
 import SpinniesManager from '../ui/SpinniesManager.js';
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../ui/logger.js');
 vi.mock('@hubspot/local-dev-lib/api/sandboxSync');
 vi.mock('@hubspot/local-dev-lib/config');
 vi.mock('../sandboxes');
 vi.mock('../ui/SpinniesManager');
-const mockedLogger = logger;
+const mockedUiLogger = uiLogger;
 const mockedInitiateSync = initiateSync;
 const mockedGetAccountId = getAccountId;
 const mockedGetAvailableSyncTypes = getAvailableSyncTypes;
@@ -73,7 +73,7 @@ describe('lib/sandboxSync', () => {
             mockedInitiateSync.mockRejectedValue(error);
             await expect(syncSandbox(mockChildAccount, mockParentAccount, mockEnv, mockSyncTasks)).rejects.toEqual(error);
             expect(mockedSpinniesFail).toHaveBeenCalled();
-            expect(mockedLogger.error).toHaveBeenCalledWith(expect.stringMatching(/Couldn't run the sync because there's another sync in progress/));
+            expect(mockedUiLogger.error).toHaveBeenCalledWith(expect.stringMatching(/Couldn't run the sync because there's another sync in progress/));
         });
         it('handles invalid user error', async () => {
             const error = mockHubSpotHttpError('', {
@@ -86,7 +86,7 @@ describe('lib/sandboxSync', () => {
             mockedInitiateSync.mockRejectedValue(error);
             await expect(syncSandbox(mockChildAccount, mockParentAccount, mockEnv, mockSyncTasks)).rejects.toEqual(error);
             expect(mockedSpinniesFail).toHaveBeenCalled();
-            expect(mockedLogger.error).toHaveBeenCalledWith(expect.stringMatching(/because your account has been removed from/));
+            expect(mockedUiLogger.error).toHaveBeenCalledWith(expect.stringMatching(/because your account has been removed from/));
         });
         it('handles not super admin error', async () => {
             const error = mockHubSpotHttpError('', {
@@ -99,7 +99,7 @@ describe('lib/sandboxSync', () => {
             mockedInitiateSync.mockRejectedValue(error);
             await expect(syncSandbox(mockChildAccount, mockParentAccount, mockEnv, mockSyncTasks)).rejects.toEqual(error);
             expect(mockedSpinniesFail).toHaveBeenCalled();
-            expect(mockedLogger.error).toHaveBeenCalledWith(expect.stringMatching(/Couldn't run the sync because you are not a super admin in/));
+            expect(mockedUiLogger.error).toHaveBeenCalledWith(expect.stringMatching(/Couldn't run the sync because you are not a super admin in/));
         });
         it('handles sandbox not found error', async () => {
             const error = mockHubSpotHttpError('', {
@@ -112,12 +112,12 @@ describe('lib/sandboxSync', () => {
             mockedInitiateSync.mockRejectedValue(error);
             await expect(syncSandbox(mockChildAccount, mockParentAccount, mockEnv, mockSyncTasks)).rejects.toEqual(error);
             expect(mockedSpinniesFail).toHaveBeenCalled();
-            expect(mockedLogger.error).toHaveBeenCalledWith(expect.stringMatching(/may have been deleted through the UI/));
+            expect(mockedUiLogger.error).toHaveBeenCalledWith(expect.stringMatching(/may have been deleted through the UI/));
         });
         it('displays slim info message when specified', async () => {
             mockedInitiateSync.mockResolvedValue({ status: 'SUCCESS' });
             await syncSandbox(mockChildAccount, mockParentAccount, mockEnv, mockSyncTasks, true);
-            expect(mockedLogger.info).not.toHaveBeenCalled();
+            expect(mockedUiLogger.info).not.toHaveBeenCalled();
             expect(mockedSpinniesSucceed).toHaveBeenCalledWith('sandboxSync', expect.objectContaining({
                 text: expect.stringMatching(/Initiated sync of object definitions from production to /),
             }));

package/lib/__tests__/sandboxes.test.js

@@ -1,11 +1,11 @@
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { getSandboxUsageLimits } from '@hubspot/local-dev-lib/api/sandboxHubs';
 import { fetchTypes } from '@hubspot/local-dev-lib/api/sandboxSync';
 import { getAccountId, getConfigAccounts } from '@hubspot/local-dev-lib/config';
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
 import { mockHubSpotHttpError } from '../testUtils.js';
 import { getSandboxTypeAsString, getHasSandboxesByType, getAvailableSyncTypes, validateSandboxUsageLimits, handleSandboxCreateError, } from '../sandboxes.js';
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../ui/logger.js');
 vi.mock('@hubspot/local-dev-lib/api/sandboxHubs');
 vi.mock('@hubspot/local-dev-lib/api/sandboxSync');
 vi.mock('@hubspot/local-dev-lib/config');
@@ -13,7 +13,7 @@ const mockedGetAccountId = getAccountId;
 const mockedGetSandboxUsageLimits = getSandboxUsageLimits;
 const mockedFetchTypes = fetchTypes;
 const mockedGetConfigAccounts = getConfigAccounts;
-const mockedLogger = logger;
+const mockedUiLogger = uiLogger;
 describe('lib/sandboxes', () => {
     describe('getSandboxTypeAsString()', () => {
         it('returns "development" for development sandbox type', () => {
@@ -101,7 +101,7 @@ describe('lib/sandboxes', () => {
                     usage: { DEVELOPER: { available: 0, limit: 1 } },
                 },
             });
-            await expect(validateSandboxUsageLimits(mockAccount, HUBSPOT_ACCOUNT_TYPES.DEVELOPMENT_SANDBOX, 'qa')).rejects.toThrow(/reached the limit of 1 development sandbox/);
+            await expect(validateSandboxUsageLimits(mockAccount, HUBSPOT_ACCOUNT_TYPES.DEVELOPMENT_SANDBOX, 'qa')).rejects.toThrow(/reached the limit of 1 developer sandboxes/);
         });
     });
     describe('handleSandboxCreateError()', () => {
@@ -117,8 +117,8 @@ describe('lib/sandboxes', () => {
                 },
             });
             expect(() => handleSandboxCreateError(error, mockEnv, mockName, mockAccountId)).toThrow(error);
-            expect(mockedLogger.error).toHaveBeenCalledWith(expect.stringMatching(/The personal access key you provided doesn't include sandbox permissions/));
-            expect(mockedLogger.info).toHaveBeenCalledWith(expect.stringMatching(/To update CLI permissions for/));
+            expect(mockedUiLogger.error).toHaveBeenCalledWith(expect.stringMatching(/The personal access key you provided doesn't include sandbox permissions/));
+            expect(mockedUiLogger.info).toHaveBeenCalledWith(expect.stringMatching(/To update CLI permissions for/));
         });
         it('handles user access not allowed error', () => {
             const error = mockHubSpotHttpError('user access not allowed error', {
@@ -129,7 +129,7 @@ describe('lib/sandboxes', () => {
                 },
             });
             expect(() => handleSandboxCreateError(error, mockEnv, mockName, mockAccountId)).toThrow(error);
-            expect(mockedLogger.error).toHaveBeenCalledWith(expect.stringMatching(/your permission set doesn't allow you to create the sandbox/));
+            expect(mockedUiLogger.error).toHaveBeenCalledWith(expect.stringMatching(/your permission set doesn't allow you to create the sandbox/));
         });
         it('handles 403 gating error', () => {
             const error = mockHubSpotHttpError('403 gating error', {
@@ -140,7 +140,7 @@ describe('lib/sandboxes', () => {
                 },
             });
             expect(() => handleSandboxCreateError(error, mockEnv, mockName, mockAccountId)).toThrow(error);
-            expect(mockedLogger.error).toHaveBeenCalledWith(expect.stringMatching(/does not have access to development sandboxes/));
+            expect(mockedUiLogger.error).toHaveBeenCalledWith(expect.stringMatching(/does not have access to development sandboxes/));
         });
     });
 });

package/lib/__tests__/serverlessLogs.test.js

@@ -2,7 +2,7 @@ import mockStdIn from 'mock-stdin';
 import { outputLogs } from '../ui/serverlessFunctionLogs.js';
 import { tailLogs } from '../serverlessLogs.js';
 vi.mock('../ui/serverlessFunctionLogs');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../ui/logger.js');
 vi.mock('../ui/SpinniesManager', () => ({
     default: {
         init: vi.fn(),

package/lib/__tests__/usageTracking.test.js

@@ -1,17 +1,17 @@
 import { trackUsage } from '@hubspot/local-dev-lib/trackUsage';
 import { isTrackingAllowed, getAccountConfig, } from '@hubspot/local-dev-lib/config';
 import { API_KEY_AUTH_METHOD } from '@hubspot/local-dev-lib/constants/auth';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { trackCommandUsage, trackHelpUsage, trackConvertFieldsUsage, trackAuthAction, trackCommandMetadataUsage, } from '../usageTracking.js';
 import packageJson from '../../package.json' with { type: 'json' };
 const version = packageJson.version;
 vi.mock('@hubspot/local-dev-lib/trackUsage');
 vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../ui/logger.js');
 const mockedTrackUsage = trackUsage;
 const mockedIsTrackingAllowed = isTrackingAllowed;
 const mockedGetAccountConfig = getAccountConfig;
-const mockedLogger = logger;
+const mockedUiLogger = uiLogger;
 describe('lib/usageTracking', () => {
     const mockPlatform = 'darwin';
     const mockNodeVersion = 'v16.14.0';
@@ -53,7 +53,7 @@ describe('lib/usageTracking', () => {
                 throw error;
             });
             await trackCommandUsage(mockCommand);
-            expect(mockedLogger.debug).toHaveBeenCalledWith(expect.stringContaining(error.message));
+            expect(mockedUiLogger.debug).toHaveBeenCalledWith(expect.stringContaining(error.message));
         });
     });
     describe('trackHelpUsage()', () => {
@@ -76,7 +76,7 @@ describe('lib/usageTracking', () => {
         });
         it('should track main help usage without command', async () => {
             await trackHelpUsage('');
-            expect(mockedLogger.debug).toHaveBeenCalledWith(expect.stringContaining('main command'));
+            expect(mockedUiLogger.debug).toHaveBeenCalledWith(expect.stringContaining('main command'));
         });
     });
     describe('trackConvertFieldsUsage()', () => {

package/lib/__tests__/validation.test.js

@@ -3,7 +3,8 @@ import { getOauthManager } from '@hubspot/local-dev-lib/oauth';
 import { accessTokenForPersonalAccessKey } from '@hubspot/local-dev-lib/personalAccessKey';
 import { validateAccount } from '../validation.js';
 vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../lib/ui/logger.js');
+vi.mock('../errorHandlers/index.js');
 vi.mock('@hubspot/local-dev-lib/oauth');
 vi.mock('@hubspot/local-dev-lib/personalAccessKey');
 vi.mock('../commonOpts');

package/lib/__tests__/yargsUtils.test.js

@@ -1,6 +1,19 @@
-import { hasFlag, makeYargsBuilder, getExclusiveConflicts, } from '../yargsUtils.js';
+import { hasFlag, makeYargsBuilder, strictEnforceBoolean, } from '../yargsUtils.js';
 import * as commonOpts from '../commonOpts.js';
 vi.mock('../commonOpts');
+vi.mock('../../lang/en.js', () => ({
+    commands: {
+        config: {
+            subcommands: {
+                set: {
+                    errors: {
+                        invalidBoolean: (option, value) => `Invalid boolean value "${value}" for --${option}. Valid values are: true, false`,
+                    },
+                },
+            },
+        },
+    },
+}));
 const argvWithFlag = ['hs', 'command', '--test'];
 const argvWithoutFlag = ['hs', 'command'];
 describe('lib/yargsUtils', () => {
@@ -36,15 +49,76 @@ describe('lib/yargsUtils', () => {
             expect(commonOpts.addCustomHelpOutput).toHaveBeenCalled();
         });
     });
-    describe('getExclusiveConflicts()', () => {
-        it('should return an object where each option conflicts with all others', () => {
-            const options = ['option1', 'option2', 'option3'];
-            const result = getExclusiveConflicts(options);
-            expect(result).toEqual({
-                option1: ['option2', 'option3'],
-                option2: ['option1', 'option3'],
-                option3: ['option1', 'option2'],
+    describe('strictEnforceBoolean()', () => {
+        it('should validate valid boolean values (true/false, case-insensitive)', () => {
+            const validArgs = [
+                ['hs', 'config', 'set', '--allow-usage-tracking=true'],
+                ['hs', 'config', 'set', '--allow-usage-tracking=false'],
+                ['hs', 'config', 'set', '--allow-usage-tracking=TRUE'],
+                ['hs', 'config', 'set', '--allow-usage-tracking=False'],
+            ];
+            validArgs.forEach(args => {
+                expect(() => strictEnforceBoolean(args, ['allow-usage-tracking'])).not.toThrow();
+                expect(strictEnforceBoolean(args, ['allow-usage-tracking'])).toBe(true);
             });
         });
+        it('should reject invalid boolean values with descriptive error messages', () => {
+            const invalidCases = [
+                {
+                    args: ['hs', 'config', 'set', '--allow-usage-tracking=yes'],
+                    value: 'yes',
+                },
+                {
+                    args: ['hs', 'config', 'set', '--allow-usage-tracking=1'],
+                    value: '1',
+                },
+                {
+                    args: ['hs', 'config', 'set', '--auto-open-browser=maybe'],
+                    value: 'maybe',
+                },
+            ];
+            invalidCases.forEach(({ args, value }) => {
+                const option = args[3].split('=')[0].replace('--', '');
+                expect(() => strictEnforceBoolean(args, [option])).toThrow(`Invalid boolean value "${value}" for --${option}. Valid values are: true, false`);
+            });
+        });
+        it('should support multiple config values in a single command', () => {
+            const args = [
+                'hs',
+                'config',
+                'set',
+                '--allow-usage-tracking=true',
+                '--auto-open-browser=false',
+                '--allow-auto-updates=true',
+                '--http-timeout=5000',
+                '--default-cms-publish-mode=draft',
+            ];
+            const booleanOptions = [
+                'allow-usage-tracking',
+                'auto-open-browser',
+                'allow-auto-updates',
+            ];
+            expect(() => strictEnforceBoolean(args, booleanOptions)).not.toThrow();
+            expect(strictEnforceBoolean(args, booleanOptions)).toBe(true);
+        });
+        it('should error when one of multiple values is invalid', () => {
+            const args = [
+                'hs',
+                'config',
+                'set',
+                '--allow-usage-tracking=true',
+                '--auto-open-browser=invalid',
+                '--http-timeout=5000',
+            ];
+            const booleanOptions = ['allow-usage-tracking', 'auto-open-browser'];
+            expect(() => strictEnforceBoolean(args, booleanOptions)).toThrow('Invalid boolean value "invalid" for --auto-open-browser. Valid values are: true, false');
+        });
+        it('should maintain backwards compatability', () => {
+            expect(() => strictEnforceBoolean(['hs', 'config', 'set', '--allow-usage-tracking=true'], ['allow-usage-tracking'])).not.toThrow();
+            // No values provided (interactive mode)
+            expect(() => strictEnforceBoolean(['hs', 'config', 'set'], ['allow-usage-tracking'])).not.toThrow();
+            // Flag without equals sign (succeed with true)
+            expect(() => strictEnforceBoolean(['hs', 'config', 'set', '--allow-usage-tracking'], ['allow-usage-tracking'])).not.toThrow();
+        });
     });
 });

package/lib/app/__tests__/migrate.test.js

@@ -1,7 +1,7 @@
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../../ui/logger.js';
 import { getCwd, sanitizeFileName } from '@hubspot/local-dev-lib/path';
 import { extractZipArchive } from '@hubspot/local-dev-lib/archive';
-import { validateUid, getProjectThemeDetails, } from '@hubspot/project-parsing-lib';
+import { validateUid } from '@hubspot/project-parsing-lib';
 import { UNMIGRATABLE_REASONS } from '@hubspot/local-dev-lib/constants/projects';
 import { MIGRATION_STATUS } from '@hubspot/local-dev-lib/types/Migration';
 import { downloadProject } from '@hubspot/local-dev-lib/api/projects';
@@ -12,8 +12,8 @@ import { poll } from '../../polling.js';
 import { CLI_UNMIGRATABLE_REASONS, continueAppMigration, initializeAppMigration, listAppsForMigration, } from '../../../api/migrate.js';
 import { lib } from '../../../lang/en.js';
 import { hasUnfiedAppsAccess } from '../../hasFeature.js';
-import { getUnmigratableReason, generateFilterAppsByProjectNameFunction, buildErrorMessageFromMigrationStatus, fetchMigrationApps, promptForAppToMigrate, selectAppToMigrate, handleMigrationSetup, getHasMigratableThemes, beginAppMigration, pollMigrationStatus, finalizeAppMigration, downloadProjectFiles, migrateApp2025_2, logInvalidAccountError, validateMigrationAppsAndThemes, } from '../migrate.js';
-vi.mock('@hubspot/local-dev-lib/logger');
+import { getUnmigratableReason, generateFilterAppsByProjectNameFunction, buildErrorMessageFromMigrationStatus, fetchMigrationApps, promptForAppToMigrate, selectAppToMigrate, handleMigrationSetup, beginAppMigration, pollMigrationStatus, finalizeAppMigration, downloadProjectFiles, migrateApp2025_2, logInvalidAccountError, validateMigrationApps, } from '../migrate.js';
+vi.mock('../../ui/logger.js');
 vi.mock('@hubspot/local-dev-lib/path');
 vi.mock('@hubspot/local-dev-lib/archive');
 vi.mock('@hubspot/project-parsing-lib');
@@ -28,7 +28,7 @@ vi.mock('../../../api/migrate');
 vi.mock('../../hasFeature');
 vi.mock('../../projects/urls');
 vi.mock('fs');
-const mockedLogger = logger;
+const mockedUiLogger = uiLogger;
 const mockedGetCwd = getCwd;
 const mockedSanitizeFileName = sanitizeFileName;
 const mockedExtractZipArchive = extractZipArchive;
@@ -40,7 +40,6 @@ const mockedListPrompt = listPrompt;
 const mockedEnsureProjectExists = ensureProjectExists;
 const mockedPoll = poll;
 const mockedListAppsForMigration = listAppsForMigration;
-const mockedGetProjectThemeDetails = getProjectThemeDetails;
 const mockedInitializeAppMigration = initializeAppMigration;
 const mockedContinueAppMigration = continueAppMigration;
 const mockedHasUnfiedAppsAccess = hasUnfiedAppsAccess;
@@ -205,58 +204,31 @@ describe('lib/app/migrate', () => {
             expect(result.migratableApps[0].projectName).toBe(PROJECT_NAME);
         });
     });
-    describe('getHasMigratableThemes', () => {
-        it('should return false when no projectConfig is provided', async () => {
-            const result = await getHasMigratableThemes();
-            expect(result).toEqual({
-                hasMigratableThemes: false,
-                migratableThemesCount: 0,
-            });
-        });
-        it('should return true when there are migratable themes', async () => {
-            mockedGetProjectThemeDetails.mockResolvedValue({
-                legacyThemeDetails: [
-                    {
-                        configFilepath: 'src/theme.json',
-                        themePath: 'src/theme',
-                        themeConfig: {
-                            secret_names: ['my-secret'],
-                        },
-                    },
-                ],
-                legacyReactThemeDetails: [],
-            });
-            const projectConfig = createLoadedProjectConfig(PROJECT_NAME);
-            const result = await getHasMigratableThemes(projectConfig);
-            expect(result).toEqual({
-                hasMigratableThemes: true,
-                migratableThemesCount: 1,
-            });
-        });
-    });
-    describe('validateMigrationAppsAndThemes', () => {
+    describe('validateMigrationApps', () => {
         const mockMigratableApp1 = createMockMigratableApp(1, 'App 1', PROJECT_NAME);
         const mockMigratableApp2 = createMockMigratableApp(2, 'App 2', PROJECT_NAME);
         it('should throw an error when multiple apps are found for a project', async () => {
             const projectConfig = createLoadedProjectConfig(PROJECT_NAME);
-            await expect(validateMigrationAppsAndThemes(APP_ID, ACCOUNT_ID, {
+            await expect(validateMigrationApps(APP_ID, ACCOUNT_ID, {
                 migratableApps: [mockMigratableApp1, mockMigratableApp2],
                 unmigratableApps: [],
-            }, false, projectConfig)).rejects.toThrow(lib.migrate.errors.project.multipleApps);
-        });
-        it('should throw an error when cms themes are found for a project', async () => {
-            const projectConfig = createLoadedProjectConfig(PROJECT_NAME);
-            await expect(validateMigrationAppsAndThemes(APP_ID, ACCOUNT_ID, { migratableApps: [mockMigratableApp1], unmigratableApps: [] }, true, projectConfig)).rejects.toThrow(lib.migrate.errors.project.themesAndAppsNotAllowed);
+            }, projectConfig)).rejects.toThrow(lib.migrate.errors.project.multipleApps);
         });
         it('should throw an error when no apps are found for a project', async () => {
             const projectConfig = createLoadedProjectConfig(PROJECT_NAME);
-            await expect(validateMigrationAppsAndThemes(APP_ID, ACCOUNT_ID, { migratableApps: [], unmigratableApps: [] }, false, projectConfig)).rejects.toThrow(lib.migrate.errors.noAppsForProject(PROJECT_NAME));
+            await expect(validateMigrationApps(APP_ID, ACCOUNT_ID, { migratableApps: [], unmigratableApps: [] }, projectConfig)).rejects.toThrow(lib.migrate.errors.noAppsForProject(PROJECT_NAME));
         });
         it('should throw an error when no migratable apps are found', async () => {
-            await expect(validateMigrationAppsAndThemes(APP_ID, ACCOUNT_ID, { migratableApps: [], unmigratableApps: mockUnmigratableApps }, false)).rejects.toThrow(/No apps in account/);
+            await expect(validateMigrationApps(APP_ID, ACCOUNT_ID, {
+                migratableApps: [],
+                unmigratableApps: mockUnmigratableApps,
+            })).rejects.toThrow(/No apps in account/);
         });
         it('should throw an error when appId is provided but not found', async () => {
-            await expect(validateMigrationAppsAndThemes(APP_ID, ACCOUNT_ID, { migratableApps: [], unmigratableApps: [] }, false)).rejects.toThrow(/No apps in account/);
+            await expect(validateMigrationApps(APP_ID, ACCOUNT_ID, {
+                migratableApps: [],
+                unmigratableApps: [],
+            })).rejects.toThrow(/No apps in account/);
         });
     });
     describe('promptForAppToMigrate', () => {
@@ -330,10 +302,6 @@ describe('lib/app/migrate', () => {
                     unmigratableApps: [],
                 },
             });
-            mockedGetProjectThemeDetails.mockResolvedValue({
-                legacyThemeDetails: [],
-                legacyReactThemeDetails: [],
-            });
             mockedListPrompt.mockResolvedValue({ appId: 1 });
             mockedConfirmPrompt.mockResolvedValue(true);
             mockedEnsureProjectExists.mockResolvedValue({ projectExists: false });
@@ -352,16 +320,11 @@ describe('lib/app/migrate', () => {
                     unmigratableApps: [],
                 },
             });
-            mockedGetProjectThemeDetails.mockResolvedValueOnce({
-                legacyThemeDetails: [],
-                legacyReactThemeDetails: [],
-            });
             const result = await handleMigrationSetup(ACCOUNT_ID, defaultOptions, projectConfig);
             expect(result).toEqual({
                 appIdToMigrate: 1,
                 projectName: PROJECT_NAME,
                 projectDest: MOCK_PROJECT_DIR,
-                isThemesMigration: false,
             });
         });
         it('should prompt for project name when not provided', async () => {
@@ -528,8 +491,8 @@ describe('lib/app/migrate', () => {
     describe('logInvalidAccountError', () => {
         it('should log the invalid account error message', () => {
             logInvalidAccountError();
-            expect(mockedLogger.error).toHaveBeenCalledWith(lib.migrate.errors.invalidAccountTypeTitle);
-            expect(mockedLogger.log).toHaveBeenCalledWith(expect.stringContaining('Only public apps created in a developer account can be converted to a project component'));
+            expect(mockedUiLogger.error).toHaveBeenCalledWith(lib.migrate.errors.invalidAccountTypeTitle);
+            expect(mockedUiLogger.log).toHaveBeenCalledWith(expect.stringContaining('Only public apps created in a developer account can be converted to a project component'));
         });
     });
 });