fix(tui): restore input after sandbox shell

Discard stale events accumulated around the suspended TUI and let the normal periodic tick refresh state after resuming.

Signed-off-by: John T. Myers <9696606+johntmyers@users.noreply.github.com>

Author: johntmyers

crates/openshell-tui/src/event.rs

@@ -118,6 +118,15 @@ impl EventHandler {
         self.rx.recv().await
     }
 
+    /// Discard events queued before or while the TUI was suspended.
+    ///
+    /// The input reader is paused before this is called, so terminal input
+    /// remains in the TTY for the resumed reader. This primarily removes stale
+    /// tick events that would otherwise run refresh work before new key input.
+    pub fn discard_pending(&mut self) {
+        while self.rx.try_recv().is_ok() {}
+    }
+
     /// Get a sender handle for dispatching events from background tasks.
     pub fn sender(&self) -> mpsc::UnboundedSender<Event> {
         self.keepalive.clone()

crates/openshell-tui/src/lib.rs

@@ -142,8 +142,7 @@ pub async fn run(
                 }
                 if app.pending_shell_connect {
                     app.pending_shell_connect = false;
-                    handle_shell_connect(&mut app, &mut terminal, &events).await?;
-                    refresh_data(&mut app).await;
+                    handle_shell_connect(&mut app, &mut terminal, &mut events).await?;
                 }
                 // --- Draft actions ---
                 if app.pending_draft_approve {
@@ -366,7 +365,7 @@ pub async fn run(
                                     handle_exec_command(
                                         &mut app,
                                         &mut terminal,
-                                        &events,
+                                        &mut events,
                                         &name,
                                         &command,
                                     )
@@ -841,7 +840,7 @@ async fn fetch_sandbox_detail(app: &mut App) {
 async fn handle_shell_connect(
     app: &mut App,
     terminal: &mut Terminal<CrosstermBackend<io::Stdout>>,
-    events: &EventHandler,
+    events: &mut EventHandler,
 ) -> Result<()> {
     let sandbox_name = match app.selected_sandbox_name() {
         Some(n) => n.to_string(),
@@ -973,7 +972,7 @@ async fn handle_shell_connect(
         }
     }
 
-    // Step 9: Resume and draw the TUI before the caller refreshes gateway data.
+    // Step 9: Resume and draw the TUI before accepting new terminal input.
     enable_raw_mode().into_diagnostic()?;
     execute!(
         terminal.backend_mut(),
@@ -985,6 +984,7 @@ async fn handle_shell_connect(
     terminal
         .draw(|frame| ui::draw(frame, app))
         .into_diagnostic()?;
+    events.discard_pending();
     events.resume();
 
     Ok(())
@@ -998,7 +998,7 @@ async fn handle_shell_connect(
 async fn handle_exec_command(
     app: &mut App,
     terminal: &mut Terminal<CrosstermBackend<io::Stdout>>,
-    events: &EventHandler,
+    events: &mut EventHandler,
     sandbox_name: &str,
     command: &str,
 ) -> Result<()> {
@@ -1140,6 +1140,7 @@ async fn handle_exec_command(
     )
     .into_diagnostic()?;
     terminal.clear().into_diagnostic()?;
+    events.discard_pending();
     events.resume();
 
     Ok(())