refactoring

src/main.rs

@@ -1,4 +1,4 @@
-use std::io::{stdout, Write};
+use std::io::{stdout, IsTerminal, Write};
 
 use clap::Parser;
 use cli::Cli;

src/tui.rs

@@ -1,5 +1,5 @@
 use std::{
-    io::{stderr, stdout, IsTerminal, Stderr, Stdout, Write},
+    io::{stderr, BufWriter, Stderr},
     ops::{Deref, DerefMut},
 };
 
@@ -10,48 +10,20 @@ use crossterm::{
 };
 use ratatui::backend::CrosstermBackend as Backend;
 use tokio::task::JoinHandle;
-use tracing::debug;
 
-pub enum OutputHandle {
-    Stdout(Stdout),
-    Stderr(std::io::BufWriter<Stderr>),
-}
-
-impl<W: Write> Write for Tui<W> {
-    fn write(&mut self, buf: &[u8]) -> std::io::Result<usize> {
-        self.terminal.backend_mut().write(buf)
-    }
-
-    fn flush(&mut self) -> std::io::Result<()> {
-        self.terminal.backend_mut().flush()
-    }
-}
-
-// FIXME: fix this (see https://blog.orhun.dev/stdout-vs-stderr/)
-pub struct Tui<W: Write> {
-    pub terminal: ratatui::Terminal<Backend<W>>,
+pub struct Tui {
+    pub terminal: ratatui::Terminal<Backend<BufWriter<Stderr>>>,
     pub task: JoinHandle<()>,
     pub frame_rate: f64,
 }
 
-impl<W: Write> Tui<W> {
+impl Tui {
     pub fn new() -> Result<Self> {
-        if stdout().is_terminal() {
-            debug!("stdout is a terminal, using stdout for TUI");
-            Ok(Self {
-                terminal: ratatui::Terminal::new(Backend::new(stdout()))?,
-                task: tokio::spawn(async {}),
-                frame_rate: 60.0,
-            })
-        } else {
-            debug!("stdout is not a terminal, using stderr for TUI");
-            let buffered_stderr = std::io::BufWriter::new(stderr());
-            Ok(Self {
-                terminal: ratatui::Terminal::new(Backend::new(buffered_stderr))?,
-                task: tokio::spawn(async {}),
-                frame_rate: 60.0,
-            })
-        }
+        Ok(Self {
+            terminal: ratatui::Terminal::new(Backend::new(BufWriter::new(stderr())))?,
+            task: tokio::spawn(async {}),
+            frame_rate: 60.0,
+        })
     }
 
     pub fn frame_rate(mut self, frame_rate: f64) -> Self {
@@ -96,7 +68,7 @@ impl<W: Write> Tui<W> {
 }
 
 impl Deref for Tui {
-    type Target = ratatui::Terminal<Backend<OutputHandle>>;
+    type Target = ratatui::Terminal<Backend<BufWriter<Stderr>>>;
 
     fn deref(&self) -> &Self::Target {
         &self.terminal