fix(mcp): return to MCP panel after adding server and enable servers by default (#250)

- After adding an MCP server (stdio or HTTP), the UI now returns to the /mcp
  home screen instead of exiting interactive mode completely
- MCP servers are now enabled/running by default when added, as the user is
  using Cortex as an MCP client
- Updated all MCP server addition paths (inline forms, card actions, modal
  actions, and slash commands) to use Running status by default

Co-authored-by: Droid Agent <droid@factory.ai>

Author: echobt

cortex-tui/src/runner/event_loop.rs

@@ -1415,8 +1415,12 @@ impl EventLoop {
                                 values,
                             } => {
                                 // Handle inline form submission
-                                self.handle_inline_form_submission(&action_id, values);
-                                self.app_state.exit_interactive_mode();
+                                // Returns true if we should stay in interactive mode
+                                let stay_open =
+                                    self.handle_inline_form_submission(&action_id, values);
+                                if !stay_open {
+                                    self.app_state.exit_interactive_mode();
+                                }
                             }
                             crate::interactive::InteractiveResult::Cancelled => {
                                 self.app_state.exit_interactive_mode();
@@ -6026,11 +6030,12 @@ impl EventLoop {
     }
 
     /// Handles inline form submission from interactive mode (for MCP config, etc.).
+    /// Returns `true` if we should stay in interactive mode (e.g., return to MCP panel).
     fn handle_inline_form_submission(
         &mut self,
         action_id: &str,
         values: std::collections::HashMap<String, String>,
-    ) {
+    ) -> bool {
         match action_id {
             "mcp-add" | "mcp-add-stdio" => {
                 // Add new stdio MCP server from inline form
@@ -6040,7 +6045,7 @@ impl EventLoop {
 
                 if name.is_empty() || command.is_empty() {
                     self.app_state.toasts.error("Name and command are required");
-                    return;
+                    return false;
                 }
 
                 // Parse args
@@ -6050,10 +6055,10 @@ impl EventLoop {
                     args_str.split_whitespace().map(|s| s.to_string()).collect()
                 };
 
-                // Create new server info
+                // Create new server info - MCP servers are enabled (Running) by default
                 let server_info = crate::modal::mcp_manager::McpServerInfo {
                     name: name.clone(),
-                    status: crate::modal::mcp_manager::McpStatus::Stopped,
+                    status: crate::modal::mcp_manager::McpStatus::Running,
                     tool_count: 0,
                     error: None,
                     requires_auth: false,
@@ -6072,6 +6077,10 @@ impl EventLoop {
                     command,
                     args
                 );
+
+                // Return to MCP panel
+                self.reopen_mcp_panel();
+                true
             }
             "mcp-add-http" => {
                 // Add new HTTP MCP server from inline form
@@ -6081,13 +6090,13 @@ impl EventLoop {
 
                 if name.is_empty() || url.is_empty() {
                     self.app_state.toasts.error("Name and URL are required");
-                    return;
+                    return false;
                 }
 
-                // Create new server info
+                // Create new server info - MCP servers are enabled (Running) by default
                 let server_info = crate::modal::mcp_manager::McpServerInfo {
                     name: name.clone(),
-                    status: crate::modal::mcp_manager::McpStatus::Stopped,
+                    status: crate::modal::mcp_manager::McpStatus::Running,
                     tool_count: 0,
                     error: None,
                     requires_auth: !api_key.is_empty(),
@@ -6106,13 +6115,26 @@ impl EventLoop {
                     url,
                     !api_key.is_empty()
                 );
+
+                // Return to MCP panel
+                self.reopen_mcp_panel();
+                true
             }
             _ => {
                 tracing::warn!("Unhandled inline form submission: {}", action_id);
+                false
             }
         }
     }
 
+    /// Re-opens the MCP panel to show the list of servers after adding a new one.
+    fn reopen_mcp_panel(&mut self) {
+        use crate::interactive::builders::build_mcp_selector;
+        let servers = self.app_state.mcp_servers.clone();
+        let interactive = build_mcp_selector(&servers);
+        self.app_state.enter_interactive_mode(interactive);
+    }
+
     /// Re-opens the settings menu with current state (to stay in menu after toggle)
     fn reopen_settings_menu(&mut self) {
         use crate::interactive::builders::{SettingsSnapshot, build_settings_selector};
@@ -7358,10 +7380,10 @@ impl EventLoop {
             return;
         }
 
-        // Add the new server
+        // Add the new server - MCP servers are enabled (Running) by default
         let server = crate::modal::mcp_manager::McpServerInfo {
             name: name.clone(),
-            status: crate::modal::mcp_manager::McpStatus::Stopped,
+            status: crate::modal::mcp_manager::McpStatus::Running,
             tool_count: 0,
             error: None,
             requires_auth: false,
@@ -8733,13 +8755,13 @@ impl EventLoop {
                     self.add_system_message(&format!("Restarted MCP server: {}", name));
                 }
                 CardAction::AddMcpServer(config) => {
-                    // Implement MCP server addition
+                    // Implement MCP server addition - MCP servers are enabled (Running) by default
                     tracing::info!("Adding MCP server: {}", config);
                     use crate::modal::mcp_manager::{McpServerInfo, McpStatus};
                     if !self.app_state.mcp_servers.iter().any(|s| s.name == config) {
                         self.app_state.mcp_servers.push(McpServerInfo {
                             name: config.clone(),
-                            status: McpStatus::Stopped,
+                            status: McpStatus::Running,
                             tool_count: 0,
                             error: None,
                             requires_auth: false,
@@ -9151,7 +9173,7 @@ impl EventLoop {
                 command,
                 args,
             } => {
-                // Implement MCP server addition
+                // Implement MCP server addition - MCP servers are enabled (Running) by default
                 use crate::modal::mcp_manager::{McpServerInfo, McpStatus};
 
                 // Check if server already exists
@@ -9163,7 +9185,7 @@ impl EventLoop {
                 // Create new server entry
                 let new_server = McpServerInfo {
                     name: name.clone(),
-                    status: McpStatus::Stopped,
+                    status: McpStatus::Running,
                     tool_count: 0,
                     error: None,
                     requires_auth: false,