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

package/lang/en.lyaml

@@ -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', () => ({

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,4 +1,4 @@
-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 } from '@hubspot/project-parsing-lib';
@@ -13,7 +13,7 @@ import { CLI_UNMIGRATABLE_REASONS, continueAppMigration, initializeAppMigration,
 import { lib } from '../../../lang/en.js';
 import { hasUnfiedAppsAccess } from '../../hasFeature.js';
 import { getUnmigratableReason, generateFilterAppsByProjectNameFunction, buildErrorMessageFromMigrationStatus, fetchMigrationApps, promptForAppToMigrate, selectAppToMigrate, handleMigrationSetup, beginAppMigration, pollMigrationStatus, finalizeAppMigration, downloadProjectFiles, migrateApp2025_2, logInvalidAccountError, validateMigrationApps, } from '../migrate.js';
-vi.mock('@hubspot/local-dev-lib/logger');
+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;
@@ -491,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'));
         });
     });
 });

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

@@ -11,7 +11,7 @@ import { poll as _poll } from '../../polling.js';
 import { migrateApp2023_2 } from '../migrate_legacy.js';
 // Mock all external dependencies
 vi.mock('@hubspot/local-dev-lib/api/appsDev');
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../ui/logger.js');
 vi.mock('@hubspot/local-dev-lib/api/projects');
 vi.mock('@hubspot/local-dev-lib/path');
 vi.mock('@hubspot/local-dev-lib/urls');

package/lib/app/migrate.js

@@ -358,6 +358,6 @@ export async function migrateApp2025_2(derivedAccountId, options, projectConfig)
 export function logInvalidAccountError() {
     uiLine();
     uiLogger.error(lib.migrate.errors.invalidAccountTypeTitle);
-    uiLogger.log(lib.migrate.errors.invalidAccountTypeDescription(uiCommandReference('hs account use'), uiCommandReference('hs auth')));
+    uiLogger.log(lib.migrate.errors.invalidAccountTypeDescription(uiCommandReference('hs account use')));
     uiLine();
 }

package/lib/app/migrate_legacy.js

@@ -1,5 +1,5 @@
 import { fetchPublicAppMetadata } from '@hubspot/local-dev-lib/api/appsDev';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { checkMigrationStatus, downloadProject, migrateApp as migrateNonProjectApp_v2023_2, } from '@hubspot/local-dev-lib/api/projects';
 import path from 'path';
 import { getCwd, sanitizeFileName } from '@hubspot/local-dev-lib/path';
@@ -9,7 +9,7 @@ import { promptUser } from '../prompts/promptUtils.js';
 import { ApiErrorContext, logError } from '../errorHandlers/index.js';
 import { EXIT_CODES } from '../enums/exitCodes.js';
 import { uiAccountDescription, uiLine, uiLink } from '../ui/index.js';
-import { i18n } from '../lang.js';
+import { commands } from '../../lang/en.js';
 import { isAppDeveloperAccount, isUnifiedAccount } from '../accountTypes.js';
 import { selectPublicAppForMigrationPrompt } from '../prompts/selectPublicAppForMigrationPrompt.js';
 import { projectNameAndDestPrompt } from '../prompts/projectNameAndDestPrompt.js';
@@ -42,9 +42,7 @@ export async function migrateApp2023_2(derivedAccountId, options, accountConfig)
         const preventProjectMigrations = selectedApp.preventProjectMigrations;
         const listingInfo = selectedApp.listingInfo;
         if (preventProjectMigrations && listingInfo) {
-            logger.error(i18n(`commands.project.subcommands.migrateApp.errors.invalidApp`, {
-                appId,
-            }));
+            uiLogger.error(commands.project.migrateApp.errors.invalidApp(appId));
             return process.exit(EXIT_CODES.ERROR);
         }
     }
@@ -58,24 +56,22 @@ export async function migrateApp2023_2(derivedAccountId, options, accountConfig)
         noLogs: true,
     });
     if (projectExists) {
-        throw new Error(i18n(`commands.project.subcommands.migrateApp.errors.projectAlreadyExists`, {
-            projectName,
-        }));
+        throw new Error(commands.project.migrateApp.errors.projectAlreadyExists(projectName));
     }
     await trackCommandMetadataUsage('migrate-app', { step: 'STARTED' }, derivedAccountId);
