@hubspot/cli 8.3.0 → 8.4.0-beta.0

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

@@ -1,118 +0,0 @@
-import { ValidateProjectTool, } from '../ValidateProjectTool.js';
-import { runCommandInDir } from '../../../utils/command.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('../../../utils/toolUsageTracking');
-vi.mock('../../../utils/feedbackTracking');
-const mockTrackToolUsage = trackToolUsage;
-const mockMcpFeedbackRequest = mcpFeedbackRequest;
-const mockRunCommandInDir = runCommandInDir;
-describe('mcp-server/tools/project/ValidateProjectTool', () => {
-    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 ValidateProjectTool(mockMcpServer);
-    });
-    describe('register', () => {
-        it('should register tool with correct parameters', () => {
-            const result = tool.register();
-            expect(mockMcpServer.registerTool).toHaveBeenCalledWith('validate-project', expect.objectContaining({
-                title: expect.stringContaining('Validate HubSpot Project'),
-                description: expect.stringContaining('Validates the HubSpot project and its configuration files.'),
-                inputSchema: expect.any(Object),
-            }), expect.any(Function));
-            expect(result).toBe(mockRegisteredTool);
-        });
-    });
-    describe('handler', () => {
-        const input = {
-            absoluteCurrentWorkingDirectory: '/test/dir',
-            absoluteProjectPath: '/test/project',
-        };
-        it('should validate project successfully', async () => {
-            mockRunCommandInDir.mockResolvedValue({
-                stdout: 'Project validation successful',
-                stderr: '',
-            });
-            const result = await tool.handler(input);
-            expect(mockRunCommandInDir).toHaveBeenCalledWith('/test/project', 'hs project validate');
-            expect(result).toEqual({
-                content: [
-                    { type: 'text', text: 'Project validation successful' },
-                    { type: 'text', text: '' },
-                ],
-            });
-        });
-        it('should handle validation with warnings', async () => {
-            mockRunCommandInDir.mockResolvedValue({
-                stdout: 'Project is valid',
-                stderr: 'Warning: some files may need updates',
-            });
-            const result = await tool.handler(input);
-            expect(result.content).toEqual([
-                { type: 'text', text: 'Project is valid' },
-                { type: 'text', text: 'Warning: some files may need updates' },
-            ]);
-        });
-        it('should handle validation errors', async () => {
-            const error = new Error('Validation failed');
-            mockRunCommandInDir.mockRejectedValue(error);
-            const result = await tool.handler(input);
-            expect(result).toEqual({
-                content: [{ type: 'text', text: 'Validation failed' }],
-            });
-        });
-        it('should handle non-Error rejection', async () => {
-            mockRunCommandInDir.mockRejectedValue('String error');
-            const result = await tool.handler(input);
-            expect(result).toEqual({
-                content: [{ type: 'text', text: 'String error' }],
-            });
-        });
-        it('should handle empty stdout and stderr', async () => {
-            mockRunCommandInDir.mockResolvedValue({
-                stdout: 'stdout',
-                stderr: 'stderr',
-            });
-            const result = await tool.handler(input);
-            expect(result.content).toEqual([
-                { type: 'text', text: 'stdout' },
-                { type: 'text', text: 'stderr' },
-            ]);
-        });
-        it('should work with different project paths', async () => {
-            mockRunCommandInDir.mockResolvedValue({
-                stdout: 'Validation complete',
-                stderr: '',
-            });
-            const differentInput = {
-                absoluteCurrentWorkingDirectory: '/test/dir',
-                absoluteProjectPath: '/different/path/to/project',
-            };
-            await tool.handler(differentInput);
-            expect(mockRunCommandInDir).toHaveBeenCalledWith('/different/path/to/project', 'hs project validate');
-        });
-        it('should handle validation errors with stderr', async () => {
-            mockRunCommandInDir.mockResolvedValue({
-                stdout: '',
-                stderr: 'Error: Missing required configuration file',
-            });
-            const result = await tool.handler(input);
-            expect(result.content).toEqual([
-                { type: 'text', text: '' },
-                { type: 'text', text: 'Error: Missing required configuration file' },
-            ]);
-        });
-    });
-});

