chore: some quality of life changes

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

core/src/shell.rs

@@ -389,7 +389,8 @@ pub fn module_output(data: &Data, module: &str) -> Option<Vec<String>> {
 	}
 	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::<Vec<String>>(),

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