@hubspot/cli 8.0.10-experimental.5 → 8.0.10-experimental.6

package/lib/errorHandlers/__tests__/index.test.js

@@ -1,278 +0,0 @@
-import { logError, debugError, ApiErrorContext, isErrorWithMessageOrReason, getErrorMessage, } from '../index.js';
-import { uiLogger } from '../../ui/logger.js';
-import { isHubSpotHttpError, isValidationError, } from '@hubspot/local-dev-lib/errors/index';
-import { getConfig } from '@hubspot/local-dev-lib/config';
-import { shouldSuppressError } from '../suppressError.js';
-import { isProjectValidationError } from '../../errors/ProjectValidationError.js';
-import { lib } from '../../../lang/en.js';
-vi.mock('../../ui/logger.js');
-vi.mock('@hubspot/local-dev-lib/errors/index');
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('../suppressError.js');
-vi.mock('../../errors/ProjectValidationError.js');
-describe('lib/errorHandlers/index', () => {
-    const uiLoggerErrorMock = uiLogger.error;
-    const uiLoggerDebugMock = uiLogger.debug;
-    const isHubSpotHttpErrorMock = isHubSpotHttpError;
-    const isValidationErrorMock = isValidationError;
-    const getConfigMock = getConfig;
-    const shouldSuppressErrorMock = shouldSuppressError;
-    const isProjectValidationErrorMock = isProjectValidationError;
-    beforeEach(() => {
-        vi.clearAllMocks();
-        isHubSpotHttpErrorMock.mockReturnValue(false);
-        isValidationErrorMock.mockReturnValue(false);
-        shouldSuppressErrorMock.mockReturnValue(false);
-        isProjectValidationErrorMock.mockReturnValue(false);
-        getConfigMock.mockReturnValue({});
-    });
-    describe('logError', () => {
-        it('logs ProjectValidationError message and returns early', () => {
-            const error = {
-                message: 'Project validation failed',
-                name: 'ProjectValidationError',
-            };
-            isProjectValidationErrorMock.mockReturnValue(true);
-            logError(error);
-            expect(uiLoggerErrorMock).toHaveBeenCalledWith('Project validation failed');
-            expect(uiLoggerErrorMock).toHaveBeenCalledTimes(1);
-        });
-        it('returns early when error should be suppressed', () => {
-            const error = new Error('Suppressed error');
-            shouldSuppressErrorMock.mockReturnValue(true);
-            logError(error);
-            expect(shouldSuppressErrorMock).toHaveBeenCalled();
-            expect(uiLoggerErrorMock).not.toHaveBeenCalled();
-        });
-        it('logs validation errors for HubSpotHttpError with validation errors', () => {
-            const mockError = {
-                formattedValidationErrors: vi
-                    .fn()
-                    .mockReturnValue('Formatted validation errors'),
-                updateContext: vi.fn(),
-                context: {},
-            };
-            isHubSpotHttpErrorMock.mockReturnValue(true);
-            isValidationErrorMock.mockReturnValue(true);
-            logError(mockError);
-            expect(mockError.formattedValidationErrors).toHaveBeenCalled();
-            expect(uiLoggerErrorMock).toHaveBeenCalledWith('Formatted validation errors');
-        });
-        it('logs error message for errors with message property', () => {
-            const error = new Error('Something went wrong');
-            logError(error);
-            expect(uiLoggerErrorMock).toHaveBeenCalledWith('Something went wrong');
-        });
-        it('logs error with both message and reason', () => {
-            const error = { message: 'Error message', reason: 'Error reason' };
-            logError(error);
-            expect(uiLoggerErrorMock).toHaveBeenCalledWith('Error message Error reason');
-        });
-        it('logs unknown error message for errors without message or reason', () => {
-            const error = { foo: 'bar' };
-            logError(error);
-            expect(uiLoggerErrorMock).toHaveBeenCalledWith(lib.errorHandlers.index.unknownErrorOccurred);
-        });
-        it('calls updateContext on HubSpotHttpError', () => {
-            const mockError = {
-                updateContext: vi.fn(),
-                context: {},
-                message: 'test',
-            };
-            isHubSpotHttpErrorMock.mockReturnValue(true);
-            logError(mockError);
-            expect(mockError.updateContext).toHaveBeenCalled();
-        });
-        describe('timeout error handling', () => {
-            it('shows config timeout message for direct ETIMEDOUT error matching default timeout', () => {
-                const mockError = {
-                    code: 'ETIMEDOUT',
-                    timeout: 15000,
-                    updateContext: vi.fn(),
-                    context: {},
-                    message: 'Timeout',
-                };
-                isHubSpotHttpErrorMock.mockReturnValue(true);
-                getConfigMock.mockReturnValue({ httpTimeout: 15000 });
-                logError(mockError);
-                expect(uiLoggerErrorMock).toHaveBeenCalledTimes(2);
-                expect(uiLoggerErrorMock).toHaveBeenNthCalledWith(1, 'Timeout');
-                expect(uiLoggerErrorMock).toHaveBeenNthCalledWith(2, lib.errorHandlers.index.configTimeoutErrorOccurred(15000, 'hs config set'));
-            });
-            it('shows generic timeout message for direct ETIMEDOUT error with custom timeout', () => {
-                const mockError = {
-                    code: 'ETIMEDOUT',
-                    timeout: 30000,
-                    updateContext: vi.fn(),
-                    context: {},
-                    message: 'Timeout',
-                };
-                isHubSpotHttpErrorMock.mockReturnValue(true);
-                getConfigMock.mockReturnValue({ httpTimeout: 15000 });
-                logError(mockError);
-                expect(uiLoggerErrorMock).toHaveBeenCalledTimes(2);
-                expect(uiLoggerErrorMock).toHaveBeenNthCalledWith(2, lib.errorHandlers.index.genericTimeoutErrorOccurred);
-            });
-            it('detects timeout error wrapped in error.cause', () => {
-                const causeError = {
-                    code: 'ETIMEDOUT',
-                    timeout: 15000,
-                    name: 'HubSpotHttpError',
-                };
-                const wrapperError = new Error('Assets unavailable');
-                Object.defineProperty(wrapperError, 'cause', {
-                    value: causeError,
-                    writable: true,
-                });
-                isHubSpotHttpErrorMock.mockImplementation(err => {
-                    return err === causeError;
-                });
-                getConfigMock.mockReturnValue({ httpTimeout: 15000 });
-                logError(wrapperError);
-                expect(uiLoggerErrorMock).toHaveBeenCalledWith('Assets unavailable');
-                expect(uiLoggerErrorMock).toHaveBeenCalledWith(lib.errorHandlers.index.configTimeoutErrorOccurred(15000, 'hs config set'));
-            });
-            it('shows generic timeout message for wrapped timeout with different timeout value', () => {
-                const causeError = {
-                    code: 'ETIMEDOUT',
-                    timeout: 60000,
-                    name: 'HubSpotHttpError',
-                };
-                const wrapperError = new Error('Assets unavailable');
-                Object.defineProperty(wrapperError, 'cause', {
-                    value: causeError,
-                    writable: true,
-                });
-                isHubSpotHttpErrorMock.mockImplementation(err => {
-                    return err === causeError;
-                });
-                getConfigMock.mockReturnValue({ httpTimeout: 15000 });
-                logError(wrapperError);
-                expect(uiLoggerErrorMock).toHaveBeenCalledTimes(2);
-                expect(uiLoggerErrorMock).toHaveBeenNthCalledWith(2, lib.errorHandlers.index.genericTimeoutErrorOccurred);
-            });
-            it('does not show timeout message for non-timeout errors', () => {
-                const error = new Error('Regular error');
-                logError(error);
-                expect(uiLoggerErrorMock).toHaveBeenCalledTimes(1);
-                expect(uiLoggerErrorMock).toHaveBeenCalledWith('Regular error');
-            });
-        });
-    });
-    describe('debugError', () => {
-        it('logs HubSpotHttpError using toString', () => {
-            const mockError = {
-                toString: vi.fn().mockReturnValue('HubSpotHttpError details'),
-            };
-            isHubSpotHttpErrorMock.mockReturnValue(true);
-            debugError(mockError);
-            expect(uiLoggerDebugMock).toHaveBeenCalledWith('HubSpotHttpError details');
-        });
-        it('logs regular error using lib.errorHandlers.index.errorOccurred', () => {
-            const error = new Error('Regular error');
-            debugError(error);
-            expect(uiLoggerDebugMock).toHaveBeenCalledWith(lib.errorHandlers.index.errorOccurred('Error: Regular error'));
-        });
-        it('logs error.cause when it is a HubSpotHttpError', () => {
-            const causeError = {
-                toString: vi.fn().mockReturnValue('Cause error details'),
-            };
-            const error = new Error('Wrapper error');
-            Object.defineProperty(error, 'cause', {
-                value: causeError,
-                writable: true,
-            });
-            isHubSpotHttpErrorMock.mockImplementation(err => {
-                return err === causeError;
-            });
-            debugError(error);
-            expect(causeError.toString).toHaveBeenCalled();
-            expect(uiLoggerDebugMock).toHaveBeenCalledWith('Cause error details');
-        });
-        it('logs error.cause using lib.errorHandlers.index.errorCause when not a HubSpotHttpError', () => {
-            const causeError = { customField: 'value' };
-            const error = new Error('Wrapper error');
-            Object.defineProperty(error, 'cause', {
-                value: causeError,
-                writable: true,
-            });
-            debugError(error);
-            expect(uiLoggerDebugMock).toHaveBeenCalledWith(expect.stringMatching(/^Cause:/));
-        });
-        it('logs context using lib.errorHandlers.index.errorContext when provided', () => {
-            const error = new Error('Error');
-            const context = new ApiErrorContext({
-                accountId: 123,
-                request: '/api/test',
-            });
-            debugError(error, context);
-            expect(uiLoggerDebugMock).toHaveBeenCalledWith(expect.stringMatching(/^Context:/));
-        });
-    });
-    describe('ApiErrorContext', () => {
-        it('creates context with all properties', () => {
-            const context = new ApiErrorContext({
-                accountId: 123,
-                request: '/api/test',
-                payload: '{"data": "value"}',
-                projectName: 'my-project',
-            });
-            expect(context.accountId).toBe(123);
-            expect(context.request).toBe('/api/test');
-            expect(context.payload).toBe('{"data": "value"}');
-            expect(context.projectName).toBe('my-project');
-        });
-        it('creates context with default values', () => {
-            const context = new ApiErrorContext();
-            expect(context.accountId).toBeUndefined();
-            expect(context.request).toBe('');
-            expect(context.payload).toBe('');
-            expect(context.projectName).toBe('');
-        });
-        it('creates context with partial properties', () => {
-            const context = new ApiErrorContext({
-                accountId: 456,
-            });
-            expect(context.accountId).toBe(456);
-            expect(context.request).toBe('');
-            expect(context.payload).toBe('');
-            expect(context.projectName).toBe('');
-        });
-    });
-    describe('isErrorWithMessageOrReason', () => {
-        it('returns true for object with message property', () => {
-            expect(isErrorWithMessageOrReason({ message: 'test' })).toBe(true);
-        });
-        it('returns true for object with reason property', () => {
-            expect(isErrorWithMessageOrReason({ reason: 'test' })).toBe(true);
-        });
-        it('returns true for object with both message and reason', () => {
-            expect(isErrorWithMessageOrReason({ message: 'msg', reason: 'rsn' })).toBe(true);
-        });
-        it('returns true for Error instances', () => {
-            expect(isErrorWithMessageOrReason(new Error('test'))).toBe(true);
-        });
-        it('returns false for null', () => {
-            expect(isErrorWithMessageOrReason(null)).toBe(false);
-        });
-        it('returns false for undefined', () => {
-            expect(isErrorWithMessageOrReason(undefined)).toBe(false);
-        });
-        it('returns false for primitive values', () => {
-            expect(isErrorWithMessageOrReason('string')).toBe(false);
-            expect(isErrorWithMessageOrReason(123)).toBe(false);
-            expect(isErrorWithMessageOrReason(true)).toBe(false);
-        });
-        it('returns false for object without message or reason', () => {
-            expect(isErrorWithMessageOrReason({ foo: 'bar' })).toBe(false);
-        });
-        it('returns false for empty object', () => {
-            expect(isErrorWithMessageOrReason({})).toBe(false);
-        });
-    });
-    describe('getErrorMessage', () => {
-        it('returns message from Error instance', () => {
-            expect(getErrorMessage(new Error('Error message'))).toBe('Error message');
-        });
-    });
-});

