diff --git a/Makefile b/Makefile index 222a058..aa2f55e 100644 --- a/Makefile +++ b/Makefile @@ -13,6 +13,12 @@ release-all: test-rust: cargo test --workspace --verbose +fmt: + cargo fmt + +fix: + cargo clippy --all --fix --allow-dirty --allow-staged + install: echo "Installing pay-respects core. Use `install-all` to install all modules." cargo install --path core diff --git a/core/src/shell.rs b/core/src/shell.rs index 458d733..a4db0b6 100644 --- a/core/src/shell.rs +++ b/core/src/shell.rs @@ -389,7 +389,8 @@ pub fn module_output(data: &Data, module: &str) -> Option> { } let break_holder = "<_PR_BR>"; Some( - String::from_utf8_lossy(&output.stdout)[..output.stdout.len() - break_holder.len()] + String::from_utf8_lossy(&output.stdout) + .trim_end_matches(break_holder) .split("<_PR_BR>") .map(|s| s.trim().to_string()) .collect::>(), diff --git a/modules.md b/modules.md index 574103c..bd10908 100644 --- a/modules.md +++ b/modules.md @@ -38,11 +38,12 @@ Your module should print: - At the end of each suggestion, append `<_PR_BR>` so pay-respects knows you are either done or adding another suggestion - **To `stderr`**: Any relevant information that should display to the user (e.g, warning for AI generated content) -An example of a shell based module that always adds a `sudo` before the command: +An example of a shell based module that always suggest adding a `sudo` or `doas`: ```sh #!/bin/sh echo "sudo $_PR_LAST_COMMAND" echo "<_PR_BR>" +echo "doas $_PR_LAST_COMMAND" ``` ## Adding a Module