fix(cortex-cli): consolidate mcp list command with ls alias

Fixes bounty issue #1399

The mcp subcommands had both 'list' and 'ls' as separate variants in the
McpSubcommand enum. This caused the help output to show them as separate
commands instead of documenting 'ls' as an alias for 'list'. This fix
removes the redundant Ls variant and adds visible_alias='ls' to the List
variant so the help correctly shows 'ls' as an alias.

Author: factorydroid

cortex-cli/src/mcp_cmd.rs

@@ -256,11 +256,8 @@ pub struct McpCli {
 #[derive(Debug, clap::Subcommand)]
 pub enum McpSubcommand {
     /// List configured MCP servers.
-    List(ListArgs),
-
-    /// List configured MCP servers (alias for list).
     #[command(visible_alias = "ls")]
-    Ls(ListArgs),
+    List(ListArgs),
 
     /// Show details for a configured MCP server.
     Get(GetArgs),
@@ -443,7 +440,7 @@ impl McpCli {
         } = self;
 
         match subcommand {
-            McpSubcommand::List(args) | McpSubcommand::Ls(args) => run_list(args).await,
+            McpSubcommand::List(args) => run_list(args).await,
             McpSubcommand::Get(args) => run_get(args).await,
             McpSubcommand::Add(args) => run_add(args).await,
             McpSubcommand::Remove(args) => run_remove(args).await,