package/lib/errorHandlers/index.d.ts

@@ -1,19 +0,0 @@
-export declare function logError(error: unknown, context?: ApiErrorContext): void;
-export declare function debugError(error: unknown, context?: ApiErrorContext): void;
-export declare class ApiErrorContext {
-    accountId?: number;
-    request?: string;
-    payload?: string;
-    projectName?: string;
-    constructor(props?: {
-        accountId?: number;
-        request?: string;
-        payload?: string;
-        projectName?: string;
-    });
-}
-export declare function isErrorWithMessageOrReason(error: unknown): error is {
-    message?: string;
-    reason?: string;
-};
-export declare function getErrorMessage(error: unknown): string;

package/lib/errorHandlers/index.js

@@ -1,99 +0,0 @@
-import { uiLogger } from '../ui/logger.js';
-import { isHubSpotHttpError, isValidationError, } from '@hubspot/local-dev-lib/errors/index';
-import { getConfig } from '@hubspot/local-dev-lib/config';
-import { shouldSuppressError } from './suppressError.js';
-import { lib } from '../../lang/en.js';
-import util from 'util';
-import { isProjectValidationError } from '../errors/ProjectValidationError.js';
-export function logError(error, context) {
-    debugError(error, context);
-    if (isProjectValidationError(error)) {
-        uiLogger.error(error.message);
-        return;
-    }
-    if (shouldSuppressError(error, context)) {
-        return;
-    }
-    if (isHubSpotHttpError(error)) {
-        if (shouldSuppressError(error, error.context)) {
-            return;
-        }
-        error.updateContext(context || {}, lib.errorHandlers.index.additionalDebugContext);
-    }
-    if (isHubSpotHttpError(error) && isValidationError(error)) {
-        uiLogger.error(error.formattedValidationErrors());
-    }
-    else if (isErrorWithMessageOrReason(error)) {
-        const message = [];
-        [error.message, error.reason].forEach(msg => {
-            if (msg) {
-                message.push(msg);
-            }
-        });
-        uiLogger.error(message.join(' '));
-    }
-    else {
-        // Unknown errors
-        uiLogger.error(lib.errorHandlers.index.unknownErrorOccurred);
-    }
-    let timeoutError = null;
-    if (isHubSpotHttpError(error) && error.code === 'ETIMEDOUT') {
-        timeoutError = error;
-    }
-    else if (error instanceof Error &&
-        isHubSpotHttpError(error.cause) &&
-        error.cause.code === 'ETIMEDOUT') {
-        timeoutError = error.cause;
-    }
-    if (timeoutError) {
-        const config = getConfig();
-        const defaultTimeout = config?.httpTimeout;
-        // Timeout was caused by the default timeout
-        if (timeoutError.timeout && defaultTimeout === timeoutError.timeout) {
-            uiLogger.error(lib.errorHandlers.index.configTimeoutErrorOccurred(timeoutError.timeout, 'hs config set'));
-        }
-        // Timeout was caused by a custom timeout set by the CLI or LDL
-        else {
-            uiLogger.error(lib.errorHandlers.index.genericTimeoutErrorOccurred);
-        }
-    }
-}
-export function debugError(error, context) {
-    if (isHubSpotHttpError(error)) {
-        uiLogger.debug(error.toString());
-    }
-    else {
-        uiLogger.debug(lib.errorHandlers.index.errorOccurred(String(error)));
-        if (error instanceof Error && error.cause) {
-            if (isHubSpotHttpError(error.cause)) {
-                uiLogger.debug(error.cause.toString());
-            }
-            else {
-                uiLogger.debug(lib.errorHandlers.index.errorCause(util.inspect(error.cause, false, null, true)));
-            }
-        }
-    }
-    if (context) {
-        uiLogger.debug(lib.errorHandlers.index.errorContext(util.inspect(context, false, null, true)));
-    }
-}
-export class ApiErrorContext {
-    accountId;
-    request;
-    payload;
-    projectName;
-    constructor(props = {}) {
-        this.accountId = props.accountId;
-        this.request = props.request || '';
-        this.payload = props.payload || '';
-        this.projectName = props.projectName || '';
-    }
-}
-export function isErrorWithMessageOrReason(error) {
-    return (typeof error === 'object' &&
-        error !== null &&
-        ('message' in error || 'reason' in error));
-}
-export function getErrorMessage(error) {
-    return error instanceof Error ? error.message : `${error}`;
-}

