@hubspot/cli 8.0.11-experimental.0 → 8.0.11-experimental.1

package/commands/filemanager/__tests__/upload.test.js

@@ -1,191 +0,0 @@
-import yargs from 'yargs';
-import fs from 'fs';
-import path from 'path';
-import * as fileManagerLib from '@hubspot/local-dev-lib/fileManager';
-import * as fileManagerApiLib from '@hubspot/local-dev-lib/api/fileManager';
-import * as pathLib from '@hubspot/local-dev-lib/path';
-import * as modulesLib from '@hubspot/local-dev-lib/cms/modules';
-import * as ignoreRulesLib from '@hubspot/local-dev-lib/ignoreRules';
-import * as configLib from '@hubspot/local-dev-lib/config';
-import { addGlobalOptions, addConfigOptions, addAccountOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
-import { uiLogger } from '../../../lib/ui/logger.js';
-import * as errorHandlers from '../../../lib/errorHandlers/index.js';
-import * as usageTrackingLib from '../../../lib/usageTracking.js';
-import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
-import fileManagerUploadCommand from '../upload.js';
-vi.mock('../../../lib/commonOpts');
-vi.mock('fs');
-vi.mock('@hubspot/local-dev-lib/fileManager');
-vi.mock('@hubspot/local-dev-lib/api/fileManager');
-vi.mock('@hubspot/local-dev-lib/path');
-vi.mock('@hubspot/local-dev-lib/cms/modules');
-vi.mock('@hubspot/local-dev-lib/ignoreRules');
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('../../../lib/errorHandlers/index.js');
-const uploadFolderSpy = vi.spyOn(fileManagerLib, 'uploadFolder');
-const uploadFileSpy = vi.spyOn(fileManagerApiLib, 'uploadFile');
-const statSyncSpy = vi.spyOn(fs, 'statSync');
-const validateSrcAndDestPathsSpy = vi.spyOn(modulesLib, 'validateSrcAndDestPaths');
-const shouldIgnoreFileSpy = vi.spyOn(ignoreRulesLib, 'shouldIgnoreFile');
-const getCwdSpy = vi.spyOn(pathLib, 'getCwd');
-const convertToUnixPathSpy = vi.spyOn(pathLib, 'convertToUnixPath');
-const trackCommandUsageSpy = vi.spyOn(usageTrackingLib, 'trackCommandUsage');
-const processExitSpy = vi.spyOn(process, 'exit');
-const logErrorSpy = vi.spyOn(errorHandlers, 'logError');
-const getConfigAccountIfExistsSpy = vi.spyOn(configLib, 'getConfigAccountIfExists');
-describe('commands/filemanager/upload', () => {
-    const yargsMock = yargs;
-    beforeEach(() => {
-        // @ts-expect-error Mock implementation
-        processExitSpy.mockImplementation(() => { });
-        getCwdSpy.mockReturnValue('/test/cwd');
-        convertToUnixPathSpy.mockImplementation(p => p.replace(/\\/g, '/'));
-        validateSrcAndDestPathsSpy.mockResolvedValue([]);
-        // Mock config to prevent reading actual config file in CI
-        getConfigAccountIfExistsSpy.mockReturnValue(undefined);
-    });
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(fileManagerUploadCommand.command).toEqual('upload <src> <dest>');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(fileManagerUploadCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            fileManagerUploadCommand.builder(yargsMock);
-            expect(addGlobalOptions).toHaveBeenCalledTimes(1);
-            expect(addGlobalOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
-        });
-    });
-    describe('handler', () => {
-        let args;
-        beforeEach(() => {
-            args = {
-                src: 'test.js',
-                dest: '/dest/test.js',
-                derivedAccountId: 123456,
-                d: false,
-                debug: false,
-            };
-        });
-        describe('validation', () => {
-            it('should error if src path is invalid', async () => {
-                statSyncSpy.mockImplementation(() => {
-                    throw new Error('File not found');
-                });
-                await fileManagerUploadCommand.handler(args);
-                expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('path to a file or folder'));
-            });
-            it('should error if src path is not a file or directory', async () => {
-                statSyncSpy.mockReturnValue({
-                    isFile: () => false,
-                    isDirectory: () => false,
-                });
-                await fileManagerUploadCommand.handler(args);
-                expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('path to a file or folder'));
-            });
-            it('should error if dest is not provided', async () => {
-                args.dest = '';
-                statSyncSpy.mockReturnValue({
-                    isFile: () => true,
-                    isDirectory: () => false,
-                });
-                await fileManagerUploadCommand.handler(args);
-                expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('destination'));
-            });
-            it('should exit if src/dest validation fails', async () => {
-                statSyncSpy.mockReturnValue({
-                    isFile: () => true,
-                    isDirectory: () => false,
-                });
-                validateSrcAndDestPathsSpy.mockResolvedValue([
-                    { id: '1', message: 'Invalid path' },
-                ]);
-                // @ts-expect-error Mock return value doesn't need full type implementation
-                uploadFileSpy.mockResolvedValue({});
-                await fileManagerUploadCommand.handler(args);
-                expect(uiLogger.error).toHaveBeenCalledWith('Invalid path');
-                expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-            });
-        });
-        describe('file upload', () => {
-            beforeEach(() => {
-                statSyncSpy.mockReturnValue({
-                    isFile: () => true,
-                    isDirectory: () => false,
-                });
-                shouldIgnoreFileSpy.mockReturnValue(false);
-            });
-            it('should track command usage for file uploads', async () => {
-                // @ts-expect-error Mock return value doesn't need full type implementation
-                uploadFileSpy.mockResolvedValue({});
-                await fileManagerUploadCommand.handler(args);
-                expect(trackCommandUsageSpy).toHaveBeenCalledWith('filemanager-upload', { type: 'file' }, 123456);
-            });
-            it('should upload a file successfully', async () => {
-                // @ts-expect-error Mock return value doesn't need full type implementation
-                uploadFileSpy.mockResolvedValue({});
-                await fileManagerUploadCommand.handler(args);
-                expect(uploadFileSpy).toHaveBeenCalledWith(123456, path.resolve('/test/cwd', 'test.js'), '/dest/test.js');
-                expect(uiLogger.success).toHaveBeenCalledWith(expect.stringContaining('test.js'));
-            });
-            it('should error if file is ignored', async () => {
-                shouldIgnoreFileSpy.mockReturnValue(true);
-                await fileManagerUploadCommand.handler(args);
-                expect(uploadFileSpy).not.toHaveBeenCalled();
-                expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('ignored'));
-            });
-            it('should handle upload errors', async () => {
-                vi.useFakeTimers();
-                const error = new Error('Upload failed');
-                uploadFileSpy.mockRejectedValue(error);
-                await fileManagerUploadCommand.handler(args);
-                await vi.runAllTimersAsync();
-                expect(uiLogger.error).toHaveBeenCalled();
-                expect(logErrorSpy).toHaveBeenCalledWith(error, expect.any(errorHandlers.ApiErrorContext));
-                vi.useRealTimers();
-            });
-        });
-        describe('folder upload', () => {
-            beforeEach(() => {
-                statSyncSpy.mockReturnValue({
-                    isFile: () => false,
-                    isDirectory: () => true,
-                });
-                // @ts-expect-error Mock return value doesn't need full type implementation
-                uploadFolderSpy.mockResolvedValue({});
-            });
-            it('should track command usage for folder uploads', async () => {
-                await fileManagerUploadCommand.handler(args);
-                expect(trackCommandUsageSpy).toHaveBeenCalledWith('filemanager-upload', { type: 'folder' }, 123456);
-            });
-            it('should upload a folder successfully', async () => {
-                await fileManagerUploadCommand.handler(args);
-                expect(uploadFolderSpy).toHaveBeenCalledWith(123456, path.resolve('/test/cwd', 'test.js'), '/dest/test.js');
-                expect(uiLogger.success).toHaveBeenCalledWith(expect.stringContaining('complete'));
-            });
-            it('should handle folder upload errors', async () => {
-                vi.useFakeTimers();
-                const error = new Error('Folder upload failed');
-                uploadFolderSpy.mockRejectedValue(error);
-                await fileManagerUploadCommand.handler(args);
-                await vi.runAllTimersAsync();
-                expect(uiLogger.error).toHaveBeenCalled();
-                expect(logErrorSpy).toHaveBeenCalledWith(error, {
-                    accountId: 123456,
-                });
-                vi.useRealTimers();
-            });
-        });
-    });
-});

