@hubspot/cli 7.6.0-beta.2 → 7.6.0-beta.4

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

@@ -0,0 +1,37 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const createConfig_1 = __importDefault(require("../createConfig"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/testAccount/createConfig', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(createConfig_1.default.command).toEqual('create-config');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(createConfig_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            const optionSpy = vi.spyOn(yargsMock, 'option');
+            createConfig_1.default.builder(yargsMock);
+            expect(yargsMock.example).toHaveBeenCalledTimes(1);
+            expect(optionSpy).toHaveBeenCalledWith('name', expect.objectContaining({
+                type: 'string',
+            }));
+            expect(optionSpy).toHaveBeenCalledWith('description', expect.objectContaining({
+                type: 'string',
+            }));
+            expect(optionSpy).toHaveBeenCalledWith('path', expect.objectContaining({
+                type: 'string',
+            }));
+        });
+    });
+});

package/commands/testAccount/__tests__/delete.test.d.ts

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

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

@@ -0,0 +1,36 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const delete_1 = __importDefault(require("../delete"));
+vi.mock('../../../lib/commonOpts');
+describe('commands/testAccount/delete', () => {
+    const yargsMock = yargs_1.default;
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(delete_1.default.command).toEqual('delete <test-account-id>');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(delete_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            delete_1.default.builder(yargsMock);
+            expect(yargsMock.example).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addTestingOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addTestingOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargsMock);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
+        });
+    });
+});

package/commands/testAccount/create.js

@@ -77,7 +77,6 @@ async function handler(args) {
         if (formatOutputAsJson) {
             jsonOutput.accountName = data.accountName;
             jsonOutput.accountId = data.id;
-            jsonOutput.personalAccessKey = data.personalAccessKey;
         }
         testAccountId = data.id;
     }
@@ -93,16 +92,30 @@ async function handler(args) {
             successStates: ['SUCCESS'],
             errorStates: [],
         });
-        SpinniesManager_1.default.succeed('createTestAccount', {
-            text: en_1.commands.testAccount.create.polling.success(testAccountConfig.accountName, testAccountId),
+    }
+    catch (err) {
+        (0, errorHandlers_1.debugError)(err);
+        SpinniesManager_1.default.fail('createTestAccount', {
+            text: en_1.commands.testAccount.create.polling.failure,
         });
     }
+    // HACK: The status endpoint sometimes returns an early success status.
+    // Sleep for an extra 5 seconds to make sure the sync is actually complete.
+    await new Promise(resolve => setTimeout(resolve, 5000));
+    try {
+        // Attempt to generate a new personal access key for the test account now that gate sync is complete.
+        const { data } = await (0, developerTestAccounts_1.generateDeveloperTestAccountPersonalAccessKey)(derivedAccountId, testAccountId);
+        jsonOutput.personalAccessKey = data.personalAccessKey;
+    }
     catch (err) {
         (0, errorHandlers_1.debugError)(err);
         SpinniesManager_1.default.fail('createTestAccount', {
             text: en_1.commands.testAccount.create.polling.failure,
         });
     }
+    SpinniesManager_1.default.succeed('createTestAccount', {
+        text: en_1.commands.testAccount.create.polling.success(testAccountConfig.accountName, testAccountId),
+    });
     if (formatOutputAsJson) {
         logger_1.uiLogger.json(jsonOutput);
     }

package/commands/testAccount/createConfig.d.ts

@@ -1,9 +1,7 @@
 import { CommonArgs, YargsCommandModule } from '../../types/Yargs';
-import { HubConfig } from '../../lib/prompts/createDeveloperTestAccountConfigPrompt';
 type CreateTestAccountConfigArgs = CommonArgs & {
     name?: string;
     description?: string;
-    tiers?: HubConfig[];
     path?: string;
 };
 declare const createTestAccountConfigCommand: YargsCommandModule<unknown, CreateTestAccountConfigArgs>;

package/commands/testAccount/createConfig.js

@@ -17,20 +17,20 @@ const validation_1 = require("../../lib/validation");
 const command = 'create-config';
 const describe = en_1.commands.testAccount.createConfig.describe;
 async function handler(args) {
-    const { derivedAccountId, name, tiers, description, path: configPath } = args;
+    const { derivedAccountId, name, description, path: configPath } = args;
     (0, usageTracking_1.trackCommandUsage)('test-account-create-config', {}, derivedAccountId);
     let accountConfigPath = configPath;
     const testAccountConfig = await (0, createDeveloperTestAccountConfigPrompt_1.createDeveloperTestAccountConfigPrompt)({
         name,
         description,
-        tiers,
     });
     if (!accountConfigPath) {
         const pathPromptResult = await (0, promptUtils_1.promptUser)({
             name: 'path',
             message: en_1.commands.testAccount.createConfig.pathPrompt,
             type: 'input',
-            default: 'test-account-config.json',
+            default: testAccountConfig.accountName.toLowerCase().replace(/\s+/g, '-') +
+                '.json',
             validate: path => {
                 if (!path) {
                     return en_1.commands.testAccount.createConfig.errors.pathError;
@@ -38,6 +38,9 @@ async function handler(args) {
                 else if (!path.endsWith('.json')) {
                     return en_1.commands.testAccount.createConfig.errors.pathFormatError;
                 }
+                else if ((0, validation_1.fileExists)(path)) {
+                    return en_1.commands.testAccount.createConfig.errors.pathExistsError;
+                }
                 return true;
             },
         });
@@ -70,17 +73,13 @@ function createTestAccountConfigBuilder(yargs) {
         type: 'string',
         description: en_1.commands.testAccount.createConfig.options.description,
     });
-    yargs.option('tiers', {
-        type: 'array',
-        description: en_1.commands.testAccount.createConfig.options.tiers,
-    });
     yargs.option('path', {
         type: 'string',
         description: en_1.commands.testAccount.createConfig.options.path,
     });
     yargs.example([
         [
-            '$0 create-config --name my-account',
+            '$0 test-account create-config --name my-account',
             en_1.commands.testAccount.createConfig.example('my-account'),
         ],
     ]);

package/commands/theme/__tests__/generate-selectors.test.d.ts

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

package/commands/theme/__tests__/generate-selectors.test.js

@@ -0,0 +1,33 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const generate_selectors_1 = __importDefault(require("../generate-selectors"));
+const positionalSpy = vi
+    .spyOn(yargs_1.default, 'positional')
+    .mockReturnValue(yargs_1.default);
+describe('commands/theme/generate-selectors', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(generate_selectors_1.default.command).toEqual('generate-selectors <path>');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(generate_selectors_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            generate_selectors_1.default.builder(yargs_1.default);
+            expect(positionalSpy).toHaveBeenCalledTimes(1);
+            expect(positionalSpy).toHaveBeenCalledWith('path', {
+                describe: expect.any(String),
+                required: true,
+                type: 'string',
+            });
+        });
+    });
+});

