mlgym-deploy 2.3.2 → 2.3.5

package/index.js

@@ -17,18 +17,20 @@ import crypto from 'crypto';
 const execAsync = promisify(exec);
 
 // Current version of this MCP server
-const CURRENT_VERSION = '2.3.2';
+const CURRENT_VERSION = '2.3.5';
 const PACKAGE_NAME = 'mlgym-deploy';
 
 // Version check state
 let versionCheckResult = null;
 let lastVersionCheck = 0;
 const VERSION_CHECK_INTERVAL = 3600000; // Check once per hour
+let updateAcknowledged = false; // Track if user has acknowledged the update
 
 // Helper to wrap tool response with update message if needed
 function wrapResponseWithUpdateMessage(response) {
   if (versionCheckResult && versionCheckResult.updateAvailable) {
-    const updateMessage = `\n╔════════════════════════════════════════════════════════════════════╗
+    const updateMessage = `
+╔════════════════════════════════════════════════════════════════════╗
 ║                  🔄 UPDATE AVAILABLE 🔄                              ║
 ║                                                                        ║
 ║  A newer version of MLGym MCP Server is available!                    ║
@@ -37,12 +39,13 @@ function wrapResponseWithUpdateMessage(response) {
 ║  Latest version:  ${versionCheckResult.latest.padEnd(52)} ║
 ║                                                                        ║
 ║  To update in Cursor:                                                 ║
-║  1. Close this terminal/tab                                          ║
+║  1. Open a new terminal tab (Cmd/Ctrl + Shift + \`)                  ║
 ║  2. Run: npm install -g ${PACKAGE_NAME}@latest                          ║
 ║  3. Restart Cursor to use the updated MCP server                    ║
 ║                                                                        ║
-║  ⚠️  Using outdated version may cause issues!                         ║
-╚════════════════════════════════════════════════════════════════════╝\n`;
+║  ⚠️  Using outdated version may cause unexpected issues!              ║
+╚════════════════════════════════════════════════════════════════════╝
+`;
 
     // Prepend update message to the response
     if (response.content && response.content[0] && response.content[0].type === 'text') {
@@ -315,6 +318,14 @@ const server = new Server(
 
 // Tool definitions
 const TOOLS = [
+  {
+    name: 'mlgym_update_acknowledge',
+    description: 'Acknowledge the update notification and continue using the current version (not recommended)',
+    inputSchema: {
+      type: 'object',
+      properties: {}
+    }
+  },
   {
     name: 'mlgym_git_configure',
     description: 'Check and configure git user settings (required before making commits)',
@@ -430,18 +441,18 @@ const TOOLS = [
   },
   {
     name: 'mlgym_auth_check',
-    description: '🚨 ALWAYS USE THIS FIRST when deploying! Check if user account exists BEFORE asking for other details. Start by ONLY asking for email.',
+    description: '🚨 ALWAYS USE THIS FIRST when deploying! Check if user account exists. First call with email only, then call again with password if needed.',
     inputSchema: {
       type: 'object',
       properties: {
         email: {
           type: 'string',
-          description: 'Email address to check (ask for this FIRST)',
+          description: 'Email address to check',
           pattern: '^[^@]+@[^@]+\\.[^@]+$'
         },
         password: {
           type: 'string',
-          description: 'Password (ONLY ask if account exists)',
+          description: 'Password (optional - add on second call to authenticate)',
           minLength: 8
         }
       },
@@ -1204,6 +1215,71 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
   const { name, arguments: args } = request.params;
 
+  // Check for updates before executing ANY tool
+  // Special handling: mlgym_version_check and mlgym_update_acknowledge bypass this check
+  if (name !== 'mlgym_version_check' && name !== 'mlgym_update_acknowledge') {
+    // Perform version check
+    await checkForUpdates();
+
+    // If update is available and not acknowledged, force user to acknowledge
+    if (versionCheckResult && versionCheckResult.updateAvailable && !updateAcknowledged) {
+      return {
+        content: [
+          {
+            type: 'text',
+            text: `╔════════════════════════════════════════════════════════════════════╗
+║                    🚨 UPDATE REQUIRED 🚨                           ║
+╚════════════════════════════════════════════════════════════════════╝
+
+A newer version of MLGym MCP Server is available!
+
+Current version: ${CURRENT_VERSION}
+Latest version:  ${versionCheckResult.latest}
+
+The MCP server requires an update before you can continue using it.
+
+Please choose one of the following options:
+
+1. **UPDATE NOW (Recommended)**
+   Open a new terminal in Cursor and run:
+   \`\`\`bash
+   npm install -g ${PACKAGE_NAME}@latest
+   \`\`\`
+   Then restart Cursor to use the updated MCP server.
+
+2. **ACKNOWLEDGE AND CONTINUE**
+   If you need to continue without updating, use the special tool:
+   \`mlgym_update_acknowledge\`
+
+   ⚠️ Warning: Continuing with an outdated version may cause:
+   - Missing features
+   - Compatibility issues
+   - Unexpected errors
+   - Deployment failures
+
+Please update the MCP server or explicitly acknowledge to continue.`
+          }
+        ]
+      };
+    }
+  }
+
+  // Handle special update acknowledgment tool
+  if (name === 'mlgym_update_acknowledge') {
+    updateAcknowledged = true;
+    return {
+      content: [
+        {
+          type: 'text',
+          text: `✓ Update acknowledgment received. You can now continue using the MCP server.
+
+⚠️ You are using version ${CURRENT_VERSION} while version ${versionCheckResult?.latest || 'unknown'} is available.
+   Some features may not work correctly. Please update as soon as possible.`
+        }
+      ]
+    };
+  }
+
   let response;
   switch (name) {
     case 'mlgym_git_configure':
@@ -1864,44 +1940,22 @@ async function checkUserExists(args) {
     }
   }
 
-  // Try to check if email exists without password (will fail but gives info)
-  try {
-    const result = await apiRequest('POST', '/api/v1/auth/login', {
-      email,
-      password: 'dummy_check'
-    }, false);
-
-    // If we get here, it means the endpoint is accessible
-    // The error message will tell us if user exists
-    if (result.error && result.error.includes('Invalid credentials')) {
-      return {
-        content: [{
-          type: 'text',
-          text: JSON.stringify({
-            status: 'success',
-            exists: true,
-            authenticated: false,
-            email: email,
-            message: 'User exists but needs password to authenticate',
-            next_step: 'Please provide password to check SSH keys'
-          }, null, 2)
-        }]
-      };
-    }
-  } catch (err) {
-    // Network or other error
-  }
-
-  // User doesn't exist
+  // Without password, we cannot definitively check if user exists
+  // Return a response that prompts for password to check
   return {
     content: [{
       type: 'text',
       text: JSON.stringify({
         status: 'success',
-        exists: false,
+        exists: 'unknown',
+        authenticated: false,
         email: email,
-        message: 'User does not exist. You can create a new account.',
-        next_step: 'Use mlgym_user_create to create a new account'
+        message: 'Cannot determine if account exists without password',
+        next_step: 'Please provide your password to check if you have an existing account, or provide full details to create a new account',
+        options: [
+          'If you have an account: Provide your password',
+          'If you are new: Provide name, password, and accept terms to create account'
+        ]
       }, null, 2)
     }]
   };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mlgym-deploy",
-  "version": "2.3.2",
+  "version": "2.3.5",
   "description": "MCP server for GitLab Backend - User creation and project deployment",
   "main": "index.js",
   "type": "module",