mirror of
https://github.com/ellie/atuin.git
synced 2024-09-11 21:18:22 +03:00
refactor
remove colours
This commit is contained in:
parent
6851b386d1
commit
1c591b223d
7
Cargo.lock
generated
7
Cargo.lock
generated
@ -90,7 +90,6 @@ dependencies = [
|
||||
"indicatif",
|
||||
"itertools",
|
||||
"log",
|
||||
"owo-colors",
|
||||
"pretty_env_logger",
|
||||
"serde",
|
||||
"serde_json",
|
||||
@ -1381,12 +1380,6 @@ version = "6.0.0"
|
||||
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||
checksum = "8e22443d1643a904602595ba1cd8f7d896afe56d26712531c5ff73a15b2fbf64"
|
||||
|
||||
[[package]]
|
||||
name = "owo-colors"
|
||||
version = "3.4.0"
|
||||
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||
checksum = "decf7381921fea4dcb2549c5667eda59b3ec297ab7e2b5fc33eac69d2e7da87b"
|
||||
|
||||
[[package]]
|
||||
name = "parking_lot"
|
||||
version = "0.11.2"
|
||||
|
@ -65,7 +65,6 @@ clap = { version = "3.1.11", features = ["derive"] }
|
||||
clap_complete = "3.1.2"
|
||||
fs-err = "2.7"
|
||||
whoami = "1.1.2"
|
||||
owo-colors = "3.4.0"
|
||||
|
||||
[dependencies.tracing-subscriber]
|
||||
version = "0.3"
|
||||
|
@ -11,7 +11,6 @@ use atuin_client::settings::Settings;
|
||||
|
||||
#[cfg(feature = "sync")]
|
||||
use atuin_client::sync;
|
||||
use owo_colors::OwoColorize;
|
||||
|
||||
#[derive(Subcommand)]
|
||||
#[clap(infer_subcommands = true)]
|
||||
@ -53,17 +52,34 @@ pub enum Cmd {
|
||||
},
|
||||
}
|
||||
|
||||
#[derive(Clone, Copy, Debug)]
|
||||
pub enum ListMode {
|
||||
Human,
|
||||
CmdOnly,
|
||||
Regular,
|
||||
}
|
||||
|
||||
impl ListMode {
|
||||
pub const fn from_flags(human: bool, cmd_only: bool) -> Self {
|
||||
if human {
|
||||
ListMode::Human
|
||||
} else if cmd_only {
|
||||
ListMode::CmdOnly
|
||||
} else {
|
||||
ListMode::Regular
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
#[allow(clippy::cast_sign_loss)]
|
||||
pub fn print_list(h: &[History], human: bool, cmd_only: bool) {
|
||||
pub fn print_list(h: &[History], list_mode: ListMode) {
|
||||
let w = std::io::stdout();
|
||||
let mut w = w.lock();
|
||||
|
||||
if human {
|
||||
print_human_list(&mut w, h);
|
||||
} else if cmd_only {
|
||||
print_cmd_only(&mut w, h);
|
||||
} else {
|
||||
print_basic(&mut w, h);
|
||||
match list_mode {
|
||||
ListMode::Human => print_human_list(&mut w, h),
|
||||
ListMode::CmdOnly => print_cmd_only(&mut w, h),
|
||||
ListMode::Regular => print_regular(&mut w, h),
|
||||
}
|
||||
|
||||
w.flush().expect("failed to flush history");
|
||||
@ -81,19 +97,12 @@ pub fn print_human_list(w: &mut StdoutLock, h: &[History]) {
|
||||
let time = h.timestamp.format("%Y-%m-%d %H:%M:%S");
|
||||
let cmd = h.command.trim();
|
||||
|
||||
let exit_color = if h.success() {
|
||||
owo_colors::AnsiColors::Green
|
||||
} else {
|
||||
owo_colors::AnsiColors::Red
|
||||
};
|
||||
let duration = duration.color(exit_color);
|
||||
|
||||
writeln!(w, "{time} · {duration}\t{cmd}").expect("failed to write history");
|
||||
}
|
||||
}
|
||||
|
||||
#[allow(clippy::cast_sign_loss)]
|
||||
pub fn print_basic(w: &mut StdoutLock, h: &[History]) {
|
||||
pub fn print_regular(w: &mut StdoutLock, h: &[History]) {
|
||||
for h in h.iter().rev() {
|
||||
let duration =
|
||||
humantime::format_duration(Duration::from_nanos(std::cmp::max(h.duration, 0) as u64))
|
||||
@ -216,14 +225,14 @@ impl Cmd {
|
||||
}
|
||||
};
|
||||
|
||||
print_list(&history, *human, *cmd_only);
|
||||
print_list(&history, ListMode::from_flags(*human, *cmd_only));
|
||||
|
||||
Ok(())
|
||||
}
|
||||
|
||||
Self::Last { human, cmd_only } => {
|
||||
let last = db.last().await?;
|
||||
print_list(&[last], *human, *cmd_only);
|
||||
print_list(&[last], ListMode::from_flags(*human, *cmd_only));
|
||||
|
||||
Ok(())
|
||||
}
|
||||
|
@ -23,6 +23,7 @@ use atuin_client::{
|
||||
};
|
||||
|
||||
use super::event::{Event, Events};
|
||||
use super::history::ListMode;
|
||||
|
||||
const VERSION: &str = env!("CARGO_PKG_VERSION");
|
||||
|
||||
@ -73,21 +74,33 @@ impl Cmd {
|
||||
db: &mut (impl Database + Send + Sync),
|
||||
settings: &Settings,
|
||||
) -> Result<()> {
|
||||
run(
|
||||
if self.interactive {
|
||||
let item = select_history(
|
||||
&self.query,
|
||||
settings.search_mode,
|
||||
settings.filter_mode,
|
||||
settings.style,
|
||||
db,
|
||||
)
|
||||
.await?;
|
||||
eprintln!("{}", item);
|
||||
} else {
|
||||
let list_mode = ListMode::from_flags(self.human, self.cmd_only);
|
||||
run_non_interactive(
|
||||
settings,
|
||||
list_mode,
|
||||
self.cwd,
|
||||
self.exit,
|
||||
self.interactive,
|
||||
self.human,
|
||||
self.exclude_exit,
|
||||
self.exclude_cwd,
|
||||
self.before,
|
||||
self.after,
|
||||
self.cmd_only,
|
||||
&self.query,
|
||||
db,
|
||||
)
|
||||
.await
|
||||
.await?;
|
||||
};
|
||||
Ok(())
|
||||
}
|
||||
}
|
||||
|
||||
@ -565,45 +578,28 @@ async fn select_history(
|
||||
// This is supposed to more-or-less mirror the command line version, so ofc
|
||||
// it is going to have a lot of args
|
||||
#[allow(clippy::too_many_arguments)]
|
||||
pub async fn run(
|
||||
async fn run_non_interactive(
|
||||
settings: &Settings,
|
||||
list_mode: ListMode,
|
||||
cwd: Option<String>,
|
||||
exit: Option<i64>,
|
||||
interactive: bool,
|
||||
human: bool,
|
||||
exclude_exit: Option<i64>,
|
||||
exclude_cwd: Option<String>,
|
||||
before: Option<String>,
|
||||
after: Option<String>,
|
||||
cmd_only: bool,
|
||||
query: &[String],
|
||||
db: &mut (impl Database + Send + Sync),
|
||||
) -> Result<()> {
|
||||
let dir = if let Some(cwd) = cwd {
|
||||
if cwd == "." {
|
||||
let dir = if cwd.as_deref() == Some(".") {
|
||||
let current = std::env::current_dir()?;
|
||||
let current = current.as_os_str();
|
||||
let current = current.to_str().unwrap();
|
||||
|
||||
Some(current.to_owned())
|
||||
} else {
|
||||
Some(cwd)
|
||||
}
|
||||
} else {
|
||||
None
|
||||
cwd
|
||||
};
|
||||
|
||||
if interactive {
|
||||
let item = select_history(
|
||||
query,
|
||||
settings.search_mode,
|
||||
settings.filter_mode,
|
||||
settings.style,
|
||||
db,
|
||||
)
|
||||
.await?;
|
||||
eprintln!("{}", item);
|
||||
} else {
|
||||
let context = current_context();
|
||||
|
||||
let results = db
|
||||
@ -674,8 +670,6 @@ pub async fn run(
|
||||
.map(std::borrow::ToOwned::to_owned)
|
||||
.collect();
|
||||
|
||||
super::history::print_list(&results, human, cmd_only);
|
||||
}
|
||||
|
||||
super::history::print_list(&results, list_mode);
|
||||
Ok(())
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user