package/lib/errorHandlers/suppressError.d.ts

@@ -1,2 +0,0 @@
-import { ApiErrorContext } from './index.js';
-export declare function shouldSuppressError(err: unknown, context?: ApiErrorContext): boolean;

package/lib/errorHandlers/suppressError.js

@@ -1,53 +0,0 @@
-import { isSpecifiedError, isMissingScopeError, } from '@hubspot/local-dev-lib/errors/index';
-import { uiLogger } from '../ui/logger.js';
-import { PLATFORM_VERSION_ERROR_TYPES } from '../constants.js';
-import { lib } from '../../lang/en.js';
-import { uiAccountDescription, uiLine, uiLink } from '../ui/index.js';
-function createPlatformVersionError(err, subCategory) {
-    let translationKey = 'unspecifiedPlatformVersion';
-    let platformVersion = 'unspecified platformVersion';
-    const errorContext = err?.data?.context;
-    if (subCategory === PLATFORM_VERSION_ERROR_TYPES.PLATFORM_VERSION_RETIRED) {
-        platformVersion = errorContext?.RETIRED_PLATFORM_VERSION ?? platformVersion;
-        translationKey = 'platformVersionRetired';
-    }
-    else if (subCategory ===
-        PLATFORM_VERSION_ERROR_TYPES.PLATFORM_VERSION_SPECIFIED_DOES_NOT_EXIST) {
-        platformVersion = errorContext?.PLATFORM_VERSION ?? platformVersion;
-        translationKey = 'nonExistentPlatformVersion';
-    }
-    uiLine();
-    uiLogger.error(lib.errorHandlers.suppressErrors.platformVersionErrors.header);
-    const errorMessage = lib.errorHandlers.suppressErrors.platformVersionErrors[translationKey];
-    uiLogger.log(typeof errorMessage === 'function'
-        ? errorMessage(platformVersion)
-        : errorMessage);
-    uiLogger.log(lib.errorHandlers.suppressErrors.platformVersionErrors.updateProject);
-    uiLogger.log(lib.errorHandlers.suppressErrors.platformVersionErrors.betaLink(uiLink(lib.errorHandlers.suppressErrors.platformVersionErrors.docsLink, 'https://developers.hubspot.com/docs/developer-tooling/platform/versioning')));
-    uiLine();
-}
-export function shouldSuppressError(err, context) {
-    if (isMissingScopeError(err)) {
-        uiLogger.error(lib.errorHandlers.suppressErrors.missingScopeError(context?.request || 'request', context?.accountId ? uiAccountDescription(context.accountId) : ''));
-        return true;
-    }
-    if (isSpecifiedError(err, {
-        subCategory: PLATFORM_VERSION_ERROR_TYPES.PLATFORM_VERSION_NOT_SPECIFIED,
-    })) {
-        createPlatformVersionError(err, PLATFORM_VERSION_ERROR_TYPES.PLATFORM_VERSION_NOT_SPECIFIED);
-        return true;
-    }
-    if (isSpecifiedError(err, {
-        subCategory: PLATFORM_VERSION_ERROR_TYPES.PLATFORM_VERSION_RETIRED,
-    })) {
-        createPlatformVersionError(err, PLATFORM_VERSION_ERROR_TYPES.PLATFORM_VERSION_RETIRED);
-        return true;
-    }
-    if (isSpecifiedError(err, {
-        subCategory: PLATFORM_VERSION_ERROR_TYPES.PLATFORM_VERSION_SPECIFIED_DOES_NOT_EXIST,
-    })) {
-        createPlatformVersionError(err, PLATFORM_VERSION_ERROR_TYPES.PLATFORM_VERSION_SPECIFIED_DOES_NOT_EXIST);
-        return true;
-    }
-    return false;
-}