package/commands/hubdb/__tests__/clear.test.d.ts

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

package/commands/hubdb/__tests__/clear.test.js

@@ -1,28 +0,0 @@
-import yargs from 'yargs';
-import { addAccountOptions, addConfigOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
-import hubdbClearCommand from '../clear.js';
-vi.mock('../../../lib/commonOpts');
-describe('commands/hubdb/clear', () => {
-    const yargsMock = yargs;
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(hubdbClearCommand.command).toEqual('clear [table-id]');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(hubdbClearCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            hubdbClearCommand.builder(yargsMock);
-            expect(addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
-        });
-    });
-});

package/commands/hubdb/__tests__/create.test.d.ts

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

package/commands/hubdb/__tests__/create.test.js

@@ -1,28 +0,0 @@
-import yargs from 'yargs';
-import { addAccountOptions, addConfigOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
-import hubdbCreateCommand from '../create.js';
-vi.mock('../../../lib/commonOpts');
-describe('commands/hubdb/create', () => {
-    const yargsMock = yargs;
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(hubdbCreateCommand.command).toEqual('create');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(hubdbCreateCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            hubdbCreateCommand.builder(yargsMock);
-            expect(addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
-        });
-    });
-});

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

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

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

@@ -1,28 +0,0 @@
-import yargs from 'yargs';
-import { addAccountOptions, addConfigOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
-import hubdbDeleteCommand from '../delete.js';
-vi.mock('../../../lib/commonOpts');
-describe('commands/hubdb/delete', () => {
-    const yargsMock = yargs;
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(hubdbDeleteCommand.command).toEqual('delete [table-id]');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(hubdbDeleteCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            hubdbDeleteCommand.builder(yargsMock);
-            expect(addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
-        });
-    });
-});

