@hubspot/cli 7.10.0 → 7.11.0-experimental.0

package/lib/projects/uieLinting.js

@@ -0,0 +1,222 @@
+import fs from 'fs';
+import path from 'path';
+import util from 'util';
+import semver from 'semver';
+import { exec as execAsync } from 'node:child_process';
+import { getProjectPackageJsonLocations, isPackageInstalled, } from '../dependencyManagement.js';
+import { commands } from '../../lang/en.js';
+import { uiLogger } from '../ui/logger.js';
+export const REQUIRED_PACKAGES_AND_MIN_VERSIONS = {
+    eslint: '9.0.0',
+    '@typescript-eslint/eslint-plugin': '8.46.4',
+    '@typescript-eslint/parser': '8.46.4',
+    'typescript-eslint': '8.46.4',
+    jiti: '2.6.1',
+};
+const ESLINT_CONFIG_FILES = [
+    'eslint.config.mts',
+    'eslint.config.ts',
+    'eslint.config.cts',
+    'eslint.config.js',
+    'eslint.config.mjs',
+    'eslint.config.cjs',
+];
+const DEPRECATED_ESLINT_CONFIG_FILES = [
+    '.eslintrc.js',
+    '.eslintrc.cjs',
+    '.eslintrc.yaml',
+    '.eslintrc.yml',
+    '.eslintrc.json',
+    '.eslintrc',
+];
+const ESLINT_CONFIG_TEMPLATE = `import { defineConfig } from "eslint/config";
+import tsParser from "@typescript-eslint/parser";
+
+export default defineConfig([
+  {
+    files: ["**/*.{js,mjs,cjs,ts,mts,cts,jsx,tsx}"],
+    languageOptions: {
+      parser: tsParser,
+      parserOptions: {
+        ecmaVersion: "latest",
+        sourceType: "module",
+        ecmaFeatures: {
+          jsx: true
+        }
+      }
+    },
+    rules: {
+      "no-console": ["warn", { allow: ["warn", "error"] }]
+    }
+  },
+]);
+`;
+function getPackageVersionFromPackageJson(directory, packageName) {
+    const packageJsonPath = path.join(directory, 'package.json');
+    try {
+        const packageJsonContent = fs.readFileSync(packageJsonPath, 'utf-8');
+        const packageJson = JSON.parse(packageJsonContent);
+        const version = packageJson.dependencies?.[packageName] ||
+            packageJson.devDependencies?.[packageName];
+        return version || null;
+    }
+    catch (error) {
+        return null;
+    }
+}
+function isPackageVersionValid(directory, packageName) {
+    const versionRange = getPackageVersionFromPackageJson(directory, packageName);
+    if (!versionRange) {
+        return false;
+    }
+    const minimumVersion = REQUIRED_PACKAGES_AND_MIN_VERSIONS[packageName];
+    if (!minimumVersion) {
+        // If no minimum version specified, any version is valid
+        return true;
+    }
+    try {
+        // Check if the version range satisfies the minimum version
+        // For ranges like ^9.0.0, ~8.5.0, >=9.0.0, etc.
+        const minVersionSatisfied = semver.satisfies(minimumVersion, versionRange, {
+            includePrerelease: true,
+        });
+        const coercedVersion = semver.minVersion(versionRange);
+        if (coercedVersion) {
+            return semver.gte(coercedVersion, minimumVersion);
+        }
+        return minVersionSatisfied;
+    }
+    catch (error) {
+        return false;
+    }
+}
+export function isEslintInstalled(directory) {
+    return isPackageInstalled(directory, 'eslint');
+}
+export function areAllLintPackagesInstalled(directory) {
+    return Object.keys(REQUIRED_PACKAGES_AND_MIN_VERSIONS).every(pkg => {
+        const installed = isPackageInstalled(directory, pkg);
+        const validVersion = isPackageVersionValid(directory, pkg);
+        return installed && validVersion;
+    });
+}
+export function getMissingLintPackages(directory) {
+    const missingPackages = [];
+    for (const packageName of Object.keys(REQUIRED_PACKAGES_AND_MIN_VERSIONS)) {
+        const isInstalled = isPackageInstalled(directory, packageName);
+        const isValidVersion = isPackageVersionValid(directory, packageName);
+        if (!isInstalled || !isValidVersion) {
+            missingPackages.push(packageName);
+        }
+    }
+    return { missingPackages };
+}
+export function hasEslintConfig(directory) {
+    return ESLINT_CONFIG_FILES.some(configFile => {
+        const configPath = path.join(directory, configFile);
+        return fs.existsSync(configPath);
+    });
+}
+export function hasDeprecatedEslintConfig(directory) {
+    return DEPRECATED_ESLINT_CONFIG_FILES.some(configFile => {
+        const configPath = path.join(directory, configFile);
+        return fs.existsSync(configPath);
+    });
+}
+export function getDeprecatedEslintConfigFiles(directory) {
+    return DEPRECATED_ESLINT_CONFIG_FILES.filter(configFile => {
+        const configPath = path.join(directory, configFile);
+        return fs.existsSync(configPath);
+    });
+}
+export function createEslintConfig(directory) {
+    const configPath = path.join(directory, 'eslint.config.mts');
+    try {
+        fs.writeFileSync(configPath, ESLINT_CONFIG_TEMPLATE, 'utf-8');
+        return path.relative(process.cwd(), configPath);
+    }
+    catch (error) {
+        uiLogger.error(commands.project.lint.failedToCreateEslintConfig(configPath));
+        throw error;
+    }
+}
+export async function lintPackagesInDirectory(directory, projectDir) {
+    const displayPath = projectDir
+        ? path.relative(projectDir, directory)
+        : directory;
+    const exec = util.promisify(execAsync);
+    const lintCommand = 'npx eslint . --color';
+    try {
+        const { stdout, stderr } = await exec(lintCommand, {
+            cwd: directory,
+            maxBuffer: 10 * 1024 * 1024, // 10MB buffer for large outputs
+        });
+        let output = `\n${displayPath}:\n`;
+        if (stdout && stdout.trim()) {
+            output += stdout;
+        }
+        else {
+            output += '  No linting issues found.\n';
+        }
+        if (stderr && stderr.trim()) {
+            output += stderr;
+        }
+        return { success: true, output };
+    }
+    catch (error) {
+        // ESLint returns exit code 1 when there are linting errors
+        // but still provides useful output in stdout/stderr
+        let output = `\n${displayPath}:\n`;
+        if (error && typeof error === 'object' && 'stdout' in error) {
+            const execError = error;
+            if (execError.stdout) {
+                output += execError.stdout;
+            }
+            if (execError.stderr) {
+                output += execError.stderr;
+            }
+        }
+        return { success: false, output };
+    }
+}
+export async function lintPackages(lintLocations, projectDir) {
+    const locations = lintLocations || (await getProjectPackageJsonLocations());
+    const results = [];
+    for (const location of locations) {
+        const result = await lintPackagesInDirectory(location, projectDir);
+        const displayPath = projectDir
+            ? path.relative(projectDir, location)
+            : location;
+        results.push({ location: displayPath, ...result });
+    }
+    const failedLocations = results.filter(r => !r.success);
+    return {
+        success: failedLocations.length === 0,
+        results,
+    };
+}
+export function displayLintResults(results) {
+    // Display all output
+    results.forEach(r => {
+        uiLogger.log(r.output);
+    });
+    // Summary
+    const failedLocations = results.filter(r => !r.success);
+    const passedLocations = results.filter(r => r.success);
+    uiLogger.log('\n' + '='.repeat(50));
+    if (passedLocations.length > 0) {
+        uiLogger.success(`Linting passed in ${passedLocations.length} ${passedLocations.length === 1 ? 'directory' : 'directories'}:`);
+        passedLocations.forEach(r => {
+            uiLogger.log(`  ✓ ${r.location}`);
+        });
+    }
+    if (failedLocations.length > 0) {
+        if (passedLocations.length > 0) {
+            uiLogger.log('');
+        }
+        uiLogger.error(`Linting failed in ${failedLocations.length} ${failedLocations.length === 1 ? 'directory' : 'directories'}:`);
+        failedLocations.forEach(r => {
+            uiLogger.log(`  ✗ ${r.location}`);
+        });
+    }
+}

