4runr-os 2.9.49 → 2.9.51

package/mk3-tui/src/app.rs

@@ -343,7 +343,7 @@ pub struct SetupPortalState {
     pub detecting: bool,
     pub detection_result: Option<DetectionResult>,
     pub error: Option<String>,
-    pub list_state: ratatui::widgets::ListState,  // CRITICAL: Must persist across frames
+    pub list_state: ratatui::widgets::ListState,
 }
 
 #[derive(Debug, Clone)]
@@ -382,7 +382,6 @@ impl Default for SetupPortalState {
 impl SetupPortalState {
     pub fn cycle_option(&mut self, direction: i8) {
         use GatewayOption::*;
-        let prev = self.selected_option.clone();
         self.selected_option = match (&self.selected_option, direction) {
             // Forward cycling (Down key, Scroll Down)
             (LocalBundle, 1) => CloudServer,
@@ -402,9 +401,6 @@ impl SetupPortalState {
             CustomUrl => 2,
         };
         self.list_state.select(Some(new_index));
-        
-        eprintln!("[SETUP-PORTAL] cycle_option direction={} prev={:?} -> new={:?} list_index={}", 
-            direction, prev, self.selected_option, new_index);
     }
     
     pub fn reset(&mut self) {
@@ -1816,7 +1812,6 @@ impl App {
     // ============================================================
     
     fn handle_setup_portal_input(&mut self, key: KeyEvent, _ws_client: Option<&WebSocketClient>) -> anyhow::Result<bool> {
-        eprintln!("[SETUP-PORTAL] INPUT key={:?} (selected_option={:?})", key.code, self.state.setup_portal.selected_option);
         // Guard: Don't process if already detecting
         if self.state.setup_portal.detecting {
             return Ok(false);
@@ -1841,13 +1836,11 @@ impl App {
             
             // Up/Down - Navigate options (immediate render for responsiveness)
             KeyCode::Up => {
-                eprintln!("[SETUP-PORTAL] KEY Up -> cycle_option(-1)");
                 self.state.setup_portal.cycle_option(-1);
                 self.request_immediate_render("setup_nav_up");
             }
             
             KeyCode::Down => {
-                eprintln!("[SETUP-PORTAL] KEY Down -> cycle_option(1)");
                 self.state.setup_portal.cycle_option(1);
                 self.request_immediate_render("setup_nav_down");
             }

package/mk3-tui/src/main.rs

@@ -591,16 +591,13 @@ fn main() -> Result<()> {
             let current_screen = app.state.navigation.current_screen().clone();
             let is_portal_active = matches!(current_screen, crate::screens::Screen::ConnectionPortal | crate::screens::Screen::SetupPortal);
             
-            // Clear terminal on screen switch OR if Setup Portal is active
-            // Setup Portal needs per-frame clear to prevent terminal buffering artifacts
+            // Clear only on screen switch (no clear on Setup Portal nav = no flicker)
             let is_switching = match (&previous_screen, &current_screen) {
                 (Some(prev), current) => prev != current,
-                (None, _) => true, // First render
+                (None, _) => true,
             };
-            
-            let force_clear_for_setup = matches!(current_screen, crate::screens::Screen::SetupPortal);
-            
-            if is_switching || force_clear_for_setup {
+
+            if is_switching {
                 terminal.clear()?;
             }
             
@@ -633,12 +630,10 @@ fn main() -> Result<()> {
                         if !app.state.setup_portal.detecting {
                             match mouse.kind {
                                 MouseEventKind::ScrollUp => {
-                                    eprintln!("[SETUP-PORTAL] MOUSE ScrollUp -> cycle_option(-1)");
                                     app.state.setup_portal.cycle_option(-1);
                                     app.request_immediate_render("setup_portal_scroll_up");
                                 }
                                 MouseEventKind::ScrollDown => {
-                                    eprintln!("[SETUP-PORTAL] MOUSE ScrollDown -> cycle_option(1)");
                                     app.state.setup_portal.cycle_option(1);
                                     app.request_immediate_render("setup_portal_scroll_down");
                                 }

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

@@ -144,7 +144,7 @@ fn render_header(f: &mut Frame, area: Rect, state: &AppState) {
     
     // Line 1: Brand + version + mode + uptime - Bug 3 fix: Use "4Runr." with dot (matches brand logo)
     // Use npm package version (2.9.24) - matches package.json
-    const PACKAGE_VERSION: &str = "2.9.49";
+    const PACKAGE_VERSION: &str = "2.9.51";
     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/setup_portal.rs

@@ -18,40 +18,22 @@ const ERROR_RED: Color = Color::Rgb(255, 69, 69);
 
 /// Render the Setup Portal screen - Full-screen standalone portal
 pub fn render(f: &mut Frame, state: &mut AppState) {
-    // Per-frame render ID so we can correlate all logs from the same draw
+    // Per-frame render ID (used for passing to content/details; debug logs removed to avoid terminal spam)
     static mut RENDER_ID: u64 = 0;
     let render_id = unsafe {
         RENDER_ID += 1;
         RENDER_ID
     };
-    eprintln!("[SETUP-PORTAL] R#{} START selected_option={:?}", render_id, state.setup_portal.selected_option);
-
-    static mut RENDER_COUNT: u64 = 0;
-    unsafe {
-        RENDER_COUNT += 1;
-        if RENDER_COUNT % 10 == 1 {
-            eprintln!("[SETUP-PORTAL] Render #{}", RENDER_COUNT);
-        }
-    }
 
     let area = f.size();
 
-    unsafe {
-        if RENDER_COUNT % 10 == 1 {
-            eprintln!("[SETUP-PORTAL] Terminal size: {}x{}", area.width, area.height);
-        }
-    }
-    
     // CRITICAL: Validate terminal size before rendering
     // This prevents character corruption when terminal size is wrong on initial render
     if area.width == 0 || area.height == 0 {
-        eprintln!("[SETUP-PORTAL] ⚠️ Terminal size invalid: {}x{} - skipping render", area.width, area.height);
         return;
     }
-    
-    // Ensure minimum size for portal (85x25)
+
     if area.width < 85 || area.height < 25 {
-        eprintln!("[SETUP-PORTAL] ⚠️ Terminal too small: {}x{} (required: 85x25)", area.width, area.height);
         render_too_small(f, area);
         return;
     }
@@ -76,15 +58,7 @@ pub fn render(f: &mut Frame, state: &mut AppState) {
         width: portal_width,
         height: portal_height,
     };
-    
-    // DEBUG: Log portal area calculations
-    unsafe {
-        if RENDER_COUNT % 10 == 1 {
-            eprintln!("[SETUP-PORTAL] Portal area: x={}, y={}, w={}, h={}", 
-                portal_area.x, portal_area.y, portal_area.width, portal_area.height);
-        }
-    }
-    
+
     // Split portal into sections
     use ratatui::layout::{Constraint, Direction, Layout};
     
@@ -111,31 +85,7 @@ pub fn render(f: &mut Frame, state: &mut AppState) {
         .direction(Direction::Vertical)
         .constraints(constraints)
         .split(portal_area);
-    
-    // DEBUG: Log layout chunks and selected option
-    unsafe {
-        if RENDER_COUNT % 10 == 1 {
-            eprintln!("[SETUP-PORTAL] Selected option: {:?}", state.setup_portal.selected_option);
-            eprintln!("[SETUP-PORTAL] Has error: {}", has_error);
-            eprintln!("[SETUP-PORTAL] Chunks count: {}", chunks.len());
-            for (i, chunk) in chunks.iter().enumerate() {
-                eprintln!("[SETUP-PORTAL]   chunks[{}]: x={}, y={}, w={}, h={}", 
-                    i, chunk.x, chunk.y, chunk.width, chunk.height);
-            }
-            // Check for overlaps
-            for i in 0..chunks.len() {
-                for j in (i+1)..chunks.len() {
-                    let a = &chunks[i];
-                    let b = &chunks[j];
-                    if a.x < b.x + b.width && a.x + a.width > b.x &&
-                       a.y < b.y + b.height && a.y + a.height > b.y {
-                        eprintln!("[SETUP-PORTAL] ⚠️ OVERLAP: chunks[{}] overlaps chunks[{}]!", i, j);
-                    }
-                }
-            }
-        }
-    }
-    
+
     render_header(f, chunks[0], state);
     render_description(f, chunks[1]);
     render_content(f, chunks[2], state, render_id);
@@ -188,14 +138,6 @@ fn render_description(f: &mut Frame, area: Rect) {
 }
 
 fn render_content(f: &mut Frame, area: Rect, state: &mut AppState, render_id: u64) {
-    static mut LOG_COUNT: u64 = 0;
-    unsafe {
-        LOG_COUNT += 1;
-        if LOG_COUNT % 10 == 1 {
-            eprintln!("[SETUP-PORTAL] R#{} render_content area: {}x{}", render_id, area.width, area.height);
-        }
-    }
-
     let chunks = Layout::default()
         .direction(Direction::Horizontal)
         .constraints([
@@ -223,8 +165,6 @@ fn render_options_list(f: &mut Frame, area: Rect, state: &mut AppState, render_i
         .position(|(_, opt)| opt == selected_option)
         .unwrap_or(0);
 
-    eprintln!("[SETUP-PORTAL] R#{} LIST selected_index={} selected_option={:?}", render_id, selected_index, state.setup_portal.selected_option);
-
     let items: Vec<ListItem> = options
         .iter()
         .map(|(name, option)| {
@@ -257,9 +197,7 @@ fn render_options_list(f: &mut Frame, area: Rect, state: &mut AppState, render_i
     f.render_widget(list, area);
 }
 
-fn render_option_details(f: &mut Frame, area: Rect, state: &AppState, render_id: u64) {
-    eprintln!("[SETUP-PORTAL] R#{} DETAILS selected_option={:?}", render_id, state.setup_portal.selected_option);
-
+fn render_option_details(f: &mut Frame, area: Rect, state: &AppState, _render_id: u64) {
     // CRITICAL: Fill entire area with background so every cell is overwritten.
     // Without this, Paragraph may not paint every cell; leftover shows previous option (corruption).
     let fill = Block::default().style(Style::default().bg(BG_PANEL));
@@ -350,8 +288,6 @@ fn render_option_details(f: &mut Frame, area: Rect, state: &AppState, render_id:
         ),
     };
 
-    eprintln!("[SETUP-PORTAL] R#{} DETAILS title='{}' lines={}", render_id, title.trim(), content.len());
-
     // Use Block to properly contain and clear the area
     let block = Block::default()
         .title(title)
@@ -370,7 +306,6 @@ fn render_option_details(f: &mut Frame, area: Rect, state: &AppState, render_id:
     let terminal_size = f.size();
     if area.width == 0 || area.height == 0 || 
        area.x >= terminal_size.width || area.y >= terminal_size.height {
-        eprintln!("[SETUP-PORTAL]   ⚠️ ERROR: Invalid area, skipping render");
         return;
     }
     

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "4runr-os",
-  "version": "2.9.49",
+  "version": "2.9.51",
   "type": "module",
-  "description": "4Runr AI Agent OS - Secure terminal interface for AI agents. v2.9.49: Setup Portal - per-frame terminal clear to fix list/details mismatch (terminal buffering). v2.9.48: Stateless list, R# debug. ⚠️ Pre-MVP / Development Phase",
+  "description": "4Runr AI Agent OS - Secure terminal interface for AI agents. v2.9.51: Setup Portal - no clear on nav (no flicker), removed debug logs. v2.9.50: Clear only when selection changed. ⚠️ Pre-MVP / Development Phase",
   "main": "dist/index.js",
   "bin": {
     "4runr": "dist/index.js",