package/commands/hubdb/__tests__/fetch.test.d.ts

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

package/commands/hubdb/__tests__/fetch.test.js

@@ -1,28 +0,0 @@
-import yargs from 'yargs';
-import { addAccountOptions, addConfigOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
-import hubdbFetchCommand from '../fetch.js';
-vi.mock('../../../lib/commonOpts');
-describe('commands/hubdb/fetch', () => {
-    const yargsMock = yargs;
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(hubdbFetchCommand.command).toEqual('fetch [table-id] [dest]');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(hubdbFetchCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            hubdbFetchCommand.builder(yargsMock);
-            expect(addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
-        });
-    });
-});

package/commands/hubdb/__tests__/list.test.d.ts

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

package/commands/hubdb/__tests__/list.test.js

@@ -1,88 +0,0 @@
-import yargs from 'yargs';
-import { fetchTables } from '@hubspot/local-dev-lib/api/hubdb';
-import { addAccountOptions, addConfigOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
-import { uiLogger } from '../../../lib/ui/logger.js';
-import { mockHubSpotHttpResponse } from '../../../lib/testUtils.js';
-import { trackCommandUsage } from '../../../lib/usageTracking.js';
-import hubdbListCommand from '../list.js';
-vi.mock('../../../lib/commonOpts');
-vi.mock('@hubspot/local-dev-lib/api/hubdb');
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('../../../lib/errorHandlers');
-const mockExit = vi
-    .spyOn(process, 'exit')
-    .mockImplementation(() => undefined);
-describe('commands/hubdb/list', () => {
-    const yargsMock = yargs;
-    const mockedFetchTables = fetchTables;
-    const mockedLogger = uiLogger;
-    const mockedTrackCommandUsage = trackCommandUsage;
-    beforeEach(() => {
-        mockedFetchTables.mockReset();
-        mockedTrackCommandUsage.mockReset();
-        vi.mocked(mockedLogger.success).mockReset();
-        vi.mocked(mockedLogger.log).mockReset();
-        vi.mocked(mockedLogger.error).mockReset();
-    });
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(hubdbListCommand.command).toEqual(['list', 'ls']);
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(hubdbListCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            hubdbListCommand.builder(yargsMock);
-            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', () => {
-        const mockArgs = {
-            derivedAccountId: 123456789,
-            d: false,
-            debug: false,
-            _: [],
-            $0: 'hs',
-        };
-        const mockTables = {
-            results: [
-                {
-                    id: 1,
-                    label: 'Test Table',
-                    name: 'test_table',
-                    columnCount: 5,
-                    rowCount: 10,
-                },
-            ],
-            total: 1,
-        };
-        it('should fetch and display tables', async () => {
-            mockedFetchTables.mockReturnValue(mockHubSpotHttpResponse(mockTables));
-            await hubdbListCommand.handler(mockArgs);
-            expect(mockedFetchTables).toHaveBeenCalledTimes(1);
-            expect(mockedFetchTables).toHaveBeenCalledWith(mockArgs.derivedAccountId);
-            expect(mockedLogger.success).toHaveBeenCalled();
-            expect(mockedLogger.log).toHaveBeenCalled();
-            expect(mockExit).toHaveBeenCalledWith(0);
-        });
-        it('should handle errors appropriately', async () => {
-            const error = new Error('Test error');
-            mockedFetchTables.mockRejectedValue(error);
-            // this is a hack because the system exit is being mocked, so the return value from fetchTables is weird and can't be unpacked by the handler
-            try {
-                await hubdbListCommand.handler(mockArgs);
-            }
-            catch (e) { }
-            expect(mockExit).toHaveBeenCalledWith(1);
-        });
-    });
-});

