4runr-os 2.4.0 → 2.4.2

package/mk3-tui/src/app.rs

@@ -29,6 +29,35 @@ pub enum AppMode {
     Main,
 }
 
+#[derive(Debug, Clone)]
+pub struct AgentListState {
+    pub agents: Vec<AgentInfo>,
+    pub selected_index: usize,
+    pub detail_view: Option<usize>,  // None = list view, Some(index) = detail popup
+}
+
+impl Default for AgentListState {
+    fn default() -> Self {
+        Self {
+            agents: Vec::new(),
+            selected_index: 0,
+            detail_view: None,
+        }
+    }
+}
+
+#[derive(Debug, Clone)]
+pub struct AgentInfo {
+    pub name: String,
+    pub description: Option<String>,
+    pub model: String,
+    pub provider: String,
+    pub system_prompt: Option<String>,
+    pub temperature: Option<f32>,
+    pub max_tokens: Option<u32>,
+    pub tools: Vec<String>,
+}
+
 #[derive(Debug, Clone)]
 pub struct AppState {
     // Navigation state (NEW - replaces simple mode)
@@ -96,6 +125,7 @@ pub struct AppState {
     pub agent_builder: AgentBuilderState,
     pub run_manager: RunManagerState,
     pub settings: SettingsState,
+    pub agent_list: AgentListState,
     
     // Local cache
     pub cache: Option<Cache>,
@@ -153,6 +183,7 @@ impl Default for AppState {
             agent_builder: AgentBuilderState::default(),
             run_manager: RunManagerState::default(),
             settings: SettingsState::default(),
+            agent_list: AgentListState::default(),
             cache: Cache::new().ok(),
             cache_loaded: false,
         }
@@ -339,6 +370,11 @@ impl App {
             return self.handle_settings_input(key);
         }
         
+        // === AGENT LIST INPUT HANDLING ===
+        if self.state.navigation.current_screen() == &Screen::AgentList {
+            return self.handle_agent_list_input(key);
+        }
+        
         // === MAIN INPUT HANDLING ===
         match key.code {
             // Typing - ALL characters go to command input (with debounce)
@@ -362,48 +398,9 @@ impl App {
                         "quit" | "exit" => return Ok(true),
                         "clear" => self.state.logs.clear(),
                         "help" => {
-                            self.state.logs.push_back("[HELP] ═══════════════════════════════════════".into());
-                            self.state.logs.push_back("[HELP] 4Runr AI Agent OS - Command Reference".into());
-                            self.state.logs.push_back("[HELP] ═══════════════════════════════════════".into());
-                            self.state.logs.push_back("".into());
-                            
-                            // Local Commands
-                            self.state.logs.push_back("[HELP] Local Commands:".into());
-                            self.state.logs.push_back("  quit, exit       - Exit application".into());
-                            self.state.logs.push_back("  clear            - Clear logs".into());
-                            self.state.logs.push_back("  help             - Show this help".into());
-                            self.state.logs.push_back("  :perf            - Show performance stats".into());
-                            self.state.logs.push_back("".into());
-                            
-                            // Navigation Commands
-                            self.state.logs.push_back("[HELP] Navigation Commands:".into());
-                            self.state.logs.push_back("  build            - Open Agent Builder (6-step wizard)".into());
-                            self.state.logs.push_back("  runs             - Open Run Manager (list, filter, sort)".into());
-                            self.state.logs.push_back("  config, settings - Open Settings (mode, AI provider)".into());
-                            self.state.logs.push_back("  ESC              - Close overlay/popup or clear input".into());
-                            self.state.logs.push_back("".into());
-                            
-                            // WebSocket Commands - Label BEFORE commands with clear separator
-                            self.state.logs.push_back("[HELP] ─────────────────────────────────────────".into());
-                            self.state.logs.push_back("[HELP] WebSocket Commands (requires connection):".into());
-                            self.state.logs.push_back("[HELP] ─────────────────────────────────────────".into());
-                            self.state.logs.push_back("  agent.list       - List all agents".into());
-                            self.state.logs.push_back("  agent.get        - Get agent details (data: {name})".into());
-                            self.state.logs.push_back("  agent.create     - Create agent (use Agent Builder)".into());
-                            self.state.logs.push_back("  agent.delete     - Delete agent (data: {name})".into());
-                            self.state.logs.push_back("  system.status    - Get system status".into());
-                            self.state.logs.push_back("  run.list         - List runs (requires gateway)".into());
-                            self.state.logs.push_back("  tool.list        - List available tools".into());
-                            self.state.logs.push_back("".into());
-                            
-                            // Screen Controls
-                            self.state.logs.push_back("[HELP] Screen Controls:".into());
-                            self.state.logs.push_back("  Agent Builder: Enter=Next, Backspace=Prev, ESC=Cancel".into());
-                            self.state.logs.push_back("  Run Manager: ↑/↓=Navigate, F=Filter, S=Sort, R=Refresh".into());
-                            self.state.logs.push_back("  Settings: ↑/↓=Navigate, Space=Toggle, Enter=Save".into());
-                            self.state.logs.push_back("".into());
-                            self.state.logs.push_back("[HELP] Press F12 to toggle performance overlay".into());
-                            self.state.logs.push_back("[HELP] ═══════════════════════════════════════".into());
+                            // Open help popup instead of logging to operations log
+                            self.push_popup(Screen::Help);
+                            self.request_render("help_command");
                         }
                         ":perf" => {
                             // Perf self-check command
@@ -591,6 +588,10 @@ impl App {
                 use crate::ui::settings;
                 settings::render(f, &self.state);
             }
+            Screen::AgentList => {
+                use crate::ui::agent_list;
+                agent_list::render(f, &self.state);
+            }
             Screen::Confirmation { message, action } => {
                 // TODO: Implement confirmation popup
                 // For now, just render the base screen
@@ -606,10 +607,13 @@ impl App {
                 let _ = message; // Suppress warning
             }
             Screen::Help => {
-                // TODO: Implement help popup
-                // For now, just render the base screen
+                // Render base screen first (dimmed)
                 use crate::ui::layout;
                 layout::render(f, &self.state);
+                
+                // Render help popup overlay
+                use crate::ui::help;
+                help::render_help(f, f.size(), &self.state);
             }
         }
     }
@@ -1034,20 +1038,17 @@ impl App {
             // Filter
             KeyCode::Char('f') | KeyCode::Char('F') => {
                 self.state.run_manager.next_filter();
-                self.add_log(format!("[RUN] Filter: {}", self.state.run_manager.filter.as_str()));
                 self.request_render("run_manager_filter");
             }
             
             // Sort
             KeyCode::Char('s') | KeyCode::Char('S') => {
                 self.state.run_manager.next_sort();
-                self.add_log(format!("[RUN] Sort: {}", self.state.run_manager.sort.as_str()));
                 self.request_render("run_manager_sort");
             }
             
             // Refresh
             KeyCode::Char('r') | KeyCode::Char('R') => {
-                self.add_log("[RUN] Refreshing run list...".to_string());
                 // TODO: Send run.list command via WebSocket
                 self.request_render("run_manager_refresh");
             }
@@ -1111,6 +1112,69 @@ impl App {
         Ok(false)
     }
     
+    // ============================================================
+    // AGENT LIST INPUT HANDLING (Step 4)
+    // ============================================================
+    
+    /// Handle input when Agent List screen is active
+    fn handle_agent_list_input(&mut self, key: KeyEvent) -> anyhow::Result<bool> {
+        use crossterm::event::KeyModifiers;
+        
+        // Ctrl+C/Q to exit
+        if key.modifiers.contains(KeyModifiers::CONTROL) {
+            match key.code {
+                KeyCode::Char('c') | KeyCode::Char('q') => return Ok(true),
+                _ => return Ok(false),
+            }
+        }
+        
+        match key.code {
+            // Navigation
+            KeyCode::Up => {
+                if self.state.agent_list.selected_index > 0 {
+                    self.state.agent_list.selected_index -= 1;
+                }
+                self.request_render("agent_list_up");
+            }
+            KeyCode::Down => {
+                let max = self.state.agent_list.agents.len().saturating_sub(1);
+                if self.state.agent_list.selected_index < max {
+                    self.state.agent_list.selected_index += 1;
+                }
+                self.request_render("agent_list_down");
+            }
+            
+            // View details / Close detail view
+            KeyCode::Enter => {
+                if self.state.agent_list.detail_view.is_some() {
+                    // Close detail view
+                    self.state.agent_list.detail_view = None;
+                } else {
+                    // Open detail view for selected agent
+                    self.state.agent_list.detail_view = Some(self.state.agent_list.selected_index);
+                }
+                self.request_render("agent_list_toggle_detail");
+            }
+            
+            // Close
+            KeyCode::Esc => {
+                if self.state.agent_list.detail_view.is_some() {
+                    // Close detail popup first
+                    self.state.agent_list.detail_view = None;
+                    self.request_render("agent_list_close_detail");
+                } else {
+                    // Close agent list
+                    self.pop_overlay();
+                    self.request_render("agent_list_close");
+                }
+            }
+            
+            _ => {}
+        }
+        
+        Ok(false)
+    }
+    
     // ============================================================
     // SETTINGS INPUT HANDLING (Step 4.8)
     // ============================================================

package/mk3-tui/src/main.rs

@@ -151,12 +151,35 @@ fn main() -> Result<()> {
                                                 })
                                                 .collect();
                                             
+                                            // Parse agents into AgentInfo structs for Agent List viewer
+                                            use crate::app::AgentInfo;
+                                            let agents: Vec<AgentInfo> = agents_array.iter()
+                                                .filter_map(|agent| {
+                                                    let obj = agent.as_object()?;
+                                                    Some(AgentInfo {
+                                                        name: obj.get("name")?.as_str()?.to_string(),
+                                                        description: obj.get("description").and_then(|d| d.as_str()).map(|s| s.to_string()),
+                                                        model: obj.get("model").and_then(|m| m.as_str()).unwrap_or("unknown").to_string(),
+                                                        provider: obj.get("provider").and_then(|p| p.as_str()).unwrap_or("unknown").to_string(),
+                                                        system_prompt: obj.get("systemPrompt").and_then(|sp| sp.as_str()).map(|s| s.to_string()),
+                                                        temperature: obj.get("temperature").and_then(|t| t.as_f64()).map(|f| f as f32),
+                                                        max_tokens: obj.get("maxTokens").and_then(|mt| mt.as_u64()).map(|u| u as u32),
+                                                        tools: obj.get("tools")
+                                                            .and_then(|t| t.as_array())
+                                                            .map(|arr| arr.iter()
+                                                                .filter_map(|v| v.as_str().map(|s| s.to_string()))
+                                                                .collect())
+                                                            .unwrap_or_default(),
+                                                    })
+                                                })
+                                                .collect();
+                                            
                                             // Update cache with agent data
                                             if let Some(cache) = &mut app.state.cache {
                                                 use crate::storage::cache::AgentData;
                                                 use std::time::{SystemTime, UNIX_EPOCH};
                                                 
-                                                let agents: Vec<AgentData> = agents_array.iter()
+                                                let cache_agents: Vec<AgentData> = agents_array.iter()
                                                     .filter_map(|agent| {
                                                         let obj = agent.as_object()?;
                                                         Some(AgentData {
@@ -169,14 +192,26 @@ fn main() -> Result<()> {
                                                     })
                                                     .collect();
                                                 
-                                                let _ = cache.update_agents(agents);
+                                                let _ = cache.update_agents(cache_agents);
                                             }
                                             
-                                            app.add_log(format!(
-                                                "✓ [{}] Loaded {} agents",
-                                                short_id,
-                                                agents_array.len()
-                                            ));
+                                            // Update agent list state
+                                            app.state.agent_list.agents = agents;
+                                            app.state.agent_list.selected_index = 0;
+                                            app.state.agent_list.detail_view = None;
+                                            
+                                            // Open AgentList overlay
+                                            app.push_overlay(Screen::AgentList);
+                                            app.request_render("agent_list_opened");
+                                            
+                                            // Only log if agents were actually loaded
+                                            if agents_array.len() > 0 {
+                                                app.add_log(format!(
+                                                    "✓ [{}] Loaded {} agents",
+                                                    short_id,
+                                                    agents_array.len()
+                                                ));
+                                            }
                                         }
                                     } else {
                                         app.add_log(format!("✓ [{}] Success", short_id));

package/mk3-tui/src/screens/mod.rs

@@ -19,6 +19,7 @@ pub enum Screen {
     AgentBuilder,
     RunManager,
     Settings,
+    AgentList,
     
     // Popup screens (small overlays)
     Confirmation { message: String, action: String },
@@ -31,7 +32,7 @@ impl Screen {
     pub fn is_overlay(&self) -> bool {
         matches!(
             self,
-            Screen::AgentBuilder | Screen::RunManager | Screen::Settings
+            Screen::AgentBuilder | Screen::RunManager | Screen::Settings | Screen::AgentList
         )
     }
     
@@ -56,6 +57,7 @@ impl Screen {
             Screen::AgentBuilder => "Agent Builder",
             Screen::RunManager => "Run Manager",
             Screen::Settings => "Settings",
+            Screen::AgentList => "Agent List",
             Screen::Confirmation { .. } => "Confirmation",
             Screen::Alert { .. } => "Alert",
             Screen::Help => "Help",