4runr-os 2.10.57 → 2.10.58

package/mk3-tui/src/app.rs

@@ -2226,34 +2226,54 @@ impl App {
         }
 
         match key.code {
-            // Navigation
+            // Navigation — list only; detail view locks to pinned run
             KeyCode::Up => {
-                self.state.run_manager.select_previous();
+                if self.state.run_manager.is_detail_view() {
+                    self.state.run_manager.detail_log_scroll_up();
+                } else {
+                    self.state.run_manager.select_previous();
+                }
                 self.request_render("run_manager_up");
             }
             KeyCode::Down => {
-                self.state.run_manager.select_next();
+                if self.state.run_manager.is_detail_view() {
+                    self.state.run_manager.detail_log_scroll_down();
+                } else {
+                    self.state.run_manager.select_next();
+                }
                 self.request_render("run_manager_down");
             }
 
-            // Filter
+            // Filter (list view only)
             KeyCode::Char('f') | KeyCode::Char('F') => {
-                self.state.run_manager.next_filter();
-                self.request_render("run_manager_filter");
+                if !self.state.run_manager.is_detail_view() {
+                    self.state.run_manager.next_filter();
+                    self.request_render("run_manager_filter");
+                }
             }
 
-            // Sort
+            // Sort (list view only)
             KeyCode::Char('s') | KeyCode::Char('S') => {
-                self.state.run_manager.next_sort();
-                self.request_render("run_manager_sort");
+                if !self.state.run_manager.is_detail_view() {
+                    self.state.run_manager.next_sort();
+                    self.request_render("run_manager_sort");
+                }
             }
 
-            // Refresh (manual)
+            // Refresh (list or pinned detail)
             KeyCode::Char('r') | KeyCode::Char('R') => {
                 if self.state.operation_mode == OperationMode::Connected {
                     if let Some(ws) = ws_client {
-                        self.begin_run_list_request(ws, false);
-                        self.add_log("[RUN] Refreshing run list from Gateway...".to_string());
+                        if self.state.run_manager.is_detail_view() {
+                            if let Some(run) = self.state.run_manager.detail_run() {
+                                let run_id = run.id.clone();
+                                self.begin_run_get_request(ws, &run_id);
+                                self.add_log("[RUN] Refreshing run detail...".to_string());
+                            }
+                        } else {
+                            self.begin_run_list_request(ws, false);
+                            self.add_log("[RUN] Refreshing run list from Gateway...".to_string());
+                        }
                     } else {
                         self.add_log("[RUN] WebSocket not connected".to_string());
                     }
@@ -2279,34 +2299,31 @@ impl App {
             // View details / Close detail view
             KeyCode::Enter => {
                 if self.state.run_manager.is_detail_view() {
-                    // Close detail view
                     self.state.run_manager.close_detail_view();
                     self.add_log("[RUN] Closed detail view".to_string());
-                } else {
-                    // Open detail view
-                    if let Some(run) = self.state.run_manager.selected_run() {
-                        let run_id = run.id.clone();
-                        let run_name = run.name.clone();
-                        self.add_log(format!("[RUN] Viewing run: {}", run_name));
-                        self.state.run_manager.toggle_detail_view();
-                        if self.state.operation_mode == OperationMode::Connected {
-                            if let Some(ws) = ws_client {
-                                let data = serde_json::json!({ "runId": run_id });
-                                if let Ok(id) = ws.send_command("run.get", Some(data)) {
-                                    self.state.pending_run_get_id = Some(id);
-                                }
-                            }
+                } else if let Some(run_id) = self.state.run_manager.open_detail_view() {
+                    if let Some(run) = self.state.run_manager.detail_run() {
+                        self.add_log(format!("[RUN] Viewing run: {}", run.name));
+                    }
+                    if self.state.operation_mode == OperationMode::Connected {
+                        if let Some(ws) = ws_client {
+                            self.begin_run_get_request(ws, &run_id);
                         }
-                    } else {
-                        self.add_log("[RUN] No run selected".to_string());
                     }
+                } else {
+                    self.add_log("[RUN] No run selected".to_string());
                 }
                 self.request_render("run_manager_view");
             }
 
-            // Cancel run
+            // Cancel run (list selection or pinned detail run)
             KeyCode::Char('c') | KeyCode::Char('C') => {
-                if let Some(run) = self.state.run_manager.selected_run() {
+                let run = if self.state.run_manager.is_detail_view() {
+                    self.state.run_manager.detail_run()
+                } else {
+                    self.state.run_manager.selected_run()
+                };
+                if let Some(run) = run {
                     if matches!(
                         run.status,
                         crate::ui::run_manager::RunStatus::Running

package/mk3-tui/src/main.rs

@@ -205,11 +205,7 @@ fn main() -> Result<()> {
                     if !due {
                         None
                     } else {
-                        let detail_run_id = if rm.is_detail_view() {
-                            rm.selected_run().map(|r| r.id.clone())
-                        } else {
-                            None
-                        };
+                        let detail_run_id = rm.detail_run_id.clone();
                         Some(detail_run_id)
                     }
                 }

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

@@ -38,8 +38,11 @@ pub struct RunManagerState {
     #[allow(dead_code)]
     pub last_refresh: Option<std::time::Instant>,
 
-    /// Detailed view state (None = list view, Some(index) = detail view)
-    pub detail_view: Option<usize>,
+    /// Full-screen detail view: locked to this run until ESC/Enter.
+    pub detail_run_id: Option<String>,
+
+    /// Log panel scroll offset in detail view.
+    pub detail_log_scroll: u16,
 
     /// Poll Gateway for run list while Run Manager is open (no manual R).
     pub auto_refresh_enabled: bool,
@@ -156,7 +159,8 @@ impl Default for RunManagerState {
             sort: RunSort::DateDesc,
             loading: false,
             last_refresh: None,
-            detail_view: None,
+            detail_run_id: None,
+            detail_log_scroll: 0,
             auto_refresh_enabled: true,
             auto_refresh_interval: std::time::Duration::from_secs(3),
         }
@@ -283,7 +287,8 @@ impl RunManagerState {
     pub fn replace_runs(&mut self, runs: Vec<RunInfo>) {
         self.runs = runs;
         self.selected_index = 0;
-        self.detail_view = None;
+        self.detail_run_id = None;
+        self.detail_log_scroll = 0;
         self.loading = false;
         self.last_refresh = Some(std::time::Instant::now());
     }
@@ -291,7 +296,7 @@ impl RunManagerState {
     /// Auto-refresh: keep selection and detail view when the selected run still exists.
     pub fn replace_runs_preserve(&mut self, runs: Vec<RunInfo>) {
         let selected_id = self.selected_run().map(|r| r.id.clone());
-        let in_detail = self.detail_view.is_some();
+        let pinned_detail_id = self.detail_run_id.clone();
         self.runs = runs;
         self.loading = false;
         self.last_refresh = Some(std::time::Instant::now());
@@ -299,7 +304,8 @@ impl RunManagerState {
         let filtered = self.filtered_runs();
         if filtered.is_empty() {
             self.selected_index = 0;
-            self.detail_view = None;
+            self.detail_run_id = None;
+            self.detail_log_scroll = 0;
             return;
         }
 
@@ -313,15 +319,21 @@ impl RunManagerState {
             self.selected_index = filtered.len() - 1;
         }
 
-        if in_detail && self.selected_run().is_none() {
-            self.detail_view = None;
+        if let Some(detail_id) = pinned_detail_id {
+            if self.runs.iter().any(|r| r.id == detail_id) {
+                self.detail_run_id = Some(detail_id);
+            } else {
+                self.detail_run_id = None;
+                self.detail_log_scroll = 0;
+            }
         }
     }
 
     pub fn clear_runs(&mut self) {
         self.runs.clear();
         self.selected_index = 0;
-        self.detail_view = None;
+        self.detail_run_id = None;
+        self.detail_log_scroll = 0;
         self.loading = false;
         self.last_refresh = None;
     }
@@ -368,28 +380,39 @@ impl RunManagerState {
         filtered.get(self.selected_index).copied()
     }
 
-    /// Toggle detail view for selected run
-    pub fn toggle_detail_view(&mut self) {
-        if self.detail_view.is_some() {
-            // Close detail view
-            self.detail_view = None;
-        } else {
-            // Open detail view for selected run
-            self.detail_view = Some(self.selected_index);
-        }
+    /// Run pinned in full-screen detail view (independent of list selection).
+    pub fn detail_run(&self) -> Option<&RunInfo> {
+        let id = self.detail_run_id.as_ref()?;
+        self.runs.iter().find(|r| &r.id == id)
+    }
+
+    /// Open full-screen detail for the currently selected run.
+    pub fn open_detail_view(&mut self) -> Option<String> {
+        let run = self.selected_run()?;
+        self.detail_run_id = Some(run.id.clone());
+        self.detail_log_scroll = 0;
+        Some(run.id.clone())
     }
 
-    /// Close detail view
+    /// Close detail view and return to list (selection unchanged).
     pub fn close_detail_view(&mut self) {
-        self.detail_view = None;
+        self.detail_run_id = None;
+        self.detail_log_scroll = 0;
     }
 
-    /// Check if in detail view
     pub fn is_detail_view(&self) -> bool {
-        self.detail_view.is_some()
+        self.detail_run_id.is_some()
     }
 
-    /// Move selection up
+    pub fn detail_log_scroll_up(&mut self) {
+        self.detail_log_scroll = self.detail_log_scroll.saturating_sub(1);
+    }
+
+    pub fn detail_log_scroll_down(&mut self) {
+        self.detail_log_scroll = self.detail_log_scroll.saturating_add(1);
+    }
+
+    /// Move selection up (list view only).
     pub fn select_previous(&mut self) {
         if self.selected_index > 0 {
             self.selected_index -= 1;
@@ -628,8 +651,8 @@ fn render_detail_view(f: &mut Frame, area: Rect, state: &RunManagerState) {
         ])
         .split(area);
 
-    // Get the selected run
-    let run = match state.selected_run() {
+    // Get the pinned run (not list selection — stays fixed until exit)
+    let run = match state.detail_run() {
         Some(r) => r,
         None => {
             let block = Block::default()
@@ -776,7 +799,9 @@ fn render_detail_view(f: &mut Frame, area: Rect, state: &RunManagerState) {
             .collect()
     };
 
-    let logs_paragraph = Paragraph::new(log_lines).wrap(Wrap { trim: false });
+    let logs_paragraph = Paragraph::new(log_lines)
+        .wrap(Wrap { trim: false })
+        .scroll((state.detail_log_scroll, 0));
     f.render_widget(logs_paragraph, logs_inner);
 
     // Footer
@@ -792,8 +817,12 @@ fn render_detail_view(f: &mut Frame, area: Rect, state: &RunManagerState) {
     let footer_text = Line::from(vec![
         Span::styled("ESC/Enter", Style::default().fg(BRAND_PURPLE).bold()),
         Span::styled(" Back to List  │  ", Style::default().fg(TEXT_DIM)),
+        Span::styled("↑/↓", Style::default().fg(CYBER_CYAN).bold()),
+        Span::styled(" Scroll logs  │  ", Style::default().fg(TEXT_DIM)),
         Span::styled("R", Style::default().fg(NEON_GREEN).bold()),
-        Span::styled(" Refresh", Style::default().fg(TEXT_DIM)),
+        Span::styled(" Refresh  │  ", Style::default().fg(TEXT_DIM)),
+        Span::styled("C", Style::default().fg(Color::Rgb(255, 69, 69)).bold()),
+        Span::styled(" Cancel run", Style::default().fg(TEXT_DIM)),
     ]);
 
     let footer_paragraph = Paragraph::new(footer_text).alignment(Alignment::Center);
@@ -813,6 +842,8 @@ fn render_actions(f: &mut Frame, area: Rect) {
     let text = Line::from(vec![
         Span::styled("↑/↓", Style::default().fg(CYBER_CYAN).bold()),
         Span::styled(" Select  │  ", Style::default().fg(TEXT_DIM)),
+        Span::styled("Enter", Style::default().fg(BRAND_PURPLE).bold()),
+        Span::styled(" Details  │  ", Style::default().fg(TEXT_DIM)),
         Span::styled("F", Style::default().fg(AMBER_WARN).bold()),
         Span::styled(" Filter  │  ", Style::default().fg(TEXT_DIM)),
         Span::styled("S", Style::default().fg(AMBER_WARN).bold()),

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "4runr-os",
-  "version": "2.10.57",
+  "version": "2.10.58",
   "type": "module",
-  "description": "4Runr AI Agent OS - Secure terminal interface for AI agents. v2.10.57: Fix mk3-tui compile (Run Manager live refresh borrow); DDoS loopback exempt; run retention + 4Runr Tools.",
+  "description": "4Runr AI Agent OS - Secure terminal interface for AI agents. v2.10.58: Run Manager detail view locks to selected run; ↑/↓ scroll logs in detail; list nav disabled until ESC.",
   "main": "dist/index.js",
   "bin": {
     "4runr": "dist/index.js",