@hubspot/cli 8.0.8-experimental.5 → 8.0.8-experimental.7

package/commands/cms/theme/preview.js

@@ -1,16 +1,14 @@
 import fs from 'fs';
 import path from 'path';
-import cliProgress from 'cli-progress';
 import { commands } from '../../../lang/en.js';
 import { getCwd } from '@hubspot/local-dev-lib/path';
-import { FILE_UPLOAD_RESULT_TYPES } from '@hubspot/local-dev-lib/constants/files';
 import { getThemeJSONPath } from '@hubspot/local-dev-lib/cms/themes';
-import { createDevServer } from '@hubspot/cms-dev-server';
-import { getUploadableFileList } from '../../../lib/upload.js';
+// Subprocess approach - don't import cms-dev-server directly to avoid React version conflicts
+// import { createDevServer } from '@hubspot/cms-dev-server';
+import { spawnDevServer } from '../../../lib/cms/devServerProcess.js';
 import { trackCommandUsage } from '../../../lib/usageTracking.js';
 import { previewPrompt, previewProjectPrompt, } from '../../../lib/prompts/previewPrompt.js';
 import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
-import { ApiErrorContext, logError } from '../../../lib/errorHandlers/index.js';
 import { getProjectConfig } from '../../../lib/projects/config.js';
 import { findProjectComponents } from '../../../lib/projects/structure.js';
 import { ComponentTypes } from '../../../types/Projects.js';
