lua-cli 3.2.0 → 3.4.0

package/dist/commands/mcp.js

@@ -9,21 +9,21 @@ import { BASE_URLS } from '../config/constants.js';
 import { safePrompt } from '../utils/prompt-handler.js';
 import { validateConfig, validateAgentConfig, } from '../utils/dev-helpers.js';
 import DeveloperApi from '../api/developer.api.service.js';
+// ─────────────────────────────────────────────────────────────────────────────
+// Main Command Entry
+// ─────────────────────────────────────────────────────────────────────────────
 /**
- * Main MCP command - manages MCP servers
+ * Main MCP command - manages MCP servers.
  *
- * Subcommands:
- * - lua mcp list       - List all MCP servers
- * - lua mcp activate   - Activate an MCP server
- * - lua mcp deactivate - Deactivate an MCP server
- * - lua mcp delete     - Delete an MCP server
- *
- * @param action - Optional action ('list', 'activate', 'deactivate', 'delete')
- * @param serverName - Optional server name for actions
- * @returns Promise that resolves when command completes
+ * Supports both interactive and non-interactive modes:
+ * - Interactive: menu-driven server management
+ * - Non-interactive: action + -i flag or positional server name
  */
-export async function mcpCommand(action, serverName) {
+export async function mcpCommand(action, serverNamePositional, cmdObj) {
     return withErrorHandling(async () => {
+        const options = {
+            serverName: cmdObj?.serverName || serverNamePositional || null,
+        };
         // Step 1: Load configuration
         const config = readSkillConfig();
         validateConfig(config);
@@ -43,352 +43,362 @@ export async function mcpCommand(action, serverName) {
             apiKey,
             developerApi,
         };
-        // Step 3: Handle action or show interactive menu
+        // Determine mode
         if (action) {
-            const normalizedAction = action.toLowerCase();
-            switch (normalizedAction) {
-                case 'list':
-                case 'ls':
-                    await listMCPServers(context);
-                    break;
-                case 'activate':
-                case 'enable':
-                    await activateMCPServer(context, serverName);
-                    break;
-                case 'deactivate':
-                case 'disable':
-                    await deactivateMCPServer(context, serverName);
-                    break;
-                case 'delete':
-                case 'rm':
-                    await deleteMCPServer(context, serverName);
-                    break;
-                default:
-                    console.error(`❌ Unknown action: "${action}"`);
-                    showUsage();
-                    process.exit(1);
-            }
+            await executeNonInteractive(context, action, options);
         }
         else {
-            // Interactive mode
             await interactiveMCPManagement(context);
         }
     }, "mcp");
 }
+// ─────────────────────────────────────────────────────────────────────────────
+// Shared Core Functions
+// ─────────────────────────────────────────────────────────────────────────────
 /**
- * Show usage information
+ * Fetches all MCP servers from the API.
  */
-function showUsage() {
-    console.log('\nUsage:');
-    console.log('  lua mcp                        - Interactive MCP server management');
-    console.log('  lua mcp list                   - List all MCP servers');
-    console.log('  lua mcp activate [name]        - Activate an MCP server');
-    console.log('  lua mcp deactivate [name]      - Deactivate an MCP server');
-    console.log('  lua mcp delete [name]          - Delete an MCP server');
-    console.log('\nAliases:');
-    console.log('  list → ls');
-    console.log('  activate → enable');
-    console.log('  deactivate → disable');
-    console.log('  delete → rm');
-}
-/**
- * Interactive MCP management
- */
-async function interactiveMCPManagement(context) {
-    let continueManaging = true;
-    while (continueManaging) {
-        console.log("\n" + "=".repeat(60));
-        console.log("🔌 MCP Server Management");
-        console.log("=".repeat(60) + "\n");
-        const actionAnswer = await safePrompt([
-            {
-                type: 'list',
-                name: 'action',
-                message: 'What would you like to do?',
-                choices: [
-                    { name: '📋 List MCP servers', value: 'list' },
-                    { name: '✅ Activate a server', value: 'activate' },
-                    { name: '⏸️  Deactivate a server', value: 'deactivate' },
-                    { name: '🗑️  Delete a server', value: 'delete' },
-                    { name: '❌ Exit', value: 'exit' }
-                ]
-            }
-        ]);
-        if (!actionAnswer)
-            return;
-        const { action } = actionAnswer;
-        switch (action) {
-            case 'list':
-                await listMCPServers(context);
-                break;
-            case 'activate':
-                await activateMCPServer(context);
-                break;
-            case 'deactivate':
-                await deactivateMCPServer(context);
-                break;
-            case 'delete':
-                await deleteMCPServer(context);
-                break;
-            case 'exit':
-                continueManaging = false;
-                console.log("\n👋 Goodbye!\n");
-                break;
-        }
-    }
-}
-/**
- * List all MCP servers
- */
-async function listMCPServers(context) {
-    writeProgress("🔄 Loading MCP servers...");
+async function fetchServersCore(context) {
     try {
         const result = await context.developerApi.getMCPServers();
         if (!result.success) {
-            console.error(`\n❌ Failed to load MCP servers: ${result.error?.message}\n`);
-            return;
+            console.error(`❌ Failed to load MCP servers: ${result.error?.message}`);
+            return null;
         }
-        const servers = result.data || [];
-        console.log("\n" + "=".repeat(60));
-        console.log("🔌 MCP Servers");
-        console.log("=".repeat(60) + "\n");
-        if (servers.length === 0) {
-            console.log("ℹ️  No MCP servers found.");
-            console.log("💡 Define MCP servers in your LuaAgent configuration and run 'lua push all'.\n");
-            return;
-        }
-        // Separate active and inactive servers
-        const activeServers = servers.filter((s) => s.active);
-        const inactiveServers = servers.filter((s) => !s.active);
-        if (activeServers.length > 0) {
-            console.log("✅ Active Servers:\n");
-            activeServers.forEach((server) => {
-                printServerInfo(server);
-            });
-        }
-        if (inactiveServers.length > 0) {
-            if (activeServers.length > 0)
-                console.log("─".repeat(40) + "\n");
-            console.log("⏸️  Inactive Servers:\n");
-            inactiveServers.forEach((server) => {
-                printServerInfo(server);
-            });
-        }
-        console.log("=".repeat(60) + "\n");
+        return result.data || [];
     }
     catch (error) {
-        console.error('\n❌ Error loading MCP servers:', error.message);
+        console.error('❌ Error loading MCP servers:', error.message);
+        return null;
     }
 }
 /**
- * Print server information
+ * Displays all MCP servers with status.
+ */
+function displayServersCore(servers) {
+    console.log("\n" + "=".repeat(60));
+    console.log("🔌 MCP Servers");
+    console.log("=".repeat(60) + "\n");
+    if (servers.length === 0) {
+        console.log("ℹ️  No MCP servers found.");
+        console.log("💡 Define MCP servers in your LuaAgent configuration and run 'lua push all'.\n");
+        return;
+    }
+    const activeServers = servers.filter((s) => s.active);
+    const inactiveServers = servers.filter((s) => !s.active);
+    if (activeServers.length > 0) {
+        console.log("✅ Active Servers:\n");
+        activeServers.forEach((server) => printServerInfo(server));
+    }
+    if (inactiveServers.length > 0) {
+        if (activeServers.length > 0)
+            console.log("─".repeat(40) + "\n");
+        console.log("⏸️  Inactive Servers:\n");
+        inactiveServers.forEach((server) => printServerInfo(server));
+    }
+    console.log("=".repeat(60));
+}
+/**
+ * Print server information.
  */
 function printServerInfo(server) {
     const statusIcon = server.active ? '🟢' : '⚪';
     console.log(`${statusIcon} ${server.name}`);
     console.log(`   Transport: ${server.transport}`);
-    if (server.transport === 'stdio') {
-        console.log(`   Command: ${server.command} ${(server.args || []).join(' ')}`);
-    }
-    else {
-        console.log(`   URL: ${server.url}`);
-    }
+    console.log(`   URL: ${server.url}`);
     console.log(`   ID: ${server.id}`);
     console.log();
 }
 /**
- * Activate an MCP server
+ * Activates an MCP server.
  */
-async function activateMCPServer(context, serverName) {
+async function activateServerCore(context, server) {
+    if (server.active) {
+        console.log(`ℹ️  Server "${server.name}" is already active.`);
+        return true;
+    }
+    writeProgress(`🔄 Activating ${server.name}...`);
     try {
-        // Get all servers
-        const result = await context.developerApi.getMCPServers();
-        if (!result.success) {
-            console.error(`\n❌ Failed to load MCP servers: ${result.error?.message}\n`);
-            return;
-        }
-        const servers = result.data || [];
-        const inactiveServers = servers.filter((s) => !s.active);
-        if (inactiveServers.length === 0) {
-            console.log("\nℹ️  All MCP servers are already active.\n");
-            return;
-        }
-        let serverToActivate;
-        if (serverName) {
-            // Find by name
-            const server = inactiveServers.find((s) => s.name.toLowerCase() === serverName.toLowerCase());
-            if (!server) {
-                console.error(`\n❌ Inactive MCP server "${serverName}" not found.\n`);
-                console.log("Available inactive servers:");
-                inactiveServers.forEach((s) => console.log(`  - ${s.name}`));
-                return;
-            }
-            serverToActivate = server;
-        }
-        else {
-            // Prompt for selection
-            const answer = await safePrompt([
-                {
-                    type: 'list',
-                    name: 'server',
-                    message: 'Select an MCP server to activate:',
-                    choices: inactiveServers.map((s) => ({
-                        name: `${s.name} (${s.transport})`,
-                        value: s
-                    }))
-                }
-            ]);
-            if (!answer)
-                return;
-            serverToActivate = answer.server;
-        }
-        writeProgress(`🔄 Activating ${serverToActivate.name}...`);
-        const activateResult = await context.developerApi.activateMCPServer(serverToActivate.id);
-        if (activateResult.success) {
-            writeSuccess(`\n✅ MCP server "${serverToActivate.name}" activated successfully!\n`);
+        const result = await context.developerApi.activateMCPServer(server.id);
+        if (result.success) {
+            writeSuccess(`✅ MCP server "${server.name}" activated successfully!`);
             writeInfo("💡 The server's tools are now available to your agent.");
+            return true;
         }
         else {
-            console.error(`\n❌ Failed to activate: ${activateResult.error?.message}\n`);
+            console.error(`❌ Failed to activate: ${result.error?.message}`);
+            return false;
         }
     }
     catch (error) {
-        console.error('\n❌ Error activating MCP server:', error.message);
+        console.error('❌ Error activating MCP server:', error.message);
+        return false;
     }
 }
 /**
- * Deactivate an MCP server
+ * Deactivates an MCP server.
  */
-async function deactivateMCPServer(context, serverName) {
+async function deactivateServerCore(context, server) {
+    if (!server.active) {
+        console.log(`ℹ️  Server "${server.name}" is already inactive.`);
+        return true;
+    }
+    writeProgress(`🔄 Deactivating ${server.name}...`);
     try {
-        // Get all servers
-        const result = await context.developerApi.getMCPServers();
-        if (!result.success) {
-            console.error(`\n❌ Failed to load MCP servers: ${result.error?.message}\n`);
-            return;
-        }
-        const servers = result.data || [];
-        const activeServers = servers.filter((s) => s.active);
-        if (activeServers.length === 0) {
-            console.log("\nℹ️  No active MCP servers to deactivate.\n");
-            return;
-        }
-        let serverToDeactivate;
-        if (serverName) {
-            // Find by name
-            const server = activeServers.find((s) => s.name.toLowerCase() === serverName.toLowerCase());
-            if (!server) {
-                console.error(`\n❌ Active MCP server "${serverName}" not found.\n`);
-                console.log("Available active servers:");
-                activeServers.forEach((s) => console.log(`  - ${s.name}`));
-                return;
-            }
-            serverToDeactivate = server;
+        const result = await context.developerApi.deactivateMCPServer(server.id);
+        if (result.success) {
+            writeSuccess(`✅ MCP server "${server.name}" deactivated successfully!`);
+            writeInfo("💡 The server's tools are no longer available to your agent.");
+            return true;
         }
         else {
-            // Prompt for selection
-            const answer = await safePrompt([
-                {
-                    type: 'list',
-                    name: 'server',
-                    message: 'Select an MCP server to deactivate:',
-                    choices: activeServers.map((s) => ({
-                        name: `${s.name} (${s.transport})`,
-                        value: s
-                    }))
-                }
-            ]);
-            if (!answer)
-                return;
-            serverToDeactivate = answer.server;
+            console.error(`❌ Failed to deactivate: ${result.error?.message}`);
+            return false;
         }
-        writeProgress(`🔄 Deactivating ${serverToDeactivate.name}...`);
-        const deactivateResult = await context.developerApi.deactivateMCPServer(serverToDeactivate.id);
-        if (deactivateResult.success) {
-            writeSuccess(`\n✅ MCP server "${serverToDeactivate.name}" deactivated successfully!\n`);
-            writeInfo("💡 The server's tools are no longer available to your agent.");
+    }
+    catch (error) {
+        console.error('❌ Error deactivating MCP server:', error.message);
+        return false;
+    }
+}
+/**
+ * Deletes an MCP server.
+ */
+async function deleteServerCore(context, server) {
+    writeProgress(`🔄 Deleting ${server.name}...`);
+    try {
+        const result = await context.developerApi.deleteMCPServer(server.id);
+        if (result.success) {
+            writeSuccess(`✅ MCP server "${server.name}" deleted successfully!`);
+            return true;
         }
         else {
-            console.error(`\n❌ Failed to deactivate: ${deactivateResult.error?.message}\n`);
+            console.error(`❌ Failed to delete: ${result.error?.message}`);
+            return false;
         }
     }
     catch (error) {
-        console.error('\n❌ Error deactivating MCP server:', error.message);
+        console.error('❌ Error deleting MCP server:', error.message);
+        return false;
     }
 }
 /**
- * Delete an MCP server
+ * Finds a server by name (case-insensitive).
  */
-async function deleteMCPServer(context, serverName) {
-    try {
-        // Get all servers
-        const result = await context.developerApi.getMCPServers();
-        if (!result.success) {
-            console.error(`\n❌ Failed to load MCP servers: ${result.error?.message}\n`);
-            return;
+function findServer(servers, serverName) {
+    const normalizedName = serverName.toLowerCase();
+    return servers.find(s => s.name.toLowerCase() === normalizedName) || null;
+}
+/**
+ * Prompts user to select a server from a list.
+ */
+async function promptServerSelection(servers, message) {
+    const answer = await safePrompt([
+        {
+            type: 'list',
+            name: 'server',
+            message,
+            choices: servers.map((s) => ({
+                name: `${s.name} (${s.transport}) ${s.active ? '🟢' : '⚪'}`,
+                value: s
+            }))
         }
-        const servers = result.data || [];
-        if (servers.length === 0) {
-            console.log("\nℹ️  No MCP servers to delete.\n");
-            return;
+    ]);
+    return answer?.server || null;
+}
+/**
+ * Show usage information.
+ */
+function showUsage() {
+    console.log('\nUsage:');
+    console.log('  lua mcp                           Interactive MCP server management');
+    console.log('  lua mcp list                      List all MCP servers');
+    console.log('  lua mcp activate -i <name>        Activate an MCP server');
+    console.log('  lua mcp deactivate -i <name>      Deactivate an MCP server');
+    console.log('  lua mcp delete -i <name>          Delete an MCP server');
+    console.log('\nNote: Positional server name is also supported (e.g., lua mcp activate myServer)');
+}
+// ─────────────────────────────────────────────────────────────────────────────
+// Non-Interactive Mode
+// ─────────────────────────────────────────────────────────────────────────────
+async function executeNonInteractive(context, action, options) {
+    const normalizedAction = action.toLowerCase();
+    // Support aliases
+    const actionMap = {
+        'list': 'list',
+        'ls': 'list',
+        'activate': 'activate',
+        'enable': 'activate',
+        'deactivate': 'deactivate',
+        'disable': 'deactivate',
+        'delete': 'delete',
+        'rm': 'delete',
+    };
+    const resolvedAction = actionMap[normalizedAction];
+    if (!resolvedAction) {
+        console.error(`❌ Invalid action: "${action}"`);
+        showUsage();
+        process.exit(1);
+    }
+    writeProgress("🔄 Loading MCP servers...");
+    const servers = await fetchServersCore(context);
+    if (!servers)
+        process.exit(1);
+    // List action doesn't require server selection
+    if (resolvedAction === 'list') {
+        displayServersCore(servers);
+        return;
+    }
+    // All other actions require server selection
+    if (!options.serverName) {
+        console.error(`❌ --server-name is required for action "${resolvedAction}"`);
+        console.log(`\nUsage: lua mcp ${resolvedAction} --server-name <name>`);
+        if (servers.length > 0) {
+            console.log('\nAvailable servers:');
+            servers.forEach(s => console.log(`  - ${s.name} ${s.active ? '(active)' : '(inactive)'}`));
         }
-        let serverToDelete;
-        if (serverName) {
-            // Find by name
-            const server = servers.find((s) => s.name.toLowerCase() === serverName.toLowerCase());
-            if (!server) {
-                console.error(`\n❌ MCP server "${serverName}" not found.\n`);
-                console.log("Available servers:");
-                servers.forEach((s) => console.log(`  - ${s.name}`));
-                return;
-            }
-            serverToDelete = server;
+        process.exit(1);
+    }
+    const selectedServer = findServer(servers, options.serverName);
+    if (!selectedServer) {
+        console.error(`❌ MCP server "${options.serverName}" not found`);
+        if (servers.length > 0) {
+            console.log('\nAvailable servers:');
+            servers.forEach(s => console.log(`  - ${s.name}`));
         }
-        else {
-            // Prompt for selection
-            const answer = await safePrompt([
-                {
-                    type: 'list',
-                    name: 'server',
-                    message: 'Select an MCP server to delete:',
-                    choices: servers.map((s) => ({
-                        name: `${s.name} (${s.transport}) ${s.active ? '🟢' : '⚪'}`,
-                        value: s
-                    }))
-                }
-            ]);
-            if (!answer)
-                return;
-            serverToDelete = answer.server;
+        process.exit(1);
+    }
+    switch (resolvedAction) {
+        case 'activate': {
+            const success = await activateServerCore(context, selectedServer);
+            if (!success)
+                process.exit(1);
+            break;
+        }
+        case 'deactivate': {
+            const success = await deactivateServerCore(context, selectedServer);
+            if (!success)
+                process.exit(1);
+            break;
         }
-        // Confirm deletion
-        console.log(`\n⚠️  You are about to delete MCP server: ${serverToDelete.name}`);
-        if (serverToDelete.active) {
-            console.log("⚠️  WARNING: This server is currently ACTIVE!");
+        case 'delete': {
+            // In non-interactive mode, delete directly without confirmation
+            const success = await deleteServerCore(context, selectedServer);
+            if (!success)
+                process.exit(1);
+            break;
         }
-        const confirmAnswer = await safePrompt([
+    }
+}
+// ─────────────────────────────────────────────────────────────────────────────
+// Interactive Mode
+// ─────────────────────────────────────────────────────────────────────────────
+async function interactiveMCPManagement(context) {
+    let continueManaging = true;
+    while (continueManaging) {
+        console.log("\n" + "=".repeat(60));
+        console.log("🔌 MCP Server Management");
+        console.log("=".repeat(60) + "\n");
+        const actionAnswer = await safePrompt([
             {
-                type: 'confirm',
-                name: 'confirm',
-                message: 'Are you sure you want to delete this MCP server?',
-                default: false
+                type: 'list',
+                name: 'action',
+                message: 'What would you like to do?',
+                choices: [
+                    { name: '📋 List MCP servers', value: 'list' },
+                    { name: '✅ Activate a server', value: 'activate' },
+                    { name: '⏸️  Deactivate a server', value: 'deactivate' },
+                    { name: '🗑️  Delete a server', value: 'delete' },
+                    { name: '❌ Exit', value: 'exit' }
+                ]
             }
         ]);
-        if (!confirmAnswer || !confirmAnswer.confirm) {
-            console.log("\n❌ Deletion cancelled.\n");
+        if (!actionAnswer)
             return;
-        }
-        writeProgress(`🔄 Deleting ${serverToDelete.name}...`);
-        const deleteResult = await context.developerApi.deleteMCPServer(serverToDelete.id);
-        if (deleteResult.success) {
-            writeSuccess(`\n✅ MCP server "${serverToDelete.name}" deleted successfully!\n`);
-        }
-        else {
-            console.error(`\n❌ Failed to delete: ${deleteResult.error?.message}\n`);
+        const { action } = actionAnswer;
+        switch (action) {
+            case 'list':
+                await listServersInteractive(context);
+                break;
+            case 'activate':
+                await activateServerInteractive(context);
+                break;
+            case 'deactivate':
+                await deactivateServerInteractive(context);
+                break;
+            case 'delete':
+                await deleteServerInteractive(context);
+                break;
+            case 'exit':
+                continueManaging = false;
+                console.log("\n👋 Goodbye!\n");
+                break;
         }
     }
-    catch (error) {
-        console.error('\n❌ Error deleting MCP server:', error.message);
+}
+async function listServersInteractive(context) {
+    writeProgress("🔄 Loading MCP servers...");
+    const servers = await fetchServersCore(context);
+    if (servers) {
+        displayServersCore(servers);
+    }
+}
+async function activateServerInteractive(context) {
+    const servers = await fetchServersCore(context);
+    if (!servers)
+        return;
+    const inactiveServers = servers.filter((s) => !s.active);
+    if (inactiveServers.length === 0) {
+        console.log("\nℹ️  All MCP servers are already active.\n");
+        return;
+    }
+    const server = await promptServerSelection(inactiveServers, 'Select an MCP server to activate:');
+    if (!server)
+        return;
+    await activateServerCore(context, server);
+}
+async function deactivateServerInteractive(context) {
+    const servers = await fetchServersCore(context);
+    if (!servers)
+        return;
+    const activeServers = servers.filter((s) => s.active);
+    if (activeServers.length === 0) {
+        console.log("\nℹ️  No active MCP servers to deactivate.\n");
+        return;
+    }
+    const server = await promptServerSelection(activeServers, 'Select an MCP server to deactivate:');
+    if (!server)
+        return;
+    await deactivateServerCore(context, server);
+}
+async function deleteServerInteractive(context) {
+    const servers = await fetchServersCore(context);
+    if (!servers)
+        return;
+    if (servers.length === 0) {
+        console.log("\nℹ️  No MCP servers to delete.\n");
+        return;
+    }
+    const server = await promptServerSelection(servers, 'Select an MCP server to delete:');
+    if (!server)
+        return;
+    // Confirm deletion in interactive mode
+    console.log(`\n⚠️  You are about to delete MCP server: ${server.name}`);
+    if (server.active) {
+        console.log("⚠️  WARNING: This server is currently ACTIVE!");
+    }
+    const confirmAnswer = await safePrompt([
+        {
+            type: 'confirm',
+            name: 'confirm',
+            message: 'Are you sure you want to delete this MCP server?',
+            default: false
+        }
+    ]);
+    if (!confirmAnswer || !confirmAnswer.confirm) {
+        console.log("\n❌ Deletion cancelled.\n");
+        return;
     }
+    await deleteServerCore(context, server);
 }
 //# sourceMappingURL=mcp.js.map