moflo 4.6.1 → 4.6.3

package/src/@claude-flow/cli/dist/src/init/mcp-generator.d.ts

@@ -1,7 +1,11 @@
 /**
  * MCP Configuration Generator
  * Creates .mcp.json for Claude Code MCP server integration
- * Handles cross-platform compatibility (Windows requires cmd /c wrapper)
+ *
+ * Note: Claude Code spawns MCP servers as child processes using the
+ * command/args from .mcp.json. On all platforms (including Windows),
+ * using `npx` directly works correctly. The previous `cmd /c` wrapper
+ * on Windows caused MCP servers to fail to start.
  */
 import type { InitOptions } from './types.js';
 /**

package/src/@claude-flow/cli/dist/src/init/mcp-generator.js

@@ -1,29 +1,18 @@
 /**
  * MCP Configuration Generator
  * Creates .mcp.json for Claude Code MCP server integration
- * Handles cross-platform compatibility (Windows requires cmd /c wrapper)
+ *
+ * Note: Claude Code spawns MCP servers as child processes using the
+ * command/args from .mcp.json. On all platforms (including Windows),
+ * using `npx` directly works correctly. The previous `cmd /c` wrapper
+ * on Windows caused MCP servers to fail to start.
  */
 /**
- * Check if running on Windows
- */
-function isWindows() {
-    return process.platform === 'win32';
-}
-/**
- * Generate platform-specific MCP server entry
- * - Windows: uses 'cmd /c npx' directly
- * - Unix: uses 'npx' directly (simple, reliable)
+ * Generate MCP server entry
+ * Uses `npx` directly on all platforms — Claude Code handles process
+ * spawning correctly without needing a cmd.exe wrapper.
  */
 function createMCPServerEntry(npxArgs, env, additionalProps = {}) {
-    if (isWindows()) {
-        return {
-            command: 'cmd',
-            args: ['/c', 'npx', '-y', ...npxArgs],
-            env,
-            ...additionalProps,
-        };
-    }
-    // Unix: direct npx invocation — simple and reliable
     return {
         command: 'npx',
         args: ['-y', ...npxArgs],
@@ -74,27 +63,14 @@ export function generateMCPJson(options) {
 export function generateMCPCommands(options) {
     const commands = [];
     const config = options.mcp;
-    if (isWindows()) {
-        if (config.claudeFlow) {
-            commands.push('claude mcp add claude-flow -- cmd /c npx -y @claude-flow/cli@latest mcp start');
-        }
-        if (config.ruvSwarm) {
-            commands.push('claude mcp add ruv-swarm -- cmd /c npx -y ruv-swarm mcp start');
-        }
-        if (config.flowNexus) {
-            commands.push('claude mcp add flow-nexus -- cmd /c npx -y flow-nexus@latest mcp start');
-        }
+    if (config.claudeFlow) {
+        commands.push('claude mcp add claude-flow -- npx -y @claude-flow/cli@latest mcp start');
     }
-    else {
-        if (config.claudeFlow) {
-            commands.push("claude mcp add claude-flow -- npx -y @claude-flow/cli@latest mcp start");
-        }
-        if (config.ruvSwarm) {
-            commands.push("claude mcp add ruv-swarm -- npx -y ruv-swarm mcp start");
-        }
-        if (config.flowNexus) {
-            commands.push("claude mcp add flow-nexus -- npx -y flow-nexus@latest mcp start");
-        }
+    if (config.ruvSwarm) {
+        commands.push('claude mcp add ruv-swarm -- npx -y ruv-swarm mcp start');
+    }
+    if (config.flowNexus) {
+        commands.push('claude mcp add flow-nexus -- npx -y flow-nexus@latest mcp start');
     }
     return commands;
 }
@@ -102,14 +78,11 @@ export function generateMCPCommands(options) {
  * Get platform-specific setup instructions
  */
 export function getPlatformInstructions() {
-    if (isWindows()) {
-        return {
-            platform: 'Windows',
-            note: 'MCP configuration uses cmd /c wrapper for npx compatibility.',
-        };
-    }
+    const platform = process.platform === 'win32'
+        ? 'Windows'
+        : process.platform === 'darwin' ? 'macOS' : 'Linux';
     return {
-        platform: process.platform === 'darwin' ? 'macOS' : 'Linux',
+        platform,
         note: 'MCP configuration uses npx directly.',
     };
 }