@hubspot/cli 7.9.0 → 7.10.0-beta.0

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

@@ -1,12 +1,18 @@
 import { UploadProjectTools } from '../UploadProjectTools.js';
+import { getAllHsProfiles } from '@hubspot/project-parsing-lib';
+import { getProjectConfig } from '../../../../lib/projects/config.js';
 import { runCommandInDir } from '../../../utils/project.js';
 import { mcpFeedbackRequest } from '../../../utils/feedbackTracking.js';
 vi.mock('@modelcontextprotocol/sdk/server/mcp.js');
+vi.mock('@hubspot/project-parsing-lib');
+vi.mock('../../../../lib/projects/config.js');
 vi.mock('../../../utils/project');
 vi.mock('../../../utils/toolUsageTracking');
 vi.mock('../../../utils/feedbackTracking');
 const mockMcpFeedbackRequest = mcpFeedbackRequest;
 const mockRunCommandInDir = runCommandInDir;
+const mockGetProjectConfig = getProjectConfig;
+const mockGetAllHsProfiles = getAllHsProfiles;
 describe('mcp-server/tools/project/UploadProjectTools', () => {
     let mockMcpServer;
     let tool;
@@ -20,6 +26,15 @@ describe('mcp-server/tools/project/UploadProjectTools', () => {
         mockRegisteredTool = {};
         mockMcpServer.registerTool.mockReturnValue(mockRegisteredTool);
         mockMcpFeedbackRequest.mockResolvedValue('');
+        mockGetProjectConfig.mockResolvedValue({
+            projectConfig: {
+                srcDir: 'src',
+                name: 'test-project',
+                platformVersion: '2025.2',
+            },
+            projectDir: '/test/project',
+        });
+        mockGetAllHsProfiles.mockResolvedValue([]);
         tool = new UploadProjectTools(mockMcpServer);
     });
     describe('register', () => {
@@ -37,6 +52,7 @@ describe('mcp-server/tools/project/UploadProjectTools', () => {
         const input = {
             absoluteCurrentWorkingDirectory: '/test/dir',
             absoluteProjectPath: '/test/project',
+            uploadMessage: 'Test upload message',
         };
         it('should upload project successfully', async () => {
             mockRunCommandInDir.mockResolvedValue({
@@ -44,7 +60,9 @@ describe('mcp-server/tools/project/UploadProjectTools', () => {
                 stderr: '',
             });
             const result = await tool.handler(input);
-            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', 'hs project upload --force-create');
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('hs project upload'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('--force-create'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('--message "Test upload message"'));
             expect(result).toEqual({
                 content: [
                     { type: 'text', text: 'Project uploaded successfully' },
@@ -68,13 +86,44 @@ describe('mcp-server/tools/project/UploadProjectTools', () => {
             mockRunCommandInDir.mockRejectedValue(error);
             await expect(tool.handler(input)).rejects.toThrow('Upload failed');
         });
-        it('should use force-create flag', async () => {
+        it('should use force-create and message flags', async () => {
             mockRunCommandInDir.mockResolvedValue({
                 stdout: 'Project created and uploaded',
                 stderr: '',
             });
             await tool.handler(input);
-            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', 'hs project upload --force-create');
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('hs project upload'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('--force-create'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('--message "Test upload message"'));
+        });
+        it('should use profiles', async () => {
+            mockRunCommandInDir.mockResolvedValue({
+                stdout: 'Project created and uploaded',
+                stderr: '',
+            });
+            await tool.handler({
+                ...input,
+                profile: 'dev',
+            });
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('hs project upload'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('--force-create'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('--message "Test upload message"'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', expect.stringContaining('--profile "dev"'));
+        });
+        it('should prompt for profile if not specified and the project requires them', async () => {
+            mockGetAllHsProfiles.mockResolvedValue(['prod', 'dev']);
+            mockRunCommandInDir.mockResolvedValue({
+                stdout: 'Project created and uploaded',
+                stderr: '',
+            });
+            const result = await tool.handler(input);
+            expect(mockRunCommandInDir).not.toHaveBeenCalled();
+            expect(result.content).toEqual([
+                {
+                    type: 'text',
+                    text: 'Ask the user which profile they would like to use for the upload.',
+                },
+            ]);
         });
         it('should handle empty stdout and stderr', async () => {
             mockRunCommandInDir.mockResolvedValue({
@@ -95,9 +144,12 @@ describe('mcp-server/tools/project/UploadProjectTools', () => {
             const differentInput = {
                 absoluteCurrentWorkingDirectory: '/test/dir',
                 absoluteProjectPath: '/different/path/to/project',
+                uploadMessage: 'Different test upload message',
             };
             await tool.handler(differentInput);
-            expect(mockRunCommandInDir).toHaveBeenCalledWith('/different/path/to/project', 'hs project upload --force-create');
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/different/path/to/project', expect.stringContaining('hs project upload'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/different/path/to/project', expect.stringContaining('--force-create'));
+            expect(mockRunCommandInDir).toHaveBeenCalledWith('/different/path/to/project', expect.stringContaining('--message "Different test upload message"'));
         });
         it('should handle very long output', async () => {
             const longOutput = 'A'.repeat(10000);

package/package.json

@@ -1,12 +1,12 @@
 {
     "name": "@hubspot/cli",
-    "version": "7.9.0",
+    "version": "7.10.0-beta.0",
     "description": "The official CLI for developing on HubSpot",
     "license": "Apache-2.0",
     "repository": "https://github.com/HubSpot/hubspot-cli",
     "type": "module",
     "dependencies": {
-        "@hubspot/local-dev-lib": "3.21.1",
+        "@hubspot/local-dev-lib": "3.21.2",
         "@hubspot/project-parsing-lib": "0.10.1",
         "@hubspot/serverless-dev-runtime": "7.0.6",
         "@hubspot/theme-preview-dev-server": "0.0.10",