fix(tui): reset terminal no matter what (#1175)

Currently if you run `rustic snapshots -i` without a repository
configured `rustic` will leave the terminal in raw mode.

This is a simple fix to make sure the terminal is properly reset in all
situations.

close #1166

Co-authored-by: aawsome <37850842+aawsome@users.noreply.github.com>

Cargo.toml

@@ -76,6 +76,7 @@ aho-corasick = "1.1.2"
 chrono = { version = "0.4", default-features = false, features = ["clock", "serde"] }
 comfy-table = "7.1.0"
 rhai = { version = "1.17", features = ["sync", "serde", "no_optimize", "no_module", "no_custom_syntax", "only_i64"] }
+scopeguard = "1.2"
 semver = { version = "1", optional = true }
 simplelog = "0.12"
 

src/commands/tui.rs

@@ -8,6 +8,7 @@ mod widgets;
 
 use crossterm::event::{KeyEvent, KeyModifiers};
 use progress::TuiProgressBars;
+use scopeguard::defer;
 use snapshots::Snapshots;
 
 use std::io;
@@ -36,12 +37,10 @@ pub fn run(group_by: SnapshotGroupCriterion) -> Result<()> {
     let terminal = init_terminal()?;
     let terminal = Arc::new(RwLock::new(terminal));
 
-    let original_hook = std::panic::take_hook();
-
-    std::panic::set_hook(Box::new(move |panic| {
+    // restore terminal (even when leaving through ?, early return, or panic)
+    defer! {
         reset_terminal().unwrap();
-        original_hook(panic);
-    }));
+    }
 
     let progress = TuiProgressBars {
         terminal: terminal.clone(),
@@ -54,9 +53,6 @@ pub fn run(group_by: SnapshotGroupCriterion) -> Result<()> {
     let app = App { snapshots };
     let res = run_app(terminal, app);
 
-    // restore terminal
-    reset_terminal()?;
-
     if let Err(err) = res {
         println!("{err:?}");
     }