feat: auto-retry if suggestion failed

This commit is contained in:
iff 2023-08-07 19:58:19 +02:00
parent b6c298d8f7
commit 1444eeab84
3 changed files with 35 additions and 13 deletions

View file

@ -13,17 +13,28 @@ fn main() {
let shell = std::env::var("_PR_SHELL").expect( let shell = std::env::var("_PR_SHELL").expect(
"No _PR_SHELL in environment. Did you aliased the binary with the correct arguments?", "No _PR_SHELL in environment. Did you aliased the binary with the correct arguments?",
); );
let last_command = shell::last_command_expanded_alias(&shell); let mut last_command = shell::last_command_expanded_alias(&shell);
let corrected_command = suggestions::suggest_command(&shell, &last_command); loop {
if let Some(corrected_command) = corrected_command { let corrected_command = suggestions::suggest_command(&shell, &last_command);
let command_difference = highlight_difference(&shell, &corrected_command, &last_command);
if let Some(highlighted_command) = command_difference { if let Some(corrected_command) = corrected_command {
suggestions::confirm_suggestion(&shell, &corrected_command, &highlighted_command); let command_difference = highlight_difference(&shell, &corrected_command, &last_command);
return; if let Some(highlighted_command) = command_difference {
let execution = suggestions::confirm_suggestion(&shell, &corrected_command, &highlighted_command);
if execution.is_ok() {
return;
}
else {
let retry_message = format!("{}", "Looking for new suggestion...".cyan().bold());
println!("\n{}\n", retry_message);
last_command = corrected_command;
}
}
} else {
break;
} }
} }
println!( println!(
"No correction found for the command: {}\n", "No correction found for the command: {}\n",
last_command.red() last_command.red()

View file

@ -35,7 +35,7 @@ pub fn highlight_difference(shell: &str, suggested_command: &str, last_command:
} }
for old in &old_entries { for old in &old_entries {
if old == entry { if old == entry {
*entry = entry.cyan().to_string(); *entry = entry.blue().to_string();
continue 'next; continue 'next;
} }
} }

View file

@ -166,7 +166,7 @@ fn compare_string(a: &str, b: &str) -> usize {
matrix[a.chars().count()][b.chars().count()] matrix[a.chars().count()][b.chars().count()]
} }
pub fn confirm_suggestion(shell: &str, command: &str, highlighted: &str) { pub fn confirm_suggestion(shell: &str, command: &str, highlighted: &str) -> Result<(), ()> {
println!{"{}\n", highlighted} println!{"{}\n", highlighted}
println!("Press enter to execute the suggestion. Or press Ctrl+C to exit."); println!("Press enter to execute the suggestion. Or press Ctrl+C to exit.");
std::io::stdin().read_line(&mut String::new()).unwrap(); std::io::stdin().read_line(&mut String::new()).unwrap();
@ -175,7 +175,7 @@ pub fn confirm_suggestion(shell: &str, command: &str, highlighted: &str) {
let _p = p.to_owned() + " "; let _p = p.to_owned() + " ";
if command.starts_with(&_p) { if command.starts_with(&_p) {
let command = command.replace(p, ""); let command = command.replace(p, "");
std::process::Command::new(p) let process = std::process::Command::new(p)
.arg(shell) .arg(shell)
.arg("-c") .arg("-c")
.arg(command) .arg(command)
@ -183,15 +183,26 @@ pub fn confirm_suggestion(shell: &str, command: &str, highlighted: &str) {
.expect("failed to execute process") .expect("failed to execute process")
.wait() .wait()
.expect("failed to wait on process"); .expect("failed to wait on process");
return;
if process.success() {
return Ok(());
} else {
return Err(());
}
} }
} }
std::process::Command::new(shell) let process = std::process::Command::new(shell)
.arg("-c") .arg("-c")
.arg(command) .arg(command)
.spawn() .spawn()
.expect("failed to execute process") .expect("failed to execute process")
.wait() .wait()
.expect("failed to wait on process"); .expect("failed to wait on process");
if process.success() {
return Ok(());
} else {
return Err(());
}
} }