@hubspot/cli 8.0.10-experimental.7 → 8.0.11-experimental.0

package/mcp-server/tools/project/__tests__/AddFeatureToProjectTool.test.js

@@ -0,0 +1,157 @@
+import { AddFeatureToProjectTool, } from '../AddFeatureToProjectTool.js';
+import { runCommandInDir } from '../../../utils/command.js';
+import { addFlag } from '../../../utils/command.js';
+import { APP_AUTH_TYPES, APP_DISTRIBUTION_TYPES, } from '../../../../lib/constants.js';
+import { mcpFeedbackRequest } from '../../../utils/feedbackTracking.js';
+import { trackToolUsage } from '../../../utils/toolUsageTracking.js';
+vi.mock('@modelcontextprotocol/sdk/server/mcp.js');
+vi.mock('../../../utils/command');
+vi.mock('../../../../lib/constants');
+vi.mock('../../../utils/toolUsageTracking');
+vi.mock('../../../utils/feedbackTracking');
+const mockTrackToolUsage = trackToolUsage;
+const mockMcpFeedbackRequest = mcpFeedbackRequest;
+const mockRunCommandInDir = runCommandInDir;
+const mockAddFlag = addFlag;
+describe('mcp-server/tools/project/AddFeatureToProject', () => {
+    let mockMcpServer;
+    let tool;
+    let mockRegisteredTool;
+    beforeEach(() => {
+        // @ts-expect-error Not mocking the whole server
+        mockMcpServer = {
+            registerTool: vi.fn(),
+        };
+        mockRegisteredTool = {};
+        mockMcpServer.registerTool.mockReturnValue(mockRegisteredTool);
+        mockMcpFeedbackRequest.mockResolvedValue('');
+        mockTrackToolUsage.mockResolvedValue(undefined);
+        tool = new AddFeatureToProjectTool(mockMcpServer);
+        // Mock addFlag to simulate command building
+        mockAddFlag.mockImplementation((command, flag, value) => `${command} --${flag} "${value}"`);
+    });
+    describe('register', () => {
+        it('should register tool with correct parameters', () => {
+            const result = tool.register();
+            expect(mockMcpServer.registerTool).toHaveBeenCalledWith('add-feature-to-project', expect.objectContaining({
+                title: 'Add feature to HubSpot Project',
+                description: expect.stringContaining('Adds a feature to an existing HubSpot project'),
+                inputSchema: expect.any(Object),
+            }), expect.any(Function));
+            expect(result).toBe(mockRegisteredTool);
+        });
+    });
+    describe('handler', () => {
+        const baseInput = {
+            absoluteCurrentWorkingDirectory: '/test/dir',
+            absoluteProjectPath: '/test/project',
+            addApp: false,
+        };
+        it('should handle successful command execution without app', async () => {
+            mockRunCommandInDir.mockResolvedValue({
+                stdout: 'Feature added successfully',
+                stderr: '',
+            });
+            const result = await tool.handler(baseInput);
+            expect(mockAddFlag).toHaveBeenCalledWith('hs project add', 'features', []);
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.any(String));
+            expect(result).toEqual({
+                content: [
+                    { type: 'text', text: 'Feature added successfully' },
+                    { type: 'text', text: '' },
+                ],
+            });
+        });
+        it('should handle successful command execution with features', async () => {
+            mockRunCommandInDir.mockResolvedValue({
+                stdout: 'Features added successfully',
+                stderr: '',
+            });
+            const input = {
+                ...baseInput,
+                features: ['card', 'settings'],
+            };
+            await tool.handler(input);
+            expect(mockAddFlag).toHaveBeenCalledWith('hs project add', 'features', [
+                'card',
+                'settings',
+            ]);
+        });
+        it('should prompt for distribution and auth when adding app without both', async () => {
+            const input = {
+                ...baseInput,
+                addApp: true,
+            };
+            const result = await tool.handler(input);
+            expect(result.content).toEqual([
+                {
+                    type: 'text',
+                    text: expect.stringContaining('Ask the user how they would you like to distribute the app'),
+                },
+                {
+                    type: 'text',
+                    text: expect.stringContaining('Ask the user which auth type they would like to use'),
+                },
+            ]);
+        });
+        it('should prompt for auth when adding app without auth', async () => {
+            const input = {
+                ...baseInput,
+                addApp: true,
+                distribution: APP_DISTRIBUTION_TYPES.PRIVATE,
+            };
+            const result = await tool.handler(input);
+            expect(result.content).toEqual([
+                {
+                    type: 'text',
+                    text: expect.stringContaining('Ask the user which auth type they would like to use'),
+                },
+            ]);
+        });
+        it('should add distribution and auth flags when provided', async () => {
+            mockRunCommandInDir.mockResolvedValue({
+                stdout: 'App feature added',
+                stderr: '',
+            });
+            const input = {
+                ...baseInput,
+                addApp: true,
+                distribution: APP_DISTRIBUTION_TYPES.MARKETPLACE,
+                auth: APP_AUTH_TYPES.OAUTH,
+                features: ['webhooks'],
+            };
+            await tool.handler(input);
+            expect(mockAddFlag).toHaveBeenCalledWith('hs project add', 'distribution', APP_DISTRIBUTION_TYPES.MARKETPLACE);
+            expect(mockAddFlag).toHaveBeenCalledWith(expect.any(String), 'auth', APP_AUTH_TYPES.OAUTH);
+            expect(mockAddFlag).toHaveBeenCalledWith(expect.any(String), 'features', [
+                'webhooks',
+            ]);
+        });
+        it('should handle command execution error', async () => {
+            const error = new Error('Command failed');
+            mockRunCommandInDir.mockRejectedValue(error);
+            const result = await tool.handler(baseInput);
+            expect(result).toEqual({
+                content: [{ type: 'text', text: 'Command failed' }],
+            });
+        });
+        it('should handle non-Error rejection', async () => {
+            mockRunCommandInDir.mockRejectedValue('String error');
+            const result = await tool.handler(baseInput);
+            expect(result).toEqual({
+                content: [{ type: 'text', text: 'String error' }],
+            });
+        });
+        it('should handle stderr in results', async () => {
+            mockRunCommandInDir.mockResolvedValue({
+                stdout: 'Success with warnings',
+                stderr: 'Warning: something happened',
+            });
+            const result = await tool.handler(baseInput);
+            expect(result.content).toEqual([
+                { type: 'text', text: 'Success with warnings' },
+                { type: 'text', text: 'Warning: something happened' },
+            ]);
+        });
+    });
+});