package/lib/errors/ProjectErrors.d.ts

@@ -1,15 +0,0 @@
-export declare class ProjectNestingError extends Error {
-    constructor(message: string);
-}
-export declare class ProjectConfigNotFoundError extends Error {
-    constructor(message: string);
-}
-export declare class ProjectValidationError extends Error {
-    constructor(message: string);
-}
-export declare class ProjectUploadError extends Error {
-    constructor(message: string, cause?: unknown);
-}
-export declare class ProjectBuildDeployError extends Error {
-    constructor(message: string);
-}

package/lib/errors/ProjectErrors.js

@@ -1,30 +0,0 @@
-export class ProjectNestingError extends Error {
-    constructor(message) {
-        super(message);
-        this.name = 'ProjectNestingError';
-    }
-}
-export class ProjectConfigNotFoundError extends Error {
-    constructor(message) {
-        super(message);
-        this.name = 'ProjectConfigNotFoundError';
-    }
-}
-export class ProjectValidationError extends Error {
-    constructor(message) {
-        super(message);
-        this.name = 'ProjectValidationError';
-    }
-}
-export class ProjectUploadError extends Error {
-    constructor(message, cause) {
-        super(message, { cause });
-        this.name = 'ProjectUploadError';
-    }
-}
-export class ProjectBuildDeployError extends Error {
-    constructor(message) {
-        super(message);
-        this.name = 'ProjectBuildDeployError';
-    }
-}