package/lib/projects/urls.js

@@ -1,18 +1,17 @@
 import { getHubSpotWebsiteOrigin } from '@hubspot/local-dev-lib/urls';
-import { getEnv } from '@hubspot/local-dev-lib/config';
-import { ENVIRONMENTS } from '@hubspot/local-dev-lib/constants/environments';
+import { getConfigAccountEnvironment } from '@hubspot/local-dev-lib/config';
 function getBaseUrl(accountId) {
-    return getHubSpotWebsiteOrigin(getEnv(accountId) === 'qa' ? ENVIRONMENTS.QA : ENVIRONMENTS.PROD);
+    return getHubSpotWebsiteOrigin(getConfigAccountEnvironment(accountId));
 }
 function getProjectHomeUrl(accountId) {
     return `${getBaseUrl(accountId)}/developer-projects/${accountId}`;
 }
 export function getProjectComponentDistributionUrl(projectName, componentName, accountId) {
-    const baseUrl = getHubSpotWebsiteOrigin(getEnv(accountId) === 'qa' ? ENVIRONMENTS.QA : ENVIRONMENTS.PROD);
+    const baseUrl = getHubSpotWebsiteOrigin(getConfigAccountEnvironment(accountId));
     return `${baseUrl}/developer-projects/${accountId}/project/${projectName}/component/${componentName}/distribution`;
 }
 export function getDeveloperOverviewUrl(accountId) {
-    const baseUrl = getHubSpotWebsiteOrigin(getEnv(accountId) === 'qa' ? ENVIRONMENTS.QA : ENVIRONMENTS.PROD);
+    const baseUrl = getHubSpotWebsiteOrigin(getConfigAccountEnvironment(accountId));
     return `${baseUrl}/developer-overview/${accountId}`;
 }
 export function getProjectDetailUrl(projectName, accountId) {
@@ -38,6 +37,6 @@ export function getLocalDevUiUrl(accountId, showWelcomeScreen) {
     return `${getBaseUrl(accountId)}/developer-projects-local-dev/${accountId}${showWelcomeScreen ? '?welcome' : ''}`;
 }
 export function getAccountHomeUrl(accountId) {
-    const baseUrl = getHubSpotWebsiteOrigin(getEnv(accountId) === 'qa' ? ENVIRONMENTS.QA : ENVIRONMENTS.PROD);
+    const baseUrl = getHubSpotWebsiteOrigin(getConfigAccountEnvironment(accountId));
     return `${baseUrl}/home?portalId=${accountId}`;
 }

package/lib/prompts/__tests__/downloadProjectPrompt.test.js

@@ -1,4 +1,4 @@
-import { getAccountId } from '@hubspot/local-dev-lib/config';
+import { getConfigAccountIfExists } from '@hubspot/local-dev-lib/config';
 import { fetchProjects } from '@hubspot/local-dev-lib/api/projects';
 import { downloadProjectPrompt } from '../downloadProjectPrompt.js';
 vi.mock('../promptUtils', () => ({
@@ -10,15 +10,17 @@ vi.mock('@hubspot/local-dev-lib/api/projects', () => ({
     }),
 }));
 vi.mock('@hubspot/local-dev-lib/config', () => ({
-    getAccountId: vi.fn().mockImplementation(() => 123456789),
-    configFileExists: vi.fn().mockImplementation(() => true),
+    getConfigAccountIfExists: vi
+        .fn()
+        .mockImplementation(() => ({ accountId: 123456789 })),
+    globalConfigFileExists: vi.fn().mockReturnValue(true),
 }));
 describe('lib/prompts/downloadProjectPrompt', () => {
     it('should honor the account passed as an option', async () => {
         const account = 'Prod';
         await downloadProjectPrompt({ account });
-        expect(getAccountId).toHaveBeenCalledTimes(1);
-        expect(getAccountId).toHaveBeenCalledWith(account);
+        expect(getConfigAccountIfExists).toHaveBeenCalledTimes(1);
+        expect(getConfigAccountIfExists).toHaveBeenCalledWith(account);
     });
     it('should fetch the projects for the correct accountId', async () => {
         const account = 'Prod';

package/lib/prompts/accountNamePrompt.js

@@ -1,4 +1,4 @@
-import { accountNameExistsInConfig } from '@hubspot/local-dev-lib/config';
+import { getConfigAccountIfExists } from '@hubspot/local-dev-lib/config';
 import { promptUser } from './promptUtils.js';
 import { lib } from '../../lang/en.js';
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
@@ -17,7 +17,7 @@ export function getCliAccountNamePromptConfig(defaultName) {
             else if (val.indexOf(' ') >= 0) {
                 return lib.prompts.accountNamePrompt.errors.spacesInName;
             }
-            return accountNameExistsInConfig(val)
+            return getConfigAccountIfExists(val)
                 ? lib.prompts.accountNamePrompt.errors.accountNameExists(val)
                 : true;
         },
@@ -54,7 +54,7 @@ export function hubspotAccountNamePrompt({ accountType, currentPortalCount = 0,
                 else if (!val.trim().length) {
                     return lib.prompts.accountNamePrompt.errors.nameRequired;
                 }
-                return accountNameExistsInConfig(val)
+                return getConfigAccountIfExists(val)
                     ? lib.prompts.accountNamePrompt.errors.accountNameExists(val)
                     : true;
             },

package/lib/prompts/accountsPrompt.d.ts

@@ -1 +1 @@
-export declare function selectAccountFromConfig(prompt?: string): Promise<string>;
+export declare function selectAccountFromConfig(prompt?: string): Promise<number>;

package/lib/prompts/accountsPrompt.js

@@ -1,17 +1,16 @@
-import { getConfigDefaultAccount, getConfigAccounts, } from '@hubspot/local-dev-lib/config';
-import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
+import { getConfigDefaultAccountIfExists, getAllConfigAccounts, } from '@hubspot/local-dev-lib/config';
 import { promptUser } from './promptUtils.js';
 import { commands } from '../../lang/en.js';
 import { uiAccountDescription } from '../ui/index.js';
 function mapAccountChoices(portals) {
     return (portals?.map(p => ({
-        name: uiAccountDescription(getAccountIdentifier(p), false),
-        value: String(p.name || getAccountIdentifier(p)),
+        name: uiAccountDescription(p.accountId, false),
+        value: p.accountId,
     })) || []);
 }
 export async function selectAccountFromConfig(prompt = '') {
-    const accountsList = getConfigAccounts();
-    const defaultAccount = getConfigDefaultAccount();
+    const accountsList = getAllConfigAccounts();
+    const defaultAccount = getConfigDefaultAccountIfExists();
     const { default: selectedDefault } = await promptUser([
         {
             type: 'list',
@@ -19,7 +18,7 @@ export async function selectAccountFromConfig(prompt = '') {
             pageSize: 20,
             message: prompt || commands.account.subcommands.use.promptMessage,
             choices: mapAccountChoices(accountsList),
-            default: defaultAccount ?? undefined,
+            default: defaultAccount?.accountId ?? undefined,
         },
     ]);
     return selectedDefault;

package/lib/prompts/confirmImportDataPrompt.js

@@ -1,8 +1,8 @@
-import { getAccountConfig } from '@hubspot/local-dev-lib/config';
+import { getConfigAccountById } from '@hubspot/local-dev-lib/config';
 import { promptUser } from './promptUtils.js';
 import { lib } from '../../lang/en.js';
 export async function confirmImportDataPrompt(targetAccountId, dataFileNames) {
-    const account = getAccountConfig(targetAccountId);
+    const account = getConfigAccountById(targetAccountId);
     const { confirmImportData } = await promptUser({
         type: 'confirm',
         name: 'confirmImportData',

package/lib/prompts/downloadProjectPrompt.d.ts

@@ -3,6 +3,7 @@ type DownloadProjectPromptResponse = {
 };
 export declare function downloadProjectPrompt(promptOptions: {
     account?: string;
+    derivedAccountId?: number;
     project?: string;
     name?: string;
 }): Promise<DownloadProjectPromptResponse>;

package/lib/prompts/downloadProjectPrompt.js

@@ -1,5 +1,5 @@
 import { promptUser } from './promptUtils.js';
-import { getAccountId } from '@hubspot/local-dev-lib/config';
+import { getConfigAccountIfExists } from '@hubspot/local-dev-lib/config';
 import { fetchProjects } from '@hubspot/local-dev-lib/api/projects';
 import { logError, ApiErrorContext } from '../errorHandlers/index.js';
 import { uiLogger } from '../ui/logger.js';
@@ -20,7 +20,10 @@ async function createProjectsList(accountId) {
     }
 }
 export async function downloadProjectPrompt(promptOptions) {
-    const accountId = getAccountId(promptOptions.account);
+    const account = promptOptions.account
+        ? getConfigAccountIfExists(promptOptions.account)
+        : undefined;
+    const accountId = account?.accountId || promptOptions.derivedAccountId || null;
     const projectsList = await createProjectsList(accountId);
     const response = await promptUser([
         {

package/lib/prompts/importDataTestAccountSelectPrompt.js

@@ -1,10 +1,9 @@
-import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
 import { promptUser } from './promptUtils.js';
-import { getConfigAccounts } from '@hubspot/local-dev-lib/config';
+import { getAllConfigAccounts } from '@hubspot/local-dev-lib/config';
 import { uiAccountDescription } from '../ui/index.js';
 import { lib } from '../../lang/en.js';
 export async function importDataTestAccountSelectPrompt(parentAccountId) {
-    const accounts = getConfigAccounts();
+    const accounts = getAllConfigAccounts();
     if (!accounts) {
         throw new Error(lib.prompts.importDataTestAccountSelectPrompt.errors.noAccountsFound);
     }
@@ -12,8 +11,8 @@ export async function importDataTestAccountSelectPrompt(parentAccountId) {
         .filter(account => account.parentAccountId === parentAccountId)
         .map(account => {
         return {
-            name: uiAccountDescription(getAccountIdentifier(account)),
-            value: getAccountIdentifier(account),
+            name: uiAccountDescription(account.accountId),
+            value: account.accountId,
         };
     })
         .filter(account => account.value !== undefined && account.name !== undefined);

package/lib/prompts/personalAccessKeyPrompt.js

@@ -1,6 +1,6 @@
 import open from 'open';
 import { OAUTH_SCOPES, DEFAULT_OAUTH_SCOPES, } from '@hubspot/local-dev-lib/constants/auth';
-import { deleteEmptyConfigFile } from '@hubspot/local-dev-lib/config';
+import { deleteConfigFileIfEmpty } from '@hubspot/local-dev-lib/config';
 import { getHubSpotWebsiteOrigin } from '@hubspot/local-dev-lib/urls';
 import { uiLogger } from '../ui/logger.js';
 import { promptUser } from './promptUtils.js';
@@ -26,7 +26,7 @@ export async function personalAccessKeyPrompt({ env, account, }) {
             PERSONAL_ACCESS_KEY_BROWSER_OPEN_PREP,
         ]);
         if (!choice) {
-            deleteEmptyConfigFile();
+            deleteConfigFileIfEmpty();
             process.exit(EXIT_CODES.SUCCESS);
         }
         if (choice ===

package/lib/prompts/projectDevTargetAccountPrompt.d.ts

@@ -1,4 +1,4 @@
-import { CLIAccount } from '@hubspot/local-dev-lib/types/Accounts';
+import { HubSpotConfigAccount } from '@hubspot/local-dev-lib/types/Accounts';
 import { DeveloperTestAccount } from '@hubspot/local-dev-lib/types/developerTestAccounts.js';
 export type ProjectDevTargetAccountPromptResponse = {
     targetAccountId: number | null;
@@ -6,7 +6,7 @@ export type ProjectDevTargetAccountPromptResponse = {
     parentAccountId?: number | null;
     notInConfigAccount?: DeveloperTestAccount | null;
 };
-export declare function selectSandboxTargetAccountPrompt(accounts: CLIAccount[], defaultAccountConfig: CLIAccount): Promise<ProjectDevTargetAccountPromptResponse>;
-export declare function selectDeveloperTestTargetAccountPrompt(accounts: CLIAccount[], defaultAccountConfig: CLIAccount): Promise<ProjectDevTargetAccountPromptResponse>;
+export declare function selectSandboxTargetAccountPrompt(accounts: HubSpotConfigAccount[], defaultAccountConfig: HubSpotConfigAccount): Promise<ProjectDevTargetAccountPromptResponse>;
+export declare function selectDeveloperTestTargetAccountPrompt(accounts: HubSpotConfigAccount[], defaultAccountConfig: HubSpotConfigAccount): Promise<ProjectDevTargetAccountPromptResponse>;
 export declare function confirmDefaultAccountPrompt(accountName: string, accountType: string): Promise<boolean>;
 export declare function confirmUseExistingDeveloperTestAccountPrompt(account: DeveloperTestAccount): Promise<boolean>;

package/lib/prompts/projectDevTargetAccountPrompt.js

@@ -1,7 +1,5 @@
-import { getAccountId } from '@hubspot/local-dev-lib/config';
 import { getSandboxUsageLimits } from '@hubspot/local-dev-lib/api/sandboxHubs';
 import { HUBSPOT_ACCOUNT_TYPES, HUBSPOT_ACCOUNT_TYPE_STRINGS, } from '@hubspot/local-dev-lib/constants/config';
-import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
 import { fetchDeveloperTestAccounts } from '@hubspot/local-dev-lib/api/developerTestAccounts';
 import { promptUser } from './promptUtils.js';
 import { lib } from '../../lang/en.js';
@@ -12,9 +10,9 @@ import { EXIT_CODES } from '../enums/exitCodes.js';
 function mapNestedAccount(accountConfig) {
     const parentAccountId = accountConfig.parentAccountId ?? null;
     return {
-        name: uiAccountDescription(getAccountIdentifier(accountConfig), false),
+        name: uiAccountDescription(accountConfig.accountId, false),
         value: {
-            targetAccountId: getAccountId(accountConfig.name),
+            targetAccountId: accountConfig.accountId,
             createNestedAccount: false,
             parentAccountId,
         },
@@ -24,7 +22,7 @@ function getNonConfigDeveloperTestAccountName(account) {
     return `${account.accountName} [${HUBSPOT_ACCOUNT_TYPE_STRINGS[HUBSPOT_ACCOUNT_TYPES.DEVELOPER_TEST]}] (${account.id})`;
 }
 export async function selectSandboxTargetAccountPrompt(accounts, defaultAccountConfig) {
-    const defaultAccountId = getAccountId(defaultAccountConfig.name);
+    const defaultAccountId = defaultAccountConfig.accountId;
     let choices = [];
     let sandboxUsage = {
         STANDARD: { used: 0, available: 0, limit: 0 },
@@ -84,7 +82,7 @@ export async function selectSandboxTargetAccountPrompt(accounts, defaultAccountC
     return selectTargetAccountPrompt(defaultAccountId, 'sandbox account', choices);
 }
 export async function selectDeveloperTestTargetAccountPrompt(accounts, defaultAccountConfig) {
-    const defaultAccountId = getAccountId(defaultAccountConfig.name);
+    const defaultAccountId = defaultAccountConfig.accountId;
     let devTestAccountsResponse;
     try {
         if (defaultAccountId) {
@@ -108,7 +106,7 @@ export async function selectDeveloperTestTargetAccountPrompt(accounts, defaultAc
     }
     const devTestAccounts = [];
     if (devTestAccountsResponse && devTestAccountsResponse.results) {
-        const accountIds = accounts.map(account => getAccountIdentifier(account));
+        const accountIds = accounts.map(account => account.accountId);
         devTestAccountsResponse.results.forEach(acct => {
             const inConfig = accountIds.includes(acct.id);
             devTestAccounts.push({

package/lib/prompts/sandboxesPrompt.js

@@ -2,23 +2,22 @@ import { promptUser } from './promptUtils.js';
 import { lib } from '../../lang/en.js';
 import { uiAccountDescription } from '../ui/index.js';
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
-import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
 import { isSandbox } from '../accountTypes.js';
-import { getConfigDefaultAccount, getConfigAccounts, } from '@hubspot/local-dev-lib/config';
+import { getConfigDefaultAccountIfExists, getAllConfigAccounts, } from '@hubspot/local-dev-lib/config';
 function mapSandboxAccountChoices(portals) {
     return (portals
         ?.filter(p => isSandbox(p))
         .map(p => ({
-        name: uiAccountDescription(getAccountIdentifier(p), false),
-        value: p.name || getAccountIdentifier(p),
+        name: uiAccountDescription(p.accountId, false),
+        value: p.accountId,
     })) || []);
 }
 function mapNonSandboxAccountChoices(portals) {
     return (portals
         ?.filter(p => !isSandbox(p))
         .map(p => ({
-        name: `${p.name} (${getAccountIdentifier(p)})`,
-        value: p.name || getAccountIdentifier(p),
+        name: `${p.name} (${p.accountId})`,
+        value: p.accountId,
     })) || []);
 }
 export async function sandboxTypePrompt() {
@@ -42,7 +41,7 @@ export async function sandboxTypePrompt() {
     ]);
 }
 export function deleteSandboxPrompt(promptParentAccount = false) {
-    const accountsList = getConfigAccounts();
+    const accountsList = getAllConfigAccounts();
     const choices = promptParentAccount
         ? mapNonSandboxAccountChoices(accountsList)
         : mapSandboxAccountChoices(accountsList);
@@ -58,7 +57,7 @@ export function deleteSandboxPrompt(promptParentAccount = false) {
             type: 'list',
             pageSize: 20,
             choices,
-            default: getConfigDefaultAccount(),
+            default: getConfigDefaultAccountIfExists()?.accountId,
         },
     ]);
 }

package/lib/prompts/setAsDefaultAccountPrompt.js

@@ -1,25 +1,26 @@
-import { updateDefaultAccount, getConfigDefaultAccount, } from '@hubspot/local-dev-lib/config';
+import { setConfigAccountAsDefault, getConfigDefaultAccountIfExists, getAllConfigAccounts, } from '@hubspot/local-dev-lib/config';
 import { promptUser } from './promptUtils.js';
 import { lib } from '../../lang/en.js';
 import { uiLogger } from '../ui/logger.js';
 export async function setAsDefaultAccountPrompt(accountName) {
     // Accounts for deprecated and new config
-    const defaultAccount = getConfigDefaultAccount();
+    const defaultAccount = getConfigDefaultAccountIfExists();
+    const accounts = getAllConfigAccounts() || [];
     const { setAsDefault } = await promptUser([
         {
             name: 'setAsDefault',
             type: 'confirm',
-            when: defaultAccount !== accountName,
+            when: accounts.length >= 1 && defaultAccount?.name !== accountName,
             message: lib.prompts.setAsDefaultAccountPrompt.setAsDefaultAccountMessage,
         },
     ]);
     uiLogger.log('');
     if (setAsDefault) {
-        updateDefaultAccount(accountName);
+        setConfigAccountAsDefault(accountName);
         uiLogger.success(lib.prompts.setAsDefaultAccountPrompt.setAsDefaultAccount(accountName));
     }
-    else {
-        uiLogger.log(lib.prompts.setAsDefaultAccountPrompt.keepingCurrentDefault(getConfigDefaultAccount()));
+    else if (defaultAccount) {
+        uiLogger.log(lib.prompts.setAsDefaultAccountPrompt.keepingCurrentDefault(defaultAccount.name));
     }
     return setAsDefault;
 }

package/lib/sandboxSync.d.ts

@@ -1,4 +1,4 @@
-import { CLIAccount } from '@hubspot/local-dev-lib/types/Accounts';
+import { HubSpotConfigAccount } from '@hubspot/local-dev-lib/types/Accounts';
 import { Environment } from '@hubspot/local-dev-lib/types/Config';
 import { SandboxSyncTask } from '../types/Sandboxes.js';
-export declare function syncSandbox(accountConfig: CLIAccount, parentAccountConfig: CLIAccount, env: Environment, syncTasks: Array<SandboxSyncTask>, slimInfoMessage?: boolean): Promise<void>;
+export declare function syncSandbox(accountConfig: HubSpotConfigAccount, parentAccountConfig: HubSpotConfigAccount, env: Environment, syncTasks: Array<SandboxSyncTask>, slimInfoMessage?: boolean): Promise<void>;

package/lib/sandboxSync.js

@@ -3,23 +3,17 @@ import { getHubSpotWebsiteOrigin } from '@hubspot/local-dev-lib/urls';
 import { uiLogger } from './ui/logger.js';
 import { initiateSync } from '@hubspot/local-dev-lib/api/sandboxSync';
 import { isSpecifiedError } from '@hubspot/local-dev-lib/errors/index';
-import { getAccountId } from '@hubspot/local-dev-lib/config';
-import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
 import { lib } from '../lang/en.js';
 import { getAvailableSyncTypes, getSandboxTypeAsString } from './sandboxes.js';
 import { debugError, logError, ApiErrorContext, } from './errorHandlers/index.js';
 import { uiAccountDescription, uiLine, uiCommandDisabledBanner, } from './ui/index.js';
 import { isDevelopmentSandbox } from './accountTypes.js';
 export async function syncSandbox(accountConfig, parentAccountConfig, env, syncTasks, slimInfoMessage = false) {
-    const id = getAccountIdentifier(accountConfig);
-    const accountId = getAccountId(id);
-    const parentId = getAccountIdentifier(parentAccountConfig);
-    const parentAccountId = getAccountId(parentId);
+    const accountId = accountConfig.accountId;
+    const parentAccountId = parentAccountConfig.accountId;
     const isDevSandbox = isDevelopmentSandbox(accountConfig);
     if (!accountId || !parentAccountId) {
-        throw new Error(lib.sandbox.sync.failure.invalidUser(accountId ? uiAccountDescription(accountId) : id.toString(), parentAccountId
-            ? uiAccountDescription(parentAccountId)
-            : parentId.toString()));
+        throw new Error(lib.sandbox.sync.failure.invalidUser(uiAccountDescription(accountId), uiAccountDescription(parentAccountId)));
     }
     SpinniesManager.init({
         succeedColor: 'white',

package/lib/sandboxes.d.ts

@@ -1,4 +1,4 @@
-import { AccountType, CLIAccount } from '@hubspot/local-dev-lib/types/Accounts';
+import { AccountType, HubSpotConfigAccount } from '@hubspot/local-dev-lib/types/Accounts';
 import { Environment } from '@hubspot/local-dev-lib/types/Config';
 import { SandboxSyncTask, SandboxAccountType } from '../types/Sandboxes.js';
 export declare const SYNC_TYPES: {
@@ -16,7 +16,7 @@ export declare const SANDBOX_TYPE_MAP_V2: {
     readonly DEVELOPMENT_SANDBOX: "DEVELOPER";
 };
 export declare function getSandboxTypeAsString(accountType?: AccountType): string;
-export declare function getHasSandboxesByType(parentAccountConfig: CLIAccount, type: AccountType): boolean;
-export declare function getAvailableSyncTypes(parentAccountConfig: CLIAccount, config: CLIAccount): Promise<Array<SandboxSyncTask>>;
-export declare function validateSandboxUsageLimits(accountConfig: CLIAccount, sandboxType: AccountType, env: Environment): Promise<void>;
+export declare function getHasSandboxesByType(parentAccountConfig: HubSpotConfigAccount, type: AccountType): boolean;
+export declare function getAvailableSyncTypes(parentAccountConfig: HubSpotConfigAccount, config: HubSpotConfigAccount): Promise<Array<SandboxSyncTask>>;
+export declare function validateSandboxUsageLimits(accountConfig: HubSpotConfigAccount, sandboxType: AccountType, env: Environment): Promise<void>;
 export declare function handleSandboxCreateError(err: unknown, env: Environment, name: string, accountId: number): never;