package/commands/theme/__tests__/marketplace-validate.test.d.ts

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

package/commands/theme/__tests__/marketplace-validate.test.js

@@ -0,0 +1,41 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const marketplace_validate_1 = __importDefault(require("../marketplace-validate"));
+vi.mock('../../../lib/commonOpts');
+const positionalSpy = vi
+    .spyOn(yargs_1.default, 'positional')
+    .mockReturnValue(yargs_1.default);
+describe('commands/theme/marketplace-validate', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(marketplace_validate_1.default.command).toEqual('marketplace-validate <path>');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(marketplace_validate_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            marketplace_validate_1.default.builder(yargs_1.default);
+            expect(positionalSpy).toHaveBeenCalledTimes(1);
+            expect(positionalSpy).toHaveBeenCalledWith('path', {
+                describe: expect.any(String),
+                required: true,
+                type: 'string',
+            });
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargs_1.default);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargs_1.default);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargs_1.default);
+        });
+    });
+});

package/commands/theme/__tests__/preview.test.d.ts

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

package/commands/theme/__tests__/preview.test.js

@@ -0,0 +1,65 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../../lib/commonOpts");
+const preview_1 = __importDefault(require("../preview"));
+vi.mock('../../../lib/commonOpts');
+const optionSpy = vi
+    .spyOn(yargs_1.default, 'option')
+    .mockReturnValue(yargs_1.default);
+describe('commands/theme/preview', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(preview_1.default.command).toEqual('preview [--src] [--dest]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(preview_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            preview_1.default.builder(yargs_1.default);
+            expect(optionSpy).toHaveBeenCalledWith('src', {
+                describe: expect.any(String),
+                type: 'string',
+                requiresArg: true,
+            });
+            expect(optionSpy).toHaveBeenCalledWith('dest', {
+                describe: expect.any(String),
+                type: 'string',
+                requiresArg: true,
+            });
+            expect(optionSpy).toHaveBeenCalledWith('notify', {
+                describe: expect.any(String),
+                alias: 'n',
+                type: 'string',
+                requiresArg: true,
+            });
+            expect(optionSpy).toHaveBeenCalledWith('no-ssl', {
+                describe: expect.any(String),
+                type: 'boolean',
+            });
+            expect(optionSpy).toHaveBeenCalledWith('port', {
+                describe: expect.any(String),
+                type: 'number',
+            });
+            expect(optionSpy).toHaveBeenCalledWith('resetSession', {
+                hidden: true,
+                type: 'boolean',
+            });
+            expect(optionSpy).toHaveBeenCalledWith('generateFieldsTypes', {
+                hidden: true,
+                type: 'boolean',
+            });
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargs_1.default);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargs_1.default);
+        });
+    });
+});

