@hubspot/cli 7.10.0-beta.2 → 7.10.0-experimental.0

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

@@ -1,18 +1,18 @@
 import { DocsSearchTool } from '../DocsSearchTool.js';
 import { http } from '@hubspot/local-dev-lib/http';
 import { isHubSpotHttpError } from '@hubspot/local-dev-lib/errors/index';
-import { getAccountIdFromCliConfig } from '../../../utils/cliConfig.js';
+import { getConfigDefaultAccountIfExists } from '@hubspot/local-dev-lib/config';
 import { mcpFeedbackRequest } from '../../../utils/feedbackTracking.js';
 vi.mock('@modelcontextprotocol/sdk/server/mcp.js');
 vi.mock('@hubspot/local-dev-lib/http');
 vi.mock('@hubspot/local-dev-lib/errors/index');
+vi.mock('@hubspot/local-dev-lib/config');
 vi.mock('../../../utils/toolUsageTracking');
-vi.mock('../../../utils/cliConfig.js');
 vi.mock('../../../utils/feedbackTracking');
 const mockMcpFeedbackRequest = mcpFeedbackRequest;
 const mockHttp = http;
-const mockGetAccountIdFromCliConfig = getAccountIdFromCliConfig;
 const mockIsHubSpotHttpError = vi.mocked(isHubSpotHttpError);
