4runr-os 2.9.45 → 2.9.46

package/mk3-tui/src/app.rs

@@ -343,6 +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
 }
 
 #[derive(Debug, Clone)]
@@ -366,11 +367,14 @@ pub struct CloudStatus {
 
 impl Default for SetupPortalState {
     fn default() -> Self {
+        let mut list_state = ratatui::widgets::ListState::default();
+        list_state.select(Some(0));  // Start with first option selected
         Self {
             selected_option: GatewayOption::LocalBundle,
             detecting: false,
             detection_result: None,
             error: None,
+            list_state,
         }
     }
 }
@@ -390,7 +394,17 @@ impl SetupPortalState {
             (CustomUrl, -1) => CloudServer,
             _ => self.selected_option.clone(),
         };
-        eprintln!("[SETUP-PORTAL] cycle_option direction={} prev={:?} -> new={:?}", direction, prev, self.selected_option);
+        
+        // CRITICAL: Update the persisted ListState index
+        let new_index = match self.selected_option {
+            LocalBundle => 0,
+            CloudServer => 1,
+            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) {
@@ -917,7 +931,7 @@ impl App {
         }
     }
 
-    pub fn render(&self, f: &mut Frame) {
+    pub fn render(&mut self, f: &mut Frame) {
         // Use new navigation system - render based on current screen
         let current = self.state.navigation.current_screen();
         
@@ -970,7 +984,7 @@ impl App {
                 // Render Setup Portal as standalone screen (not overlay)
                 // This eliminates double-rendering and improves performance
                 use crate::ui::setup_portal;
-                setup_portal::render(f, &self.state);
+                setup_portal::render(f, &mut self.state);
             }
             Screen::Confirmation { message, action } => {
                 // TODO: Implement confirmation popup

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.45";
+    const PACKAGE_VERSION: &str = "2.9.46";
     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

@@ -17,7 +17,7 @@ const BG_PANEL: Color = Color::Rgb(18, 18, 25);
 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: &AppState) {
+pub fn render(f: &mut Frame, state: &mut AppState) {
     // DEBUG: Every frame log the selection state used for this draw (no throttle)
     eprintln!("[SETUP-PORTAL] FRAME selected_option={:?}", state.setup_portal.selected_option);
 
@@ -182,7 +182,7 @@ fn render_description(f: &mut Frame, area: Rect) {
     f.render_widget(text, inner);
 }
 
-fn render_content(f: &mut Frame, area: Rect, state: &AppState) {
+fn render_content(f: &mut Frame, area: Rect, state: &mut AppState) {
     // DEBUG: Log content area
     static mut LOG_COUNT: u64 = 0;
     unsafe {
@@ -216,7 +216,7 @@ fn render_content(f: &mut Frame, area: Rect, state: &AppState) {
     render_option_details(f, chunks[1], state);
 }
 
-fn render_options_list(f: &mut Frame, area: Rect, state: &AppState) {
+fn render_options_list(f: &mut Frame, area: Rect, state: &mut AppState) {
     // DEBUG: Log options list rendering
     static mut LOG_COUNT: u64 = 0;
     unsafe {
@@ -274,9 +274,11 @@ fn render_options_list(f: &mut Frame, area: Rect, state: &AppState) {
         })
         .collect();
     
-    // Create ListState with selected index
-    let mut list_state = ListState::default();
-    list_state.select(Some(selected_index));
+    // CRITICAL: Use the PERSISTED ListState from app state (not a new one each frame)
+    // This is required for Ratatui's stateful widget to maintain selection across frames
+    let list_state_selected = state.setup_portal.list_state.selected();
+    eprintln!("[SETUP-PORTAL] LIST About to render: list_state.selected()={:?} selected_index={} selected_option={:?}", 
+        list_state_selected, selected_index, state.setup_portal.selected_option);
     
     // Use Block to properly contain and clear the area
     let block = Block::default()
@@ -289,7 +291,8 @@ fn render_options_list(f: &mut Frame, area: Rect, state: &AppState) {
         .block(block)
         .highlight_style(Style::default().fg(CYBER_CYAN).bold().bg(BG_PANEL));
     
-    f.render_stateful_widget(list, area, &mut list_state);
+    // CRITICAL: Pass the mutable reference to the persisted ListState
+    f.render_stateful_widget(list, area, &mut state.setup_portal.list_state);
 }
 
 fn render_option_details(f: &mut Frame, area: Rect, state: &AppState) {

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "4runr-os",
-  "version": "2.9.45",
+  "version": "2.9.46",
   "type": "module",
-  "description": "4Runr AI Agent OS - Secure terminal interface for AI agents. v2.9.45: Setup Portal debug logging for navigation (input/state/frame). v2.9.44: Setup Portal fill-background fix for duplication and corruption. ⚠️ Pre-MVP / Development Phase",
+  "description": "4Runr AI Agent OS - Secure terminal interface for AI agents. v2.9.46: Setup Portal navigation fix - persisted ListState so selection matches visual. v2.9.45: Debug logging. v2.9.44: Fill-background fix. ⚠️ Pre-MVP / Development Phase",
   "main": "dist/index.js",
   "bin": {
     "4runr": "dist/index.js",