mlgym-deploy 3.3.29 → 3.3.32

package/index.js

@@ -18,7 +18,7 @@ import crypto from 'crypto';
 const execAsync = promisify(exec);
 
 // Current version of this MCP server - INCREMENT FOR WORKFLOW FIXES
-const CURRENT_VERSION = '3.3.29';  // Fix SSH key per-user when using MLGYM_TOKEN env var
+const CURRENT_VERSION = '3.3.32';  // Fix misleading hostname description - URL is always {app_id}.eu{1,2,3}.ezb.net
 const PACKAGE_NAME = 'mlgym-deploy';
 
 // Debug logging configuration - ENABLED BY DEFAULT
@@ -1225,10 +1225,17 @@ async function detectDeploymentStrategy(projectPath) {
     log.info('MCP >>> [detectDeploymentStrategy] Both files found, analyzing compose...');
 
     try {
-      const { webServiceCount, usesLocalDockerfile, totalServiceCount } = await analyzeComposeFile(composePath);
+      const { webServiceCount, usesLocalDockerfile, totalServiceCount, slowStartServices } = await analyzeComposeFile(composePath);
 
       log.info(`MCP >>> [analyzeCompose] Total services: ${totalServiceCount}, Web services: ${webServiceCount}, Uses local Dockerfile: ${usesLocalDockerfile}`);
 
+      // Warn about slow-starting services
+      if (slowStartServices && slowStartServices.length > 0) {
+        log.warning(`⚠ WARNING: Detected services with slow startup: ${slowStartServices.join(', ')}`);
+        log.warning(`⏱ These services (Cassandra, MongoDB, Elasticsearch, etc.) may take 2-5 minutes to become healthy.`);
+        log.warning(`   Your application will not be accessible until ALL services are running.`);
+      }
+
       // SIMPLE: Only 1 web service, no other services, uses local Dockerfile
       // → docker-compose is just convenience wrapper
       if (totalServiceCount === 1 && webServiceCount === 1 && usesLocalDockerfile) {
@@ -1240,7 +1247,7 @@ async function detectDeploymentStrategy(projectPath) {
       // COMPLEX: Multiple services (web + database, etc.)
       // → MUST use docker-compose for orchestration
       log.info('MCP >>> [detectDeploymentStrategy] Strategy: dockercompose (multi-service)');
-      return { type: 'dockercompose', reason: 'Application requires multiple services (web + database/cache/etc)' };
+      return { type: 'dockercompose', reason: 'Application requires multiple services (web + database/cache/etc)', slowStartServices };
     } catch (err) {
       log.error('MCP >>> [detectDeploymentStrategy] Analysis failed, defaulting to dockercompose:', err.message);
       return { type: 'dockercompose', reason: 'Multiple deployment files found' };
@@ -1531,6 +1538,7 @@ async function analyzeComposeFile(composePath) {
   let inServicesSection = false;
   let currentService = '';
   let currentServiceIsWeb = false;
+  let slowStartServices = []; // Track services with known slow startup
 
   for (const line of lines) {
     const trimmed = line.trim();
@@ -1567,6 +1575,15 @@ async function analyzeComposeFile(composePath) {
 
       currentService = trimmed.split(':')[0].trim();
       currentServiceIsWeb = false;
+
+      // Check for services with known slow startup times
+      const serviceLower = currentService.toLowerCase();
+      if (serviceLower.includes('cassandra') || serviceLower.includes('elasticsearch') ||
+          serviceLower.includes('mongodb') || serviceLower.includes('kafka') ||
+          serviceLower.includes('zookeeper') || serviceLower.includes('spark')) {
+        slowStartServices.push(currentService);
+      }
+
       continue;
     }
 
@@ -1579,6 +1596,18 @@ async function analyzeComposeFile(composePath) {
     if (trimmed === 'build: .' || trimmed === 'build: ./') {
       usesLocalDockerfile = true;
     }
+
+    // Also detect slow-start services by image name
+    if (trimmed.startsWith('image:')) {
+      const imageName = trimmed.substring(6).trim().toLowerCase();
+      if (imageName.includes('cassandra') || imageName.includes('elasticsearch') ||
+          imageName.includes('mongodb') || imageName.includes('kafka') ||
+          imageName.includes('zookeeper') || imageName.includes('spark')) {
+        if (!slowStartServices.includes(currentService)) {
+          slowStartServices.push(currentService);
+        }
+      }
+    }
   }
 
   // Count last service
@@ -1589,7 +1618,7 @@ async function analyzeComposeFile(composePath) {
     }
   }
 
-  return { webServiceCount, usesLocalDockerfile, totalServiceCount };
+  return { webServiceCount, usesLocalDockerfile, totalServiceCount, slowStartServices };
 }
 
 // Helper to check if file exists
@@ -1934,7 +1963,9 @@ async function initProject(args) {
         if (status === 'deployed' || status === 'running') {
           deploymentStatus = {
             status: 'deployed',
-            url: statusResult.data.url || `https://${hostname}.ezb.net`,
+            // Use deployment_url from backend response, NOT a fallback hostname pattern
+            // The correct format is: https://{coolify_app_id}.{eu1|eu2|eu3}.ezb.net
+            url: statusResult.data.url || project.deployment_url || null,
             message: 'Application successfully deployed'
           };
           break;
@@ -3668,7 +3699,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             },
             hostname: {
               type: 'string',
-              description: 'Deployment hostname/subdomain (optional, defaults to project_name). Will be accessible at https://<hostname>.ezb.net',
+              description: 'Custom hostname (optional). The actual deployment URL will be https://{app_id}.eu{1,2,3}.ezb.net based on the assigned server.',
               pattern: '^[a-z][a-z0-9-]*[a-z0-9]$',
               minLength: 3,
               maxLength: 63

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mlgym-deploy",
-  "version": "3.3.29",
+  "version": "3.3.32",
   "description": "MCP server for MLGym - Complete deployment management: deploy, configure, monitor, and rollback applications",
   "main": "index.js",
   "type": "module",