+const mockGetConfigDefaultAccountIfExists = getConfigDefaultAccountIfExists;
 describe('mcp-server/tools/project/DocsSearchTool', () => {
     let mockMcpServer;
     let tool;
@@ -45,7 +45,7 @@ describe('mcp-server/tools/project/DocsSearchTool', () => {
             absoluteCurrentWorkingDirectory: '/foo',
         };
         it('should return auth error message when no account ID is found', async () => {
-            mockGetAccountIdFromCliConfig.mockReturnValue(null);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue(undefined);
             const result = await tool.handler(mockInput);
             expect(result).toEqual({
                 content: [
@@ -57,7 +57,9 @@ describe('mcp-server/tools/project/DocsSearchTool', () => {
             });
         });
         it('should return successful results when docs are found', async () => {
-            mockGetAccountIdFromCliConfig.mockReturnValue(12345);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue({
+                accountId: 12345,
+            });
             const mockResponse = {
                 results: [
                     {
@@ -81,7 +83,7 @@ describe('mcp-server/tools/project/DocsSearchTool', () => {
                 data: mockResponse,
             });
             const result = await tool.handler(mockInput);
-            expect(mockGetAccountIdFromCliConfig).toHaveBeenCalledWith('/foo');
+            expect(mockGetConfigDefaultAccountIfExists).toHaveBeenCalled();
             expect(mockHttp.post).toHaveBeenCalledWith(12345, {
                 url: 'dev/docs/llms/v1/docs-search',
                 data: {
@@ -109,7 +111,9 @@ describe('mcp-server/tools/project/DocsSearchTool', () => {
             expect(resultText).toContain('Test content 2');
         });
         it('should return no results message when no documentation is found', async () => {
-            mockGetAccountIdFromCliConfig.mockReturnValue(12345);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue({
+                accountId: 12345,
+            });
             const mockResponse = {
                 results: [],
             };
@@ -128,7 +132,9 @@ describe('mcp-server/tools/project/DocsSearchTool', () => {
             });
         });
         it('should return no results message when results is null', async () => {
-            mockGetAccountIdFromCliConfig.mockReturnValue(12345);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue({
+                accountId: 12345,
+            });
             const mockResponse = {
                 results: null,
             };
@@ -147,7 +153,9 @@ describe('mcp-server/tools/project/DocsSearchTool', () => {
             });
         });
         it('should handle HubSpot HTTP errors', async () => {
-            mockGetAccountIdFromCliConfig.mockReturnValue(12345);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue({
+                accountId: 12345,
+            });
             const mockError = {
                 toString: () => 'HubSpot API Error: 404 Not Found',
             };
@@ -164,7 +172,9 @@ describe('mcp-server/tools/project/DocsSearchTool', () => {
             });
         });
         it('should handle generic errors', async () => {
-            mockGetAccountIdFromCliConfig.mockReturnValue(12345);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue({
+                accountId: 12345,
+            });
             const mockError = new Error('Network error');
             mockHttp.post.mockRejectedValue(mockError);
             mockIsHubSpotHttpError.mockReturnValue(false);
@@ -179,7 +189,9 @@ describe('mcp-server/tools/project/DocsSearchTool', () => {
             });
         });
         it('should handle non-Error rejections', async () => {
-            mockGetAccountIdFromCliConfig.mockReturnValue(12345);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue({
+                accountId: 12345,
+            });
             mockHttp.post.mockRejectedValue('String error');
             mockIsHubSpotHttpError.mockReturnValue(false);
             const result = await tool.handler(mockInput);

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

@@ -1,6 +1,6 @@
 import { GetApiUsagePatternsByAppIdTool } from '../GetApiUsagePatternsByAppIdTool.js';
 import { z } from 'zod';
-import { getAccountId } from '@hubspot/local-dev-lib/config';
+import { getConfigDefaultAccountIfExists } from '@hubspot/local-dev-lib/config';
 import { http } from '@hubspot/local-dev-lib/http';
 import { isHubSpotHttpError } from '@hubspot/local-dev-lib/errors/index';
 import { mcpFeedbackRequest } from '../../../utils/feedbackTracking.js';
@@ -11,7 +11,7 @@ vi.mock('@hubspot/local-dev-lib/errors/index');
 vi.mock('@hubspot/local-dev-lib/config');
 vi.mock('../../../utils/feedbackTracking');
 const mockMcpFeedbackRequest = mcpFeedbackRequest;
-const mockGetAccountId = getAccountId;
+const mockGetConfigDefaultAccountIfExists = getConfigDefaultAccountIfExists;
 const mockHttp = http;
 const mockIsHubSpotHttpError = isHubSpotHttpError;
 describe('mcp-server/tools/project/GetApiUsagePatternsByAppIdTool', () => {
@@ -89,7 +89,9 @@ describe('mcp-server/tools/project/GetApiUsagePatternsByAppIdTool', () => {
             endDate: '2025-12-31',
         };
         beforeEach(() => {
-            mockGetAccountId.mockReturnValue(123456789);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue({
+                accountId: 123456789,
+            });
             mockIsHubSpotHttpError.mockReturnValue(false);
         });
         it('should return API usage patterns successfully', async () => {
@@ -118,7 +120,7 @@ describe('mcp-server/tools/project/GetApiUsagePatternsByAppIdTool', () => {
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
             mockHttp.get.mockResolvedValue(mockResponse);
             const result = await tool.handler(input);
-            expect(mockGetAccountId).toHaveBeenCalledWith();
+            expect(mockGetConfigDefaultAccountIfExists).toHaveBeenCalledWith();
             expect(mockHttp.get).toHaveBeenCalledWith(123456789, {
                 url: 'app/feature/utilization/public/v3/insights/app/12345/usage-patterns',
                 params: {
@@ -136,7 +138,7 @@ describe('mcp-server/tools/project/GetApiUsagePatternsByAppIdTool', () => {
             });
         });
         it('should return error when account ID cannot be determined', async () => {
-            mockGetAccountId.mockReturnValue(null);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue(undefined);
             const result = await tool.handler(input);
             expect(result).toEqual({
                 content: [

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

@@ -1,5 +1,5 @@
 import { GetApplicationInfoTool } from '../GetApplicationInfoTool.js';
-import { getAccountId } from '@hubspot/local-dev-lib/config';
+import { getConfigDefaultAccountIfExists } from '@hubspot/local-dev-lib/config';
 import { http } from '@hubspot/local-dev-lib/http';
 import { isHubSpotHttpError } from '@hubspot/local-dev-lib/errors/index';
 import { mcpFeedbackRequest } from '../../../utils/feedbackTracking.js';
@@ -10,7 +10,7 @@ vi.mock('@hubspot/local-dev-lib/errors/index');
 vi.mock('@hubspot/local-dev-lib/config');
 vi.mock('../../../utils/feedbackTracking');
 const mockMcpFeedbackRequest = mcpFeedbackRequest;
-const mockGetAccountId = getAccountId;
+const mockGetConfigDefaultAccountIfExists = getConfigDefaultAccountIfExists;
 const mockHttp = http;
 const mockIsHubSpotHttpError = isHubSpotHttpError;
 describe('mcp-server/tools/project/GetApplicationInfoTool', () => {
@@ -42,7 +42,9 @@ describe('mcp-server/tools/project/GetApplicationInfoTool', () => {
     describe('handler', () => {
         const input = { absoluteCurrentWorkingDirectory: '/test/dir' };
         beforeEach(() => {
-            mockGetAccountId.mockReturnValue(123456789);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue({
+                accountId: 123456789,
+            });
             mockIsHubSpotHttpError.mockReturnValue(false);
         });
         it('should return application information successfully', async () => {
@@ -67,7 +69,7 @@ describe('mcp-server/tools/project/GetApplicationInfoTool', () => {
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
             mockHttp.get.mockResolvedValue(mockResponse);
             const result = await tool.handler(input);
-            expect(mockGetAccountId).toHaveBeenCalledWith();
+            expect(mockGetConfigDefaultAccountIfExists).toHaveBeenCalledWith();
             expect(mockHttp.get).toHaveBeenCalledWith(123456789, {
                 url: 'app/feature/utilization/public/v3/insights/apps',
             });
@@ -81,7 +83,7 @@ describe('mcp-server/tools/project/GetApplicationInfoTool', () => {
             });
         });
         it('should return error when account ID cannot be determined', async () => {
-            mockGetAccountId.mockReturnValue(null);
+            mockGetConfigDefaultAccountIfExists.mockReturnValue(undefined);
             const result = await tool.handler(input);
             expect(result).toEqual({
                 content: [

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

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

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

@@ -0,0 +1,305 @@
+import { GetBuildLogsTool } from '../GetBuildLogsTool.js';
+import { http } from '@hubspot/local-dev-lib/http';
+import { isHubSpotHttpError } from '@hubspot/local-dev-lib/errors/index';
+import { getProjectConfig, validateProjectConfig, } from '../../../../lib/projects/config.js';
+import { mcpFeedbackRequest } from '../../../utils/feedbackTracking.js';
+import { getConfigDefaultAccountIfExists } from '@hubspot/local-dev-lib/config';
+vi.mock('@modelcontextprotocol/sdk/server/mcp.js');
+vi.mock('../../../utils/toolUsageTracking');
+vi.mock('../../../utils/cliConfig');
+vi.mock('@hubspot/local-dev-lib/http');
+vi.mock('@hubspot/local-dev-lib/errors/index');
+vi.mock('../../../../lib/projects/config.js');
+vi.mock('../../../utils/feedbackTracking');
+vi.mock('@hubspot/local-dev-lib/config');
+const mockMcpFeedbackRequest = mcpFeedbackRequest;
+const mockGetConfigDefaultAccountIfExists = getConfigDefaultAccountIfExists;
+const mockHttpGet = http.get;
+const mockIsHubSpotHttpError = isHubSpotHttpError;
+const mockGetProjectConfig = getProjectConfig;
+const mockValidateProjectConfig = validateProjectConfig;
+const TEST_ACCOUNT_ID = 123456789;
+const TEST_PROJECT_NAME = 'test-project';
+const TEST_PROJECT_PATH = '/test/project';
+const TEST_WORKING_DIR = '/test';
+const TEST_BUILD_ID = 123;
+const TEST_PLATFORM_VERSION = '2025.2';
+function createMockLog(overrides = {}) {
+    return {
+        lineNumber: 1,
+        logLevel: 'INFO',
+        message: 'Starting build validation',
+        pipelineStepId: 1,
+        pipelineSubstepId: 'test-step',
+        pipelineSubstepName: 'test-component',
+        timestamp: Date.now(),
+        ...overrides,
+    };
+}
+function createMockLogs() {
+    return {
+        pipelineStage: 'BUILD',
+        pipelineStepId: 2,
+        projectName: TEST_PROJECT_NAME,
+        logs: [
+            {
+                pipelineStepId: 2,
+                pipelineSubstepId: 'test-app-id',
+                pipelineStage: 'BUILD',
+                projectName: TEST_PROJECT_NAME,
+                logs: [
+                    createMockLog({
+                        lineNumber: 1,
+                        logLevel: 'INFO',
+                        message: "Starting build validation for app 'Test App'",
+                        pipelineSubstepName: 'test-app',
+                        pipelineSubstepId: 'test-app-id',
+                    }),
+                    createMockLog({
+                        lineNumber: 2,
+                        logLevel: 'INFO',
+                        message: "Successfully validated app configuration for 'Test App'",
+                        pipelineSubstepName: 'test-app',
+                        pipelineSubstepId: 'test-app-id',
+                    }),
+                ],
+            },
+            {
+                pipelineStepId: 2,
+                pipelineSubstepId: 'test-card-id',
+                pipelineStage: 'BUILD',
+                projectName: TEST_PROJECT_NAME,
+                logs: [
+                    createMockLog({
+                        lineNumber: 1,
+                        logLevel: 'INFO',
+                        message: 'Starting card build...',
+                        pipelineSubstepName: 'test-card',
+                        pipelineSubstepId: 'test-card-id',
+                    }),
+                    createMockLog({
+                        lineNumber: 2,
+                        logLevel: 'INFO',
+                        message: 'Validating card configuration...',
+                        pipelineSubstepName: 'test-card',
+                        pipelineSubstepId: 'test-card-id',
+                    }),
+                    createMockLog({
+                        lineNumber: 3,
+                        logLevel: 'ERROR',
+                        message: 'Failed to build card. The preview image file /app/cards/assets/preview.png was not found. Make sure the file exists in /app/cards and try again.',
+                        pipelineSubstepName: 'test-card',
+                        pipelineSubstepId: 'test-card-id',
+                    }),
+                ],
+            },
+            {
+                pipelineStepId: 2,
+                pipelineSubstepId: 'test-webhook-id',
+                pipelineStage: 'BUILD',
+                projectName: TEST_PROJECT_NAME,
+                logs: [
+                    createMockLog({
+                        lineNumber: 1,
+                        logLevel: 'WARN',
+                        message: 'Deprecated API usage detected in webhook configuration',
+                        pipelineSubstepName: 'test-webhook',
+                        pipelineSubstepId: 'test-webhook-id',
+                    }),
+                ],
+            },
+        ],
+    };
+}
+function createLogsResponse(data) {
+    return {
+        data,
+    };
+}
+function createMockProjectConfig(overrides = {}) {
+    return {
+        name: TEST_PROJECT_NAME,
+        srcDir: 'src',
+        platformVersion: TEST_PLATFORM_VERSION,
+        ...overrides,
+    };
+}
+function expectTextContent(result, ...expectedStrings) {
+    expectedStrings.forEach(str => {
+        expect(result.content[1].text).toContain(str);
+    });
+}
+describe('mcp-server/tools/project/GetBuildLogsTool', () => {
+    let mockMcpServer;
+    let tool;
+    let mockRegisteredTool;
+    beforeEach(() => {
+        vi.clearAllMocks();
+        // @ts-expect-error Not mocking the whole thing
+        mockMcpServer = {
+            registerTool: vi.fn(),
+        };
+        mockRegisteredTool = {};
+        mockMcpServer.registerTool.mockReturnValue(mockRegisteredTool);
+        mockMcpFeedbackRequest.mockResolvedValue('');
+        tool = new GetBuildLogsTool(mockMcpServer);
+        // Default mock implementations
+        mockGetConfigDefaultAccountIfExists.mockReturnValue({
+            accountId: TEST_ACCOUNT_ID,
+        });
+        mockIsHubSpotHttpError.mockReturnValue(false);
+        mockGetProjectConfig.mockResolvedValue({
+            projectConfig: createMockProjectConfig(),
+            projectDir: TEST_PROJECT_PATH,
+        });
+        mockValidateProjectConfig.mockImplementation(() => { });
+    });
+    describe('register', () => {
+        it('should register tool with correct parameters', () => {
+            const result = tool.register();
+            expect(mockMcpServer.registerTool).toHaveBeenCalledWith('get-build-logs', expect.objectContaining({
+                title: 'Get HubSpot Project Build Logs',
+                description: expect.stringContaining('Retrieves build logs for a specific HubSpot project build'),
+                inputSchema: expect.anything(),
+            }), expect.any(Function));
+            expect(result).toBe(mockRegisteredTool);
+        });
+    });
+    describe('handler', () => {
+        const baseInput = {
+            absoluteProjectPath: TEST_PROJECT_PATH,
+            absoluteCurrentWorkingDirectory: TEST_WORKING_DIR,
+            buildId: TEST_BUILD_ID,
+            logLevel: 'ALL',
+        };
+        describe('error handling', () => {
+            it('should return error when account ID cannot be determined', async () => {
+                mockGetConfigDefaultAccountIfExists.mockReturnValue(undefined);
+                const result = await tool.handler(baseInput);
+                expect(result).toEqual({
+                    content: [
+                        {
+                            type: 'text',
+                            text: TEST_WORKING_DIR,
+                        },
+                        {
+                            type: 'text',
+                            text: 'No account ID found. Please run `hs account auth` to configure an account, or set a default account with `hs account use <account>`',
+                        },
+                    ],
+                });
+                expect(mockHttpGet).not.toHaveBeenCalled();
+            });
+            it('should handle HubSpot HTTP errors', async () => {
+                const error = new Error('API Error');
+                mockIsHubSpotHttpError.mockReturnValue(true);
+                mockHttpGet.mockRejectedValue(error);
+                const result = await tool.handler(baseInput);
+                expectTextContent(result, 'API Error');
+            });
+            it('should handle generic errors', async () => {
+                const error = new Error('Generic error');
+                mockIsHubSpotHttpError.mockReturnValue(false);
+                mockHttpGet.mockRejectedValue(error);
+                const result = await tool.handler(baseInput);
+                expectTextContent(result, 'Generic error');
+            });
+            it('should handle empty logs', async () => {
+                const emptyLogsResponse = {
+                    pipelineStage: 'BUILD',
+                    pipelineStepId: 2,
+                    projectName: TEST_PROJECT_NAME,
+                    logs: [],
+                };
+                mockHttpGet.mockResolvedValue(
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                createLogsResponse(emptyLogsResponse));
+                const result = await tool.handler(baseInput);
+                expectTextContent(result, `No logs found for build #${TEST_BUILD_ID}`, TEST_PROJECT_NAME);
+            });
+        });
+        describe('log fetching', () => {
+            it('should fetch and display all logs', async () => {
+                const mockLogs = createMockLogs();
+                mockHttpGet.mockResolvedValue(
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                createLogsResponse(mockLogs));
+                const result = await tool.handler(baseInput);
+                expect(mockGetProjectConfig).toHaveBeenCalledWith(TEST_PROJECT_PATH);
+                expect(mockHttpGet).toHaveBeenCalledWith(TEST_ACCOUNT_ID, {
+                    url: `dfs/logging/v1/logs/projects/${TEST_PROJECT_NAME}/builds/${TEST_BUILD_ID}`,
+                });
+                expectTextContent(result, `Logs for build #${TEST_BUILD_ID}`, TEST_PROJECT_NAME, 'ALL level', 'Starting build validation', 'Successfully validated', 'Failed to build card', 'Deprecated API usage');
+            });
+            it('should filter logs by ERROR level', async () => {
+                const mockLogs = createMockLogs();
+                mockHttpGet.mockResolvedValue(
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                createLogsResponse(mockLogs));
+                const result = await tool.handler({
+                    ...baseInput,
+                    logLevel: 'ERROR',
+                });
+                expectTextContent(result, 'ERROR level', 'Failed to build card', 'preview.png was not found');
+                expect(result.content[1].text).not.toContain('Starting build');
+                expect(result.content[1].text).not.toContain('Deprecated API');
+            });
+            it('should filter logs by WARN level', async () => {
+                const mockLogs = createMockLogs();
+                mockHttpGet.mockResolvedValue(
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                createLogsResponse(mockLogs));
+                const result = await tool.handler({
+                    ...baseInput,
+                    logLevel: 'WARN',
+                });
+                expectTextContent(result, 'WARN level', 'Deprecated API usage detected');
+                expect(result.content[1].text).not.toContain('Starting build');
+                expect(result.content[1].text).not.toContain('Failed to build card');
+            });
+            it('should filter logs by INFO level', async () => {
+                const mockLogs = createMockLogs();
+                mockHttpGet.mockResolvedValue(
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                createLogsResponse(mockLogs));
+                const result = await tool.handler({
+                    ...baseInput,
+                    logLevel: 'INFO',
+                });
+                expectTextContent(result, 'INFO level', 'Starting build validation', 'Successfully validated');
+                expect(result.content[1].text).not.toContain('Failed to build card');
+                expect(result.content[1].text).not.toContain('Deprecated API');
+            });
+            it('should show all logs when no logs match filter', async () => {
+                const infoOnlyResponse = {
+                    pipelineStage: 'BUILD',
+                    pipelineStepId: 2,
+                    projectName: TEST_PROJECT_NAME,
+                    logs: [
+                        {
+                            pipelineStepId: 2,
+                            pipelineSubstepId: 'info-only-id',
+                            pipelineStage: 'BUILD',
+                            projectName: TEST_PROJECT_NAME,
+                            logs: [
+                                createMockLog({
+                                    logLevel: 'INFO',
+                                    message: 'Info message only',
+                                    pipelineSubstepId: 'info-only-id',
+                                }),
+                            ],
+                        },
+                    ],
+                };
+                mockHttpGet.mockResolvedValue(
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                createLogsResponse(infoOnlyResponse));
+                const result = await tool.handler({
+                    ...baseInput,
+                    logLevel: 'ERROR',
+                });
+                expectTextContent(result, `No ERROR level logs found`, `build #${TEST_BUILD_ID}`, 'Showing all logs instead', 'Info message only');
+            });
+        });
+    });
+});

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

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