refactor: replace .unwrap() with .expect() on infallible calls

The four .unwrap() calls all sit on values that cannot fail in
practice (regex literals known at compile time, and a serde_json
value built from a json! literal). Switch to .expect() so the panic
message documents the invariant if it ever does fire.

Author: arhtur007

src/config.rs

@@ -143,7 +143,7 @@ impl Default for ReactionTiming {
 // --- loading ---
 
 fn expand_env_vars(raw: &str) -> String {
-    let re = Regex::new(r"\$\{(\w+)\}").unwrap();
+    let re = Regex::new(r"\$\{(\w+)\}").expect("env var regex literal is valid");
     re.replace_all(raw, |caps: &regex::Captures| {
         std::env::var(&caps[1]).unwrap_or_default()
     })

src/discord.rs

@@ -88,7 +88,8 @@ impl EventHandler for Handler {
         });
         let prompt_with_sender = format!(
             "<sender_context>\n{}\n</sender_context>\n\n{}",
-            serde_json::to_string(&sender_ctx).unwrap(),
+            serde_json::to_string(&sender_ctx)
+                .expect("sender_ctx is built from a serde_json::json! literal and cannot fail to serialize"),
             prompt
         );
 
@@ -331,13 +332,14 @@ fn compose_display(tool_lines: &[String], text: &str) -> String {
 }
 
 fn strip_mention(content: &str) -> String {
-    let re = regex::Regex::new(r"<@[!&]?\d+>").unwrap();
+    let re = regex::Regex::new(r"<@[!&]?\d+>").expect("mention regex literal is valid");
     re.replace_all(content, "").trim().to_string()
 }
 
 fn shorten_thread_name(prompt: &str) -> String {
     // Shorten GitHub URLs: https://github.com/owner/repo/issues/123 → owner/repo#123
-    let re = regex::Regex::new(r"https?://github\.com/([^/]+/[^/]+)/(issues|pull)/(\d+)").unwrap();
+    let re = regex::Regex::new(r"https?://github\.com/([^/]+/[^/]+)/(issues|pull)/(\d+)")
+        .expect("github url regex literal is valid");
     let shortened = re.replace_all(prompt, "$1#$3");
     let name: String = shortened.chars().take(40).collect();
     if name.len() < shortened.len() {