fix(agents): sort agent list alphabetically for deterministic output

Fixes bounty issue #1510

The agent list command returned agents in non-deterministic order because
HashMap iteration and fs::read_dir() don't guarantee ordering. This caused
CI snapshot failures when comparing outputs across runs.

Changes:
- Sort filtered agents by name in run_list() before displaying
- Sort results in AgentRegistry::list(), list_primary(), list_subagents()
- Sort results in AgentRegistry::names()

Author: factorydroid

cortex-agents/src/registry.rs

@@ -135,29 +135,37 @@ impl AgentRegistry {
 
     /// List all agents.
     pub async fn list(&self) -> Vec<AgentInfo> {
-        self.agents.read().await.values().cloned().collect()
+        let mut agents: Vec<AgentInfo> = self.agents.read().await.values().cloned().collect();
+        agents.sort_by(|a, b| a.name.cmp(&b.name));
+        agents
     }
 
     /// List primary agents (user-facing).
     pub async fn list_primary(&self) -> Vec<AgentInfo> {
-        self.agents
+        let mut agents: Vec<AgentInfo> = self
+            .agents
             .read()
             .await
             .values()
             .filter(|a| !a.hidden && matches!(a.mode, AgentMode::Primary | AgentMode::All))
             .cloned()
-            .collect()
+            .collect();
+        agents.sort_by(|a, b| a.name.cmp(&b.name));
+        agents
     }
 
     /// List sub-agents.
     pub async fn list_subagents(&self) -> Vec<AgentInfo> {
-        self.agents
+        let mut agents: Vec<AgentInfo> = self
+            .agents
             .read()
             .await
             .values()
             .filter(|a| matches!(a.mode, AgentMode::Subagent | AgentMode::All))
             .cloned()
-            .collect()
+            .collect();
+        agents.sort_by(|a, b| a.name.cmp(&b.name));
+        agents
     }
 
     /// Unregister an agent.
@@ -172,7 +180,9 @@ impl AgentRegistry {
 
     /// Get agent names.
     pub async fn names(&self) -> Vec<String> {
-        self.agents.read().await.keys().cloned().collect()
+        let mut names: Vec<String> = self.agents.read().await.keys().cloned().collect();
+        names.sort();
+        names
     }
 
     /// Get agent by name if it's a valid subagent.

cortex-cli/src/agent_cmd.rs

@@ -668,7 +668,7 @@ async fn run_list(args: ListArgs) -> Result<()> {
     let agents = load_all_agents()?;
 
     // Filter agents
-    let filtered: Vec<_> = agents
+    let mut filtered: Vec<_> = agents
         .iter()
         .filter(|a| {
             // Filter by visibility
@@ -686,6 +686,9 @@ async fn run_list(args: ListArgs) -> Result<()> {
         })
         .collect();
 
+    // Sort agents alphabetically by name for deterministic output
+    filtered.sort_by(|a, b| a.name.cmp(&b.name));
+
     if args.json {
         let json = serde_json::to_string_pretty(&filtered)?;
         println!("{json}");