package/lib/errors/ProjectValidationError.d.ts

@@ -1,4 +0,0 @@
-export default class ProjectValidationError extends Error {
-    constructor(message: string, options?: ErrorOptions);
-}
-export declare function isProjectValidationError(err: unknown): err is ProjectValidationError;

package/lib/errors/ProjectValidationError.js

@@ -1,9 +0,0 @@
-export default class ProjectValidationError extends Error {
-    constructor(message, options) {
-        super(message, options);
-        this.name = 'ProjectValidationError';
-    }
-}
-export function isProjectValidationError(err) {
-    return err instanceof ProjectValidationError;
-}

package/lib/errors/PromptExitError.d.ts

@@ -1,4 +0,0 @@
-export declare class PromptExitError extends Error {
-    exitCode: number;
-    constructor(message: string, exitCode: number);
-}

package/lib/errors/PromptExitError.js

@@ -1,8 +0,0 @@
-export class PromptExitError extends Error {
-    exitCode;
-    constructor(message, exitCode) {
-        super(message);
-        this.name = 'PromptExitError';
-        this.exitCode = exitCode;
-    }
-}

package/lib/filesystem.d.ts

@@ -1,2 +0,0 @@
-export declare function resolveLocalPath(filepath?: string): string;
-export declare function isPathFolder(path: string): boolean;

