4runr-os 2.5.0 → 2.5.2

package/mk3-tui/src/app.rs

@@ -682,17 +682,20 @@ impl App {
     }
     
     /// Close all overlays and popups (return to base screen)
+    #[allow(dead_code)]
     pub fn close_all_overlays(&mut self) {
         self.state.navigation.close_all();
         self.request_render("close_all");
     }
     
     /// Get the currently visible screen
+    #[allow(dead_code)]
     pub fn current_screen(&self) -> &Screen {
         self.state.navigation.current_screen()
     }
     
     /// Check if we're on the base screen (no overlays/popups)
+    #[allow(dead_code)]
     pub fn is_on_base_screen(&self) -> bool {
         self.state.navigation.is_on_base()
     }

package/mk3-tui/src/main.rs

@@ -187,14 +187,17 @@ fn main() -> Result<()> {
                                                 let _ = cache.update_agents(cache_agents);
                                             }
                                             
-                                            // Update agent list state and open AgentList overlay
+                                            // Update agent list state
                                             app.state.agent_list.agents = agents;
                                             app.state.agent_list.selected_index = 0;
                                             app.state.agent_list.detail_view = None;
                                             
+                                            // Only open AgentList overlay if we're on Main screen (not during boot)
                                             use crate::screens::Screen;
