bug fix: fixed table rendering in mcp servers list

Author: sizzlebop

BIBBLE_GLAMOUR_ROADMAP.md

@@ -198,7 +198,7 @@ src/ui/
 ╰─────────────────────────────────────────╯
 
 🔥 Model: Claude 3.7 Sonnet   ⚡ MCP: 5 servers connected
-💫 Version: 1.3.9              🚀 Ready for magic!
+💫 Version: 1.3.10              🚀 Ready for magic!
 ```
 
 ### Chat Interface

OVERVIEW.md

@@ -2,7 +2,7 @@
 
 Bibble is a sophisticated command-line interface (CLI) chatbot application that integrates with multiple language model providers and supports the Model Context Protocol (MCP) for enhanced functionality through external tools. Built with TypeScript, it provides a robust terminal-based AI assistant experience with comprehensive tool integration.
 
-**Version**: 1.3.9
+**Version**: 1.3.10
 **Author**: Pink Pixel
 **NPM Package**: @pinkpixel/bibble
 *Last updated: August 28, 2025*

src/commands/config.ts

@@ -292,14 +292,26 @@ async function manageMcpServers(): Promise<void> {
     ],
   }]);
 
-  const actions = {
-    list: () => listMcpServers(servers),
-    add: () => addMcpServer(),
-    remove: () => removeMcpServer(servers),
-    edit: () => editMcpServer(servers),
-  };
-
-  if (actions[action]) await actions[action]();
+  // Handle actions properly with explicit cancel check
+  switch (action) {
+    case "list":
+      listMcpServers(servers);
+      break;
+    case "add":
+      await addMcpServer();
+      break;
+    case "remove":
+      await removeMcpServer(servers);
+      break;
+    case "edit":
+      await editMcpServer(servers);
+      break;
+    case "cancel":
+      console.log(terminal.info("Operation cancelled."));
+      break;
+    default:
+      console.log(terminal.warning(`Unknown action: ${action}`));
+  }
 }
 
 function listMcpServers(servers: any[]): void {
@@ -308,30 +320,44 @@ function listMcpServers(servers: any[]): void {
     return;
   }
 
-  // Transform servers for our beautiful table
-  const mcpServerData = servers.map(server => ({
-    name: server.name,
-    command: `${server.command} ${server.args.join(" ")}`,
-    environment: server.env ? Object.keys(server.env).join(", ") || "None" : "None",
-    status: server.enabled ? "enabled" : "disabled"
-  }));
-
-  // Create a beautiful table using our table system
-  const table = new BibbleTable({
-    head: ['Server', 'Command', 'Environment', 'Status'],
-    style: 'fancy'
+  // Transform servers for our beautiful table with defensive programming
+  const mcpServerData = servers.map(server => {
+    const args = Array.isArray(server.args) ? server.args : [];
+    return {
+      name: server.name || 'Unknown',
+      command: `${server.command || 'Unknown'} ${args.join(" ")}`,
+      environment: server.env ? Object.keys(server.env).join(", ") || "None" : "None",
+      status: server.enabled ? "enabled" : "disabled"
+    };
   });
 
-  mcpServerData.forEach(server => {
-    table.addRow([
-      server.name,
-      server.command,
-      server.environment,
-      server.status
-    ]);
-  });
+  try {
+    // Create table with explicit column widths like the config table does
+    const table = new BibbleTable({
+      head: ['Server', 'Command', 'Environment', 'Status'],
+      style: 'fancy',
+      colWidths: [20, 50, 15, 10] // Explicit column widths
+    });
 
-  console.log(`\n${t.h2('MCP Servers')} ${brandSymbols.lightning}\n${table.toString()}`);
+    mcpServerData.forEach(server => {
+      table.addRow([
+        server.name,
+        server.command,
+        server.environment,
+        server.status
+      ]);
+    });
+
+    console.log(`\n${t.h2('MCP Servers')} ${brandSymbols.lightning}\n${table.toString()}`);
+    
+  } catch (tableError) {
+    // Fallback to simple list if anything goes wrong
+    console.log('Table rendering failed, using simple list');
+    console.log(`\n${t.h2('MCP Servers')} ${brandSymbols.lightning}`);
+    mcpServerData.forEach((server, i) => {
+      console.log(`${i + 1}. ${t.cyan(server.name)} - ${server.command} (${server.status})`);
+    });
+  }
 }
 
 /**