package/mcp-server/utils/__tests__/command.test.d.ts

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

package/mcp-server/utils/__tests__/command.test.js

@@ -1,275 +0,0 @@
-import fs from 'fs';
-import path from 'path';
-const mockExecAsync = vi.fn();
-vi.mock('node:child_process');
-vi.mock('util', () => ({
-    default: {
-        promisify: () => mockExecAsync,
-    },
-    promisify: () => mockExecAsync,
-}));
-vi.mock('fs');
-vi.mock('path');
-// Import after mocks are set up
-const { addFlag, runCommandInDir } = await import('../command.js');
-const mockExistsSync = vi.mocked(fs.existsSync);
-const mockMkdirSync = vi.mocked(fs.mkdirSync);
-const mockResolve = vi.mocked(path.resolve);
-describe('mcp-server/utils/command', () => {
-    describe('addFlag', () => {
-        it('should add string flag to command', () => {
-            const result = addFlag('hs project create', 'name', 'test-project');
-            expect(result).toBe('hs project create --name "test-project"');
-        });
-        it('should add number flag to command', () => {
-            const result = addFlag('hs project deploy', 'build', 123);
-            expect(result).toBe('hs project deploy --build "123"');
-        });
-        it('should add boolean flag to command', () => {
-            const result = addFlag('hs project upload', 'watch', true);
-            expect(result).toBe('hs project upload --watch "true"');
-        });
-        it('should add array flag to command', () => {
-            const result = addFlag('hs project create', 'features', [
-                'card',
-                'settings',
-            ]);
-            expect(result).toBe('hs project create --features "card" "settings"');
-        });
-        it('should handle empty array', () => {
-            const result = addFlag('hs project create', 'features', []);
-            expect(result).toBe('hs project create --features ');
-        });
-        it('should handle array with one item', () => {
-            const result = addFlag('hs project create', 'features', ['card']);
-            expect(result).toBe('hs project create --features "card"');
-        });
-        it('should handle special characters in string values', () => {
-            const result = addFlag('hs project create', 'name', 'my-project with spaces');
-            expect(result).toBe('hs project create --name "my-project with spaces"');
-        });
-        it('should handle special characters in array values', () => {
-            const result = addFlag('hs project create', 'features', [
-                'card with spaces',
-                'settings',
-            ]);
-            expect(result).toBe('hs project create --features "card with spaces" "settings"');
-        });
-    });
-    describe('runCommandInDir', () => {
-        const mockDirectory = '/test/directory';
-        const mockCommand = 'npm install';
-        const mockResolvedPath = '/resolved/test/directory';
-        beforeEach(() => {
-            mockResolve.mockReturnValue(mockResolvedPath);
-        });
-        it('should run command in existing directory', async () => {
-            const expectedResult = {
-                stdout: 'command output',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            const result = await runCommandInDir(mockDirectory, mockCommand);
-            expect(mockExistsSync).toHaveBeenCalledWith(mockDirectory);
-            expect(mockMkdirSync).not.toHaveBeenCalled();
-            expect(mockResolve).toHaveBeenCalledWith(mockDirectory);
-            expect(mockExecAsync).toHaveBeenCalledWith(mockCommand, expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-            expect(result).toEqual(expectedResult);
-        });
-        it('should create directory if it does not exist', async () => {
-            const expectedResult = {
-                stdout: 'command output',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(false);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            const result = await runCommandInDir(mockDirectory, mockCommand);
-            expect(mockExistsSync).toHaveBeenCalledWith(mockDirectory);
-            expect(mockMkdirSync).toHaveBeenCalledWith(mockDirectory);
-            expect(mockResolve).toHaveBeenCalledWith(mockDirectory);
-            expect(mockExecAsync).toHaveBeenCalledWith(mockCommand, expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-            expect(result).toEqual(expectedResult);
-        });
-        it('should propagate execAsync errors', async () => {
-            const error = new Error('Command failed');
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockRejectedValue(error);
-            await expect(runCommandInDir(mockDirectory, mockCommand)).rejects.toThrow('Command failed');
-            expect(mockExecAsync).toHaveBeenCalledWith(mockCommand, expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-        });
-        it('should handle stderr in results', async () => {
-            const expectedResult = {
-                stdout: 'some output',
-                stderr: 'warning message',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            const result = await runCommandInDir(mockDirectory, mockCommand);
-            expect(result.stdout).toBe('some output');
-            expect(result.stderr).toBe('warning message');
-        });
-        it('should add --disable-usage-tracking flag to hs commands', async () => {
-            const hsCommand = 'hs project upload';
-            const expectedResult = {
-                stdout: 'success',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            await runCommandInDir(mockDirectory, hsCommand);
-            expect(mockExecAsync).toHaveBeenCalledWith('hs project upload --disable-usage-tracking "true"', expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-        });
-        it('should not add --disable-usage-tracking flag to non-hs commands', async () => {
-            const nonHsCommand = 'npm install';
-            const expectedResult = {
-                stdout: 'success',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            await runCommandInDir(mockDirectory, nonHsCommand);
-            expect(mockExecAsync).toHaveBeenCalledWith('npm install', expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-        });
-        it('should add --disable-usage-tracking flag to hs commands with existing flags', async () => {
-            const hsCommand = 'hs project upload --profile prod';
-            const expectedResult = {
-                stdout: 'success',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            await runCommandInDir(mockDirectory, hsCommand);
-            expect(mockExecAsync).toHaveBeenCalledWith('hs project upload --profile prod --disable-usage-tracking "true"', expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-        });
-        it('should handle hs commands that start with whitespace', async () => {
-            const hsCommand = 'hs init';
-            const expectedResult = {
-                stdout: 'success',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            await runCommandInDir(mockDirectory, hsCommand);
-            expect(mockExecAsync).toHaveBeenCalledWith('hs init --disable-usage-tracking "true"', expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-        });
-        it('should use npx -p @hubspot/cli when HUBSPOT_MCP_STANDALONE is true', async () => {
-            const originalEnv = process.env.HUBSPOT_MCP_STANDALONE;
-            process.env.HUBSPOT_MCP_STANDALONE = 'true';
-            const hsCommand = 'hs project upload';
-            const expectedResult = {
-                stdout: 'success',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            await runCommandInDir(mockDirectory, hsCommand);
-            expect(mockExecAsync).toHaveBeenCalledWith('npx -y -p @hubspot/cli hs project upload --disable-usage-tracking "true"', expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-            // Restore original env
-            if (originalEnv === undefined) {
-                delete process.env.HUBSPOT_MCP_STANDALONE;
-            }
-            else {
-                process.env.HUBSPOT_MCP_STANDALONE = originalEnv;
-            }
-        });
-        it('should use regular hs command when HUBSPOT_MCP_STANDALONE is not set', async () => {
-            const originalEnv = process.env.HUBSPOT_MCP_STANDALONE;
-            delete process.env.HUBSPOT_MCP_STANDALONE;
-            const hsCommand = 'hs project upload';
-            const expectedResult = {
-                stdout: 'success',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            await runCommandInDir(mockDirectory, hsCommand);
-            expect(mockExecAsync).toHaveBeenCalledWith('hs project upload --disable-usage-tracking "true"', expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-            // Restore original env
-            if (originalEnv !== undefined) {
-                process.env.HUBSPOT_MCP_STANDALONE = originalEnv;
-            }
-        });
-        it('should use npx -p @hubspot/cli for hs commands with flags in standalone mode', async () => {
-            const originalEnv = process.env.HUBSPOT_MCP_STANDALONE;
-            process.env.HUBSPOT_MCP_STANDALONE = 'true';
-            const hsCommand = 'hs project upload --profile prod';
-            const expectedResult = {
-                stdout: 'success',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            await runCommandInDir(mockDirectory, hsCommand);
-            expect(mockExecAsync).toHaveBeenCalledWith('npx -y -p @hubspot/cli hs project upload --profile prod --disable-usage-tracking "true"', expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-            // Restore original env
-            if (originalEnv === undefined) {
-                delete process.env.HUBSPOT_MCP_STANDALONE;
-            }
-            else {
-                process.env.HUBSPOT_MCP_STANDALONE = originalEnv;
-            }
-        });
-        it('should use pinned CLI version when HUBSPOT_CLI_VERSION is set in standalone mode', async () => {
-            const originalStandaloneEnv = process.env.HUBSPOT_MCP_STANDALONE;
-            const originalVersionEnv = process.env.HUBSPOT_CLI_VERSION;
-            process.env.HUBSPOT_MCP_STANDALONE = 'true';
-            process.env.HUBSPOT_CLI_VERSION = '8.1.0';
-            const hsCommand = 'hs project upload';
-            const expectedResult = {
-                stdout: 'success',
-                stderr: '',
-            };
-            mockExistsSync.mockReturnValue(true);
-            mockExecAsync.mockResolvedValue(expectedResult);
-            await runCommandInDir(mockDirectory, hsCommand);
-            expect(mockExecAsync).toHaveBeenCalledWith('npx -y -p @hubspot/cli@8.1.0 hs project upload --disable-usage-tracking "true"', expect.objectContaining({
-                cwd: mockResolvedPath,
-                env: expect.any(Object),
-            }));
-            // Restore original env
-            if (originalStandaloneEnv === undefined) {
-                delete process.env.HUBSPOT_MCP_STANDALONE;
-            }
-            else {
-                process.env.HUBSPOT_MCP_STANDALONE = originalStandaloneEnv;
-            }
-            if (originalVersionEnv === undefined) {
-                delete process.env.HUBSPOT_CLI_VERSION;
-            }
-            else {
-                process.env.HUBSPOT_CLI_VERSION = originalVersionEnv;
-            }
-        });
-    });
-});

package/mcp-server/utils/__tests__/content.test.d.ts

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

package/mcp-server/utils/__tests__/content.test.js

@@ -1,164 +0,0 @@
-import { formatTextContents, formatTextContent } from '../content.js';
-import { mcpFeedbackRequest } from '../feedbackTracking.js';
-vi.mock('../feedbackTracking');
-const mockMcpFeedbackRequest = mcpFeedbackRequest;
-describe('mcp-server/utils/content', () => {
-    describe('formatTextContent', () => {
-        it('should format a single text string', () => {
-            const result = formatTextContent('Test message');
-            expect(result).toEqual({
-                type: 'text',
-                text: 'Test message',
-            });
-        });
-        it('should handle empty string', () => {
-            const result = formatTextContent('');
-            expect(result).toEqual({
-                type: 'text',
-                text: '',
-            });
-        });
-        it('should handle multiline text', () => {
-            const multilineText = 'Line 1\nLine 2\nLine 3';
-            const result = formatTextContent(multilineText);
-            expect(result).toEqual({
-                type: 'text',
-                text: multilineText,
-            });
-        });
-    });
-    describe('formatTextContents', () => {
-        it('should format single output without feedback', async () => {
-            mockMcpFeedbackRequest.mockResolvedValue('');
-            const result = await formatTextContents('Test output');
-            expect(mockMcpFeedbackRequest).toHaveBeenCalled();
-            expect(result).toEqual({
-                content: [
-                    {
-                        type: 'text',
-                        text: 'Test output',
-                    },
-                ],
-            });
-        });
-        it('should format multiple outputs without feedback', async () => {
-            mockMcpFeedbackRequest.mockResolvedValue('');
-            const result = await formatTextContents('First output', 'Second output', 'Third output');
-            expect(result).toEqual({
-                content: [
-                    { type: 'text', text: 'First output' },
-                    { type: 'text', text: 'Second output' },
-                    { type: 'text', text: 'Third output' },
-                ],
-            });
-        });
-        it('should append feedback message when feedback is returned', async () => {
-            const feedbackMessage = 'Please share your feedback!';
-            mockMcpFeedbackRequest.mockResolvedValue(feedbackMessage);
-            const result = await formatTextContents('Command output');
-            expect(mockMcpFeedbackRequest).toHaveBeenCalled();
-            expect(result).toEqual({
-                content: [
-                    { type: 'text', text: 'Command output' },
-                    { type: 'text', text: feedbackMessage },
-                ],
-            });
-        });
-        it('should append feedback to multiple outputs', async () => {
-            const feedbackMessage = 'Feedback request message';
-            mockMcpFeedbackRequest.mockResolvedValue(feedbackMessage);
-            const result = await formatTextContents('First output', 'Second output');
-            expect(result).toEqual({
-                content: [
-                    { type: 'text', text: 'First output' },
-                    { type: 'text', text: 'Second output' },
-                    { type: 'text', text: feedbackMessage },
-                ],
-            });
-        });
-        it('should filter out undefined outputs', async () => {
-            mockMcpFeedbackRequest.mockResolvedValue('');
-            const result = await formatTextContents('First output', undefined, 'Third output', undefined);
-            expect(result).toEqual({
-                content: [
-                    { type: 'text', text: 'First output' },
-                    { type: 'text', text: 'Third output' },
-                ],
-            });
-        });
-        it('should not call mcpFeedbackRequest when no outputs provided', async () => {
-            mockMcpFeedbackRequest.mockResolvedValue('Feedback message');
-            const result = await formatTextContents();
-            expect(mockMcpFeedbackRequest).not.toHaveBeenCalled();
-            expect(result).toEqual({
-                content: [],
-            });
-        });
-        it('should call mcpFeedbackRequest but not add feedback when all outputs are undefined and feedback is empty', async () => {
-            mockMcpFeedbackRequest.mockResolvedValue('');
-            const result = await formatTextContents(undefined, undefined);
-            // mcpFeedbackRequest is still called because outputs.length > 0, but content is empty
-            expect(mockMcpFeedbackRequest).toHaveBeenCalled();
-            expect(result).toEqual({
-                content: [],
-            });
-        });
-        it('should handle empty string outputs and still request feedback', async () => {
-            mockMcpFeedbackRequest.mockResolvedValue('');
-            const result = await formatTextContents('');
-            expect(mockMcpFeedbackRequest).toHaveBeenCalled();
-            expect(result).toEqual({
-                content: [{ type: 'text', text: '' }],
-            });
-        });
-        it('should not add feedback when empty string is returned from mcpFeedbackRequest', async () => {
-            mockMcpFeedbackRequest.mockResolvedValue('');
-            const result = await formatTextContents('Output 1', 'Output 2');
-            // Should not have a third content item with empty feedback
-            expect(result.content).toHaveLength(2);
-            expect(result).toEqual({
-                content: [
-                    { type: 'text', text: 'Output 1' },
-                    { type: 'text', text: 'Output 2' },
-                ],
-            });
-        });
-        it('should handle feedback request with realistic threshold message', async () => {
-            const realisticFeedback = `
-
-    ---
-
-    IMPORTANT - FEEDBACK REQUEST FOR THIS RESPONSE ONLY:
-    This message appears only once. If you see this text in the tool output, you MUST include the following feedback request in your response to the user. If this text is NOT in the tool output, do NOT ask for feedback.
-
-    Say: "📝 **Before we continue, would you mind sharing feedback on your experience with the HubSpot Developer MCP?** Your input helps us improve: [Share feedback here](https://app.hubspot.com/l/product-updates/in-beta?rollout=239890)"`;
-            mockMcpFeedbackRequest.mockResolvedValue(realisticFeedback);
-            const result = await formatTextContents('Project created successfully');
-            expect(result.content).toHaveLength(2);
-            expect(result.content[0]).toEqual({
-                type: 'text',
-                text: 'Project created successfully',
-            });
-            expect(result.content[1]).toEqual({
-                type: 'text',
-                text: realisticFeedback,
-            });
-            expect(result.content[1].text).toContain('FEEDBACK REQUEST');
-        });
-        it('should not require absoluteCurrentWorkingDirectory parameter', async () => {
-            mockMcpFeedbackRequest.mockResolvedValue('');
-            const result = await formatTextContents('Test output');
-            expect(mockMcpFeedbackRequest).toHaveBeenCalled();
-            expect(result).toEqual({
-                content: [{ type: 'text', text: 'Test output' }],
-            });
-        });
-        it('should await mcpFeedbackRequest and handle async properly', async () => {
-            const feedbackPromise = Promise.resolve('Async feedback message');
-            mockMcpFeedbackRequest.mockReturnValue(feedbackPromise);
-            const result = await formatTextContents('Output');
-            expect(result.content).toHaveLength(2);
-            expect(result.content[1].text).toBe('Async feedback message');
-        });
-    });
-});

package/mcp-server/utils/__tests__/feedbackTracking.test.d.ts

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

package/mcp-server/utils/__tests__/feedbackTracking.test.js

@@ -1,69 +0,0 @@
-import { mcpFeedbackRequest } from '../feedbackTracking.js';
-import { getStateValue, setStateValue, } from '@hubspot/local-dev-lib/config/state';
-import { FEEDBACK_URL } from '../../../lib/constants.js';
-vi.mock('@hubspot/local-dev-lib/config/state');
-vi.mock('@hubspot/local-dev-lib/logger');
-const mockGetStateValue = getStateValue;
-const mockSetStateValue = setStateValue;
-describe('mcp-server/utils/feedbackTracking', () => {
-    describe('mcpFeedbackRequest', () => {
-        it('should return empty string when not at a threshold', async () => {
-            mockGetStateValue.mockReturnValue(10);
-            const result = await mcpFeedbackRequest();
-            expect(mockSetStateValue).toHaveBeenCalledWith('mcpTotalToolCalls', 11);
-            expect(result).toBe(undefined);
-        });
-        it('should return feedback message at threshold 50', async () => {
-            mockGetStateValue.mockReturnValue(49);
-            const result = await mcpFeedbackRequest();
-            expect(mockSetStateValue).toHaveBeenCalledWith('mcpTotalToolCalls', 50);
-            expect(result).toContain('FEEDBACK REQUEST');
-            expect(result).toContain(FEEDBACK_URL);
-        });
-        it('should return feedback message at threshold 250', async () => {
-            mockGetStateValue.mockReturnValue(249);
-            const result = await mcpFeedbackRequest();
-            expect(mockSetStateValue).toHaveBeenCalledWith('mcpTotalToolCalls', 250);
-            expect(result).toContain('FEEDBACK REQUEST');
-        });
-        it('should return feedback message at threshold 550', async () => {
-            mockGetStateValue.mockReturnValue(549);
-            const result = await mcpFeedbackRequest();
-            expect(mockSetStateValue).toHaveBeenCalledWith('mcpTotalToolCalls', 550);
-            expect(result).toContain('FEEDBACK REQUEST');
-        });
-        it('should return feedback message at threshold 1050', async () => {
-            mockGetStateValue.mockReturnValue(1049);
-            const result = await mcpFeedbackRequest();
-            expect(mockSetStateValue).toHaveBeenCalledWith('mcpTotalToolCalls', 1050);
-            expect(result).toContain('FEEDBACK REQUEST');
-        });
-        it('should not return feedback at threshold + 1', async () => {
-            mockGetStateValue.mockReturnValue(50);
-            const result = await mcpFeedbackRequest();
-            expect(mockSetStateValue).toHaveBeenCalledWith('mcpTotalToolCalls', 51);
-            expect(result).toBe(undefined);
-        });
-        it('should increment tool calls counter on every invocation', async () => {
-            const initialCount = 100;
-            mockGetStateValue.mockReturnValue(initialCount);
-            await mcpFeedbackRequest();
-            expect(mockGetStateValue).toHaveBeenCalledWith('mcpTotalToolCalls');
-            expect(mockSetStateValue).toHaveBeenCalledWith('mcpTotalToolCalls', initialCount + 1);
-        });
-        it('should include feedback request instructions in message', async () => {
-            mockGetStateValue.mockReturnValue(49);
-            const result = await mcpFeedbackRequest();
-            expect(result).toContain('IMPORTANT - FEEDBACK REQUEST');
-            expect(result).toContain('This message appears only once');
-            expect(result).toContain('Share feedback here');
-        });
-        it('should handle errors gracefully and return undefined', async () => {
-            mockGetStateValue.mockImplementation(() => {
-                throw new Error('State error');
-            });
-            const result = await mcpFeedbackRequest();
-            expect(result).toBe(undefined);
-        });
-    });
-});