package/lang/en.d.ts

@@ -837,6 +837,7 @@ Global configuration replaces hubspot.config.yml, and you will be prompted to mi
             readonly success: (derivedTargets: string[]) => string;
             readonly errors: {
                 readonly needsNode20: "This feature requires node >=20";
+                readonly errorParsingJsonFIle: (filename: string, errorMessage: string) => string;
             };
             readonly spinners: {
                 readonly failedToConfigure: "Failed to configure the HubSpot mcp server.";
@@ -1833,7 +1834,6 @@ ${string}`;
             readonly options: {
                 readonly name: "The name of the test account";
                 readonly description: "The description of the test account";
-                readonly tiers: "The tiers of the test account";
                 readonly path: "The path to the test account config";
             };
             readonly example: (name: string) => string;
@@ -2847,9 +2847,22 @@ Run ${string} to upgrade to version ${string}`;
         readonly createDeveloperTestAccountConfigPrompt: {
             readonly namePrompt: "[--name] What is the name of the test account?";
             readonly descriptionPrompt: "[--description] What is the description of the test account?";
-            readonly tiersPrompt: "[--tiers] Which product tiers should the test account have?";
+            readonly useDefaultAccountLevelsPrompt: {
+                readonly message: "Would you like to create a default Test Account, or customize your own?";
+                readonly default: "Default (All Hubs, ENTERPRISE)";
+                readonly manual: "Customize my own";
+            };
+            readonly tiersPrompt: "Select an option per hub to customize tiers. If left blank, default is ENTERPRISE";
+            readonly hubTypes: {
+                readonly marketing: "Marketing";
+                readonly ops: "Ops";
+                readonly service: "Service";
+                readonly sales: "Sales";
+                readonly content: "Content";
+            };
             readonly errors: {
                 readonly tiersError: "Cannot have more than one tier per hub";
+                readonly nameRequired: "The name may not be blank. Please add a name for the Test Account.";
             };
         };
         readonly accountNamePrompt: {

package/lang/en.js

@@ -846,6 +846,7 @@ exports.commands = {
             success: (derivedTargets) => `You can now use the HubSpot CLI MCP Server in ${derivedTargets.join(', ')}.  ${chalk_1.default.bold('You may need to restart these tools to apply the changes')}.`,
             errors: {
                 needsNode20: `This feature requires node >=20`,
+                errorParsingJsonFIle: (filename, errorMessage) => `Unable to update ${chalk_1.default.bold(filename)} due to invalid JSON: ${errorMessage}`,
             },
             spinners: {
                 failedToConfigure: 'Failed to configure the HubSpot mcp server.',
@@ -1831,7 +1832,6 @@ exports.commands = {
             options: {
                 name: 'The name of the test account',
                 description: 'The description of the test account',
-                tiers: 'The tiers of the test account',
                 path: 'The path to the test account config',
             },
             example: (name) => `Create a test account config file with the name "${name}"`,
@@ -2842,9 +2842,22 @@ exports.lib = {
         createDeveloperTestAccountConfigPrompt: {
             namePrompt: '[--name] What is the name of the test account?',
             descriptionPrompt: '[--description] What is the description of the test account?',
-            tiersPrompt: '[--tiers] Which product tiers should the test account have?',
+            useDefaultAccountLevelsPrompt: {
+                message: 'Would you like to create a default Test Account, or customize your own?',
+                default: 'Default (All Hubs, ENTERPRISE)',
+                manual: 'Customize my own',
+            },
+            tiersPrompt: 'Select an option per hub to customize tiers. If left blank, default is ENTERPRISE',
+            hubTypes: {
+                marketing: 'Marketing',
+                ops: 'Ops',
+                service: 'Service',
+                sales: 'Sales',
+                content: 'Content',
+            },
             errors: {
                 tiersError: 'Cannot have more than one tier per hub',
+                nameRequired: 'The name may not be blank. Please add a name for the Test Account.',
             },
         },
         accountNamePrompt: {

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

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

package/lib/__tests__/accountTypes.test.js

@@ -0,0 +1,100 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const config_1 = require("@hubspot/local-dev-lib/constants/config");
+const accountTypes_1 = require("../accountTypes");
+const STANDARD_ACCOUNT = {
+    name: 'standard-account',
+    accountId: 123,
+    accountType: config_1.HUBSPOT_ACCOUNT_TYPES.STANDARD,
+    env: 'prod',
+};
+const DEVELOPMENT_SANDBOX_ACCOUNT = {
+    name: 'development-sandbox-account',
+    accountId: 456,
+    accountType: config_1.HUBSPOT_ACCOUNT_TYPES.DEVELOPMENT_SANDBOX,
+    env: 'prod',
+};
+const STANDARD_SANDBOX_ACCOUNT = {
+    name: 'sandbox-account',
+    accountId: 456,
+    accountType: config_1.HUBSPOT_ACCOUNT_TYPES.STANDARD_SANDBOX,
+    env: 'prod',
+};
+const DEVELOPER_TEST_ACCOUNT = {
+    name: 'developer-test-account',
+    accountId: 789,
+    accountType: config_1.HUBSPOT_ACCOUNT_TYPES.DEVELOPER_TEST,
+    env: 'prod',
+};
+const APP_DEVELOPER_ACCOUNT = {
+    name: 'app-developer-account',
+    accountId: 1011,
+    accountType: config_1.HUBSPOT_ACCOUNT_TYPES.APP_DEVELOPER,
+    env: 'prod',
+};
+describe('lib/accountTypes', () => {
+    describe('isStandardAccount()', () => {
+        it('should return true if the account is a standard account', () => {
+            const result = (0, accountTypes_1.isStandardAccount)(STANDARD_ACCOUNT);
+            expect(result).toBe(true);
+        });
+        it('should return false if the account is not a standard account', () => {
+            const result = (0, accountTypes_1.isStandardAccount)(DEVELOPER_TEST_ACCOUNT);
+            expect(result).toBe(false);
+        });
+    });
+    describe('isSandbox()', () => {
+        it('should return true if the account is a standard sandbox account', () => {
+            const result = (0, accountTypes_1.isSandbox)(STANDARD_SANDBOX_ACCOUNT);
+            expect(result).toBe(true);
+        });
+        it('should return true if the account is a development sandbox account', () => {
+            const result = (0, accountTypes_1.isSandbox)(DEVELOPMENT_SANDBOX_ACCOUNT);
+            expect(result).toBe(true);
+        });
+        it('should return false if the account is not a sandbox account', () => {
+            const result = (0, accountTypes_1.isSandbox)(STANDARD_ACCOUNT);
+            expect(result).toBe(false);
+        });
+    });
+    describe('isStandardSandbox()', () => {
+        it('should return true if the account is a standard sandbox account', () => {
+            const result = (0, accountTypes_1.isStandardSandbox)(STANDARD_SANDBOX_ACCOUNT);
+            expect(result).toBe(true);
+        });
+        it('should return false if the account is not a standard sandbox account', () => {
+            const result = (0, accountTypes_1.isStandardSandbox)(DEVELOPMENT_SANDBOX_ACCOUNT);
+            expect(result).toBe(false);
+        });
+    });
+    describe('isDevelopmentSandbox()', () => {
+        it('should return true if the account is a development sandbox account', () => {
+            const result = (0, accountTypes_1.isDevelopmentSandbox)(DEVELOPMENT_SANDBOX_ACCOUNT);
+            expect(result).toBe(true);
+        });
+        it('should return false if the account is not a development sandbox account', () => {
+            const result = (0, accountTypes_1.isDevelopmentSandbox)(STANDARD_ACCOUNT);
+            expect(result).toBe(false);
+        });
+    });
+    describe('isDeveloperTestAccount()', () => {
+        it('should return true if the account is a developer test account', () => {
+            const result = (0, accountTypes_1.isDeveloperTestAccount)(DEVELOPER_TEST_ACCOUNT);
+            expect(result).toBe(true);
+        });
+        it('should return false if the account is not a developer test account', () => {
+            const result = (0, accountTypes_1.isDeveloperTestAccount)(STANDARD_ACCOUNT);
+            expect(result).toBe(false);
+        });
+    });
+    describe('isAppDeveloperAccount()', () => {
+        it('should return true if the account is an app developer account', () => {
+            const result = (0, accountTypes_1.isAppDeveloperAccount)(APP_DEVELOPER_ACCOUNT);
+            expect(result).toBe(true);
+        });
+        it('should return false if the account is not an app developer account', () => {
+            const result = (0, accountTypes_1.isAppDeveloperAccount)(STANDARD_ACCOUNT);
+            expect(result).toBe(false);
+        });
+    });
+});

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

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