snow-flow 2.8.7 → 2.8.9

package/dist/cli.js

@@ -417,6 +417,36 @@ async function executeClaudeCode(prompt) {
         // Check for MCP config
         const mcpConfigPath = (0, path_1.join)(process.cwd(), '.mcp.json');
         const hasMcpConfig = (0, fs_2.existsSync)(mcpConfigPath);
+        // Auto-start MCP servers if they're not running
+        if (hasMcpConfig) {
+            cliLogger.info('🔧 Checking MCP server status...');
+            try {
+                const { MCPServerManager } = await Promise.resolve().then(() => __importStar(require('./utils/mcp-server-manager.js')));
+                const manager = new MCPServerManager();
+                await manager.initialize();
+                const systemStatus = manager.getSystemStatus();
+                if (systemStatus.running === 0) {
+                    cliLogger.info('🚀 Starting MCP servers automatically for swarm operation...');
+                    await manager.startAllServers();
+                    const newStatus = manager.getSystemStatus();
+                    cliLogger.info(`✅ Started ${newStatus.running}/${newStatus.total} MCP servers`);
+                }
+                else if (systemStatus.running < systemStatus.total) {
+                    cliLogger.info(`⚠️  Only ${systemStatus.running}/${systemStatus.total} MCP servers running`);
+                    cliLogger.info('🔄 Starting remaining servers...');
+                    await manager.startAllServers();
+                    const newStatus = manager.getSystemStatus();
+                    cliLogger.info(`✅ All ${newStatus.running}/${newStatus.total} MCP servers running`);
+                }
+                else {
+                    cliLogger.info(`✅ All ${systemStatus.running} MCP servers already running`);
+                }
+            }
+            catch (error) {
+                cliLogger.warn('⚠️  Could not auto-start MCP servers:', error instanceof Error ? error.message : error);
+                cliLogger.info('💡 You may need to run "npm run mcp:start" manually');
+            }
+        }
         // Launch Claude Code with MCP config and skip permissions to avoid raw mode issues
         const claudeArgs = hasMcpConfig
             ? ['--mcp-config', '.mcp.json', '.', '--dangerously-skip-permissions']
@@ -2365,7 +2395,7 @@ program
                 await manager.initialize();
                 console.log('📡 Starting all ServiceNow MCP servers...');
                 await manager.startAllServers();
-                const status = manager.getServerStatus();
+                const status = manager.getServerList();
                 const running = status.filter((s) => s.status === 'running').length;
                 const total = status.length;
                 console.log(chalk_1.default.green(`✅ Started ${running}/${total} MCP servers successfully!`));
@@ -4058,7 +4088,7 @@ async function handleMCPStart(manager, options) {
     else {
         console.log('📡 Starting all configured MCP servers...');
         await manager.startAllServers();
-        const status = manager.getServerStatus();
+        const status = manager.getServerList();
         const running = status.filter((s) => s.status === 'running').length;
         const total = status.length;
         console.log(`\n✅ Started ${running}/${total} MCP servers`);
@@ -4110,12 +4140,12 @@ async function handleMCPRestart(manager, options) {
         await manager.stopAllServers();
         await manager.startAllServers();
         const running = manager.getRunningServersCount();
-        const total = manager.getServerStatus().length;
+        const total = manager.getServerList().length;
         console.log(`✅ Restarted ${running}/${total} MCP servers`);
     }
 }
 async function handleMCPStatus(manager, options) {
-    const servers = manager.getServerStatus();
+    const servers = manager.getServerList();
     console.log('\n📊 MCP Server Status');
     console.log('═'.repeat(80));
     if (servers.length === 0) {
@@ -4184,7 +4214,7 @@ async function handleMCPLogs(manager, options) {
     }
 }
 async function handleMCPList(manager, options) {
-    const servers = manager.getServerStatus();
+    const servers = manager.getServerList();
     console.log('\n📋 Configured MCP Servers');
     console.log('═'.repeat(80));
     if (servers.length === 0) {

package/dist/health/system-health.js

@@ -254,7 +254,7 @@ class SystemHealth extends events_1.EventEmitter {
     async checkMCP() {
         const startTime = Date.now();
         try {
-            const serverStatuses = this.mcpManager.getServerStatus();
+            const serverStatuses = this.mcpManager.getServerList();
             const totalServers = serverStatuses.length;
             const healthyServers = serverStatuses.filter(s => s.status === 'running').length;
             const unhealthyServers = serverStatuses.filter(s => s.status === 'error').length;

package/dist/mcp/servicenow-deployment-mcp.js

@@ -643,9 +643,15 @@ class ServiceNowDeploymentMCP {
                         has_preview: true,
                         category: args.category || 'custom',
                     });
-                    deploymentMethod = 'direct_api';
-                    deploymentSuccess = true;
-                    this.logger.info('✅ Direct deployment successful');
+                    // ONLY set success if the operation actually succeeded
+                    if (result?.success) {
+                        deploymentMethod = 'direct_api';
+                        deploymentSuccess = true;
+                        this.logger.info('✅ Direct deployment successful');
+                    }
+                    else {
+                        throw new Error(`Widget creation failed: ${result?.error || 'Unknown error'}`);
+                    }
                 }
                 catch (error) {
                     directError = error;
@@ -706,8 +712,15 @@ class ServiceNowDeploymentMCP {
                         roles: '',
                         servicenow: false
                     });
-                    deploymentMethod = 'table_record';
-                    deploymentSuccess = true;
+                    // ONLY set success if the operation actually succeeded
+                    if (result?.success) {
+                        deploymentMethod = 'table_record';
+                        deploymentSuccess = true;
+                        this.logger.info('✅ Fallback deployment successful');
+                    }
+                    else {
+                        throw new Error(`Table record creation failed: ${result?.error || 'Unknown error'}`);
+                    }
                     // createRecord should already return a ServiceNowAPIResponse structure
                     // No need to wrap it again
                 }

package/dist/mcp/start-all-mcp-servers.d.ts

@@ -1,6 +1,11 @@
 #!/usr/bin/env node
 /**
- * Start all ServiceNow MCP servers (WITH SINGLETON PROTECTION)
+ * ⚠️ DEPRECATED: Legacy MCP Server Starter
+ *
+ * This script is DEPRECATED and will be removed in v3.0.0
+ * Use MCPServerManager instead for proper server management.
+ *
+ * Migration: Use scripts/start-mcp-proper.js or MCPServerManager directly
  */
 export {};
 //# sourceMappingURL=start-all-mcp-servers.d.ts.map

package/dist/mcp/start-all-mcp-servers.js

@@ -1,93 +1,74 @@
 #!/usr/bin/env node
 "use strict";
 /**
- * Start all ServiceNow MCP servers (WITH SINGLETON PROTECTION)
+ * ⚠️ DEPRECATED: Legacy MCP Server Starter
+ *
+ * This script is DEPRECATED and will be removed in v3.0.0
+ * Use MCPServerManager instead for proper server management.
+ *
+ * Migration: Use scripts/start-mcp-proper.js or MCPServerManager directly
  */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
-const child_process_1 = require("child_process");
 const logger_js_1 = require("../utils/logger.js");
-const mcp_singleton_lock_js_1 = require("../utils/mcp-singleton-lock.js");
-const logger = new logger_js_1.Logger('MCPServerLauncher');
+const logger = new logger_js_1.Logger('DeprecatedMCPLauncher');
 async function startAllServers() {
-    // 🔒 SINGLETON CHECK - Prevent duplicate instances
-    const singletonLock = (0, mcp_singleton_lock_js_1.getMCPSingletonLock)();
-    if (!singletonLock.acquire()) {
-        logger.error('❌ MCP servers already running. Exiting.');
+    logger.warn('⚠️ DEPRECATED: This start-all-mcp-servers.ts script is deprecated!');
+    logger.warn('   Please use MCPServerManager or scripts/start-mcp-proper.js instead');
+    logger.warn('   This provides proper process management and singleton protection');
+    console.log('\n🔄 Redirecting to proper MCPServerManager...\n');
+    try {
+        // Redirect to proper approach
+        const { MCPServerManager } = await Promise.resolve().then(() => __importStar(require('../utils/mcp-server-manager.js')));
+        const manager = new MCPServerManager();
+        await manager.initialize();
+        await manager.startAllServers();
+        logger.info('✅ Servers started using proper MCPServerManager');
+    }
+    catch (error) {
+        logger.error('❌ Failed to start servers:', error.message);
+        logger.info('💡 Consider migrating to: scripts/start-mcp-proper.js');
         process.exit(1);
     }
-    logger.info('✅ Starting all ServiceNow MCP servers (singleton protected)...');
-    const servers = [
-        {
-            name: 'ServiceNow MCP Server',
-            script: 'dist/mcp/servicenow-mcp-server.js'
-        },
-        {
-            name: 'ServiceNow Deployment MCP',
-            script: 'dist/mcp/servicenow-deployment-mcp.js'
-        },
-        {
-            name: 'ServiceNow Intelligent MCP',
-            script: 'dist/mcp/servicenow-intelligent-mcp.js'
-        },
-        {
-            name: 'ServiceNow Update Set MCP',
-            script: 'dist/mcp/servicenow-update-set-mcp.js'
-        },
-        {
-            name: 'ServiceNow Advanced Features MCP',
-            script: 'dist/mcp/advanced/servicenow-advanced-features-mcp.js'
-        },
-        {
-            name: 'ServiceNow Operations MCP',
-            script: 'dist/mcp/servicenow-operations-mcp.js'
-        },
-        {
-            name: 'ServiceNow Reporting Analytics MCP',
-            script: 'dist/mcp/servicenow-reporting-analytics-mcp.js'
-        },
-        {
-            name: 'ServiceNow Security Compliance MCP',
-            script: 'dist/mcp/servicenow-security-compliance-mcp.js'
-        },
-        {
-            name: 'ServiceNow Platform Development MCP',
-            script: 'dist/mcp/servicenow-platform-development-mcp.js'
-        },
-        {
-            name: 'ServiceNow Automation MCP',
-            script: 'dist/mcp/servicenow-automation-mcp.js'
-        },
-        {
-            name: 'ServiceNow Integration MCP',
-            script: 'dist/mcp/servicenow-integration-mcp.js'
-        },
-        {
-            name: 'Snow-Flow MCP',
-            script: 'dist/mcp/snow-flow-mcp.js'
-        }
-    ];
-    const processes = servers.map(server => {
-        logger.info(`Starting ${server.name}...`);
-        const proc = (0, child_process_1.spawn)('node', [server.script], {
-            stdio: 'inherit',
-            env: { ...process.env }
-        });
-        proc.on('error', (error) => {
-            logger.error(`Failed to start ${server.name}:`, error);
-        });
-        proc.on('exit', (code) => {
-            logger.info(`${server.name} exited with code ${code}`);
-        });
-        return proc;
-    });
-    // Handle graceful shutdown
+    // If we get here, MCPServerManager started successfully
+    logger.info('🎉 Migration successful! Using proper MCPServerManager now.');
+    // Keep process alive to monitor servers (handled by MCPServerManager)
     process.on('SIGINT', () => {
-        logger.info('Shutting down all MCP servers...');
-        processes.forEach(proc => proc.kill());
+        logger.info('👋 Shutting down...');
         process.exit(0);
     });
-    logger.info('All MCP servers started successfully!');
-    logger.info('Press Ctrl+C to stop all servers');
 }
 startAllServers().catch(error => {
     logger.error('Failed to start MCP servers:', error);

package/dist/utils/mcp-server-manager.d.ts

@@ -59,10 +59,6 @@ export declare class MCPServerManager extends EventEmitter {
      * Stop all running MCP servers
      */
     stopAllServers(): Promise<void>;
-    /**
-     * Get status of all servers
-     */
-    getServerStatus(): MCPServer[];
     /**
      * Get status of a specific server
      */
@@ -83,6 +79,25 @@ export declare class MCPServerManager extends EventEmitter {
      * Remove a server configuration
      */
     removeServer(name: string): Promise<void>;
+    /**
+     * Get list of all servers (for monitoring/status display)
+     */
+    getServerList(): MCPServer[];
+    /**
+     * Get server status by name
+     */
+    getServerStatus(name: string): MCPServer | undefined;
+    /**
+     * Get overall system status
+     */
+    getSystemStatus(): {
+        total: number;
+        running: number;
+        stopped: number;
+        starting: number;
+        error: number;
+        health: string;
+    };
     /**
      * Cleanup - stop all servers and clean up resources
      */

package/dist/utils/mcp-server-manager.js

@@ -304,12 +304,6 @@ class MCPServerManager extends events_1.EventEmitter {
         }));
         await Promise.all(promises);
     }
-    /**
-     * Get status of all servers
-     */
-    getServerStatus() {
-        return Array.from(this.servers.values());
-    }
     /**
      * Get status of a specific server
      */
@@ -367,6 +361,34 @@ class MCPServerManager extends events_1.EventEmitter {
         }));
         await this.saveConfiguration(configs);
     }
+    /**
+     * Get list of all servers (for monitoring/status display)
+     */
+    getServerList() {
+        return Array.from(this.servers.values());
+    }
+    /**
+     * Get server status by name
+     */
+    getServerStatus(name) {
+        return this.servers.get(name);
+    }
+    /**
+     * Get overall system status
+     */
+    getSystemStatus() {
+        const servers = this.getServerList();
+        const running = servers.filter(s => s.status === 'running').length;
+        const total = servers.length;
+        return {
+            total,
+            running,
+            stopped: servers.filter(s => s.status === 'stopped').length,
+            starting: servers.filter(s => s.status === 'starting').length,
+            error: servers.filter(s => s.status === 'error').length,
+            health: running === total ? 'healthy' : running > 0 ? 'partial' : 'down'
+        };
+    }
     /**
      * Cleanup - stop all servers and clean up resources
      */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "snow-flow",
-  "version": "2.8.7",
+  "version": "2.8.9",
   "description": "Snow-Flow: ServiceNow Advanced Intelligence Platform - 100+ real MCP tools with AI-powered swarm orchestration and neural networks. Dynamic task categorization using AI. Machine learning for incident classification, change risk prediction, and anomaly detection. Zero Mock Data, 100% Real API Integration.",
   "main": "dist/index.js",
   "type": "commonjs",
@@ -23,6 +23,8 @@
     "reset-mcp:restart": "node scripts/reset-mcp-servers.js --restart",
     "cleanup-mcp": "node scripts/cleanup-mcp-servers.js",
     "mcp:clean": "node scripts/cleanup-mcp-servers.js && npm run build",
+    "mcp:start": "node scripts/start-mcp-proper.js",
+    "mcp:start-proper": "node scripts/start-mcp-proper.js",
     "postbuild-disabled": "npm run setup-mcp",
     "postinstall": "node scripts/postinstall.js",
     "version": "node scripts/update-version.js && npm run build && git add src/version.ts",