@hubspot/cli 7.9.0 → 7.9.1-experimental.0

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/createDeveloperTestAccountConfigPrompt.d.ts

@@ -14,5 +14,10 @@ export type HubConfig = {
 export declare function createDeveloperTestAccountConfigPrompt(args?: {
     name?: string;
     description?: string;
+    marketingLevel?: AccountLevel;
+    opsLevel?: AccountLevel;
+    serviceLevel?: AccountLevel;
+    salesLevel?: AccountLevel;
+    contentLevel?: AccountLevel;
 }, supportFlags?: boolean): Promise<DeveloperTestAccountConfig>;
 export {};

package/lib/prompts/createDeveloperTestAccountConfigPrompt.js

@@ -49,15 +49,17 @@ const TEST_ACCOUNT_TIERS = [
     new Separator(),
 ];
 export async function createDeveloperTestAccountConfigPrompt(args = {}, supportFlags = true) {
-    const { name, description } = args;
-    let accountName = name;
-    let accountDescription = description;
-    let accountLevelsArray = [];
-    if (!accountName) {
-        const namePromptResult = await promptUser({
+    const hasAnyTierLevels = !!(args.marketingLevel ||
+        args.opsLevel ||
+        args.serviceLevel ||
+        args.salesLevel ||
+        args.contentLevel);
+    const result = await promptUser([
+        {
             name: 'accountName',
             message: lib.prompts.createDeveloperTestAccountConfigPrompt.namePrompt(supportFlags),
             type: 'input',
+            when: !args.name,
             validate: value => {
                 if (!value) {
                     return lib.prompts.createDeveloperTestAccountConfigPrompt.errors
@@ -65,58 +67,67 @@ export async function createDeveloperTestAccountConfigPrompt(args = {}, supportF
                 }
                 return true;
             },
-        });
-        accountName = namePromptResult.accountName;
-    }
-    if (!accountDescription) {
-        const descriptionPromptResult = await promptUser({
+        },
+        {
             name: 'description',
             message: lib.prompts.createDeveloperTestAccountConfigPrompt.descriptionPrompt(supportFlags),
             type: 'input',
-        });
-        accountDescription = descriptionPromptResult.description;
-    }
-    const useDefaultAccountLevelsPromptResult = await promptUser({
-        name: 'useDefaultAccountLevels',
-        message: lib.prompts.createDeveloperTestAccountConfigPrompt
-            .useDefaultAccountLevelsPrompt.message,
-        type: 'list',
-        choices: [
-            {
-                name: lib.prompts.createDeveloperTestAccountConfigPrompt
-                    .useDefaultAccountLevelsPrompt.default,
-                value: 'default',
-            },
-            {
-                name: lib.prompts.createDeveloperTestAccountConfigPrompt
-                    .useDefaultAccountLevelsPrompt.manual,
-                value: 'manual',
-            },
-        ],
-    });
-    if (useDefaultAccountLevelsPromptResult.useDefaultAccountLevels === 'default') {
-        accountLevelsArray = [
-            { hub: 'MARKETING', tier: AccountTiers.ENTERPRISE },
-            { hub: 'OPS', tier: AccountTiers.ENTERPRISE },
-            { hub: 'SERVICE', tier: AccountTiers.ENTERPRISE },
-            { hub: 'SALES', tier: AccountTiers.ENTERPRISE },
-            { hub: 'CONTENT', tier: AccountTiers.ENTERPRISE },
-        ];
+            when: !args.description,
+        },
+    ]);
+    const accountName = args.name || result.accountName;
+    const description = args.description || result.description;
+    let accountLevels = {};
+    if (hasAnyTierLevels) {
+        accountLevels = {
+            marketingLevel: args.marketingLevel || 'ENTERPRISE',
+            opsLevel: args.opsLevel || 'ENTERPRISE',
+            serviceLevel: args.serviceLevel || 'ENTERPRISE',
+            salesLevel: args.salesLevel || 'ENTERPRISE',
+            contentLevel: args.contentLevel || 'ENTERPRISE',
+        };
     }
     else {
-        const accountLevelsPromptResult = await promptUser({
-            name: 'testAccountLevels',
-            message: lib.prompts.createDeveloperTestAccountConfigPrompt.tiersPrompt,
-            type: 'checkbox',
-            pageSize: 13,
-            choices: TEST_ACCOUNT_TIERS,
-            loop: false,
-            validate: choices => {
-                if (choices?.length < Object.keys(hubs).length) {
-                    return lib.prompts.createDeveloperTestAccountConfigPrompt.errors
-                        .allHubsRequired;
-                }
-                else {
+        const tierChoiceResult = await promptUser({
+            name: 'useDefaultAccountLevels',
+            message: lib.prompts.createDeveloperTestAccountConfigPrompt
+                .useDefaultAccountLevelsPrompt.message,
+            type: 'list',
+            choices: [
+                {
+                    name: lib.prompts.createDeveloperTestAccountConfigPrompt
+                        .useDefaultAccountLevelsPrompt.default,
+                    value: 'default',
+                },
+                {
+                    name: lib.prompts.createDeveloperTestAccountConfigPrompt
+                        .useDefaultAccountLevelsPrompt.manual,
+                    value: 'manual',
+                },
+            ],
+        });
+        if (tierChoiceResult.useDefaultAccountLevels === 'default') {
+            accountLevels = {
+                marketingLevel: 'ENTERPRISE',
+                opsLevel: 'ENTERPRISE',
+                serviceLevel: 'ENTERPRISE',
+                salesLevel: 'ENTERPRISE',
+                contentLevel: 'ENTERPRISE',
+            };
+        }
+        else {
+            const tierResult = await promptUser({
+                name: 'testAccountLevels',
+                message: lib.prompts.createDeveloperTestAccountConfigPrompt.tiersPrompt,
+                type: 'checkbox',
+                pageSize: 13,
+                choices: TEST_ACCOUNT_TIERS,
+                loop: false,
+                validate: choices => {
+                    if (choices?.length < Object.keys(hubs).length) {
+                        return lib.prompts.createDeveloperTestAccountConfigPrompt.errors
+                            .allHubsRequired;
+                    }
                     const hubMap = {};
                     for (const choice of choices) {
                         const { hub } = choice.value;
@@ -126,21 +137,20 @@ export async function createDeveloperTestAccountConfigPrompt(args = {}, supportF
                         }
                         hubMap[hub] = true;
                     }
-                }
-                return true;
-            },
-        });
-        accountLevelsArray = accountLevelsPromptResult.testAccountLevels;
+                    return true;
+                },
+            });
+            accountLevels = tierResult.testAccountLevels.reduce((acc, level) => {
+                const { hub: hubName, tier: hubTier } = level;
+                const hubLevel = hubs[hubName];
+                acc[hubLevel] = hubTier;
+                return acc;
+            }, {});
+        }
     }
-    const accountLevels = accountLevelsArray.reduce((acc, level) => {
-        const { hub: hubName, tier: hubTier } = level;
-        const hubLevel = hubs[hubName];
-        acc[hubLevel] = hubTier;
-        return acc;
-    }, {});
     return {
-        accountName: accountName,
-        description: accountDescription,
+        accountName,
+        description,
         ...accountLevels,
     };
 }

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;

package/lib/sandboxes.js

@@ -1,9 +1,8 @@
 import { getSandboxUsageLimits } from '@hubspot/local-dev-lib/api/sandboxHubs';
 import { fetchTypes } from '@hubspot/local-dev-lib/api/sandboxSync';
-import { getAccountId, getConfigAccounts } from '@hubspot/local-dev-lib/config';
+import { getAllConfigAccounts } from '@hubspot/local-dev-lib/config';
 import { getHubSpotWebsiteOrigin } from '@hubspot/local-dev-lib/urls';
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
-import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
 import { isMissingScopeError, isSpecifiedError, } from '@hubspot/local-dev-lib/errors/index';
 import { uiLogger } from './ui/logger.js';
 import { lib } from '../lang/en.js';
@@ -33,9 +32,8 @@ export function getSandboxTypeAsString(accountType) {
     return 'standard';
 }
 export function getHasSandboxesByType(parentAccountConfig, type) {
-    const id = getAccountIdentifier(parentAccountConfig);
-    const parentPortalId = getAccountId(id);
-    const accountsList = getConfigAccounts() || [];
+    const parentPortalId = parentAccountConfig.accountId;
+    const accountsList = getAllConfigAccounts();
     for (const portal of accountsList) {
         if ((portal.parentAccountId !== null ||
             portal.parentAccountId !== undefined) &&
@@ -49,10 +47,8 @@ export function getHasSandboxesByType(parentAccountConfig, type) {
 }
 // Fetches available sync types for a given sandbox portal
 export async function getAvailableSyncTypes(parentAccountConfig, config) {
-    const parentId = getAccountIdentifier(parentAccountConfig);
-    const parentPortalId = getAccountId(parentId);
-    const id = getAccountIdentifier(config);
-    const portalId = getAccountId(id);
+    const parentPortalId = parentAccountConfig.accountId;
+    const portalId = config.accountId;
     if (!parentPortalId || !portalId) {
         throw new Error(lib.sandbox.sync.failure.syncTypeFetch);
     }
@@ -63,8 +59,7 @@ export async function getAvailableSyncTypes(parentAccountConfig, config) {
     return syncTypes.map(t => ({ type: t.name }));
 }
 export async function validateSandboxUsageLimits(accountConfig, sandboxType, env) {
-    const id = getAccountIdentifier(accountConfig);
-    const accountId = getAccountId(id);
+    const accountId = accountConfig.accountId;
     if (!accountId) {
         throw new Error(lib.sandbox.create.failure.usageLimitsFetch);
     }

package/lib/serverlessLogs.js

@@ -3,7 +3,7 @@ import chalk from 'chalk';
 import { uiLogger } from './ui/logger.js';
 import { isHubSpotHttpError, isMissingScopeError, } from '@hubspot/local-dev-lib/errors/index';
 import { SCOPE_GROUPS, PERSONAL_ACCESS_KEY_AUTH_METHOD, } from '@hubspot/local-dev-lib/constants/auth';
-import { getAccountConfig } from '@hubspot/local-dev-lib/config';
+import { getConfigAccountById } from '@hubspot/local-dev-lib/config';
 import { fetchScopeData } from '@hubspot/local-dev-lib/api/localDevAuth';
 import { outputLogs } from './ui/serverlessFunctionLogs.js';
 import { logError, ApiErrorContext } from './errorHandlers/index.js';
@@ -33,7 +33,7 @@ function handleUserInput() {
     });
 }
 async function verifyAccessKeyAndUserAccess(accountId, scopeGroup) {
-    const accountConfig = getAccountConfig(accountId);
+    const accountConfig = getConfigAccountById(accountId);
     if (!accountConfig) {
         return;
     }