package/lib/filesystem.js

@@ -1,19 +0,0 @@
-import path from 'path';
-import { getCwd } from '@hubspot/local-dev-lib/path';
-import { FOLDER_DOT_EXTENSIONS } from '@hubspot/local-dev-lib/constants/extensions';
-export function resolveLocalPath(filepath) {
-    return filepath && typeof filepath === 'string'
-        ? path.resolve(getCwd(), filepath)
-        : // Use CWD if optional filepath is not passed.
-            getCwd();
-}
-export function isPathFolder(path) {
-    const splitPath = path.split('/');
-    const fileOrFolderName = splitPath[splitPath.length - 1];
-    const splitName = fileOrFolderName.split('.');
-    if (splitName.length > 1 &&
-        FOLDER_DOT_EXTENSIONS.indexOf(splitName[1]) === -1) {
-        return false;
-    }
-    return true;
-}

package/lib/generateSelectors.d.ts

@@ -1,20 +0,0 @@
-export declare function getMaxFieldsDepth(): number;
-export declare function findFieldsJsonPath(basePath: string): string | null;
-export declare function combineThemeCss(basePath: string, cssString?: string): string | null;
-type Field = {
-    name: string;
-    children: Field[];
-    selectors?: string[];
-    inherited_value?: {
-        property_value_paths?: {
-            [key: string]: string;
-        };
-    };
-};
-export declare function setPreviewSelectors(fields: Field[], fieldPath: string[], selectors: string[], depth?: number): Field[];
-export declare function generateInheritedSelectors(fields: Field[]): Field[];
-type SelectorsMap = {
-    [key: string]: string[] | undefined;
-};
-export declare function generateSelectorsMap(fields: Field[], fieldKey?: string[]): SelectorsMap;
-export {};