4runr-os 2.9.45 → 2.9.47

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.47";
     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 {
@@ -238,13 +238,14 @@ fn render_options_list(f: &mut Frame, area: Rect, state: &AppState) {
         ("Custom URL", GatewayOption::CustomUrl),
     ];
     
-    // Find selected index
+    // Find selected index from canonical source of truth (selected_option)
     let selected_index = options.iter()
         .position(|(_, opt)| opt == &state.setup_portal.selected_option)
         .unwrap_or(0);
 
-    // DEBUG: Every frame - exact state used for list (so we can match what's drawn)
-    eprintln!("[SETUP-PORTAL] LIST selected_index={} selected_option={:?}", selected_index, state.setup_portal.selected_option);
+    // CRITICAL: Force ListState to match selected_option every frame.
+    // This prevents drift (e.g. from widget internals or multiple render paths).
+    state.setup_portal.list_state.select(Some(selected_index));
 
     unsafe {
         if LOG_COUNT % 10 == 1 {
@@ -274,10 +275,6 @@ 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));
-    
     // Use Block to properly contain and clear the area
     let block = Block::default()
         .borders(Borders::ALL)
@@ -289,7 +286,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.47",
   "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.47: Setup Portal - sync ListState from selected_option every frame so highlight never drifts. v2.9.46: Persisted ListState. ⚠️ Pre-MVP / Development Phase",
   "main": "dist/index.js",
   "bin": {
     "4runr": "dist/index.js",