package/mcp-server/tools/project/__tests__/CreateProjectTool.test.d.ts

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

package/mcp-server/tools/project/__tests__/CreateProjectTool.test.js

@@ -0,0 +1,131 @@
+import { CreateProjectTool, } from '../CreateProjectTool.js';
+import { runCommandInDir } from '../../../utils/command.js';
+import { addFlag } from '../../../utils/command.js';
+import { APP_DISTRIBUTION_TYPES, EMPTY_PROJECT, PROJECT_WITH_APP, } from '../../../../lib/constants.js';
+import { mcpFeedbackRequest } from '../../../utils/feedbackTracking.js';
+import { trackToolUsage } from '../../../utils/toolUsageTracking.js';
+vi.mock('@modelcontextprotocol/sdk/server/mcp.js');
+vi.mock('../../../utils/command');
+vi.mock('../../../../lib/constants');
+vi.mock('../../../../lib/projects/create/v2');
+vi.mock('../../../utils/toolUsageTracking');
+vi.mock('../../../utils/feedbackTracking');
+const mockTrackToolUsage = trackToolUsage;
+const mockMcpFeedbackRequest = mcpFeedbackRequest;
+const mockRunCommandInDir = runCommandInDir;
+const mockAddFlag = addFlag;
+describe('mcp-server/tools/project/CreateProjectTool', () => {
+    let mockMcpServer;
+    let tool;
+    let mockRegisteredTool;
+    beforeEach(() => {
+        // @ts-expect-error Not mocking the whole thing
+        mockMcpServer = {
+            registerTool: vi.fn(),
+        };
+        mockRegisteredTool = {};
+        mockMcpServer.registerTool.mockReturnValue(mockRegisteredTool);
+        mockMcpFeedbackRequest.mockResolvedValue('');
+        mockTrackToolUsage.mockResolvedValue(undefined);
+        tool = new CreateProjectTool(mockMcpServer);
+        // Mock addFlag to simulate command building
+        mockAddFlag.mockImplementation((command, flag, value) => `${command} --${flag} "${value}"`);
+    });
+    describe('register', () => {
+        it('should register tool with correct parameters', () => {
+            const result = tool.register();
+            expect(mockMcpServer.registerTool).toHaveBeenCalledWith('create-project', expect.objectContaining({
+                title: 'Create HubSpot Project',
+                description: expect.stringContaining('Creates a HubSpot project with the provided name'),
+                inputSchema: expect.any(Object),
+            }), expect.any(Function));
+            expect(result).toBe(mockRegisteredTool);
+        });
+    });
+    describe('handler', () => {
+        const baseInput = {
+            absoluteCurrentWorkingDirectory: '/test/workspace',
+            name: 'test-project',
+            destination: './test-dest',
+            projectBase: EMPTY_PROJECT,
+        };
+        it('should handle successful command execution', async () => {
+            mockRunCommandInDir.mockResolvedValue({
+                stdout: 'Project created successfully',
+                stderr: '',
+            });
+            const result = await tool.handler(baseInput);
+            expect(mockAddFlag).toHaveBeenCalledWith(expect.any(String), 'name', 'test-project');
+            expect(mockAddFlag).toHaveBeenCalledWith(expect.any(String), 'dest', './test-dest');
+            expect(mockAddFlag).toHaveBeenCalledWith(expect.any(String), 'project-base', EMPTY_PROJECT);
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/workspace', expect.any(String));
+            expect(result).toEqual({
+                content: [
+                    { type: 'text', text: 'Project created successfully' },
+                    { type: 'text', text: '' },
+                ],
+            });
+        });
+        it('should handle command execution error', async () => {
+            const error = new Error('Command failed');
+            mockRunCommandInDir.mockRejectedValue(error);
+            const result = await tool.handler(baseInput);
+            expect(result).toEqual({
+                content: [{ type: 'text', text: 'Command failed' }],
+            });
+        });
+        it('should prompt for distribution and auth when creating app project without both', async () => {
+            const input = {
+                ...baseInput,
+                projectBase: PROJECT_WITH_APP,
+            };
+            const result = await tool.handler(input);
+            expect(result.content).toEqual([
+                {
+                    type: 'text',
+                    text: expect.stringContaining('Ask the user how they would you like to distribute the app?'),
+                },
+                {
+                    type: 'text',
+                    text: expect.stringContaining('Ask the user which auth type they would like to use?'),
+                },
+            ]);
+        });
+        it('should prompt for auth when creating app project without auth', async () => {
+            const input = {
+                ...baseInput,
+                projectBase: PROJECT_WITH_APP,
+                distribution: APP_DISTRIBUTION_TYPES.PRIVATE,
+            };
+            const result = await tool.handler(input);
+            expect(result.content).toEqual([
+                {
+                    type: 'text',
+                    text: expect.stringContaining('Ask the user which auth type they would like to use?'),
+                },
+            ]);
+        });
+        it('should add features flag when features are provided', async () => {
+            mockRunCommandInDir.mockResolvedValue({
+                stdout: 'Project created',
+                stderr: '',
+            });
+            const input = {
+                ...baseInput,
+                features: ['card', 'settings'],
+            };
+            await tool.handler(input);
+            expect(mockAddFlag).toHaveBeenCalledWith(expect.any(String), 'features', [
+                'card',
+                'settings',
+            ]);
+        });
+        it('should handle non-Error rejection', async () => {
+            mockRunCommandInDir.mockRejectedValue('String error');
+            const result = await tool.handler(baseInput);
+            expect(result).toEqual({
+                content: [{ type: 'text', text: 'String error' }],
+            });
+        });
+    });
+});

package/mcp-server/tools/project/__tests__/CreateTestAccountTool.test.d.ts

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