refactor: replace Box<dyn Error> with anyhow::Result

Author: tlipinski

Cargo.lock

@@ -21,6 +21,7 @@ crossterm = { version = "0.29.0" }
 unicode-width = "0.2.2"
 itertools = "0.14.0"
 regex = "1.12.3"
+anyhow = "1.0.102"
 
 [target.'cfg(target_os = "macos")'.dependencies]
 crossterm = { version = "0.29.0", features = ["use-dev-tty"] }

Cargo.toml

@@ -10,6 +10,7 @@ use crate::rura_widget::RuraWidget;
 use crate::theme::Theme;
 use crate::uicmd::{KeyBindings, UiCmd, to_ui_command};
 use KeyCode::{Enter, Esc, F};
+use anyhow::Result;
 use crossterm::event::KeyCode::Char;
 use crossterm::event::{KeyCode, KeyModifiers};
 use crossterm::tty::IsTty;
@@ -25,7 +26,6 @@ use ratatui::text::{Line, Text};
 use ratatui::widgets::{Block, BorderType, Paragraph, Widget};
 use ratatui::{DefaultTerminal, Frame};
 use serde::{Deserialize, Serialize};
-use std::error::Error;
 use std::io::{Read, Write, stdin};
 use std::process::{Command, Stdio};
 use std::sync::mpsc::{Receiver, Sender};
@@ -133,7 +133,7 @@ impl App {
         }
     }
 
-    pub fn run(mut self, terminal: &mut DefaultTerminal) -> Result<String, Box<dyn Error>> {
+    pub fn run(mut self, terminal: &mut DefaultTerminal) -> Result<String> {
         while !self.exit {
             terminal.draw(|frame| self.render(frame, frame.area()))?;
 
@@ -550,7 +550,7 @@ impl App {
 fn handle_command_task(
     command_rx: Receiver<(String, String)>,
     action_tx: Sender<Action>,
-) -> Result<(), Box<dyn Error>> {
+) -> Result<()> {
     loop {
         if let Ok((command, stdin)) = command_rx.recv() {
             info!("executing command: '{command}'");
@@ -598,7 +598,7 @@ fn handle_command_task(
     }
 }
 
-fn handle_input_task(tx: Sender<Action>) -> Result<(), Box<dyn Error>> {
+fn handle_input_task(tx: Sender<Action>) -> Result<()> {
     loop {
         if let Ok(event) = event::read() {
             debug!("event: {:?}", event);
@@ -607,7 +607,7 @@ fn handle_input_task(tx: Sender<Action>) -> Result<(), Box<dyn Error>> {
     }
 }
 
-fn read_stdin_task(file_opt: Option<String>, tx: Sender<Action>) -> Result<(), Box<dyn Error>> {
+fn read_stdin_task(file_opt: Option<String>, tx: Sender<Action>) -> Result<()> {
     if let Some(file) = file_opt {
         info!("reading file {file}");
         let file_content = std::fs::read_to_string(file);
@@ -641,11 +641,7 @@ fn read_stdin_task(file_opt: Option<String>, tx: Sender<Action>) -> Result<(), B
     }
 }
 
-fn reset_highlight_task(
-    rx: Receiver<()>,
-    tx: Sender<Action>,
-    duration_ms: u64,
-) -> Result<(), Box<dyn Error>> {
+fn reset_highlight_task(rx: Receiver<()>, tx: Sender<Action>, duration_ms: u64) -> Result<()> {
     loop {
         if let Ok(_) = rx.recv() {
             thread::sleep(Duration::from_millis(duration_ms));

src/app.rs

@@ -1,14 +1,10 @@
+use anyhow::Result;
 use log::{debug, error};
-use std::error::Error;
 use std::sync::mpsc::{Receiver, TryRecvError};
 use std::thread::sleep;
 use std::time::Duration;
 
-pub fn debouncer_task<F>(
-    rx: Receiver<()>,
-    duration: Duration,
-    on_debounce: F,
-) -> Result<(), Box<dyn Error + Send + Sync>>
+pub fn debouncer_task<F>(rx: Receiver<()>, duration: Duration, on_debounce: F) -> Result<()>
 where
     F: Fn() -> () + Send + 'static,
 {

src/debouncer.rs

@@ -13,11 +13,11 @@ mod uicmd;
 use crate::app::App;
 use crate::config::load_config;
 use crate::history::History;
+use anyhow::Result;
 use clap::Parser;
 use env_logger::{Builder, Target};
 use log::{LevelFilter, error, info};
 use props::APP_NAME;
-use std::error::Error;
 use std::fs;
 use std::fs::OpenOptions;
 use std::process::exit;
@@ -76,7 +76,7 @@ struct Args {
     last: bool,
 }
 
-fn run(args: Args, config: config::Config) -> Result<(), Box<dyn Error>> {
+fn run(args: Args, config: config::Config) -> Result<()> {
     info!("Starting TUI");
     let mut terminal = ratatui::init();
 

src/main.rs

@@ -3,16 +3,15 @@ use crate::history::History;
 use crate::rura::{ExecuteType, Part, Rura};
 use crate::theme::Theme;
 use crate::uicmd::{KeyBindings, UiCmd, to_ui_command};
+use anyhow::Result;
 use crossterm::event::Event;
-use error::Error;
 use itertools::Itertools;
 use log::info;
 use ratatui::buffer::Buffer;
 use ratatui::layout::{Position, Rect};
 use ratatui::prelude::{Line, Style, Widget};
 use ratatui::style::Styled;
 use ratatui::text::StyledGrapheme;
-use std::error;
 use std::sync::mpsc::Sender;
 use tui_input::backend::crossterm::EventHandler;
 use tui_input::{Input, InputRequest};
@@ -183,7 +182,7 @@ impl RuraWidget {
         }
     }
 
-    pub fn execute(&mut self, execute_type: ExecuteType) -> Result<Option<String>, Box<dyn Error>> {
+    pub fn execute(&mut self, execute_type: ExecuteType) -> Result<Option<String>> {
         if self.command_input.value().is_empty() {
             return Ok(None);
         }