@@ -73,67 +71,16 @@ async function determineSrcAndDest(args) {
 async function handler(args) {
     const { derivedAccountId, noSsl, resetSession, port, generateFieldsTypes } = args;
     const { absoluteSrc, dest } = await determineSrcAndDest(args);
-    const filePaths = await getUploadableFileList(absoluteSrc, false);
-    function startProgressBar(numFiles) {
-        const initialUploadProgressBar = new cliProgress.SingleBar({
-            gracefulExit: true,
-            format: '[{bar}] {percentage}% | {value}/{total} | {label}',
-            hideCursor: true,
-        }, cliProgress.Presets.rect);
-        initialUploadProgressBar.start(numFiles, 0, {
-            label: commands.cms.subcommands.theme.subcommands.preview
-                .initialUploadProgressBar.start,
-        });
-        let uploadsHaveStarted = false;
-        const uploadOptions = {
-            onAttemptCallback: () => {
-                /* Intentionally blank */
-            },
-            onSuccessCallback: () => {
-                initialUploadProgressBar.increment();
-                if (!uploadsHaveStarted) {
-                    uploadsHaveStarted = true;
-                    initialUploadProgressBar.update(0, {
-                        label: commands.cms.subcommands.theme.subcommands.preview
-                            .initialUploadProgressBar.uploading,
-                    });
-                }
-            },
-            onFirstErrorCallback: () => {
-                /* Intentionally blank */
-            },
-            onRetryCallback: () => {
-                /* Intentionally blank */
-            },
-            onFinalErrorCallback: () => initialUploadProgressBar.increment(),
-            onFinishCallback: (results) => {
-                initialUploadProgressBar.update(numFiles, {
-                    label: commands.cms.subcommands.theme.subcommands.preview
-                        .initialUploadProgressBar.finish,
-                });
-                initialUploadProgressBar.stop();
-                results.forEach(result => {
-                    if (result.resultType == FILE_UPLOAD_RESULT_TYPES.FAILURE) {
-                        uiLogger.error(commands.cms.subcommands.theme.subcommands.preview.errors.uploadFailed(result.file, dest));
-                        logError(result.error, new ApiErrorContext({
-                            accountId: derivedAccountId,
-                            request: dest,
-                            payload: result.file,
-                        }));
-                    }
-                });
-            },
-        };
-        return uploadOptions;
-    }
     trackCommandUsage('preview', {}, derivedAccountId);
-    if (port) {
-        process.env['PORT'] = port.toString();
-    }
-    createDevServer(absoluteSrc, false, '', '', !noSsl, generateFieldsTypes, {
-        filePaths,
+    // Spawn dev server as subprocess to isolate React versions
+    // Calls createDevServer as library function to support standalone themes
+    spawnDevServer({
+        absoluteSrc,
+        accountName: derivedAccountId?.toString(),
+        noSsl,
+        port,
+        generateFieldsTypes,
         resetSession: resetSession || false,
-        startProgressBar,
         dest,
     });
 }

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

@@ -7,20 +7,14 @@ 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();
+import startCommand from '../start.js';
 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');
@@ -42,7 +36,6 @@ describe('commands/mcp/start', () => {
         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', () => {

package/commands/mcp/start.js

@@ -28,6 +28,7 @@ async function startMcpServer(aiAgent) {
         uiLogger.debug(commands.mcp.start.startingServer);
         uiLogger.debug(commands.mcp.start.stopInstructions);
         const args = [serverPath];
+        // Start the server using ts-node
         const child = spawn(`node`, args, {
             stdio: 'inherit',
             env: {

package/lang/en.d.ts

@@ -43,11 +43,6 @@ export declare const commands: {
             checkOutConfig: (configPath: string) => string;
             pressEnterToInstall: (accountName: string) => string;
             pressKeyToExit: string;
-            installingApp: (appName: string, accountName: string) => string;
-            installInstructions: string;
-            browserFailedToOpen: (url: string) => string;
-            pollingTimeout: (minutes: number) => string;
-            pressEnterToContinueSetup: string;
             prompts: {
                 selectOptionV2: string;
                 options: {
@@ -1295,8 +1290,6 @@ export declare const commands: {
             prompts: {
                 targets: string;
                 targetsRequired: string;
-                standaloneMode: string;
-                cliVersion: string;
             };
         };
         start: {

package/lang/en.js

@@ -51,11 +51,6 @@ export const commands = {
             checkOutConfig: (configPath) => `Check out ${chalk.cyan(configPath)} for the full configuration.`,
             pressEnterToInstall: (accountName) => `? Press ${chalk.bold('<enter>')} to continue installing and previewing this app in ${chalk.bold(accountName)}`,
             pressKeyToExit: `Press any key to exit...`,
-            installingApp: (appName, accountName) => `Installing ${chalk.bold(appName)} in ${chalk.bold(accountName)}...`,
-            installInstructions: `We'll take you to your HubSpot account and walk you through installing your app.`,
-            browserFailedToOpen: (url) => `⚠️  Failed to open browser automatically. Please open this URL manually:\n${chalk.cyan(url)}`,
-            pollingTimeout: (minutes) => `⚠️  Installation polling timed out after ${minutes} minutes. The app may still be installing in the background.`,
-            pressEnterToContinueSetup: `Press ${chalk.bold('<enter>')} to continue with card setup...`,
             prompts: {
                 selectOptionV2: 'Choose a component type to get started',
                 options: {
@@ -1311,8 +1306,6 @@ export const commands = {
             prompts: {
                 targets: '[--client] Which tools would you like to add the HubSpot CLI MCP server to?',
                 targetsRequired: 'Must choose at least one app to configure.',
-                standaloneMode: 'Do you want to run in standalone mode? (This will use npx @hubspot/cli instead of the installed hs command)',
-                cliVersion: 'Specify a CLI version to pin (leave blank for latest):',
             },
         },
         start: {
@@ -3083,7 +3076,7 @@ export const lib = {
             updateSucceeded: (latestVersion) => `Successfully updated HubSpot CLI to version ${chalk.bold(latestVersion)}`,
             notInstalledGlobally: 'Cannot auto-update the HubSpot CLI because NPM is not installed globally',
             updateFailed: (latestVersion) => `Failed to update HubSpot CLI to version ${chalk.bold(latestVersion)}`,
-            enableAutoUpdatesMessage: `The HubSpot CLI can automatically keep itself up to date.\n\nThis helps ensure compatibility with the HubSpot platform. You can change this later at any time.\n\nRun ${uiCommandReference('hs config set --allow-auto-updates=true')}`,
+            enableAutoUpdatesMessage: `The HubSpot CLI can automatically keep itself up to date.\n\nThis helps ensure compatibility with the HubSpot platform. You can change this later at any time.\n\nRun${uiCommandReference('hs config set --allow-auto-updates=true')}`,
         },
     },
     projectProfiles: {

package/lib/__tests__/commandSuggestion.test.js

@@ -71,8 +71,6 @@ describe('lib/commandSuggestion', () => {
             // Create a mock yargs builder with strict method
             const mockYargsBuilder = {
                 strict: vi.fn().mockReturnThis(),
-                help: vi.fn().mockReturnThis(),
-                version: vi.fn().mockReturnThis(),
             };
             await commandModule.builder(mockYargsBuilder);
             expect(mockYargsBuilder.strict).toHaveBeenCalledWith(false);

package/lib/cms/devServerProcess.d.ts

@@ -0,0 +1,13 @@
+import { ChildProcess } from 'child_process';
+interface DevServerOptions {
+    absoluteSrc: string;
+    accountName?: string;
+    configPath?: string;
+    noSsl?: boolean;
+    port?: number;
+    generateFieldsTypes?: boolean;
+    resetSession?: boolean;
+    dest?: string;
+}
+export declare function spawnDevServer(options: DevServerOptions): ChildProcess;
+export {};

package/lib/cms/devServerProcess.js

@@ -0,0 +1,122 @@
+import { spawn } from 'child_process';
+export function spawnDevServer(options) {
+    const { absoluteSrc, accountName, noSsl, port, generateFieldsTypes, resetSession, dest, } = options;
+    // Build a Node script that imports and calls createDevServer
+    // This bypasses the CLI's project directory requirement while still isolating React versions
+    const script = `
+    const { createDevServer } = await import('@hubspot/cms-dev-server');
+    const { walk } = await import('@hubspot/local-dev-lib/fs');
+    const { createIgnoreFilter } = await import('@hubspot/local-dev-lib/ignoreRules');
+    const { isAllowedExtension } = await import('@hubspot/local-dev-lib/path');
+    const { FILE_UPLOAD_RESULT_TYPES } = await import('@hubspot/local-dev-lib/constants/files');
+    const cliProgress = (await import('cli-progress')).default;
+
+    ${dest
+        ? `
+    // Get uploadable files for preview (inlined from lib/upload.ts)
+    let filePaths = [];
+    try {
+      filePaths = await walk(${JSON.stringify(absoluteSrc)});
+    } catch (e) {
+      console.error('Error walking directory:', e);
+    }
+    filePaths = filePaths
+      .filter(file => isAllowedExtension(file))
+      .filter(createIgnoreFilter(false));
+
+    // Create progress bar for initial upload
+    function startProgressBar(numFiles) {
+      const initialUploadProgressBar = new cliProgress.SingleBar(
+        {
+          gracefulExit: true,
+          format: '[{bar}] {percentage}% | {value}/{total} | {label}',
+          hideCursor: true,
+        },
+        cliProgress.Presets.rect
+      );
+      initialUploadProgressBar.start(numFiles, 0, {
+        label: 'Preparing upload...',
+      });
+      let uploadsHaveStarted = false;
+      return {
+        onAttemptCallback: () => {},
+        onSuccessCallback: () => {
+          initialUploadProgressBar.increment();
+          if (!uploadsHaveStarted) {
+            uploadsHaveStarted = true;
+            initialUploadProgressBar.update(0, {
+              label: 'Uploading files...',
+            });
+          }
+        },
+        onFirstErrorCallback: () => {},
+        onRetryCallback: () => {},
+        onFinalErrorCallback: () => initialUploadProgressBar.increment(),
+        onFinishCallback: (results) => {
+          initialUploadProgressBar.update(numFiles, {
+            label: 'Upload complete',
+          });
+          initialUploadProgressBar.stop();
+          results.forEach(result => {
+            if (result.resultType == FILE_UPLOAD_RESULT_TYPES.FAILURE) {
+              console.error(\`Failed to upload \${result.file}\`);
+            }
+          });
+        },
+      };
+    }
+
+    const themePreviewOptions = {
+      filePaths,
+      startProgressBar,
+      resetSession: ${Boolean(resetSession)},
+      dest: ${JSON.stringify(dest)},
+    };
+    `
+        : 'const themePreviewOptions = undefined;'}
+
+    createDevServer(
+      ${JSON.stringify(absoluteSrc)},
+      false, // storybook
+      '', // config (uses env var HUBSPOT_CONFIG_PATH)
+      ${JSON.stringify(accountName || '')},
+      ${!noSsl}, // sslEnabled
+      ${Boolean(generateFieldsTypes)}, // fieldGenEnabled
+      themePreviewOptions
+    );
+  `;
+    // Set environment variables
+    const env = { ...process.env };
+    if (port) {
+        env.PORT = port.toString();
+    }
+    // Spawn Node with the inline script
+    const devServer = spawn('node', ['--input-type=module', '-e', script], {
+        stdio: 'inherit',
+        env,
+        cwd: absoluteSrc,
+    });
+    // Handle process events
+    devServer.on('error', error => {
+        console.error('Failed to start dev server:', error);
+        process.exit(1);
+    });
+    devServer.on('exit', (code, signal) => {
+        if (code !== 0 && code !== null) {
+            console.error(`Dev server exited with code ${code}`);
+            process.exit(code);
+        }
+        if (signal) {
+            console.error(`Dev server killed with signal ${signal}`);
+            process.exit(1);
+        }
+    });
+    // Handle CLI termination
+    process.on('SIGINT', () => {
+        devServer.kill('SIGINT');
+    });
+    process.on('SIGTERM', () => {
+        devServer.kill('SIGTERM');
+    });
+    return devServer;
+}

package/lib/commandSuggestion.js

@@ -18,6 +18,12 @@ export const commandSuggestionMappings = {
     'theme generate-selectors': 'hs cms theme generate-selectors',
     'theme marketplace-validate': 'hs cms theme marketplace-validate',
     'theme preview': 'hs cms theme preview',
+    'custom-object schema create': 'hs custom-object create-schema',
+    'custom-object schema delete': 'hs custom-object delete-schema',
+    'custom-object schema fetch-all': 'hs custom-object fetch-all-schemas',
+    'custom-object schema fetch': 'hs custom-object fetch-schema',
+    'custom-object schema list': 'hs custom-object list-schemas',
+    'custom-object schema update': 'hs custom-object update-schema',
 };
 function createCommandSuggestionHandler(newCommand) {
     return () => {
@@ -29,7 +35,7 @@ function createCommandSuggestion(oldCommand, newCommand) {
     return {
         command: oldCommand,
         builder: async (yargs) => {
-            return yargs.strict(false).help(false).version(false);
+            return yargs.strict(false);
         },
         handler: createCommandSuggestionHandler(newCommand),
     };

package/lib/getStartedV2Actions.d.ts

@@ -35,16 +35,3 @@ export declare function uploadAndDeployAction({ accountId, projectDest, }: {
     projectDest: string;
 }): Promise<UploadAndDeployResult>;
 export declare function trackGetStartedUsage(params: Record<string, unknown>, accountId: number): Promise<void>;
-export type PollAppInstallationOptions = {
-    accountId: number;
-    projectId: number;
-    appUid: string;
-    requiredScopes?: string[];
-    optionalScopes?: string[];
-    timeoutMs?: number;
-    intervalMs?: number;
-    onTimeout?: () => void;
-};
-export declare function pollAppInstallation({ accountId, projectId, appUid, requiredScopes, optionalScopes, timeoutMs, // 2 minutes
-intervalMs, // 2 seconds
-onTimeout, }: PollAppInstallationOptions): Promise<void>;

package/lib/getStartedV2Actions.js

@@ -1,7 +1,6 @@
 import fs from 'fs-extra';
 import path from 'path';
 import { fetchPublicAppsForPortal } from '@hubspot/local-dev-lib/api/appsDev';
-import { fetchAppInstallationData } from '@hubspot/local-dev-lib/api/localDevAuth';
 import { fetchProject } from '@hubspot/local-dev-lib/api/projects';
 import { getConfigAccountEnvironment } from '@hubspot/local-dev-lib/config';
 import { cloneGithubRepo } from '@hubspot/local-dev-lib/github';
@@ -145,55 +144,3 @@ export async function uploadAndDeployAction({ accountId, projectDest, }) {
 export function trackGetStartedUsage(params, accountId) {
     return trackCommandMetadataUsage('get-started', params, accountId);
 }
-export async function pollAppInstallation({ accountId, projectId, appUid, requiredScopes = [], optionalScopes = [], timeoutMs = 2 * 60 * 1000, // 2 minutes
-intervalMs = 2000, // 2 seconds
-onTimeout, }) {
-    return new Promise((resolve, reject) => {
-        let consecutiveErrors = 0;
-        const MAX_CONSECUTIVE_ERRORS = 5;
-        let pollInterval = null;
-        let pollTimeout = null;
-        const cleanup = () => {
-            if (pollInterval) {
-                clearTimeout(pollInterval);
-                pollInterval = null;
-            }
-            if (pollTimeout) {
-                clearTimeout(pollTimeout);
-                pollTimeout = null;
-            }
-        };
-        pollTimeout = setTimeout(() => {
-            cleanup();
-            if (onTimeout) {
-                onTimeout();
-            }
-            resolve(); // Resolve instead of reject to allow continuing with timeout state
-        }, timeoutMs);
-        const poll = async () => {
-            try {
-                const { data } = await fetchAppInstallationData(accountId, projectId, appUid, requiredScopes, optionalScopes);
-                // Reset error counter on successful fetch
-                consecutiveErrors = 0;
-                if (data.isInstalledWithScopeGroups) {
-                    cleanup();
-                    resolve();
-                }
-                else if (pollInterval) {
-                    pollInterval = setTimeout(poll, intervalMs);
-                }
-            }
-            catch (error) {
-                consecutiveErrors++;
-                if (consecutiveErrors >= MAX_CONSECUTIVE_ERRORS) {
-                    cleanup();
-                    reject(new Error(`Failed to check app installation status after ${MAX_CONSECUTIVE_ERRORS} consecutive errors`, { cause: error }));
-                }
-                else if (pollInterval !== null) {
-                    pollInterval = setTimeout(poll, intervalMs);
-                }
-            }
-        };
-        pollInterval = setTimeout(poll, 0);
-    });
-}

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

@@ -132,21 +132,6 @@ describe('lib/mcp/setup', () => {
             });
             expect(mockedLogError).toHaveBeenCalledWith(error);
         });
-        it('should pass through environment variables in command', async () => {
-            const mockMcpCommandWithEnv = {
-                command: 'test-command',
-                args: ['--arg1'],
-                env: { HUBSPOT_MCP_STANDALONE: 'true' },
-            };
-            mockedExecAsync.mockResolvedValueOnce({
-                stdout: 'codex version 1.0.0',
-                stderr: '',
-            });
-            mockedExecAsync.mockResolvedValueOnce({ stdout: '', stderr: '' });
-            const result = await setupCodex(mockMcpCommandWithEnv);
-            expect(result).toBe(true);
-            expect(mockedExecAsync).toHaveBeenCalledWith('codex mcp add "HubSpotDev" --env HUBSPOT_MCP_STANDALONE=true -- test-command --arg1 --ai-agent codex');
-        });
     });
     describe('setupGemini', () => {
         const mockMcpCommand = {

package/lib/mcp/setup.d.ts

@@ -5,7 +5,6 @@ export declare const supportedTools: {
 interface McpCommand {
     command: string;
     args: string[];
-    env?: Record<string, string>;
 }
 export declare function addMcpServerToConfig(targets: string[] | undefined): Promise<string[]>;
 export declare function setupVsCode(mcpCommand?: McpCommand): Promise<boolean>;

package/lib/mcp/setup.js

@@ -47,56 +47,23 @@ export async function addMcpServerToConfig(targets) {
         else {
             derivedTargets = targets;
         }
-        // Prompt for standalone mode
-        const { useStandaloneMode } = await promptUser({
-            name: 'useStandaloneMode',
-            type: 'confirm',
-            message: commands.mcp.setup.prompts.standaloneMode,
-            default: false,
-        });
-        const { cliVersion } = useStandaloneMode
-            ? await promptUser({
-                name: 'cliVersion',
-                type: 'input',
-                message: commands.mcp.setup.prompts.cliVersion,
-                validate: (v) => !v || /^[\d]+\.[\d]+\.[\d]+([-+][\w.]+)?$/.test(v.trim())
-                    ? true
-                    : 'Please enter a valid semver version (e.g. 8.0.1) or leave blank for latest.',
-            })
-            : { cliVersion: '' };
-        const cliPackage = cliVersion
-            ? `@hubspot/cli@${cliVersion}`
-            : '@hubspot/cli';
-        const standaloneEnv = {
-            HUBSPOT_MCP_STANDALONE: 'true',
-        };
-        if (cliVersion) {
-            standaloneEnv.HUBSPOT_CLI_VERSION = cliVersion;
-        }
-        const mcpCommand = useStandaloneMode
-            ? {
-                command: 'npx',
-                args: ['-y', '-p', cliPackage, 'hs', 'mcp', 'start'],
-                env: standaloneEnv,
-            }
-            : defaultMcpCommand;
         if (derivedTargets.includes(claudeCode)) {
-            await runSetupFunction(() => setupClaudeCode(mcpCommand));
+            await runSetupFunction(setupClaudeCode);
         }
         if (derivedTargets.includes(cursor)) {
-            await runSetupFunction(() => setupCursor(mcpCommand));
+            await runSetupFunction(setupCursor);
         }
         if (derivedTargets.includes(windsurf)) {
-            await runSetupFunction(() => setupWindsurf(mcpCommand));
+            await runSetupFunction(setupWindsurf);
         }
         if (derivedTargets.includes(vscode)) {
-            await runSetupFunction(() => setupVsCode(mcpCommand));
+            await runSetupFunction(setupVsCode);
         }
         if (derivedTargets.includes(codex)) {
-            await runSetupFunction(() => setupCodex(mcpCommand));
+            await runSetupFunction(setupCodex);
         }
         if (derivedTargets.includes(gemini)) {
-            await runSetupFunction(() => setupGemini(mcpCommand));
+            await runSetupFunction(setupGemini);
         }
         uiLogger.info(commands.mcp.setup.success(derivedTargets));
         return derivedTargets;
@@ -154,7 +121,10 @@ function setupMcpConfigFile(config) {
         if (!mcpConfig.mcpServers) {
             mcpConfig.mcpServers = {};
         }
-        mcpConfig.mcpServers[mcpServerName] = config.mcpCommand;
+        // Add or update HubSpot CLI MCP server
+        mcpConfig.mcpServers[mcpServerName] = {
+            ...config.mcpCommand,
+        };
         // Write the updated config
         fs.writeFileSync(config.configPath, JSON.stringify(mcpConfig, null, 2));
         SpinniesManager.succeed('spinner', {
@@ -175,12 +145,10 @@ export async function setupVsCode(mcpCommand = defaultMcpCommand) {
         SpinniesManager.add('vsCode', {
             text: commands.mcp.setup.spinners.configuringVsCode,
         });
-        const commandWithAgent = buildCommandWithAgentString(mcpCommand, vscode);
-        const configObject = {
+        const mcpConfig = JSON.stringify({
             name: mcpServerName,
-            ...commandWithAgent,
-        };
-        const mcpConfig = JSON.stringify(configObject);
+            ...buildCommandWithAgentString(mcpCommand, vscode),
+        });
         await execAsync(`code --add-mcp ${JSON.stringify(mcpConfig)}`);
         SpinniesManager.succeed('vsCode', {
             text: commands.mcp.setup.spinners.configuredVsCode,
@@ -204,27 +172,25 @@ export async function setupVsCode(mcpCommand = defaultMcpCommand) {
     }
 }
 export async function setupClaudeCode(mcpCommand = defaultMcpCommand) {
-    SpinniesManager.add('claudeCode', {
-        text: commands.mcp.setup.spinners.configuringClaudeCode,
-    });
     try {
-        // Check if claude command is available
-        await execAsync('claude --version');
-    }
-    catch (e) {
-        SpinniesManager.fail('claudeCode', {
-            text: commands.mcp.setup.spinners.claudeCodeNotFound,
+        SpinniesManager.add('claudeCode', {
+            text: commands.mcp.setup.spinners.configuringClaudeCode,
         });
-        return false;
-    }
-    try {
+        try {
+            // Check if claude command is available
+            await execAsync('claude --version');
+        }
+        catch (e) {
+            SpinniesManager.fail('claudeCode', {
+                text: commands.mcp.setup.spinners.claudeCodeNotFound,
+            });
+            return false;
+        }
         // Run claude mcp add command
-        const commandWithAgent = buildCommandWithAgentString(mcpCommand, claudeCode);
-        const configObject = {
+        const mcpConfig = JSON.stringify({
             type: 'stdio',
-            ...commandWithAgent,
-        };
-        const mcpConfig = JSON.stringify(configObject);
+            ...buildCommandWithAgentString(mcpCommand, claudeCode),
+        });
         const { stdout } = await execAsync('claude mcp list');
         if (stdout.includes(mcpServerName)) {
             SpinniesManager.update('claudeCode', {
@@ -282,7 +248,7 @@ export async function setupCodex(mcpCommand = defaultMcpCommand) {
             return false;
         }
         const mcpCommandWithAgent = buildCommandWithAgentString(mcpCommand, codex);
-        await execAsync(`codex mcp add "${mcpServerName}"${buildEnvFlagString(mcpCommand)} -- ${mcpCommandWithAgent.command} ${mcpCommandWithAgent.args.join(' ')}`);
+        await execAsync(`codex mcp add "${mcpServerName}" -- ${mcpCommandWithAgent.command} ${mcpCommandWithAgent.args.join(' ')}`);
         SpinniesManager.succeed('codexSpinner', {
             text: commands.mcp.setup.spinners.configuredCodex,
         });
@@ -311,7 +277,7 @@ export async function setupGemini(mcpCommand = defaultMcpCommand) {
             return false;
         }
         const mcpCommandWithAgent = buildCommandWithAgentString(mcpCommand, gemini);
-        await execAsync(`gemini mcp add -s user${buildEnvFlagString(mcpCommand)} "${mcpServerName}" ${mcpCommandWithAgent.command} ${mcpCommandWithAgent.args.join(' ')}`);
+        await execAsync(`gemini mcp add -s user "${mcpServerName}" ${mcpCommandWithAgent.command} ${mcpCommandWithAgent.args.join(' ')}`);
         SpinniesManager.succeed('geminiSpinner', {
             text: commands.mcp.setup.spinners.configuredGemini,
         });
@@ -330,16 +296,3 @@ function buildCommandWithAgentString(mcpCommand, agent) {
     mcpCommandCopy.args.push('--ai-agent', agent);
     return mcpCommandCopy;
 }
-function buildEnvFlagString(mcpCommand) {
-    const envFlags = [];
-    if (mcpCommand.env) {
-        const env = Object.entries(mcpCommand.env);
-        env.forEach(([key, value]) => {
-            envFlags.push(`--env ${key}=${value}`);
-        });
-    }
-    if (envFlags.length === 0) {
-        return '';
-    }
-    return ` ${envFlags.join(' ')}`;
-}

package/mcp-server/tools/cms/HsCreateFunctionTool.js

@@ -1,7 +1,7 @@
 import { Tool } from '../../types.js';
 import { z } from 'zod';
 import { absoluteCurrentWorkingDirectory } from '../project/constants.js';
-import { runCommandInDir } from '../../utils/command.js';
+import { runCommandInDir } from '../../utils/project.js';
 import { formatTextContents, formatTextContent } from '../../utils/content.js';
 import { trackToolUsage } from '../../utils/toolUsageTracking.js';
 import { addFlag } from '../../utils/command.js';

package/mcp-server/tools/cms/HsCreateModuleTool.js

@@ -1,7 +1,7 @@
 import { Tool } from '../../types.js';
 import { z } from 'zod';
 import { absoluteCurrentWorkingDirectory } from '../project/constants.js';
-import { runCommandInDir } from '../../utils/command.js';
+import { runCommandInDir } from '../../utils/project.js';
 import { formatTextContents, formatTextContent } from '../../utils/content.js';
 import { trackToolUsage } from '../../utils/toolUsageTracking.js';
 import { addFlag } from '../../utils/command.js';