fix(input): share input reader across app restarts

Switching repos (enter submodule/worktree) rebuilds the whole App.
Previously each App spun up its own input-reader thread, so during the
teardown window the outgoing reader could read and drop the first
keystroke after a switch (its channel receiver was already gone).

Create the Input once in main and thread it into each Gitui, mirroring
how the terminal is already reused across restarts. Keystrokes read
during the swap are buffered in the channel and delivered to the new
App instead of being lost. Fixes the swallowed keypress for both
worktree and submodule switching.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BMQxBdr5f3B3jmDtdvGtTu

Author: stevenpollack

CHANGELOG.md

@@ -18,6 +18,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * support rewording non-HEAD commits when `commit.gpgsign` is enabled (gpg format only) [[@guerinoni](https://github.com/guerinoni)] ([#2959](https://github.com/gitui-org/gitui/pull/2959))
 
 ### Fixes
+* first keystroke after switching to a submodule or worktree was dropped
 * crash when opening submodule ([#2895](https://github.com/gitui-org/gitui/issues/2895))
 * when staging the last file in a directory, the first item after the directory is no longer skipped [[@Tillerino](https://github.com/Tillerino)] ([#2748](https://github.com/gitui-org/gitui/issues/2748))
 * index-out-of-bounds panic when unstaging lines near the end of a diff ([#2953](https://github.com/gitui-org/gitui/issues/2953))

src/gitui.rs

@@ -37,12 +37,11 @@ impl Gitui {
 		theme: Theme,
 		key_config: &KeyConfig,
 		updater: Updater,
+		input: &Input,
 	) -> Result<Self, anyhow::Error> {
 		let (tx_git, rx_git) = unbounded();
 		let (tx_app, rx_app) = unbounded();
 
-		let input = Input::new();
-
 		let (rx_ticker, rx_watcher) = match updater {
 			Updater::NotifyWatcher => {
 				let repo_watcher = RepoWatcher::new(
@@ -212,7 +211,7 @@ mod tests {
 	use ratatui::{backend::TestBackend, Terminal};
 
 	use crate::{
-		args::CliArgs, gitui::Gitui, keys::KeyConfig,
+		args::CliArgs, gitui::Gitui, input::Input, keys::KeyConfig,
 		ui::style::Theme, AsyncNotification, Updater,
 	};
 
@@ -248,10 +247,16 @@ mod tests {
 
 		let theme = Theme::init(&PathBuf::new());
 		let key_config = KeyConfig::default();
+		let input = Input::new();
 
-		let mut gitui =
-			Gitui::new(cliargs, theme, &key_config, Updater::Ticker)
-				.unwrap();
+		let mut gitui = Gitui::new(
+			cliargs,
+			theme,
+			&key_config,
+			Updater::Ticker,
+			&input,
+		)
+		.unwrap();
 
 		let mut terminal =
 			Terminal::new(TestBackend::new(90, 12)).unwrap();

src/main.rs

@@ -97,7 +97,7 @@ use crossterm::{
 	ExecutableCommand,
 };
 use gitui::Gitui;
-use input::InputEvent;
+use input::{Input, InputEvent};
 use keys::KeyConfig;
 use ratatui::backend::CrosstermBackend;
 use scopeguard::defer;
@@ -184,6 +184,12 @@ fn main() -> Result<()> {
 	let mut terminal =
 		start_terminal(io::stdout(), &cliargs.repo_path)?;
 
+	// the input reader is created once and shared across app restarts
+	// (e.g. switching to a submodule or worktree). recreating it per
+	// app would let the outgoing reader consume and drop the first
+	// keystroke after a switch; a shared reader buffers it instead.
+	let input = Input::new();
+
 	let updater = if cliargs.notify_watcher {
 		Updater::NotifyWatcher
 	} else {
@@ -199,6 +205,7 @@ fn main() -> Result<()> {
 			theme.clone(),
 			&key_config,
 			updater,
+			&input,
 			&mut terminal,
 		)?;
 
@@ -226,9 +233,11 @@ fn run_app(
 	theme: Theme,
 	key_config: &KeyConfig,
 	updater: Updater,
+	input: &Input,
 	terminal: &mut Terminal,
 ) -> Result<QuitState, anyhow::Error> {
-	let mut gitui = Gitui::new(cliargs, theme, key_config, updater)?;
+	let mut gitui =
+		Gitui::new(cliargs, theme, key_config, updater, input)?;
 
 	log::trace!("app start: {} ms", app_start.elapsed().as_millis());