@hubspot/cli 7.9.0 → 7.9.1-experimental.0

package/lib/buildAccount.js

@@ -1,6 +1,5 @@
 import { getAccessToken, updateConfigWithAccessToken, } from '@hubspot/local-dev-lib/personalAccessKey';
-import { accountNameExistsInConfig, updateAccountConfig, writeConfig, getAccountId, } from '@hubspot/local-dev-lib/config';
-import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
+import { getConfigAccountIfExists, updateConfigAccount, } from '@hubspot/local-dev-lib/config';
 import { uiLogger } from './ui/logger.js';
 import { createDeveloperTestAccount, fetchDeveloperTestAccountGateSyncStatus, generateDeveloperTestAccountPersonalAccessKey, } from '@hubspot/local-dev-lib/api/developerTestAccounts';
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
@@ -28,8 +27,8 @@ export async function saveAccountToConfig(accountId, accountName, env, personalA
     if (!updatedConfig?.name) {
         const nameForConfig = accountName.toLowerCase().split(' ').join('-');
         validName = nameForConfig;
-        const invalidAccountName = accountNameExistsInConfig(nameForConfig);
-        if (invalidAccountName) {
+        const existingAccount = getConfigAccountIfExists(nameForConfig);
+        if (existingAccount) {
             if (!force) {
                 uiLogger.log('');
                 uiLogger.warn(lib.prompts.accountNamePrompt.errors.accountNameExists(nameForConfig));
@@ -42,13 +41,10 @@ export async function saveAccountToConfig(accountId, accountName, env, personalA
             }
         }
     }
-    updateAccountConfig({
+    updateConfigAccount({
         ...updatedConfig,
-        env: updatedConfig?.env,
-        tokenInfo: updatedConfig?.auth?.tokenInfo,
         name: validName,
     });
-    writeConfig();
     uiLogger.log('');
     return validName;
 }
@@ -83,8 +79,7 @@ export async function createDeveloperTestAccountV2(parentAccountId, testAccountC
     return result;
 }
 export async function buildDeveloperTestAccount(testAccountName, parentAccountConfig, env, portalLimit, useV2 = false) {
-    const id = getAccountIdentifier(parentAccountConfig);
-    const parentAccountId = getAccountId(id);
+    const parentAccountId = parentAccountConfig.accountId;
     let testAccountConfig = {
         accountName: testAccountName,
     };
@@ -141,8 +136,7 @@ export async function buildSandbox(sandboxName, parentAccountConfig, sandboxType
     const sandboxTypeKey = sandboxType === HUBSPOT_ACCOUNT_TYPES.STANDARD_SANDBOX
         ? 'standard'
         : 'developer';
-    const id = getAccountIdentifier(parentAccountConfig);
-    const parentAccountId = getAccountId(id);
+    const parentAccountId = parentAccountConfig.accountId;
     if (!parentAccountId) {
         throw new Error(lib.sandbox.create[sandboxTypeKey].loading.fail(''));
     }
@@ -182,8 +176,7 @@ export async function buildV2Sandbox(sandboxName, parentAccountConfig, sandboxTy
     const sandboxTypeKey = sandboxType === HUBSPOT_ACCOUNT_TYPES.STANDARD_SANDBOX
         ? 'standard'
         : 'developer';
-    const id = getAccountIdentifier(parentAccountConfig);
-    const parentAccountId = getAccountId(id);
+    const parentAccountId = parentAccountConfig.accountId;
     if (!parentAccountId) {
         throw new Error(lib.sandbox.create[sandboxTypeKey].loading.fail(''));
     }

package/lib/commonOpts.js

@@ -3,7 +3,7 @@ import yargsParser from 'yargs-parser';
 import { LOG_LEVEL, setLogLevel } from '@hubspot/local-dev-lib/logger';
 import { uiLogger } from './ui/logger.js';
 import { DEFAULT_CMS_PUBLISH_MODE, CMS_PUBLISH_MODE, } from '@hubspot/local-dev-lib/constants/files';
-import { getAccountConfig, getAndLoadConfigIfNeeded, } from '@hubspot/local-dev-lib/config';
+import { getConfigAccountById, getConfig } from '@hubspot/local-dev-lib/config';
 import { debugError } from './errorHandlers/index.js';
 import { EXIT_CODES } from './enums/exitCodes.js';
 import { uiCommandReference } from './ui/index.js';
@@ -149,16 +149,13 @@ export function setCLILogLevel(options) {
         setLogLevel(LOG_LEVEL.ERROR);
         SpinniesManager.setDisableOutput(true);
     }
-    else if (debug) {
+    else if (debug || networkDebug) {
         setLogLevel(LOG_LEVEL.DEBUG);
+        process.env.HUBSPOT_NETWORK_LOGGING = 'true';
     }
     else {
         setLogLevel(LOG_LEVEL.LOG);
     }
-    if (networkDebug) {
-        process.env.HUBSPOT_NETWORK_LOGGING = 'true';
-        setLogLevel(LOG_LEVEL.DEBUG);
-    }
 }
 export function getCommandName(argv) {
     return String(argv && argv._ && argv._[0]) || '';
@@ -171,14 +168,14 @@ export function getCmsPublishMode(options) {
     }
     // 2. config[account].defaultCmsPublishMode
     if (options.derivedAccountId) {
-        const accountConfig = getAccountConfig(options.derivedAccountId);
+        const accountConfig = getConfigAccountById(options.derivedAccountId);
         if (accountConfig && accountConfig.defaultCmsPublishMode) {
             return accountConfig.defaultCmsPublishMode;
         }
     }
     // 3. config.defaultCmsPublishMode
     // 4. DEFAULT_CMS_PUBLISH_MODE
-    const config = getAndLoadConfigIfNeeded();
+    const config = getConfig();
     return ((config && config.defaultCmsPublishMode) ||
         DEFAULT_CMS_PUBLISH_MODE);
 }

package/lib/configMigrate.d.ts

@@ -1,2 +1,2 @@
-export declare function handleMigration(deprecatedConfigPath?: string, hideWarning?: boolean): Promise<boolean>;
-export declare function handleMerge(deprecatedConfigPath?: string, force?: boolean, hideWarning?: boolean): Promise<boolean>;
+export declare function handleMigration(hideWarning?: boolean): Promise<boolean>;
+export declare function handleMerge(force?: boolean, hideWarning?: boolean): Promise<boolean>;

package/lib/configMigrate.js

@@ -1,8 +1,10 @@
-import { getDeprecatedConfig, getGlobalConfig, getConfigPath, migrateConfig, mergeConfigProperties, mergeExistingConfigs, } from '@hubspot/local-dev-lib/config/migrate';
+import { getConfigAtPath, migrateConfigAtPath, mergeConfigProperties, mergeConfigAccounts, archiveConfigAtPath, } from '@hubspot/local-dev-lib/config/migrate';
 import { ARCHIVED_HUBSPOT_CONFIG_YAML_FILE_NAME } from '@hubspot/local-dev-lib/constants/config';
 import { promptUser } from './prompts/promptUtils.js';
 import { lib } from '../lang/en.js';
 import { uiLogger } from './ui/logger.js';
+import { getConfig, getLocalConfigFilePathIfExists, } from '@hubspot/local-dev-lib/config';
+import { debugError } from './errorHandlers/index.js';
 async function promptRenameOrOmitAccount(accountName, accountId) {
     const { shouldRename } = await promptUser({
         name: 'shouldRename',
@@ -24,7 +26,7 @@ async function promptNewAccountName(account, globalConfig, renamedAccounts) {
             if (value === account.name) {
                 return lib.configMigrate.handleAccountNameConflicts.errors.sameName;
             }
-            const existingAccount = globalConfig.accounts?.some(acc => acc.name === value);
+            const existingAccount = globalConfig.accounts.some((acc) => acc.name === value);
             const renamedAccount = renamedAccounts.some(acc => acc.name === value);
             if (existingAccount || renamedAccount) {
                 return lib.configMigrate.handleAccountNameConflicts.errors.nameAlreadyInConfig(value);
@@ -34,9 +36,10 @@ async function promptNewAccountName(account, globalConfig, renamedAccounts) {
     });
     return newAccountName;
 }
-export async function handleMigration(deprecatedConfigPath, hideWarning) {
+export async function handleMigration(hideWarning) {
+    const deprecatedConfigPath = process.env.HUBSPOT_CONFIG_PATH || getLocalConfigFilePathIfExists();
     if (!hideWarning) {
-        uiLogger.warn(lib.configMigrate.deprecatedConfigWarning(deprecatedConfigPath || getConfigPath(undefined, false)));
+        uiLogger.warn(lib.configMigrate.deprecatedConfigWarning(deprecatedConfigPath || ''));
         uiLogger.log('');
     }
     uiLogger.log(lib.configMigrate.handleMigration.description(ARCHIVED_HUBSPOT_CONFIG_YAML_FILE_NAME));
@@ -49,13 +52,20 @@ export async function handleMigration(deprecatedConfigPath, hideWarning) {
     if (!shouldMigrateConfig) {
         return false;
     }
-    const deprecatedConfig = getDeprecatedConfig(deprecatedConfigPath);
-    migrateConfig(deprecatedConfig);
+    migrateConfigAtPath(deprecatedConfigPath || '');
+    try {
+        archiveConfigAtPath(deprecatedConfigPath);
+    }
+    catch (error) {
+        debugError(error);
+        uiLogger.error(lib.configMigrate.errors.archive(deprecatedConfigPath));
+        return true;
+    }
     uiLogger.success(lib.configMigrate.handleMigration.success);
     return true;
 }
 async function handleMergeConfigProperties(globalConfig, deprecatedConfig, force) {
-    const { initialConfig, conflicts, } = mergeConfigProperties(globalConfig, deprecatedConfig, force);
+    const { configWithMergedProperties, conflicts, } = mergeConfigProperties(globalConfig, deprecatedConfig, force);
     if (conflicts.length > 0) {
         const properties = conflicts.map(c => c.property);
         const propertyList = properties.length <= 2
@@ -68,15 +78,15 @@ async function handleMergeConfigProperties(globalConfig, deprecatedConfig, force
             const { shouldOverwrite } = await promptUser({
                 name: 'shouldOverwrite',
                 type: 'confirm',
-                message: lib.configMigrate.handleMergeConfigProperties.mergeConfigConflictPrompt(property, newValue.toString(), oldValue.toString()),
+                message: lib.configMigrate.handleMergeConfigProperties.mergeConfigConflictPrompt(property, newValue.toString(), oldValue?.toString() || ''),
             });
             if (shouldOverwrite) {
                 // @ts-expect-error Cannot reconcile CLIConfig_NEW and CLIConfig_DEPRECATED
-                initialConfig[property] = oldValue;
+                configWithMergedProperties[property] = oldValue;
             }
         }
     }
-    return initialConfig;
+    return configWithMergedProperties;
 }
 async function handleAccountNameConflicts(globalConfig, deprecatedConfig, force) {
     if (!deprecatedConfig.portals?.length || !globalConfig.accounts?.length) {
@@ -84,7 +94,7 @@ async function handleAccountNameConflicts(globalConfig, deprecatedConfig, force)
     }
     const accountsWithConflictsToRemove = new Set();
     const renamedAccounts = [];
-    const accountsNotYetInGlobal = deprecatedConfig.portals.filter(portal => portal.portalId &&
+    const accountsNotYetInGlobal = (deprecatedConfig.portals || []).filter(portal => portal.portalId &&
         !globalConfig.accounts?.some(acc => acc.accountId === portal.portalId));
     const accountsWithConflicts = accountsNotYetInGlobal.filter(localAccount => globalConfig.accounts?.some(globalAccount => globalAccount.name === localAccount.name));
     if (accountsWithConflicts.length > 0) {
@@ -120,12 +130,13 @@ async function handleAccountNameConflicts(globalConfig, deprecatedConfig, force)
         }
         deprecatedConfig.portals.push(...renamedAccounts);
     }
-    const cleanedPortals = deprecatedConfig.portals.filter(portal => !accountsWithConflictsToRemove.has(portal));
+    const cleanedPortals = (deprecatedConfig.portals || []).filter(portal => !accountsWithConflictsToRemove.has(portal));
     return { ...deprecatedConfig, portals: cleanedPortals };
 }
-export async function handleMerge(deprecatedConfigPath, force, hideWarning) {
+export async function handleMerge(force, hideWarning) {
+    const deprecatedConfigPath = process.env.HUBSPOT_CONFIG_PATH || getLocalConfigFilePathIfExists();
     if (!hideWarning) {
-        uiLogger.warn(lib.configMigrate.deprecatedConfigWarning(deprecatedConfigPath || getConfigPath(undefined, false)));
+        uiLogger.warn(lib.configMigrate.deprecatedConfigWarning(deprecatedConfigPath || ''));
         uiLogger.log('');
     }
     uiLogger.log(lib.configMigrate.handleMerge.description(ARCHIVED_HUBSPOT_CONFIG_YAML_FILE_NAME));
@@ -140,19 +151,32 @@ export async function handleMerge(deprecatedConfigPath, force, hideWarning) {
             return true; // exit with "true" so the user is shown a success message instead of an error
         }
     }
-    const deprecatedConfig = getDeprecatedConfig(deprecatedConfigPath);
-    const globalConfig = getGlobalConfig();
-    if (!deprecatedConfig || !globalConfig) {
+    let deprecatedConfig;
+    let globalConfig;
+    try {
+        deprecatedConfig = getConfigAtPath(deprecatedConfigPath || '');
+        globalConfig = getConfig();
+    }
+    catch (error) {
+        debugError(error);
         return true;
     }
     const mergedConfig = await handleMergeConfigProperties(globalConfig, deprecatedConfig, force);
     const cleanedDeprecatedConfig = await handleAccountNameConflicts(mergedConfig, deprecatedConfig, force);
-    const { skippedAccountIds } = mergeExistingConfigs(mergedConfig, cleanedDeprecatedConfig);
+    const { skippedAccountIds } = mergeConfigAccounts(mergedConfig, cleanedDeprecatedConfig);
     if (skippedAccountIds.length > 0) {
         uiLogger.log('');
         uiLogger.log(lib.configMigrate.handleMerge.skippedExistingAccounts(skippedAccountIds));
         uiLogger.log('');
     }
+    try {
+        archiveConfigAtPath(deprecatedConfigPath);
+    }
+    catch (error) {
+        debugError(error);
+        uiLogger.error(lib.configMigrate.errors.archive(deprecatedConfigPath));
+        return true;
+    }
     uiLogger.success(lib.configMigrate.handleMerge.success);
     return true;
 }

package/lib/configOptions.js

@@ -1,4 +1,4 @@
-import { updateAllowUsageTracking, updateAllowAutoUpdates, updateDefaultCmsPublishMode, updateHttpTimeout, isConfigFlagEnabled, updateAutoOpenBrowser, } from '@hubspot/local-dev-lib/config';
+import { updateAllowUsageTracking, updateAllowAutoUpdates, updateDefaultCmsPublishMode, updateHttpTimeout, updateAutoOpenBrowser, getConfig, } from '@hubspot/local-dev-lib/config';
 import { CMS_PUBLISH_MODE } from '@hubspot/local-dev-lib/constants/files';
 import { commaSeparatedValues } from '@hubspot/local-dev-lib/text';
 import { trackCommandUsage } from './usageTracking.js';
@@ -110,5 +110,6 @@ export async function setAutoOpenBrowser({ accountId, autoOpenBrowser, }) {
         : lib.configOptions.setAutoOpenBrowser.disabled);
 }
 export function isAutoOpenBrowserEnabled() {
-    return isConfigFlagEnabled('autoOpenBrowser', true);
+    const config = getConfig();
+    return config?.autoOpenBrowser !== false; // Default to true
 }

package/lib/constants.d.ts

@@ -139,3 +139,4 @@ export declare const LEGACY_PRIVATE_APP_FILE = "app.json";
 export declare const THEME_FILE = "theme.json";
 export declare const CMS_ASSETS_FILE = "cms-assets.json";
 export declare const LEGACY_CONFIG_FILES: string[];
+export declare const ACCOUNT_LEVEL_CHOICES: readonly ["FREE", "STARTER", "PROFESSIONAL", "ENTERPRISE"];

package/lib/constants.js

@@ -135,3 +135,9 @@ export const LEGACY_CONFIG_FILES = [
     LEGACY_PRIVATE_APP_FILE,
     LEGACY_PUBLIC_APP_FILE,
 ];
+export const ACCOUNT_LEVEL_CHOICES = [
+    'FREE',
+    'STARTER',
+    'PROFESSIONAL',
+    'ENTERPRISE',
+];

package/lib/dependencyManagement.d.ts

@@ -1,6 +1,12 @@
-export declare function installPackages({ packages, installLocations, }: {
+export declare function installPackages({ packages, installLocations, dev, }: {
     packages?: string[];
     installLocations?: string[];
+    dev?: boolean;
 }): Promise<void>;
-export declare function getProjectPackageJsonLocations(dir?: string): Promise<string[]>;
+export declare function updatePackages({ packages, installLocations, }: {
+    packages?: string[];
+    installLocations?: string[];
+}): Promise<void>;
+export declare function getProjectPackageJsonLocations(dir?: string, isUpdate?: boolean): Promise<string[]>;
+export declare function isPackageInstalled(directory: string, packageName: string): boolean;
 export declare function hasMissingPackages(directory: string): Promise<boolean>;

package/lib/dependencyManagement.js

@@ -6,19 +6,21 @@ import { walk } from '@hubspot/local-dev-lib/fs';
 import { getProjectConfig } from './projects/config.js';
 import { commands } from '../lang/en.js';
 import SpinniesManager from './ui/SpinniesManager.js';
-import { isGloballyInstalled, executeInstall, DEFAULT_PACKAGE_MANAGER, } from './npm.js';
+import { isGloballyInstalled, executeInstall, executeUpdate, DEFAULT_PACKAGE_MANAGER, } from './npm.js';
 class NoPackageJsonFilesError extends Error {
-    constructor(projectName) {
-        super(commands.project.installDeps.noPackageJsonInProject(projectName));
+    constructor(projectName, isUpdate = false) {
+        super(isUpdate
+            ? commands.project.updateDeps.noPackageJsonInProject(projectName)
+            : commands.project.installDeps.noPackageJsonInProject(projectName));
     }
 }
-export async function installPackages({ packages, installLocations, }) {
+export async function installPackages({ packages, installLocations, dev = false, }) {
     const installDirs = installLocations || (await getProjectPackageJsonLocations());
     await Promise.all(installDirs.map(async (dir) => {
-        await installPackagesInDirectory(dir, packages);
+        await installPackagesInDirectory(dir, packages, dev);
     }));
 }
-async function installPackagesInDirectory(directory, packages) {
+async function installPackagesInDirectory(directory, packages, dev = false) {
     const spinner = `installingDependencies-${directory}`;
     const relativeDir = path.relative(process.cwd(), directory);
     SpinniesManager.init();
@@ -28,7 +30,8 @@ async function installPackagesInDirectory(directory, packages) {
             : commands.project.installDeps.installingDependencies(relativeDir),
     });
     try {
-        await executeInstall(packages, null, { cwd: directory });
+        const flags = dev && packages && packages.length > 0 ? '--save-dev' : null;
+        await executeInstall(packages, flags, { cwd: directory });
         SpinniesManager.succeed(spinner, {
             text: commands.project.installDeps.installationSuccessful(relativeDir),
         });
@@ -42,26 +45,60 @@ async function installPackagesInDirectory(directory, packages) {
         });
     }
 }
-export async function getProjectPackageJsonLocations(dir) {
+export async function updatePackages({ packages, installLocations, }) {
+    const installDirs = installLocations || (await getProjectPackageJsonLocations(undefined, true));
+    await Promise.all(installDirs.map(async (dir) => {
+        await updatePackagesInDirectory(dir, packages);
+    }));
+}
+async function updatePackagesInDirectory(directory, packages) {
+    const spinner = `updatingDependencies-${directory}`;
+    const relativeDir = path.relative(process.cwd(), directory);
+    SpinniesManager.init();
+    SpinniesManager.add(spinner, {
+        text: packages && packages.length
+            ? commands.project.updateDeps.updatingDependenciesToLocation(`[${packages.join(', ')}]`, relativeDir)
+            : commands.project.updateDeps.updatingDependencies(relativeDir),
+    });
+    try {
+        await executeUpdate(packages, null, { cwd: directory });
+        SpinniesManager.succeed(spinner, {
+            text: commands.project.updateDeps.updateSuccessful(relativeDir),
+        });
+    }
+    catch (e) {
+        SpinniesManager.fail(spinner, {
+            text: commands.project.updateDeps.updatingDependenciesFailed(relativeDir),
+        });
+        throw new Error(commands.project.updateDeps.updatingDependenciesFailed(relativeDir), {
+            cause: e,
+        });
+    }
+}
+export async function getProjectPackageJsonLocations(dir, isUpdate = false) {
     const projectConfig = await getProjectConfig(dir);
     if (!projectConfig ||
         !projectConfig.projectDir ||
         !projectConfig.projectConfig) {
-        throw new Error(commands.project.installDeps.noProjectConfig);
+        throw new Error(isUpdate
+            ? commands.project.updateDeps.noProjectConfig
+            : commands.project.installDeps.noProjectConfig);
     }
     const { projectDir, projectConfig: { srcDir, name }, } = projectConfig;
     if (!(await isGloballyInstalled(DEFAULT_PACKAGE_MANAGER))) {
-        throw new Error(commands.project.installDeps.packageManagerNotInstalled(DEFAULT_PACKAGE_MANAGER));
+        throw new Error(isUpdate
+            ? commands.project.updateDeps.packageManagerNotInstalled(DEFAULT_PACKAGE_MANAGER)
+            : commands.project.installDeps.packageManagerNotInstalled(DEFAULT_PACKAGE_MANAGER));
     }
     if (!fs.existsSync(projectConfig.projectDir) ||
         !fs.existsSync(path.join(projectDir, srcDir))) {
-        throw new NoPackageJsonFilesError(name);
+        throw new NoPackageJsonFilesError(name, isUpdate);
     }
     const packageJsonFiles = (await walk(path.join(projectDir, srcDir))).filter(file => file.includes('package.json') &&
         !file.includes('node_modules') &&
         !file.includes('.vite'));
     if (packageJsonFiles.length === 0) {
-        throw new NoPackageJsonFilesError(name);
+        throw new NoPackageJsonFilesError(name, isUpdate);
     }
     const packageParentDirs = [];
     packageJsonFiles.forEach(packageJsonFile => {
@@ -70,6 +107,32 @@ export async function getProjectPackageJsonLocations(dir) {
     });
     return packageParentDirs;
 }
+function isPackageInPackageJson(directory, packageName) {
+    const packageJsonPath = path.join(directory, 'package.json');
+    try {
+        const packageJsonContent = fs.readFileSync(packageJsonPath, 'utf-8');
+        const packageJson = JSON.parse(packageJsonContent);
+        return !!((packageJson.dependencies && packageJson.dependencies[packageName]) ||
+            (packageJson.devDependencies && packageJson.devDependencies[packageName]));
+    }
+    catch (error) {
+        return false;
+    }
+}
+function isPackageInNodeModules(directory, packageName) {
+    const packagePath = path.join(directory, 'node_modules', packageName);
+    try {
+        return fs.existsSync(packagePath);
+    }
+    catch (error) {
+        return false;
+    }
+}
+export function isPackageInstalled(directory, packageName) {
+    const inPackageJson = isPackageInPackageJson(directory, packageName);
+    const actuallyInstalled = isPackageInNodeModules(directory, packageName);
+    return inPackageJson && actuallyInstalled;
+}
 export async function hasMissingPackages(directory) {
     const exec = util.promisify(execAsync);
     const { stdout } = await exec(`npm install --ignore-scripts --dry-run`, {

package/lib/developerTestAccounts.d.ts

@@ -1,6 +1,6 @@
-import { CLIAccount } from '@hubspot/local-dev-lib/types/Accounts';
+import { HubSpotConfigAccount } from '@hubspot/local-dev-lib/types/Accounts';
 import { FetchDeveloperTestAccountsResponse } from '@hubspot/local-dev-lib/types/developerTestAccounts.js';
 import { Environment } from '@hubspot/local-dev-lib/types/Config';
-export declare function getHasDevTestAccounts(appDeveloperAccountConfig: CLIAccount): boolean;
-export declare function validateDevTestAccountUsageLimits(accountConfig: CLIAccount): Promise<FetchDeveloperTestAccountsResponse | null>;
+export declare function getHasDevTestAccounts(appDeveloperAccountConfig: HubSpotConfigAccount): boolean;
+export declare function validateDevTestAccountUsageLimits(accountConfig: HubSpotConfigAccount): Promise<FetchDeveloperTestAccountsResponse | null>;
 export declare function handleDeveloperTestAccountCreateError(err: unknown, accountId: number, env: Environment, portalLimit: number): never;

package/lib/developerTestAccounts.js

@@ -1,6 +1,5 @@
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
-import { getAccountId, getConfigAccounts } from '@hubspot/local-dev-lib/config';
-import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
+import { getAllConfigAccounts } from '@hubspot/local-dev-lib/config';
 import { fetchDeveloperTestAccounts } from '@hubspot/local-dev-lib/api/developerTestAccounts';
 import { isMissingScopeError, isSpecifiedError, } from '@hubspot/local-dev-lib/errors/index';
 import { uiLogger } from './ui/logger.js';
@@ -9,9 +8,8 @@ import { lib } from '../lang/en.js';
 import { uiAccountDescription } from './ui/index.js';
 import { logError } from './errorHandlers/index.js';
 export function getHasDevTestAccounts(appDeveloperAccountConfig) {
-    const id = getAccountIdentifier(appDeveloperAccountConfig);
-    const parentPortalId = getAccountId(id);
-    const accountsList = getConfigAccounts();
+    const parentPortalId = appDeveloperAccountConfig.accountId;
+    const accountsList = getAllConfigAccounts();
     if (!accountsList) {
         return false;
     }
@@ -25,8 +23,7 @@ export function getHasDevTestAccounts(appDeveloperAccountConfig) {
     return false;
 }
 export async function validateDevTestAccountUsageLimits(accountConfig) {
-    const id = getAccountIdentifier(accountConfig);
-    const accountId = getAccountId(id);
+    const accountId = accountConfig.accountId;
     if (!accountId) {
         return null;
     }

package/lib/doctor/DiagnosticInfoBuilder.d.ts

@@ -44,7 +44,7 @@ export interface DiagnosticInfo extends FilesInfo {
     warningCount?: number;
 }
 export declare class DiagnosticInfoBuilder {
-    accountId: number | null;
+    accountId: number | undefined;
     readonly configSettings: {
         [key: string]: unknown;
     };

package/lib/doctor/DiagnosticInfoBuilder.js

@@ -3,8 +3,8 @@ import { fetchProject } from '@hubspot/local-dev-lib/api/projects';
 import path from 'path';
 import { pkg } from '../jsonLoader.js';
 import { uiLogger } from '../ui/logger.js';
-import { getAccountId, getDefaultAccountOverrideFilePath, isConfigFlagEnabled, } from '@hubspot/local-dev-lib/config';
-import { getAccountConfig, getConfigPath } from '@hubspot/local-dev-lib/config';
+import { isConfigFlagEnabled, getConfigFilePath, getConfigDefaultAccountIfExists, } from '@hubspot/local-dev-lib/config';
+import { getDefaultAccountOverrideFilePath } from '@hubspot/local-dev-lib/config/defaultAccountOverride';
 import { getAccessToken } from '@hubspot/local-dev-lib/personalAccessKey';
 import { walk } from '@hubspot/local-dev-lib/fs';
 import util from 'util';
@@ -33,15 +33,18 @@ export class DiagnosticInfoBuilder {
     files;
     processInfo;
     constructor(processInfo) {
-        this.accountId = getAccountId();
-        const accountConfig = getAccountConfig(this.accountId);
+        const accountConfig = getConfigDefaultAccountIfExists();
+        this.accountId = accountConfig?.accountId;
         this.configSettings = {
             httpUseLocalhost: isConfigFlagEnabled('httpUseLocalhost'),
         };
         this.env = accountConfig?.env;
         this.authType = accountConfig?.authType;
         this.accountType = accountConfig?.accountType;
-        this.personalAccessKey = accountConfig?.personalAccessKey;
+        this.personalAccessKey =
+            accountConfig && 'personalAccessKey' in accountConfig
+                ? accountConfig.personalAccessKey
+                : undefined;
         this.processInfo = processInfo;
     }
     async generateDiagnosticInfo() {
@@ -58,7 +61,7 @@ export class DiagnosticInfoBuilder {
             platform,
             arch,
             path: mainModule?.path,
-            config: getConfigPath(),
+            config: getConfigFilePath(),
             defaultAccountOverrideFile: getDefaultAccountOverrideFilePath(),
             configSettings: this.configSettings,
             versions: {

package/lib/doctor/Doctor.js

@@ -1,5 +1,6 @@
 import { uiLogger } from '../ui/logger.js';
-import { getAccountId, getCWDAccountOverride, } from '@hubspot/local-dev-lib/config';
+import { getConfigDefaultAccountIfExists } from '@hubspot/local-dev-lib/config';
+import { getDefaultAccountOverrideAccountId } from '@hubspot/local-dev-lib/config/defaultAccountOverride';
 import SpinniesManager from '../ui/SpinniesManager.js';
 import { hasMissingPackages } from '../dependencyManagement.js';
 import { getLatestCliVersion } from '../npm.js';
@@ -25,7 +26,7 @@ export class Doctor {
     diagnosticInfoBuilder;
     constructor(diagnosticInfoBuilder = new DiagnosticInfoBuilder(process)) {
         SpinniesManager.init();
-        this.accountId = getAccountId();
+        this.accountId = getConfigDefaultAccountIfExists()?.accountId ?? null;
         this.diagnosticInfoBuilder = diagnosticInfoBuilder;
     }
     async diagnose() {
@@ -88,7 +89,7 @@ export class Doctor {
             });
             this.diagnosis?.addDefaultAccountOverrideFileSection({
                 type: 'warning',
-                message: lib.doctor.defaultAccountOverrideFileChecks.overrideAccountId(getCWDAccountOverride()),
+                message: lib.doctor.defaultAccountOverrideFileChecks.overrideAccountId(getDefaultAccountOverrideAccountId()),
             });
         }
     }

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

@@ -1,8 +1,8 @@
 vi.mock('@hubspot/local-dev-lib/config');
 import { Diagnosis } from '../Diagnosis.js';
-import { getAccountConfig as __getAccountConfig } from '@hubspot/local-dev-lib/config';
+import { getConfigAccountIfExists as __getConfigAccountIfExists } from '@hubspot/local-dev-lib/config';
 import stripAnsi from 'strip-ansi';
-const getAccountConfig = __getAccountConfig;
+const getConfigAccountIfExists = __getConfigAccountIfExists;
 describe('lib/doctor/Diagnosis', () => {
     const diagnosticInfo = {
         account: {},
@@ -31,7 +31,8 @@ describe('lib/doctor/Diagnosis', () => {
     };
     const accountId = 123456;
     beforeEach(() => {
-        getAccountConfig.mockReturnValue({
+        getConfigAccountIfExists.mockReturnValue({
+            accountId,
             accountType: 'STANDARD',
             name: 'Standard Account',
         });