-    logger.log('');
+    uiLogger.log('');
     uiLine();
-    logger.warn(`${i18n(`commands.project.subcommands.migrateApp.warning.title`)}\n`);
-    logger.log(i18n(`commands.project.subcommands.migrateApp.warning.projectConversion`));
-    logger.log(`${i18n(`commands.project.subcommands.migrateApp.warning.appConfig`)}\n`);
-    logger.log(`${i18n(`commands.project.subcommands.migrateApp.warning.buildAndDeploy`)}\n`);
-    logger.log(`${i18n(`commands.project.subcommands.migrateApp.warning.existingApps`)}\n`);
-    logger.log(i18n(`commands.project.subcommands.migrateApp.warning.copyApp`));
+    uiLogger.warn(commands.project.migrateApp.warning.title);
+    uiLogger.log(commands.project.migrateApp.warning.projectConversion);
+    uiLogger.log(commands.project.migrateApp.warning.appConfig);
+    uiLogger.log(commands.project.migrateApp.warning.buildAndDeploy);
+    uiLogger.log(commands.project.migrateApp.warning.existingApps);
+    uiLogger.log(commands.project.migrateApp.warning.copyApp);
     uiLine();
     const { shouldCreateApp } = await promptUser({
         name: 'shouldCreateApp',
         type: 'confirm',
-        message: i18n(`commands.project.subcommands.migrateApp.createAppPrompt`),
+        message: commands.project.migrateApp.createAppPrompt,
     });
     process.stdin.resume();
     if (!shouldCreateApp) {
@@ -84,12 +80,12 @@ export async function migrateApp2023_2(derivedAccountId, options, accountConfig)
     try {
         SpinniesManager.init();
         SpinniesManager.add('migrateApp', {
-            text: i18n(`commands.project.subcommands.migrateApp.migrationStatus.inProgress`),
+            text: commands.project.migrateApp.migrationStatus.inProgress(),
         });
         handleKeypress(async (key) => {
             if ((key.ctrl && key.name === 'c') || key.name === 'q') {
                 SpinniesManager.remove('migrateApp');
-                logger.log(i18n(`commands.project.subcommands.migrateApp.migrationInterrupted`));
+                uiLogger.log(commands.project.migrateApp.migrationInterrupted);
                 return process.exit(EXIT_CODES.SUCCESS);
             }
         });
@@ -104,20 +100,20 @@ export async function migrateApp2023_2(derivedAccountId, options, accountConfig)
             const { data: zippedProject } = await downloadProject(derivedAccountId, projectName, 1);
             await extractZipArchive(zippedProject, sanitizeFileName(projectName), path.resolve(absoluteDestPath), { includesRootDir: true, hideLogs: true });
             SpinniesManager.succeed('migrateApp', {
-                text: i18n(`commands.project.subcommands.migrateApp.migrationStatus.done`),
+                text: commands.project.migrateApp.migrationStatus.done(),
                 succeedColor: 'white',
             });
-            logger.log('');
+            uiLogger.log('');
             uiLine();
-            logger.success(i18n(`commands.project.subcommands.migrateApp.migrationStatus.success`));
-            logger.log('');
-            logger.log(uiLink(i18n(`commands.project.subcommands.migrateApp.projectDetailsLink`), `${baseUrl}/developer-projects/${derivedAccountId}/project/${encodeURIComponent(project.name)}`));
+            uiLogger.success(commands.project.migrateApp.migrationStatus.success());
+            uiLogger.log('');
+            uiLogger.log(uiLink(commands.project.migrateApp.projectDetailsLink, `${baseUrl}/developer-projects/${derivedAccountId}/project/${encodeURIComponent(project.name)}`));
             process.exit(EXIT_CODES.SUCCESS);
         }
     }
     catch (error) {
         SpinniesManager.fail('migrateApp', {
-            text: i18n(`commands.project.subcommands.migrateApp.migrationStatus.failure`),
+            text: commands.project.migrateApp.migrationStatus.failure(),
             failColor: 'white',
         });
         throw error;