package/commands/mcp/__tests__/setup.test.d.ts

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

package/commands/mcp/__tests__/setup.test.js

@@ -1,26 +0,0 @@
-import setupCommand from '../setup.js';
-vi.mock('../../../lib/commonOpts');
-describe('commands/mcp/setup', () => {
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(setupCommand.command).toEqual(['setup']);
-        });
-    });
-    describe('describe', () => {
-        it('should be undefined to keep the command hidden', () => {
-            expect(setupCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should be defined as a function', () => {
-            expect(setupCommand.builder).toBeDefined();
-            expect(typeof setupCommand.builder).toBe('function');
-        });
-    });
-    describe('handler', () => {
-        it('should be defined', () => {
-            expect(setupCommand.handler).toBeDefined();
-            expect(typeof setupCommand.handler).toBe('function');
-        });
-    });
-});

package/commands/mcp/__tests__/start.test.d.ts

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

package/commands/mcp/__tests__/start.test.js

@@ -1,144 +0,0 @@
-import { spawn } from 'node:child_process';
-import fs from 'fs';
-import { EventEmitter } from 'events';
-import * as configLib from '@hubspot/local-dev-lib/config';
-import { uiLogger } from '../../../lib/ui/logger.js';
-import * as errorHandlers from '../../../lib/errorHandlers/index.js';
-import * as usageTrackingLib from '../../../lib/usageTracking.js';
-import * as processLib from '../../../lib/process.js';
-import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
-// Create a mock execAsync function before importing the module
-const execAsyncMock = vi.fn();
-vi.mock('yargs');
-vi.mock('../../../lib/commonOpts');
-vi.mock('node:child_process');
-vi.mock('node:util', () => ({
-    promisify: vi.fn(() => execAsyncMock),
-}));
-vi.mock('fs');
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('../../../lib/errorHandlers/index.js');
-vi.mock('../../../lib/process.js');
-// Import after mocks are set up
-const startCommand = await import('../start.js').then(m => m.default);
-const spawnSpy = vi.mocked(spawn);
-const existsSyncSpy = vi.spyOn(fs, 'existsSync');
-const trackCommandUsageSpy = vi.spyOn(usageTrackingLib, 'trackCommandUsage');
-const handleExitSpy = vi.spyOn(processLib, 'handleExit');
-const logErrorSpy = vi.spyOn(errorHandlers, 'logError');
-const processExitSpy = vi.spyOn(process, 'exit');
-const getConfigAccountIfExistsSpy = vi.spyOn(configLib, 'getConfigAccountIfExists');
-class MockChildProcess extends EventEmitter {
-    kill = vi.fn();
-}
-describe('commands/mcp/start', () => {
-    let mockChildProcess;
-    beforeEach(() => {
-        mockChildProcess = new MockChildProcess();
-        // @ts-expect-error Mock implementation
-        spawnSpy.mockReturnValue(mockChildProcess);
-        existsSyncSpy.mockReturnValue(true);
-        // @ts-expect-error Mock implementation
-        processExitSpy.mockImplementation(() => { });
-        // Mock config to prevent reading actual config file in CI
-        getConfigAccountIfExistsSpy.mockReturnValue(undefined);
-        execAsyncMock.mockClear();
-    });
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(startCommand.command).toEqual('start');
-        });
-    });
-    describe('describe', () => {
-        it('should be undefined to keep the command hidden', () => {
-            expect(startCommand.describe).toBeUndefined();
-        });
-    });
-    describe('builder', () => {
-        it('should be defined as a function', () => {
-            expect(startCommand.builder).toBeDefined();
-            expect(typeof startCommand.builder).toBe('function');
-        });
-    });
-    describe('handler', () => {
-        let args;
-        beforeEach(() => {
-            args = {
-                aiAgent: 'test-agent',
-                derivedAccountId: 123456,
-                d: false,
-                debug: false,
-            };
-        });
-        it('should track command usage', async () => {
-            await startCommand.handler(args);
-            expect(trackCommandUsageSpy).toHaveBeenCalledWith('mcp-start', {}, 123456);
-        });
-        it('should check if server file exists', async () => {
-            await startCommand.handler(args);
-            expect(existsSyncSpy).toHaveBeenCalledWith(expect.stringContaining('server.js'));
-        });
-        it('should error if server file does not exist', async () => {
-            existsSyncSpy.mockReturnValue(false);
-            await startCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('not found'));
-            expect(spawnSpy).not.toHaveBeenCalled();
-        });
-        it('should spawn server process with correct arguments', async () => {
-            await startCommand.handler(args);
-            expect(spawnSpy).toHaveBeenCalledWith('node', [expect.stringContaining('server.js')], expect.objectContaining({
-                stdio: 'inherit',
-                env: expect.objectContaining({
-                    HUBSPOT_MCP_AI_AGENT: 'test-agent',
-                }),
-            }));
-        });
-        it('should use "unknown" as default AI agent', async () => {
-            delete args.aiAgent;
-            await startCommand.handler(args);
-            expect(spawnSpy).toHaveBeenCalledWith('node', expect.any(Array), expect.objectContaining({
-                env: expect.objectContaining({
-                    HUBSPOT_MCP_AI_AGENT: 'unknown',
-                }),
-            }));
-        });
-        it('should log debug messages when starting', async () => {
-            await startCommand.handler(args);
-            expect(uiLogger.debug).toHaveBeenCalledWith(expect.stringContaining('Starting'));
-            expect(uiLogger.debug).toHaveBeenCalledWith(expect.stringContaining('stop'));
-        });
-        it('should handle child process errors', async () => {
-            await startCommand.handler(args);
-            const error = new Error('Process error');
-            mockChildProcess.emit('error', error);
-            expect(logErrorSpy).toHaveBeenCalledWith(error);
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('Failed to start'));
-        });
-        it('should handle child process close', async () => {
-            await startCommand.handler(args);
-            mockChildProcess.emit('close');
-            expect(uiLogger.info).toHaveBeenCalledWith(expect.stringMatching(/stopped/i));
-        });
-        it('should register exit handler', async () => {
-            await startCommand.handler(args);
-            expect(handleExitSpy).toHaveBeenCalledWith(expect.any(Function));
-        });
-        it('should kill child process on exit', async () => {
-            await startCommand.handler(args);
-            const exitHandler = handleExitSpy.mock.calls[0][0];
-            exitHandler({ isSIGHUP: false });
-            expect(uiLogger.info).toHaveBeenCalledWith(expect.stringContaining('Shutting'));
-            expect(mockChildProcess.kill).toHaveBeenCalledWith('SIGTERM');
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.SUCCESS);
-        });
-        it('should handle exceptions during server start', async () => {
-            const error = new Error('Spawn failed');
-            spawnSpy.mockImplementation(() => {
-                throw error;
-            });
-            await startCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('Failed to start'));
-            expect(logErrorSpy).toHaveBeenCalledWith(error);
-        });
-    });
-});

package/commands/project/__tests__/add.test.d.ts

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