-                                            app.push_overlay(Screen::AgentList);
-                                            app.request_render("agent_list_opened");
+                                            if app.state.navigation.current_screen() == &Screen::Main {
+                                                app.push_overlay(Screen::AgentList);
+                                                app.request_render("agent_list_opened");
+                                            }
                                         }
                                     } else {
                                         app.add_log(format!("✓ [{}] Success", short_id));

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

@@ -50,6 +50,7 @@ impl Screen {
     }
     
     /// Returns the screen name for display
+    #[allow(dead_code)]
     pub fn name(&self) -> &str {
         match self {
             Screen::Boot => "Boot",
@@ -143,12 +144,14 @@ impl NavigationState {
     }
     
     /// Close all overlays and popups (return to base screen)
+    #[allow(dead_code)]
     pub fn close_all(&mut self) {
         self.overlay_stack.clear();
         self.popup_stack.clear();
     }
     
     /// Check if we're currently on the base screen (no overlays/popups)
+    #[allow(dead_code)]
     pub fn is_on_base(&self) -> bool {
         self.overlay_stack.is_empty() && self.popup_stack.is_empty()
     }

package/mk3-tui/src/storage/cache.rs

@@ -121,6 +121,7 @@ impl Cache {
     }
     
     /// Add a single agent
+    #[allow(dead_code)]
     pub fn add_agent(&mut self, agent: AgentData) -> Result<(), std::io::Error> {
         self.data.agents.push(agent);
         self.data.last_updated = Self::now();
@@ -128,6 +129,7 @@ impl Cache {
     }
     
     /// Remove an agent by name
+    #[allow(dead_code)]
     pub fn remove_agent(&mut self, name: &str) -> Result<(), std::io::Error> {
         self.data.agents.retain(|a| a.name != name);
         self.data.last_updated = Self::now();
@@ -137,11 +139,13 @@ impl Cache {
     // === Run Operations ===
     
     /// Get all cached runs
+    #[allow(dead_code)]
     pub fn get_runs(&self) -> Vec<RunData> {
         self.data.runs.clone()
     }
     
     /// Update runs cache
+    #[allow(dead_code)]
     pub fn update_runs(&mut self, runs: Vec<RunData>) -> Result<(), std::io::Error> {
         self.data.runs = runs;
         self.data.last_updated = Self::now();
@@ -149,6 +153,7 @@ impl Cache {
     }
     
     /// Add a single run
+    #[allow(dead_code)]
     pub fn add_run(&mut self, run: RunData) -> Result<(), std::io::Error> {
         self.data.runs.push(run);
         self.data.last_updated = Self::now();
@@ -156,6 +161,7 @@ impl Cache {
     }
     
     /// Update a run by ID
+    #[allow(dead_code)]
     pub fn update_run(&mut self, id: &str, run: RunData) -> Result<(), std::io::Error> {
         if let Some(existing) = self.data.runs.iter_mut().find(|r| r.id == id) {
             *existing = run;
@@ -173,6 +179,7 @@ impl Cache {
     }
     
     /// Update system status cache
+    #[allow(dead_code)]
     pub fn update_system_status(&mut self, status: SystemStatusData) -> Result<(), std::io::Error> {
         self.data.system_status = Some(status);
         self.data.last_updated = Self::now();
@@ -182,22 +189,26 @@ impl Cache {
     // === Cache Management ===
     
     /// Get cache age in seconds
+    #[allow(dead_code)]
     pub fn get_age(&self) -> u64 {
         Self::now().saturating_sub(self.data.last_updated)
     }
     
     /// Check if cache is stale (older than threshold)
+    #[allow(dead_code)]
     pub fn is_stale(&self, threshold_secs: u64) -> bool {
         self.get_age() > threshold_secs
     }
     
     /// Clear all cached data
+    #[allow(dead_code)]
     pub fn clear(&mut self) -> Result<(), std::io::Error> {
         self.data = CacheData::default();
         self.save_to_file()
     }
     
     /// Get cache data reference
+    #[allow(dead_code)]
     pub fn data(&self) -> &CacheData {
         &self.data
     }

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

@@ -3,4 +3,4 @@
 
 pub mod cache;
 
-pub use cache::{Cache, CacheData};
+pub use cache::Cache;

package/mk3-tui/src/ui/agent_builder.rs

@@ -7,6 +7,7 @@ use crate::app::AppState;
 
 // === 4RUNR BRAND COLORS (matching layout.rs) ===
 const BRAND_PURPLE: Color = Color::Rgb(138, 43, 226);
+#[allow(dead_code)]
 const BRAND_VIOLET: Color = Color::Rgb(148, 103, 189);
 const CYBER_CYAN: Color = Color::Rgb(0, 255, 255);
 const NEON_GREEN: Color = Color::Rgb(57, 255, 20);
@@ -108,7 +109,7 @@ struct CostEstimate {
     is_free: bool,
 }
 
-fn calculate_cost(provider: &str, model: &str, max_tokens: u32) -> CostEstimate {
+fn calculate_cost(provider: &str, model: &str, _max_tokens: u32) -> CostEstimate {
     let pricing = get_model_pricing(provider, model);
     let is_free = pricing.input_cost == 0.0 && pricing.output_cost == 0.0;
     

package/mk3-tui/src/ui/agent_list.rs

@@ -57,9 +57,8 @@ fn render_list_view(f: &mut Frame, area: Rect, state: &AppState) {
         .border_style(Style::default().fg(TEXT_MUTED))
         .style(Style::default().bg(BG_PANEL));
     
-    f.render_widget(content_block, chunks[1]);
-    
     let table_area = content_block.inner(chunks[1]);
+    f.render_widget(content_block, chunks[1]);
     
     if state.agent_list.agents.is_empty() {
         // Show "No agents" message
@@ -146,7 +145,6 @@ fn render_detail_popup(f: &mut Frame, area: Rect, state: &AppState) {
 }
 
 fn render_agent_detail(f: &mut Frame, area: Rect, agent: &AgentInfo) {
-    use ratatui::layout::{Constraint, Direction, Layout};
     
     // Calculate popup size (70% width, 80% height, centered)
     let popup_width = (area.width * 70 / 100).max(50);
@@ -181,9 +179,8 @@ fn render_agent_detail(f: &mut Frame, area: Rect, agent: &AgentInfo) {
         .border_style(Style::default().fg(CYBER_CYAN))
         .style(Style::default().bg(BG_PANEL));
     
-    f.render_widget(detail_block, popup_area);
-    
     let inner = detail_block.inner(popup_area);
+    f.render_widget(detail_block, popup_area);
     
     // Create detail text
     let mut detail_lines = vec![

package/mk3-tui/src/ui/help.rs

@@ -138,8 +138,9 @@ fn render_command_list(f: &mut Frame, area: Rect) {
     
     let nav_list = List::new(nav_items);
     
+    let nav_inner = nav_block.inner(chunks[0]);
     f.render_widget(nav_block, chunks[0]);
-    f.render_widget(nav_list, nav_block.inner(chunks[0]));
+    f.render_widget(nav_list, nav_inner);
     
     // Local Commands section
     let local_block = Block::default()
@@ -158,8 +159,9 @@ fn render_command_list(f: &mut Frame, area: Rect) {
     
     let local_list = List::new(local_items);
     
+    let local_inner = local_block.inner(chunks[1]);
     f.render_widget(local_block, chunks[1]);
-    f.render_widget(local_list, local_block.inner(chunks[1]));
+    f.render_widget(local_list, local_inner);
     
     // WebSocket Commands section
     let ws_block = Block::default()
@@ -181,8 +183,9 @@ fn render_command_list(f: &mut Frame, area: Rect) {
     
     let ws_list = List::new(ws_items);
     
+    let ws_inner = ws_block.inner(chunks[2]);
     f.render_widget(ws_block, chunks[2]);
-    f.render_widget(ws_list, ws_block.inner(chunks[2]));
+    f.render_widget(ws_list, ws_inner);
 }
 
 fn render_command_details(f: &mut Frame, area: Rect) {
@@ -242,11 +245,12 @@ fn render_command_details(f: &mut Frame, area: Rect) {
         ]),
     ];
     
+    let desc_inner = desc_block.inner(chunks[0]);
     f.render_widget(desc_block, chunks[0]);
     f.render_widget(
         Paragraph::new(desc_text)
             .wrap(Wrap { trim: false }),
-        desc_block.inner(chunks[0])
+        desc_inner
     );
     
     // Screen controls
@@ -300,10 +304,11 @@ fn render_command_details(f: &mut Frame, area: Rect) {
         ]),
     ];
     
+    let controls_inner = controls_block.inner(chunks[1]);
     f.render_widget(controls_block, chunks[1]);
     f.render_widget(
         Paragraph::new(controls_text)
             .wrap(Wrap { trim: false }),
-        controls_block.inner(chunks[1])
+        controls_inner
     );
 }

package/mk3-tui/src/ui/layout.rs

@@ -120,7 +120,7 @@ fn render_header(f: &mut Frame, area: Rect, state: &AppState) {
     
     // Line 1: Brand + version + uptime - Bug 3 fix: Use "4Runr." with dot (matches brand logo)
     // Use npm package version (2.3.5) - matches package.json
-    const PACKAGE_VERSION: &str = "2.5.0";
+    const PACKAGE_VERSION: &str = "2.5.2";
     let brand_line = Line::from(vec![
         Span::styled("4Runr.", Style::default().fg(BRAND_PURPLE).add_modifier(Modifier::BOLD)),
         Span::styled(" AI AGENT OS", Style::default().fg(BRAND_VIOLET)),

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

@@ -7,8 +7,10 @@ pub mod run_manager;
 pub mod safe_viewport;
 pub mod settings;
 
-// Re-export screen states
+// Re-export screen states (used in app.rs)
+#[allow(dead_code)]
 pub use agent_builder::AgentBuilderState;
+#[allow(dead_code)]
 pub use run_manager::RunManagerState;
+#[allow(dead_code)]
 pub use settings::SettingsState;
-

package/mk3-tui/src/ui/run_manager.rs

@@ -31,9 +31,11 @@ pub struct RunManagerState {
     pub sort: RunSort,
     
     /// Loading state
+    #[allow(dead_code)]
     pub loading: bool,
     
     /// Last refresh time
+    #[allow(dead_code)]
     pub last_refresh: Option<std::time::Instant>,
     
     /// Detailed view state (None = list view, Some(index) = detail view)
@@ -61,10 +63,12 @@ pub struct RunInfo {
 
 #[derive(Debug, Clone, PartialEq)]
 pub enum RunStatus {
+    #[allow(dead_code)]
     Pending,
     Running,
     Completed,
     Failed,
+    #[allow(dead_code)]
     Cancelled,
 }
 

package/mk3-tui/src/websocket.rs

@@ -73,11 +73,14 @@ pub enum WsClientMessage {
     Event(EventMessage),
     Error(String),
     ConnectionLost,
+    #[allow(dead_code)]
     Reconnecting,
+    #[allow(dead_code)]
     ReconnectFailed(String),
 }
 
 pub struct WebSocketClient {
+    #[allow(dead_code)]
     url: String,
     tx: mpsc::UnboundedSender<WsMessage>,
     rx: Arc<Mutex<mpsc::UnboundedReceiver<WsClientMessage>>>,
@@ -211,6 +214,7 @@ impl WebSocketClient {
     }
     
     /// Retry pending commands after reconnection
+    #[allow(dead_code)]
     pub fn retry_pending_commands(&self) -> Result<Vec<String>> {
         let mut pending = self.pending_commands.lock().unwrap();
         let mut retried_ids = Vec::new();
@@ -239,6 +243,7 @@ impl WebSocketClient {
     }
     
     /// Clear pending commands (e.g., on explicit disconnect)
+    #[allow(dead_code)]
     pub fn clear_pending_commands(&self) {
         self.pending_commands.lock().unwrap().clear();
     }

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "4runr-os",
-  "version": "2.5.0",
+  "version": "2.5.2",
   "type": "module",
-  "description": "4Runr AI Agent OS - Secure terminal interface for AI agents. v2.5.0: NEW professional help popup interface, NEW Agent List viewer with detail popup for agent.list command. Enhanced UX with clean navigation and organized displays. Built with Rust + Ratatui. ⚠️ Pre-MVP / Development Phase",
+  "description": "4Runr AI Agent OS - Secure terminal interface for AI agents. v2.5.2: Fixed loading screen being overshadowed by agent list during boot. Resolved all compilation warnings. Professional help popup interface, Agent List viewer with detail popup. Enhanced UX with clean navigation. Built with Rust + Ratatui. ⚠️ Pre-MVP / Development Phase",
   "main": "dist/index.js",
   "bin": {
     "4runr": "dist/index.js",