@hubspot/cli 7.6.0-beta.6 → 7.6.0-beta.7

package/README.md

@@ -4,10 +4,6 @@
 
 A CLI for HubSpot developers to enable local development and automation. [Learn more about building on HubSpot](https://developers.hubspot.com).
 
-## Contributing
-
-For more information on developing, see the [Contributing Guide](CONTRIBUTING.md).
-
 ## Getting started
 
 For more information on using these tools, see [Local Development Tooling: Getting Started](https://developers.hubspot.com/docs/cms/guides/getting-started-with-local-development)

package/commands/__tests__/testAccount.test.js

@@ -2,6 +2,7 @@ import yargs from 'yargs';
 import testAccountCreateCommand from '../testAccount/create.js';
 import testAccountCreateConfigCommand from '../testAccount/createConfig.js';
 import testAccountDeleteCommand from '../testAccount/delete.js';
+import testAccountImportDataCommand from '../testAccount/importData.js';
 import testAccountCommands from '../testAccount.js';
 vi.mock('../testAccount/create');
 vi.mock('../testAccount/createConfig');
@@ -37,6 +38,7 @@ describe('commands/testAccount', () => {
             testAccountCreateCommand,
             testAccountCreateConfigCommand,
             testAccountDeleteCommand,
+            testAccountImportDataCommand,
         ];
         it('should demand the command takes one positional argument', () => {
             testAccountCommands.builder(yargs);

package/commands/project/__tests__/devUnifiedFlow.test.js

@@ -326,26 +326,6 @@ describe('unifiedProjectDevFlow', () => {
                 projectName: mockProject.name,
             }));
         });
-        it('should detect GitHub linked projects', async () => {
-            const githubLinkedProject = {
-                ...mockProject,
-                sourceIntegration: { source: 'GITHUB' },
-            };
-            ensureProjectExists.mockResolvedValue({
-                projectExists: true,
-                project: githubLinkedProject,
-            });
-            await unifiedProjectDevFlow({
-                args: mockArgs,
-                targetProjectAccountId: mockTargetProjectAccountId,
-                providedTargetTestingAccountId: mockProvidedTargetTestingAccountId,
-                projectConfig: mockProjectConfig,
-                projectDir: mockProjectDir,
-            });
-            expect(LocalDevProcess).toHaveBeenCalledWith(expect.objectContaining({
-                isGithubLinked: true,
-            }));
-        });
     });
     describe('local dev process setup', () => {
         it('should initialize LocalDevProcess with correct parameters', async () => {
@@ -359,7 +339,6 @@ describe('unifiedProjectDevFlow', () => {
             expect(LocalDevProcess).toHaveBeenCalledWith({
                 initialProjectNodes: mockProjectNodes,
                 debug: mockArgs.debug,
-                isGithubLinked: false,
                 profile: mockArgs.profile,
                 targetProjectAccountId: mockTargetProjectAccountId,
                 targetTestingAccountId: mockProvidedTargetTestingAccountId,

package/commands/project/dev/unifiedFlow.d.ts

@@ -10,5 +10,5 @@ type UnifiedProjectDevFlowArgs = {
     projectDir: string;
     profileConfig?: HsProfileFile;
 };
-export declare function unifiedProjectDevFlow({ args, targetProjectAccountId, providedTargetTestingAccountId, projectConfig, projectDir, profileConfig, }: UnifiedProjectDevFlowArgs): Promise<void>;
+export declare function unifiedProjectDevFlow({ args, targetProjectAccountId, providedTargetTestingAccountId, projectConfig, projectDir, }: UnifiedProjectDevFlowArgs): Promise<void>;
 export {};

package/commands/project/dev/unifiedFlow.js

@@ -19,7 +19,7 @@ import { uiLine } from '../../../lib/ui/index.js';
 import { uiLogger } from '../../../lib/ui/logger.js';
 import { commands } from '../../../lang/en.js';
 import LocalDevWebsocketServer from '../../../lib/projects/localDev/LocalDevWebsocketServer.js';
-export async function unifiedProjectDevFlow({ args, targetProjectAccountId, providedTargetTestingAccountId, projectConfig, projectDir, profileConfig, }) {
+export async function unifiedProjectDevFlow({ args, targetProjectAccountId, providedTargetTestingAccountId, projectConfig, projectDir, }) {
     const env = getValidEnv(getEnv(targetProjectAccountId));
     let projectNodes;
     // Get IR
@@ -53,7 +53,7 @@ export async function unifiedProjectDevFlow({ args, targetProjectAccountId, prov
     const accounts = getConfigAccounts();
     const accountIsCombined = await isUnifiedAccount(targetProjectAccountConfig);
     const targetProjectAccountIsTestAccountOrSandbox = isTestAccountOrSandbox(targetProjectAccountConfig);
-    if (!accountIsCombined && !profileConfig) {
+    if (!accountIsCombined) {
         uiLogger.error(commands.project.dev.errors.accountNotCombined);
         process.exit(EXIT_CODES.ERROR);
     }
@@ -105,11 +105,9 @@ export async function unifiedProjectDevFlow({ args, targetProjectAccountId, prov
         allowCreate: false,
         noLogs: true,
     });
-    let isGithubLinked = false;
     let project = uploadedProject;
     SpinniesManager.init();
     if (projectExists && project) {
-        isGithubLinked = Boolean(project.sourceIntegration && project.sourceIntegration.source === 'GITHUB');
         await compareLocalProjectToDeployed(projectConfig, targetProjectAccountId, project.deployedBuild?.buildId, projectNodes);
     }
     else {
@@ -120,7 +118,6 @@ export async function unifiedProjectDevFlow({ args, targetProjectAccountId, prov
     const localDevProcess = new LocalDevProcess({
         initialProjectNodes: projectNodes,
         debug: args.debug,
-        isGithubLinked,
         profile: args.profile,
         targetProjectAccountId,
         targetTestingAccountId: targetTestingAccountId,

package/commands/testAccount/__tests__/importData.test.js

@@ -0,0 +1,93 @@
+import yargs from 'yargs';
+import { addAccountOptions, addConfigOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
+import testAccountImportDataCommand from '../importData.js';
+import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
+import { handleImportData, handleTargetTestAccountSelectionFlow, } from '../../../lib/importData.js';
+import { trackCommandUsage } from '../../../lib/usageTracking.js';
+import { getImportDataRequest } from '@hubspot/local-dev-lib/crm';
+import { logError } from '../../../lib/errorHandlers/index.js';
+import { importDataFilePathPrompt } from '../../../lib/prompts/importDataFilePathPrompt.js';
+import { confirmImportDataPrompt } from '../../../lib/prompts/confirmImportDataPrompt.js';
+vi.mock('../../../lib/commonOpts');
+vi.mock('../../../lib/importData');
+vi.mock('../../../lib/usageTracking');
+vi.mock('@hubspot/local-dev-lib/crm');
+vi.mock('../../../lib/errorHandlers/index');
+vi.mock('../../../lib/prompts/importDataFilePathPrompt');
+vi.mock('../../../lib/prompts/confirmImportDataPrompt');
+describe('commands/testAccount/importData', () => {
+    const yargsMock = yargs;
+    const mockExit = vi
+        .spyOn(process, 'exit')
+        .mockImplementation(() => undefined);
+    const mockHandleImportData = vi.mocked(handleImportData);
+    const mockHandleTargetTestAccountSelectionFlow = vi.mocked(handleTargetTestAccountSelectionFlow);
+    const mockTrackCommandUsage = vi.mocked(trackCommandUsage);
+    const mockGetImportDataRequest = vi.mocked(getImportDataRequest);
+    const mockLogError = vi.mocked(logError);
+    const mockImportDataFilePathPrompt = vi.mocked(importDataFilePathPrompt);
+    const mockConfirmImportDataPrompt = vi.mocked(confirmImportDataPrompt);
+    beforeEach(() => {
+        mockExit.mockReset();
+        mockHandleImportData.mockReset();
+        mockHandleTargetTestAccountSelectionFlow.mockReset();
+        mockTrackCommandUsage.mockReset();
+        mockGetImportDataRequest.mockReset();
+        mockLogError.mockReset();
+        mockImportDataFilePathPrompt.mockReset();
+        mockConfirmImportDataPrompt.mockReset();
+    });
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(testAccountImportDataCommand.command).toEqual('import-data');
+        });
+    });
+    // describe('describe', () => {
+    //   it('should provide a description', () => {
+    //     expect(testAccountImportDataCommand.describe).toBeDefined();
+    //   });
+    // });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            testAccountImportDataCommand.builder(yargsMock);
+            expect(yargsMock.example).toHaveBeenCalledTimes(1);
+            expect(yargsMock.options).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+    describe('handler', () => {
+        it('should complete the flow given the correct args', async () => {
+            const targetAccountId = 123456789;
+            const mockArgs = {
+                d: false,
+                debug: false,
+                _: [],
+                $0: 'hs',
+                derivedAccountId: targetAccountId,
+                userProvidedAccount: 'test-account',
+                filePath: 'test-file.json',
+                skipConfirm: true,
+            };
+            mockHandleTargetTestAccountSelectionFlow.mockResolvedValue(targetAccountId);
+            mockGetImportDataRequest.mockReturnValue({
+                importRequest: {},
+                dataFileNames: ['test-file.json'],
+            });
+            mockHandleImportData.mockResolvedValue();
+            await testAccountImportDataCommand.handler(mockArgs);
+            expect(mockHandleTargetTestAccountSelectionFlow).toHaveBeenCalledWith(123456789, 'test-account');
+            expect(mockGetImportDataRequest).toHaveBeenCalledWith('test-file.json');
+            expect(mockHandleImportData).toHaveBeenCalledTimes(1);
+            expect(mockTrackCommandUsage).toHaveBeenCalledTimes(1);
+            expect(mockLogError).not.toHaveBeenCalled();
+            expect(mockImportDataFilePathPrompt).not.toHaveBeenCalled();
+            expect(mockConfirmImportDataPrompt).not.toHaveBeenCalled();
+            expect(mockExit).toHaveBeenCalledWith(EXIT_CODES.SUCCESS);
+        });
+    });
+});

package/commands/testAccount/importData.d.ts

@@ -0,0 +1,9 @@
+import { AccountArgs, CommonArgs, ConfigArgs, EnvironmentArgs, YargsCommandModule } from '../../types/Yargs.js';
+export declare const command = "import-data";
+export declare const describe: undefined;
+type CrmImportDataArgs = CommonArgs & ConfigArgs & AccountArgs & EnvironmentArgs & {
+    filePath: string | undefined;
+    skipConfirm: boolean | undefined;
+};
+declare const crmImportDataCommand: YargsCommandModule<unknown, CrmImportDataArgs>;
+export default crmImportDataCommand;

package/commands/testAccount/importData.js

@@ -0,0 +1,61 @@
+import { getImportDataRequest } from '@hubspot/local-dev-lib/crm';
+import { logError } from '../../lib/errorHandlers/index.js';
+import { makeYargsBuilder } from '../../lib/yargsUtils.js';
+import { trackCommandUsage } from '../../lib/usageTracking.js';
+import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import { importDataFilePathPrompt } from '../../lib/prompts/importDataFilePathPrompt.js';
+import { handleImportData, handleTargetTestAccountSelectionFlow, } from '../../lib/importData.js';
+import { confirmImportDataPrompt } from '../../lib/prompts/confirmImportDataPrompt.js';
+import { commands } from '../../lang/en.js';
+export const command = 'import-data';
+export const describe = undefined; // commands.testAccount.subcommands.importData.describe;
+async function handler(args) {
+    const { derivedAccountId, userProvidedAccount, filePath: providedFilePath, skipConfirm, } = args;
+    trackCommandUsage('crm-import-data', {}, derivedAccountId);
+    let targetAccountId;
+    try {
+        targetAccountId = await handleTargetTestAccountSelectionFlow(derivedAccountId, userProvidedAccount);
+        const filePath = providedFilePath || (await importDataFilePathPrompt());
+        const { importRequest, dataFileNames } = getImportDataRequest(filePath);
+        const confirmImportData = skipConfirm ||
+            (await confirmImportDataPrompt(targetAccountId, dataFileNames));
+        if (!confirmImportData) {
+            process.exit(EXIT_CODES.SUCCESS);
+        }
+        await handleImportData(targetAccountId, dataFileNames, importRequest);
+    }
+    catch (error) {
+        logError(error);
+        process.exit(EXIT_CODES.ERROR);
+    }
+    process.exit(EXIT_CODES.SUCCESS);
+}
+function crmImportDataBuilder(yargs) {
+    yargs.example([['$0 test-account import-data']]);
+    yargs.options({
+        'file-path': {
+            type: 'string',
+            describe: commands.testAccount.subcommands.importData.options.filePath.describe,
+            positional: false,
+        },
+        'skip-confirm': {
+            type: 'boolean',
+            describe: commands.testAccount.subcommands.importData.options.skipConfirm
+                .describe,
+        },
+    });
+    return yargs;
+}
+const builder = makeYargsBuilder(crmImportDataBuilder, command, describe, {
+    useGlobalOptions: true,
+    useConfigOptions: true,
+    useAccountOptions: true,
+    useEnvironmentOptions: true,
+});
+const crmImportDataCommand = {
+    command,
+    describe,
+    builder,
+    handler,
+};
+export default crmImportDataCommand;

package/commands/testAccount.js

@@ -1,5 +1,6 @@
 import createTestAccountCommand from './testAccount/create.js';
 import createTestAccountConfigCommand from './testAccount/createConfig.js';
+import importDataCommand from './testAccount/importData.js';
 import deleteTestAccountCommand from './testAccount/delete.js';
 import { makeYargsBuilder } from '../lib/yargsUtils.js';
 import { commands } from '../lang/en.js';
@@ -10,6 +11,7 @@ function testAccountBuilder(yargs) {
         .command(createTestAccountCommand)
         .command(createTestAccountConfigCommand)
         .command(deleteTestAccountCommand)
+        .command(importDataCommand)
         .demandCommand(1, '');
     return yargs;
 }

package/lang/en.d.ts

@@ -1,3 +1,4 @@
+import { CLIAccount } from '@hubspot/local-dev-lib/types/Accounts';
 export declare const commands: {
     readonly generalErrors: {
         readonly srcIsProject: (src: string, command: string) => string;
@@ -976,7 +977,7 @@ Profiles enable you to reference variables in your component configuration files
             readonly logs: {
                 readonly betaMessage: "HubSpot projects local development";
                 readonly placeholderAccountSelection: "Using default account as target account (for now)";
-                readonly learnMoreLocalDevServer: "Learn more about the projects local dev server";
+                readonly learnMoreLocalDevServer: string;
                 readonly accountTypeInformation: "Testing in a developer test account is strongly recommended, but you can use a sandbox account if your plan allows you to create one.";
                 readonly learnMoreMessage: `
 Visit our ${string} to learn more.`;
@@ -1803,6 +1804,19 @@ ${string}`;
     };
     readonly testAccount: {
         readonly describe: "Commands for working with test accounts.";
+        readonly subcommands: {
+            readonly importData: {
+                readonly describe: "Import data into the CRM";
+                readonly options: {
+                    readonly skipConfirm: {
+                        readonly describe: "Skip the confirmation prompt";
+                    };
+                    readonly filePath: {
+                        readonly describe: "The path to the JSON file containing the import schema";
+                    };
+                };
+            };
+        };
         readonly create: {
             readonly describe: "Create a test account from a config file";
             readonly configPathPrompt: "[--config-path] Enter the path to the test account config: ";
@@ -2690,6 +2704,16 @@ Run ${string} to upgrade to version ${string}`;
     readonly boxen: {
         readonly failedToLoad: "Failed to load boxen util.";
     };
+    readonly importData: {
+        readonly errors: {
+            readonly incorrectAccountType: (derivedAccountId: number) => string;
+            readonly failedToImportData: "Failed to import data into portal.";
+            readonly notDeveloperTestAccount: "The account is not a developer test account.";
+            readonly noAccountConfig: (accountId: number) => string;
+        };
+        readonly inProgress: (portalId: number, fileNames: string[]) => string;
+        readonly viewImportLink: (baseUrl: string, accountId: number, importId: string) => string;
+    };
     readonly ui: {
         readonly betaTag: string;
         readonly betaWarning: {
@@ -2849,6 +2873,19 @@ Run ${string} to upgrade to version ${string}`;
         };
     };
     readonly prompts: {
+        readonly importDataFilePathPrompt: {
+            readonly promptContext: `To view the JSON schema for data imports, visit ${string}`;
+            readonly promptMessage: "[--file-path] Select the JSON file that will be used to import your data.";
+        };
+        readonly confirmImportDataPrompt: {
+            readonly message: (dataFileNames: string[], cliAccount: CLIAccount | null) => string;
+        };
+        readonly importDataTestAccountSelectPrompt: {
+            readonly errors: {
+                readonly noAccountsFound: "No accounts found.";
+                readonly noChildTestAccountsFound: (parentAccountId: number) => string;
+            };
+        };
         readonly projectDevTargetAccountPrompt: {
             readonly createNewSandboxOption: "<Test on a new development sandbox>";
             readonly createNewDeveloperTestAccountOption: "<Test on a new developer test account>";

package/lang/en.js

@@ -5,6 +5,7 @@ import { ARCHIVED_HUBSPOT_CONFIG_YAML_FILE_NAME, GLOBAL_CONFIG_PATH, DEFAULT_HUB
 import { uiAccountDescription, uiBetaTag, uiCommandReference, uiLink, UI_COLORS, } from '../lib/ui/index.js';
 import { getProjectDetailUrl, getProjectSettingsUrl, getLocalDevUiUrl, } from '../lib/projects/urls.js';
 import { PROJECT_CONFIG_FILE, PROJECT_WITH_APP } from '../lib/constants.js';
+import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
 export const commands = {
     generalErrors: {
         srcIsProject: (src, command) => `"${src}" is in a project folder. Did you mean "hs project ${command}"?`,
@@ -976,7 +977,7 @@ export const commands = {
             logs: {
                 betaMessage: 'HubSpot projects local development',
                 placeholderAccountSelection: 'Using default account as target account (for now)',
-                learnMoreLocalDevServer: 'Learn more about the projects local dev server',
+                learnMoreLocalDevServer: uiLink('Learn more about the projects local dev server', 'https://developers.hubspot.com/docs/platform/project-cli-commands#start-a-local-development-server'),
                 accountTypeInformation: 'Testing in a developer test account is strongly recommended, but you can use a sandbox account if your plan allows you to create one.',
                 learnMoreMessage: `\nVisit our ${uiLink('docs on Developer Test and Sandbox accounts', 'https://developers.hubspot.com/docs/getting-started/account-types')} to learn more.`,
             },
@@ -1798,6 +1799,19 @@ export const commands = {
     },
     testAccount: {
         describe: 'Commands for working with test accounts.',
+        subcommands: {
+            importData: {
+                describe: 'Import data into the CRM',
+                options: {
+                    skipConfirm: {
+                        describe: 'Skip the confirmation prompt',
+                    },
+                    filePath: {
+                        describe: 'The path to the JSON file containing the import schema',
+                    },
+                },
+            },
+        },
         create: {
             describe: 'Create a test account from a config file',
             configPathPrompt: '[--config-path] Enter the path to the test account config: ',
@@ -2498,7 +2512,7 @@ export const lib = {
         },
     },
     AppDevModeInterface: {
-        defaultMarketplaceAppWarning: (installCount) => `\n\nYour marketplace app is currently installed in ${chalk.bold(`${installCount} ${installCount === 1 ? 'account' : 'accounts'}`)}. Any uploaded changes will impact your app's users. We strongly recommend creating a copy of this app to test your changes before proceding.`,
+        defaultMarketplaceAppWarning: (installCount) => `Your marketplace app is currently installed in ${chalk.bold(`${installCount} ${installCount === 1 ? 'account' : 'accounts'}`)}. Any uploaded changes will impact your app's users. We strongly recommend creating a copy of this app to test your changes before proceding.`,
         autoInstallDeclined: 'You must install your app on your target test account to proceed with local development.',
         autoInstallSuccess: (appName, targetTestAccountId) => `Successfully installed app ${appName} on account ${uiAccountDescription(targetTestAccountId)}\n`,
         autoInstallError: (appName, targetTestAccountId) => `Error installing app ${appName} on account ${uiAccountDescription(targetTestAccountId)}. You may still be able to install your app in your browser.`,
@@ -2682,6 +2696,16 @@ export const lib = {
     boxen: {
         failedToLoad: 'Failed to load boxen util.',
     },
+    importData: {
+        errors: {
+            incorrectAccountType: (derivedAccountId) => `The account ${uiAccountDescription(derivedAccountId)} is not a standard account, developer test account, or app developer account.`,
+            failedToImportData: 'Failed to import data into portal.',
+            notDeveloperTestAccount: 'The account is not a developer test account.',
+            noAccountConfig: (accountId) => `No account config found for ${uiAccountDescription(accountId)}`,
+        },
+        inProgress: (portalId, fileNames) => `Importing data into ${uiAccountDescription(portalId)} from [${fileNames.join(', ')}]`,
+        viewImportLink: (baseUrl, accountId, importId) => `Data import currently processing. You can view the status of your import ${uiLink('here', `${baseUrl}/import/${accountId}/post/${importId}`)}`,
+    },
     ui: {
         betaTag: chalk.bold('[BETA]'),
         betaWarning: {
@@ -2841,6 +2865,19 @@ export const lib = {
         },
     },
     prompts: {
+        importDataFilePathPrompt: {
+            promptContext: `To view the JSON schema for data imports, visit ${uiLink('the docs', 'https://developers.hubspot.com/docs/guides/api/crm/imports')}`,
+            promptMessage: '[--file-path] Select the JSON file that will be used to import your data.',
+        },
+        confirmImportDataPrompt: {
+            message: (dataFileNames, cliAccount) => `You are importing [${dataFileNames.join(', ')}] into ${uiAccountDescription(getAccountIdentifier(cliAccount))}. Continue?`,
+        },
+        importDataTestAccountSelectPrompt: {
+            errors: {
+                noAccountsFound: 'No accounts found.',
+                noChildTestAccountsFound: (parentAccountId) => `No developer test accounts found under the parent account ${uiAccountDescription(parentAccountId)}`,
+            },
+        },
         projectDevTargetAccountPrompt: {
             createNewSandboxOption: '<Test on a new development sandbox>',
             createNewDeveloperTestAccountOption: '<Test on a new developer test account>',

package/lib/__tests__/importData.test.d.ts

@@ -0,0 +1 @@
+export {};

package/lib/__tests__/importData.test.js

@@ -0,0 +1,89 @@
+import { uiLogger } from '../ui/logger.js';
+import { createImport } from '@hubspot/local-dev-lib/api/crm';
+import { getAccountConfig, getAccountId } from '@hubspot/local-dev-lib/config';
+import { handleImportData, handleTargetTestAccountSelectionFlow, } from '../importData.js';
+import { lib } from '../../lang/en.js';
+import { isDeveloperTestAccount, isStandardAccount, isAppDeveloperAccount, } from '../accountTypes.js';
+import { importDataTestAccountSelectPrompt } from '../prompts/importDataTestAccountSelectPrompt.js';
+vi.mock('../ui/logger');
+vi.mock('@hubspot/local-dev-lib/api/crm');
+vi.mock('@hubspot/local-dev-lib/config');
+vi.mock('../accountTypes');
+vi.mock('../prompts/importDataTestAccountSelectPrompt');
+describe('lib/importData', () => {
+    const mockUiLogger = vi.mocked(uiLogger);
+    const mockCreateImport = vi.mocked(createImport);
+    const mockGetAccountConfig = vi.mocked(getAccountConfig);
+    const mockGetAccountId = vi.mocked(getAccountId);
+    const mockIsDeveloperTestAccount = vi.mocked(isDeveloperTestAccount);
+    const mockIsStandardAccount = vi.mocked(isStandardAccount);
+    const mockIsAppDeveloperAccount = vi.mocked(isAppDeveloperAccount);
+    const mockImportDataTestAccountSelectPrompt = vi.mocked(importDataTestAccountSelectPrompt);
+    beforeEach(() => {
+        mockUiLogger.info.mockReset();
+        mockUiLogger.success.mockReset();
+        mockUiLogger.error.mockReset();
+        mockCreateImport.mockReset();
+        mockGetAccountConfig.mockReset();
+        mockGetAccountId.mockReset();
+        mockIsDeveloperTestAccount.mockReset();
+        mockIsStandardAccount.mockReset();
+        mockIsAppDeveloperAccount.mockReset();
+        mockImportDataTestAccountSelectPrompt.mockReset();
+    });
+    describe('handleImportData', () => {
+        const targetAccountId = 123456789;
+        const dataFileNames = ['test-file.json'];
+        const importRequest = {
+            name: 'test-import',
+        };
+        it('should log the correct success message', async () => {
+            // @ts-expect-error - mockCreateImport is not typed correctly
+            mockCreateImport.mockResolvedValue({
+                data: { id: '123' },
+            });
+            await handleImportData(targetAccountId, dataFileNames, importRequest);
+            expect(mockUiLogger.success).toHaveBeenCalledWith(lib.importData.viewImportLink('https://app.hubspot.com', targetAccountId, '123'));
+        });
+        it('should log the correct error message', async () => {
+            mockCreateImport.mockRejectedValue(new Error('test-error'));
+            // weird because we catch the error, log a specific message, and then throw it again
+            await expect(handleImportData(targetAccountId, dataFileNames, importRequest)).rejects.toThrow('test-error');
+            expect(mockUiLogger.error).toHaveBeenCalledWith(lib.importData.errors.failedToImportData);
+        });
+    });
+    describe('handleTargetTestAccountSelectionFlow', () => {
+        const userProvidedAccountId = '1234';
+        const derivedAccountId = 123456789;
+        it('should error if the userProvidedAccountId is not the right account type', async () => {
+            mockGetAccountConfig.mockReturnValue({});
+            mockGetAccountId.mockReturnValue(1234);
+            mockIsDeveloperTestAccount.mockReturnValue(false);
+            await expect(handleTargetTestAccountSelectionFlow(derivedAccountId, userProvidedAccountId)).rejects.toThrow(lib.importData.errors.notDeveloperTestAccount);
+        });
+        it('should error if the derivedAccountId belongs to the wrong account type', async () => {
+            mockGetAccountConfig.mockReturnValue({});
+            mockIsDeveloperTestAccount.mockReturnValue(false);
+            mockIsStandardAccount.mockReturnValue(false);
+            mockIsAppDeveloperAccount.mockReturnValue(false);
+            await expect(handleTargetTestAccountSelectionFlow(derivedAccountId, undefined)).rejects.toThrow(lib.importData.errors.incorrectAccountType(derivedAccountId));
+        });
+        it('should return the derivedAccountId if it is a developer test account', async () => {
+            mockGetAccountConfig.mockReturnValue({});
+            mockIsDeveloperTestAccount.mockReturnValue(true);
+            const result = await handleTargetTestAccountSelectionFlow(derivedAccountId, undefined);
+            expect(result).toBe(derivedAccountId);
+        });
+        it('should return the result of the importDataTestAccountSelectPrompt if the derivedAccountId is a standard or app developer account', async () => {
+            mockGetAccountConfig.mockReturnValue({});
+            mockIsDeveloperTestAccount.mockReturnValue(false);
+            mockIsStandardAccount.mockReturnValue(true);
+            mockIsAppDeveloperAccount.mockReturnValue(true);
+            mockImportDataTestAccountSelectPrompt.mockResolvedValue({
+                selectedAccountId: 890223,
+            });
+            const result = await handleTargetTestAccountSelectionFlow(derivedAccountId, undefined);
+            expect(result).toBe(890223);
+        });
+    });
+});

package/lib/accountTypes.js

@@ -1,6 +1,5 @@
 import { HUBSPOT_ACCOUNT_TYPES } from '@hubspot/local-dev-lib/constants/config';
-import { hasFeature } from './hasFeature.js';
-import { FEATURES } from './constants.js';
+import { hasUnfiedAppsAccess } from './hasFeature.js';
 import { getAccountIdentifier } from '@hubspot/local-dev-lib/config/getAccountIdentifier';
 function isAccountType(accountConfig, accountType) {
     return Boolean(accountConfig.accountType && accountType.includes(accountConfig.accountType));
@@ -40,5 +39,5 @@ export async function isUnifiedAccount(account) {
     if (!accountId) {
         return false;
     }
-    return hasFeature(accountId, FEATURES.UNIFIED_APPS);
+    return hasUnfiedAppsAccess(accountId);
 }

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

@@ -11,7 +11,7 @@ import { ensureProjectExists } from '../../projects/ensureProjectExists.js';
 import { poll } from '../../polling.js';
 import { CLI_UNMIGRATABLE_REASONS, continueMigration, initializeMigration, listAppsForMigration, listThemesForMigration, } from '../../../api/migrate.js';
 import { lib } from '../../../lang/en.js';
-import { hasFeature } from '../../hasFeature.js';
+import { hasUnfiedAppsAccess } from '../../hasFeature.js';
 import { getUnmigratableReason, generateFilterAppsByProjectNameFunction, buildErrorMessageFromMigrationStatus, fetchMigrationApps, promptForAppToMigrate, selectAppToMigrate, handleMigrationSetup, beginMigration, pollMigrationStatus, finalizeMigration, downloadProjectFiles, migrateApp2025_2, logInvalidAccountError, validateMigrationAppsAndThemes, } from '../migrate.js';
 vi.mock('@hubspot/local-dev-lib/logger');
 vi.mock('@hubspot/local-dev-lib/path');
@@ -43,7 +43,7 @@ const mockedListAppsForMigration = listAppsForMigration;
 const mockedListThemesForMigration = listThemesForMigration;
 const mockedInitializeMigration = initializeMigration;
 const mockedContinueMigration = continueMigration;
-const mockedHasFeature = hasFeature;
+const mockedHasUnfiedAppsAccess = hasUnfiedAppsAccess;
 const mockedFs = fs;
 const createMockMigratableApp = (id, name, projectName) => ({
     appId: id,
@@ -82,7 +82,7 @@ describe('lib/app/migrate', () => {
         mockedGetCwd.mockReturnValue(MOCK_CWD);
         mockedSanitizeFileName.mockImplementation(name => name);
         mockedValidateUid.mockReturnValue(undefined);
-        mockedHasFeature.mockResolvedValue(true);
+        mockedHasUnfiedAppsAccess.mockResolvedValue(true);
         mockedFs.renameSync.mockImplementation(() => { });
     });
     describe('getUnmigratableReason', () => {
@@ -472,10 +472,10 @@ describe('lib/app/migrate', () => {
             unstable: false,
         };
         beforeEach(() => {
-            mockedHasFeature.mockResolvedValue(true);
+            mockedHasUnfiedAppsAccess.mockResolvedValue(true);
         });
         it('should throw an error when account is not ungated for unified apps', async () => {
-            mockedHasFeature.mockResolvedValueOnce(false);
+            mockedHasUnfiedAppsAccess.mockResolvedValueOnce(false);
             await expect(migrateApp2025_2(ACCOUNT_ID, options)).rejects.toThrowError(/isn't enrolled in the required product beta to access this command./);
         });
         it('should throw an error when projectConfig is invalid', async () => {

package/lib/app/migrate.js

@@ -16,8 +16,7 @@ import { DEFAULT_POLLING_STATUS_LOOKUP, poll } from '../polling.js';
 import { checkMigrationStatusV2, CLI_UNMIGRATABLE_REASONS, continueMigration, initializeMigration, isMigrationStatus, listAppsForMigration, listThemesForMigration, } from '../../api/migrate.js';
 import fs from 'fs';
 import { lib } from '../../lang/en.js';
-import { hasFeature } from '../hasFeature.js';
-import { FEATURES } from '../constants.js';
+import { hasUnfiedAppsAccess } from '../hasFeature.js';
 import { getProjectBuildDetailUrl, getProjectDetailUrl, } from '../projects/urls.js';
 import { uiLogger } from '../ui/logger.js';
 export function getUnmigratableReason(reasonCode, projectName, accountId) {
@@ -340,7 +339,7 @@ export async function downloadProjectFiles(derivedAccountId, projectName, buildI
 }
 export async function migrateApp2025_2(derivedAccountId, options, projectConfig) {
     SpinniesManager.init();
-    const ungatedForUnifiedApps = await hasFeature(derivedAccountId, FEATURES.UNIFIED_APPS);
+    const ungatedForUnifiedApps = await hasUnfiedAppsAccess(derivedAccountId);
     if (!ungatedForUnifiedApps) {
         throw new Error(lib.migrate.errors.notUngatedForUnifiedApps(uiAccountDescription(derivedAccountId)));
     }

package/lib/constants.d.ts

@@ -87,6 +87,7 @@ export declare const LOCAL_DEV_UI_MESSAGE_SEND_TYPES: {
     UPDATE_PROJECT_NODES: string;
     UPDATE_APP_DATA: string;
     UPDATE_PROJECT_DATA: string;
+    UPDATE_UPLOAD_WARNINGS: string;
 };
 export declare const LOCAL_DEV_UI_MESSAGE_RECEIVE_TYPES: {
     UPLOAD: string;

package/lib/constants.js

@@ -79,6 +79,7 @@ export const LOCAL_DEV_UI_MESSAGE_SEND_TYPES = {
     UPDATE_PROJECT_NODES: 'server:updateProjectNodes',
     UPDATE_APP_DATA: 'server:updateAppData',
     UPDATE_PROJECT_DATA: 'server:updateProjectData',
+    UPDATE_UPLOAD_WARNINGS: 'server:updateUploadWarnings',
 };
 export const LOCAL_DEV_UI_MESSAGE_RECEIVE_TYPES = {
     UPLOAD: 'client:upload',

package/lib/hasFeature.d.ts

@@ -1,3 +1,4 @@
 import { FEATURES } from './constants.js';
 import { ValueOf } from '@hubspot/local-dev-lib/types/Utils';
 export declare function hasFeature(accountId: number, feature: ValueOf<typeof FEATURES>): Promise<boolean>;
+export declare function hasUnfiedAppsAccess(accountId: number): Promise<boolean>;