@hubspot/cli 7.8.12-experimental.0 → 7.9.0-beta.0

package/lib/doctor/Doctor.js

@@ -1,4 +1,4 @@
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { getAccountId, getCWDAccountOverride, } from '@hubspot/local-dev-lib/config';
 import SpinniesManager from '../ui/SpinniesManager.js';
 import { hasMissingPackages } from '../dependencyManagement.js';
@@ -13,9 +13,8 @@ import { PORT_MANAGER_SERVER_PORT } from '@hubspot/local-dev-lib/constants/ports
 import { accessTokenForPersonalAccessKey, authorizedScopesForPortalAndUser, scopesOnAccessToken, } from '@hubspot/local-dev-lib/personalAccessKey';
 import { isSpecifiedError } from '@hubspot/local-dev-lib/errors/index';
 import { getHubSpotWebsiteOrigin } from '@hubspot/local-dev-lib/urls';
-import { uiCommandReference } from '../ui/index.js';
 import pkg from '../../package.json' with { type: 'json' };
-import { i18n } from '../lang.js';
+import { lib } from '../../lang/en.js';
 import { uiLink } from '../ui/index.js';
 const minMajorNodeVersion = 18;
 export class Doctor {
@@ -31,7 +30,7 @@ export class Doctor {
     }
     async diagnose() {
         SpinniesManager.add('runningDiagnostics', {
-            text: i18n(`lib.doctor.runningDiagnostics`),
+            text: lib.doctor.runningDiagnostics,
         });
         this.diagnosticInfo =
             await this.diagnosticInfoBuilder.generateDiagnosticInfo();
@@ -48,7 +47,7 @@ export class Doctor {
         this.performDefaultAccountOverrideFileChecks();
         this.performCliConfigSettingsChecks();
         SpinniesManager.succeed('runningDiagnostics', {
-            text: i18n(`lib.doctor.diagnosticsComplete`),
+            text: lib.doctor.diagnosticsComplete,
             succeedColor: 'white',
         });
         this.diagnosticInfo.diagnosis = this.diagnosis.toString();
@@ -74,10 +73,8 @@ export class Doctor {
         if (!this.diagnosticInfo?.config) {
             this.diagnosis?.addCLIConfigSection({
                 type: 'error',
-                message: i18n(`lib.doctor.diagnosis.cliConfig.noConfigFile`),
-                secondaryMessaging: i18n(`lib.doctor.diagnosis.cliConfig.noConfigFileSecondary`, {
-                    command: uiCommandReference('hs init'),
-                }),
+                message: lib.doctor.diagnosis.cliConfig.noConfigFile,
+                secondaryMessaging: lib.doctor.diagnosis.cliConfig.noConfigFileSecondary('hs init'),
             });
             return [];
         }
@@ -87,15 +84,11 @@ export class Doctor {
         if (this.diagnosticInfo?.defaultAccountOverrideFile) {
             this.diagnosis?.addDefaultAccountOverrideFileSection({
                 type: 'warning',
-                message: i18n(`lib.doctor.defaultAccountOverrideFileChecks.overrideActive`, {
-                    defaultAccountOverrideFile: this.diagnosticInfo.defaultAccountOverrideFile,
-                }),
+                message: lib.doctor.defaultAccountOverrideFileChecks.overrideActive(this.diagnosticInfo.defaultAccountOverrideFile),
             });
             this.diagnosis?.addDefaultAccountOverrideFileSection({
                 type: 'warning',
-                message: i18n(`lib.doctor.defaultAccountOverrideFileChecks.overrideAccountId`, {
-                    overrideAccountId: getCWDAccountOverride(),
-                }),
+                message: lib.doctor.defaultAccountOverrideFileChecks.overrideAccountId(getCWDAccountOverride()),
             });
         }
     }
@@ -103,8 +96,8 @@ export class Doctor {
         if (this.diagnosticInfo?.configSettings.httpUseLocalhost) {
             this.diagnosis?.addCLIConfigSection({
                 type: 'warning',
-                message: i18n(`lib.doctor.diagnosis.cliConfig.settings.httpUseLocalhost`),
-                secondaryMessaging: i18n(`lib.doctor.diagnosis.cliConfig.settings.httpUseLocalhostSecondary`),
+                message: lib.doctor.diagnosis.cliConfig.settings.httpUseLocalhost,
+                secondaryMessaging: lib.doctor.diagnosis.cliConfig.settings.httpUseLocalhostSecondary,
             });
         }
     }
@@ -115,25 +108,20 @@ export class Doctor {
             const missingScopes = (await authorizedScopesForPortalAndUser(this.accountId)).filter((data) => data.userAuthorized && !pakScopes.has(data.scopeGroup.name));
             this.diagnosis?.addCLIConfigSection({
                 type: 'success',
-                message: i18n(`lib.doctor.accountChecks.active`),
+                message: lib.doctor.accountChecks.active,
             });
-            const linkToPakUI = uiLink(i18n(`lib.doctor.accountChecks.pak.viewScopes`), `${getHubSpotWebsiteOrigin(this.diagnosticInfoBuilder?.env || 'PROD')}/personal-access-key/${this.diagnosticInfo?.account.accountId}`);
+            const linkToPakUI = uiLink(lib.doctor.accountChecks.pak.viewScopes, `${getHubSpotWebsiteOrigin(this.diagnosticInfoBuilder?.env || 'PROD')}/personal-access-key/${this.diagnosticInfo?.account.accountId}`);
             if (missingScopes.length > 0) {
                 this.diagnosis?.addCLIConfigSection({
                     type: 'warning',
-                    message: i18n(`lib.doctor.accountChecks.pak.incomplete`),
-                    secondaryMessaging: i18n(`lib.doctor.accountChecks.pak.incompleteSecondary`, {
-                        command: uiCommandReference(`hs auth`),
-                        link: linkToPakUI,
-                    }),
+                    message: lib.doctor.accountChecks.pak.incomplete,
+                    secondaryMessaging: lib.doctor.accountChecks.pak.incompleteSecondary(linkToPakUI),
                 });
             }
             else {
                 this.diagnosis?.addCLIConfigSection({
                     type: 'success',
-                    message: i18n(`lib.doctor.accountChecks.pak.valid`, {
-                        link: linkToPakUI,
-                    }),
+                    message: lib.doctor.accountChecks.pak.valid(linkToPakUI),
                 });
             }
         }
@@ -151,10 +139,8 @@ export class Doctor {
             if (portalNotActive) {
                 this.diagnosis?.addCLIConfigSection({
                     type: 'error',
-                    message: i18n(`lib.doctor.accountChecks.inactive`),
-                    secondaryMessaging: i18n(`lib.doctor.accountChecks.inactiveSecondary`, {
-                        command: uiCommandReference(`hs accounts clean`),
-                    }),
+                    message: lib.doctor.accountChecks.inactive,
+                    secondaryMessaging: lib.doctor.accountChecks.inactiveSecondary('hs accounts clean'),
                 });
             }
             else if (isSpecifiedError(error, {
@@ -164,20 +150,18 @@ export class Doctor {
             })) {
                 this.diagnosis?.addCLIConfigSection({
                     type: 'success',
-                    message: i18n(`lib.doctor.accountChecks.active`),
+                    message: lib.doctor.accountChecks.active,
                 });
                 this.diagnosis?.addCLIConfigSection({
                     type: 'error',
-                    message: i18n(`lib.doctor.accountChecks.pak.invalid`),
-                    secondaryMessaging: i18n(`lib.doctor.accountChecks.pak.invalidSecondary`, {
-                        command: uiCommandReference(`hs auth`),
-                    }),
+                    message: lib.doctor.accountChecks.pak.invalid,
+                    secondaryMessaging: lib.doctor.accountChecks.pak.invalidSecondary,
                 });
             }
             else {
                 this.diagnosis?.addCLIConfigSection({
                     type: 'error',
-                    message: i18n(`lib.doctor.accountChecks.unableToDetermine`),
+                    message: lib.doctor.accountChecks.unableToDetermine,
                 });
             }
         }
@@ -186,7 +170,7 @@ export class Doctor {
         if (!this.diagnosticInfo?.versions.node) {
             return this.diagnosis?.addCliSection({
                 type: 'error',
-                message: i18n(`lib.doctor.nodeChecks.unableToDetermine`),
+                message: lib.doctor.nodeChecks.unableToDetermine,
             });
         }
         const nodeVersion = this.diagnosticInfo?.versions.node?.split('.');
@@ -194,16 +178,12 @@ export class Doctor {
         if (!currentNodeMajor || parseInt(currentNodeMajor) < minMajorNodeVersion) {
             return this.diagnosis?.addCliSection({
                 type: 'warning',
-                message: i18n(`lib.doctor.nodeChecks.minimumNotMet`, {
-                    nodeVersion: this.diagnosticInfo?.versions.node,
-                }),
+                message: lib.doctor.nodeChecks.minimumNotMet(this.diagnosticInfo?.versions.node),
             });
         }
         this.diagnosis?.addCliSection({
             type: 'success',
-            message: i18n(`lib.doctor.nodeChecks.success`, {
-                nodeVersion: this.diagnosticInfo?.versions.node,
-            }),
+            message: lib.doctor.nodeChecks.success(this.diagnosticInfo?.versions.node),
         });
     }
     async checkIfNpmIsInstalled() {
@@ -211,14 +191,12 @@ export class Doctor {
         if (!npmVersion) {
             return this.diagnosis?.addCliSection({
                 type: 'error',
-                message: i18n(`lib.doctor.npmChecks.notInstalled`),
+                message: lib.doctor.npmChecks.notInstalled,
             });
         }
         this.diagnosis?.addCliSection({
             type: 'success',
-            message: i18n(`lib.doctor.npmChecks.installed`, {
-                npmVersion,
-            }),
+            message: lib.doctor.npmChecks.installed(npmVersion),
         });
     }
     async checkCLIVersion() {
@@ -232,32 +210,22 @@ export class Doctor {
         catch (e) {
             return this.diagnosis?.addCliSection({
                 type: 'error',
-                message: i18n(`lib.doctor.hsChecks.unableToDetermine`),
-                secondaryMessaging: i18n(`lib.doctor.hsChecks.unableToDetermineSecondary`, {
-                    command: uiCommandReference(`hs --version`),
-                    link: uiLink(i18n(`lib.doctor.hsChecks.unableToDetermineSecondaryLink`), `https://www.npmjs.com/package/${pkg.name}?activeTab=versions`),
-                }),
+                message: lib.doctor.hsChecks.unableToDetermine,
+                secondaryMessaging: lib.doctor.hsChecks.unableToDetermineSecondary('hs --version', `https://www.npmjs.com/package/${pkg.name}?activeTab=versions`),
             });
         }
         if (latestCLIVersion !== pkg.version && nextCliVersion !== pkg.version) {
             const onNextTag = pkg.version.includes('beta');
             this.diagnosis?.addCliSection({
                 type: 'warning',
-                message: i18n(`lib.doctor.hsChecks.notLatest`, {
-                    hsVersion: pkg.version,
-                }),
-                secondaryMessaging: i18n(`lib.doctor.hsChecks.notLatestSecondary`, {
-                    hsVersion: onNextTag ? nextCliVersion : latestCLIVersion,
-                    command: uiCommandReference(`npm install -g ${pkg.name}`),
-                }),
+                message: lib.doctor.hsChecks.notLatest(pkg.version),
+                secondaryMessaging: lib.doctor.hsChecks.notLatestSecondary(`npm install -g ${pkg.name}`, onNextTag ? nextCliVersion : latestCLIVersion),
             });
         }
         else {
             this.diagnosis?.addCliSection({
                 type: 'success',
-                message: i18n(`lib.doctor.hsChecks.latest`, {
-                    hsVersion: pkg.version,
-                }),
+                message: lib.doctor.hsChecks.latest(pkg.version),
             });
         }
     }
@@ -271,12 +239,8 @@ export class Doctor {
                     foundError = true;
                     this.diagnosis?.addProjectSection({
                         type: 'warning',
-                        message: i18n(`lib.doctor.projectDependenciesChecks.missingDependencies`, {
-                            dir: packageDirName,
-                        }),
-                        secondaryMessaging: i18n(`lib.doctor.projectDependenciesChecks.missingDependenciesSecondary`, {
-                            command: uiCommandReference('hs project install-deps'),
-                        }),
+                        message: lib.doctor.projectDependenciesChecks.missingDependencies(packageDirName),
+                        secondaryMessaging: lib.doctor.projectDependenciesChecks.missingDependenciesSecondary('hs project install-deps'),
                     });
                 }
             }
@@ -285,26 +249,22 @@ export class Doctor {
                 if (!(await this.isValidJsonFile(packageFile))) {
                     this.diagnosis?.addProjectSection({
                         type: 'error',
-                        message: i18n(`lib.doctor.files.invalidJson`, {
-                            filename: packageFile,
-                        }),
+                        message: lib.doctor.files.invalidJson(packageFile),
                     });
                 }
                 else {
                     this.diagnosis?.addProjectSection({
                         type: 'error',
-                        message: i18n(`lib.doctor.projectDependenciesChecks.unableToDetermine`, {
-                            dir: packageDirName,
-                        }),
+                        message: lib.doctor.projectDependenciesChecks.unableToDetermine(packageDirName),
                     });
                 }
-                logger.debug(e);
+                uiLogger.debug(e instanceof Error ? e.message : String(e));
             }
         }
         if (!foundError) {
             this.diagnosis?.addProjectSection({
                 type: 'success',
-                message: i18n(`lib.doctor.projectDependenciesChecks.success`),
+                message: lib.doctor.projectDependenciesChecks.success,
             });
         }
     }
@@ -327,16 +287,14 @@ export class Doctor {
                 foundError = true;
                 this.diagnosis?.addProjectSection({
                     type: 'error',
-                    message: i18n(`lib.doctor.files.invalidJson`, {
-                        filename: jsonFile,
-                    }),
+                    message: lib.doctor.files.invalidJson(jsonFile),
                 });
             }
         }
         if (!foundError) {
             this.diagnosis?.addProjectSection({
                 type: 'success',
-                message: i18n(`lib.doctor.files.validJson`),
+                message: lib.doctor.files.validJson,
             });
         }
     }
@@ -344,20 +302,14 @@ export class Doctor {
         if (await isPortManagerPortAvailable()) {
             this.diagnosis?.addProjectSection({
                 type: 'success',
-                message: i18n(`lib.doctor.port.available`, {
-                    port: PORT_MANAGER_SERVER_PORT,
-                }),
+                message: lib.doctor.port.available(PORT_MANAGER_SERVER_PORT),
             });
             return;
         }
         this.diagnosis?.addProjectSection({
             type: 'warning',
-            message: i18n(`lib.doctor.port.inUse`, {
-                port: PORT_MANAGER_SERVER_PORT,
-            }),
-            secondaryMessaging: i18n(`lib.doctor.port.inUseSecondary`, {
-                command: uiCommandReference('hs project dev'),
-            }),
+            message: lib.doctor.port.inUse(PORT_MANAGER_SERVER_PORT),
+            secondaryMessaging: lib.doctor.port.inUseSecondary('hs project dev'),
         });
     }
 }

package/lib/doctor/__tests__/Doctor.test.js

@@ -6,7 +6,7 @@ import { HubSpotHttpError } from '@hubspot/local-dev-lib/models/HubSpotHttpError
 import { AxiosError } from 'axios';
 import { isSpecifiedError as _isSpecifiedError } from '@hubspot/local-dev-lib/errors/index';
 import { promisify as _promisify } from 'util';
-vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('../../ui/logger.js');
 vi.mock('../Diagnosis');
 vi.mock('../../ui/SpinniesManager');
 vi.mock('../DiagnosticInfoBuilder');
@@ -336,7 +336,7 @@ describe('lib/doctor/Doctor', () => {
                 // @ts-expect-error Testing private method
                 expect(doctor.diagnosis.addProjectSection).toHaveBeenCalledWith({
                     type: 'error',
-                    message: expect.stringMatching(/invalid JSON in/),
+                    message: expect.stringMatching(/Invalid JSON in/),
                 });
             });
             it('should add error section if it is unable to determine if dependencies need installed', async () => {
@@ -359,7 +359,7 @@ describe('lib/doctor/Doctor', () => {
                 expect(doctor.diagnosis?.addProjectSection).toHaveBeenCalledWith({
                     type: 'warning',
                     message: 'Port 8080 is in use',
-                    secondaryMessaging: expect.stringMatching(/Make sure it is available if before running/),
+                    secondaryMessaging: expect.stringMatching(/Make sure it is available before running `hs project dev`/),
                 });
             });
             it('should add success section if port is available', async () => {
@@ -388,7 +388,7 @@ describe('lib/doctor/Doctor', () => {
                 // @ts-expect-error Testing private method
                 expect(doctor.diagnosis.addProjectSection).toHaveBeenCalledWith({
                     type: 'error',
-                    message: expect.stringMatching(/invalid JSON in/),
+                    message: expect.stringMatching(/Invalid JSON in/),
                 });
             });
         });

package/lib/errorHandlers/index.js

@@ -1,15 +1,14 @@
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { isHubSpotHttpError, isValidationError, } from '@hubspot/local-dev-lib/errors/index';
 import { getConfig } from '@hubspot/local-dev-lib/config';
 import { shouldSuppressError } from './suppressError.js';
-import { i18n } from '../lang.js';
+import { lib } from '../../lang/en.js';
 import util from 'util';
-import { uiCommandReference } from '../ui/index.js';
 import { isProjectValidationError } from '../errors/ProjectValidationError.js';
 export function logError(error, context) {
     debugError(error, context);
     if (isProjectValidationError(error)) {
-        logger.error(error.message);
+        uiLogger.error(error.message);
         return;
     }
     if (shouldSuppressError(error, context)) {
@@ -24,7 +23,7 @@ export function logError(error, context) {
         error.updateContext(context);
     }
     if (isHubSpotHttpError(error) && isValidationError(error)) {
-        logger.error(error.formattedValidationErrors());
+        uiLogger.error(error.formattedValidationErrors());
     }
     else if (isErrorWithMessageOrReason(error)) {
         const message = [];
@@ -33,44 +32,37 @@ export function logError(error, context) {
                 message.push(msg);
             }
         });
-        logger.error(message.join(' '));
+        uiLogger.error(message.join(' '));
     }
     else {
         // Unknown errors
-        logger.error(i18n(`lib.errorHandlers.index.unknownErrorOccurred`));
+        uiLogger.error(lib.errorHandlers.index.unknownErrorOccurred);
     }
     if (isHubSpotHttpError(error) && error.code === 'ETIMEDOUT') {
         const config = getConfig();
         const defaultTimeout = config?.httpTimeout;
         // Timeout was caused by the default timeout
         if (error.timeout && defaultTimeout === error.timeout) {
-            logger.error(i18n(`lib.errorHandlers.index.configTimeoutErrorOccurred`, {
-                timeout: error.timeout,
-                configSetCommand: uiCommandReference('hs config set'),
-            }));
+            uiLogger.error(lib.errorHandlers.index.configTimeoutErrorOccurred(error.timeout, 'hs config set'));
         }
         // Timeout was caused by a custom timeout set by the CLI or LDL
         else {
-            logger.error(i18n(`lib.errorHandlers.index.genericTimeoutErrorOccurred`));
+            uiLogger.error(lib.errorHandlers.index.genericTimeoutErrorOccurred);
         }
     }
 }
 export function debugError(error, context) {
     if (isHubSpotHttpError(error)) {
-        logger.debug(error.toString());
+        uiLogger.debug(error.toString());
     }
     else {
-        logger.debug(i18n(`lib.errorHandlers.index.errorOccurred`, { error: String(error) }));
+        uiLogger.debug(lib.errorHandlers.index.errorOccurred(String(error)));
     }
     if (error instanceof Error && error.cause && !isHubSpotHttpError(error)) {
-        logger.debug(i18n(`lib.errorHandlers.index.errorCause`, {
-            cause: util.inspect(error.cause, false, null, true),
-        }));
+        uiLogger.debug(lib.errorHandlers.index.errorCause(util.inspect(error.cause, false, null, true)));
     }
     if (context) {
-        logger.debug(i18n(`lib.errorHandlers.index.errorContext`, {
-            context: util.inspect(context, false, null, true),
-        }));
+        uiLogger.debug(lib.errorHandlers.index.errorContext(util.inspect(context, false, null, true)));
     }
 }
 export class ApiErrorContext {

package/lib/errorHandlers/suppressError.js

@@ -1,8 +1,8 @@
 import { isSpecifiedError, isMissingScopeError, } from '@hubspot/local-dev-lib/errors/index';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from '../ui/logger.js';
 import { PLATFORM_VERSION_ERROR_TYPES } from '../constants.js';
-import { i18n } from '../lang.js';
-import { uiAccountDescription, uiLine, uiLink, uiCommandReference, } from '../ui/index.js';
+import { lib } from '../../lang/en.js';
+import { uiAccountDescription, uiLine, uiLink } from '../ui/index.js';
 function createPlatformVersionError(err, subCategory) {
     let translationKey = 'unspecifiedPlatformVersion';
     let platformVersion = 'unspecified platformVersion';
@@ -17,25 +17,18 @@ function createPlatformVersionError(err, subCategory) {
         translationKey = 'nonExistentPlatformVersion';
     }
     uiLine();
-    logger.error(i18n(`lib.errorHandlers.suppressErrors.platformVersionErrors.header`));
-    logger.log(i18n(`lib.errorHandlers.suppressErrors.platformVersionErrors.${translationKey}`, {
-        platformVersion,
-    }));
-    logger.log(i18n(`lib.errorHandlers.suppressErrors.platformVersionErrors.updateProject`));
-    logger.log(i18n(`lib.errorHandlers.suppressErrors.platformVersionErrors.betaLink`, {
-        docsLink: uiLink(i18n(`lib.errorHandlers.suppressErrors.platformVersionErrors.docsLink`), 'https://developers.hubspot.com/docs/developer-tooling/platform/versioning'),
-    }));
+    uiLogger.error(lib.errorHandlers.suppressErrors.platformVersionErrors.header);
+    const errorMessage = lib.errorHandlers.suppressErrors.platformVersionErrors[translationKey];
+    uiLogger.log(typeof errorMessage === 'function'
+        ? errorMessage(platformVersion)
+        : errorMessage);
+    uiLogger.log(lib.errorHandlers.suppressErrors.platformVersionErrors.updateProject);
+    uiLogger.log(lib.errorHandlers.suppressErrors.platformVersionErrors.betaLink(uiLink(lib.errorHandlers.suppressErrors.platformVersionErrors.docsLink, 'https://developers.hubspot.com/docs/developer-tooling/platform/versioning')));
     uiLine();
 }
 export function shouldSuppressError(err, context) {
     if (isMissingScopeError(err)) {
-        logger.error(i18n(`lib.errorHandlers.suppressErrors.missingScopeError`, {
-            accountName: context?.accountId
-                ? uiAccountDescription(context.accountId)
-                : '',
-            request: context?.request || 'request',
-            authCommand: uiCommandReference('hs auth'),
-        }));
+        uiLogger.error(lib.errorHandlers.suppressErrors.missingScopeError(context?.request || 'request', context?.accountId ? uiAccountDescription(context.accountId) : ''));
         return true;
     }
     if (isSpecifiedError(err, {

package/lib/generateSelectors.js

@@ -11,7 +11,7 @@ export function getMaxFieldsDepth() {
 export function findFieldsJsonPath(basePath) {
     const _path = basePath.endsWith('/') ? basePath.slice(0, -1) : basePath;
     if (!fs.existsSync(_path)) {
-        uiLogger.error(commands.theme.subcommands.generateSelectors.errors.invalidPath(basePath));
+        uiLogger.error(commands.cms.subcommands.theme.subcommands.generateSelectors.errors.invalidPath(basePath));
         process.exit(EXIT_CODES.ERROR);
     }
     const files = fs.readdirSync(_path);

package/lib/lang.js

@@ -2,7 +2,7 @@ import util from 'util';
 import path, { dirname } from 'path';
 import fs from 'fs-extra';
 import yaml from 'js-yaml';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from './ui/logger.js';
 import { interpolate } from './interpolation.js';
 import { fileURLToPath } from 'url';
 const __filename = fileURLToPath(import.meta.url);
@@ -22,10 +22,11 @@ function loadLanguageFromYaml() {
         // Fall back to using the default language file
         locale = languageFileExists ? nodeLocale : 'en';
         languageObj = yaml.load(fs.readFileSync(path.join(__dirname, `../lang/${locale}.lyaml`), 'utf8'));
-        logger.debug('Loaded language data: ', util.inspect(languageObj, true, 999, true));
+        uiLogger.debug('Loaded language data: ' + util.inspect(languageObj, true, 999, true));
     }
     catch (e) {
-        logger.error('Error loading language data: ', e);
+        uiLogger.error('Error loading language data: ' +
+            (e instanceof Error ? e.message : String(e)));
     }
 }
 function getTextValue(lookupDotNotation) {
@@ -42,8 +43,8 @@ function getTextValue(lookupDotNotation) {
         });
     }
     catch (e) {
-        logger.debug(`Unable to access language property: ${lookupProps.join('.')}. Failed to access prop "${previouslyCheckedProp}".`);
-        logger.error('Unable to access language property.');
+        uiLogger.debug(`Unable to access language property: ${lookupProps.join('.')}. Failed to access prop "${previouslyCheckedProp}".`);
+        uiLogger.error('Unable to access language property.');
         return missingTextData;
     }
     if (!textValue) {

package/lib/links.js

@@ -2,7 +2,7 @@ import open from 'open';
 import { getEnv } from '@hubspot/local-dev-lib/config';
 import { ENVIRONMENTS } from '@hubspot/local-dev-lib/constants/environments';
 import { getHubSpotWebsiteOrigin } from '@hubspot/local-dev-lib/urls';
-import { logger } from '@hubspot/local-dev-lib/logger';
+import { uiLogger } from './ui/logger.js';
 import { getTableContents, getTableHeader } from './ui/table.js';
 const COMMON_SITE_LINKS = {
     APPS_MARKETPLACE: {
@@ -87,17 +87,17 @@ export function logSiteLinks(accountId) {
         l.url,
     ]);
     linksAsArray.unshift(getTableHeader(['Shortcut', '', 'Url']));
-    logger.log(getTableContents(linksAsArray));
+    uiLogger.log(getTableContents(linksAsArray));
 }
 export function openLink(accountId, shortcut) {
     const match = Object.values(COMMON_SITE_LINKS).find(l => l.shortcut === shortcut || (l.alias && l.alias === shortcut));
     if (!match) {
-        logger.error(`We couldn't find a shortcut matching ${shortcut}.  Type 'hs open list' to see a list of available shortcuts`);
+        uiLogger.error(`We couldn't find a shortcut matching ${shortcut}.  Type 'hs open list' to see a list of available shortcuts`);
         return;
     }
     const baseUrl = getHubSpotWebsiteOrigin(getEnv() === 'qa' ? ENVIRONMENTS.QA : ENVIRONMENTS.PROD);
     open(match.getUrl(accountId, baseUrl), { url: true });
-    logger.success(`We opened ${match.getUrl(accountId, baseUrl)} in your browser`);
+    uiLogger.success(`We opened ${match.getUrl(accountId, baseUrl)} in your browser`);
 }
 export function getProductUpdatesUrl(rolloutId, accountId) {
     const baseUrl = getHubSpotWebsiteOrigin(getEnv() === 'qa' ? ENVIRONMENTS.QA : ENVIRONMENTS.PROD);

package/lib/middleware/__test__/commandTargetingUtils.test.js

@@ -0,0 +1,99 @@
+import * as cliConfig from '@hubspot/local-dev-lib/config';
+import { isTargetedCommand, shouldLoadConfigForCommand, shouldRunAccountValidationForCommand, shouldRunConfigValidationForCommand, } from '../commandTargetingUtils.js';
+vi.mock('@hubspot/local-dev-lib/config');
+const configFileExistsSpy = vi.spyOn(cliConfig, 'configFileExists');
+const targetCommandMap = {
+    init: true,
+    account: {
+        use: true,
+    },
+    project: {
+        deploy: true,
+    },
+};
+describe('lib/middleware/commandTargetingUtils', () => {
+    describe('isTargetedCommand()', () => {
+        it('should return true for a direct target command', () => {
+            const result = isTargetedCommand(['init'], targetCommandMap);
+            expect(result).toBe(true);
+        });
+        it('should return false for a command with a subcommand', () => {
+            const result = isTargetedCommand(['account'], targetCommandMap);
+            expect(result).toBe(false);
+        });
+        it('should return true for a nested target command', () => {
+            const result = isTargetedCommand(['project', 'deploy'], targetCommandMap);
+            expect(result).toBe(true);
+        });
+        it('should return false for a non-existent command', () => {
+            const result = isTargetedCommand(['nonexistent'], targetCommandMap);
+            expect(result).toBe(false);
+        });
+        it('should return false for a non-targeted command', () => {
+            const result = isTargetedCommand(['auth'], targetCommandMap);
+            expect(result).toBe(false);
+        });
+        it('should return false for a command with a targeted subcommand', () => {
+            const result = isTargetedCommand(['project'], targetCommandMap);
+            expect(result).toBe(false);
+        });
+        it('should return false for a non-existent subcommand', () => {
+            const result = isTargetedCommand(['auth', 'nonexistent'], targetCommandMap);
+            expect(result).toBe(false);
+        });
+        it('should return false for an empty command array', () => {
+            const result = isTargetedCommand([], targetCommandMap);
+            expect(result).toBe(false);
+        });
+    });
+    describe('shouldLoadConfigForCommand()', () => {
+        it('should return true for a command that requires a config file', () => {
+            const result = shouldLoadConfigForCommand(['account', 'use']);
+            expect(result).toBe(true);
+        });
+        it('should return false for a command that does not require a config file', () => {
+            const result = shouldLoadConfigForCommand(['init']);
+            expect(result).toBe(false);
+        });
+        it('should return false when the user is trying to migrate the global config', () => {
+            configFileExistsSpy.mockReturnValue(false);
+            const result = shouldLoadConfigForCommand(['config', 'migrate']);
+            expect(result).toBe(false);
+        });
+        it('should return true when the global config file does exist', () => {
+            configFileExistsSpy.mockReturnValue(true);
+            const result = shouldLoadConfigForCommand(['account', 'use']);
+            expect(result).toBe(true);
+        });
+    });
+    describe('shouldRunAccountValidationForCommand()', () => {
+        it('should return true for a command that requires a config file', () => {
+            const result = shouldRunAccountValidationForCommand([
+                'project',
+                'upload',
+            ]);
+            expect(result).toBe(true);
+        });
+        it('should return false for a command that does not require account validation', () => {
+            const result1 = shouldRunAccountValidationForCommand(['init']);
+            const result2 = shouldRunAccountValidationForCommand(['auth']);
+            const result3 = shouldRunAccountValidationForCommand([
+                'accounts',
+                'list',
+            ]);
+            expect(result1).toBe(false);
+            expect(result2).toBe(false);
+            expect(result3).toBe(false);
+        });
+    });
+    describe('shouldRunConfigValidationForCommand()', () => {
+        it('should return true for a command that requires a config file', () => {
+            const result = shouldRunConfigValidationForCommand(['account', 'use']);
+            expect(result).toBe(true);
+        });
+        it('should return false for a command that does not require config validation', () => {
+            const result = shouldRunConfigValidationForCommand(['init']);
+            expect(result).toBe(false);
+        });
+    });
+});