refactor: replace unwrap calls with descriptive expect messages in test suites

Author: fajarhide

src/hooks/normalize.rs

@@ -450,18 +450,18 @@ mod tests {
 
     #[test]
     fn test_extract_array_content() {
-        let json: Value = serde_json::from_str(
+        let json: serde_json::Value = serde_json::from_str(
             r#"[{"type":"text","text":"hello"},{"type":"text","text":"world"}]"#,
         )
-        .unwrap();
-        let content = extract_value_content(&json).unwrap();
+        .expect("Valid JSON");
+        let content = extract_value_content(&json).expect("Content exists");
         assert_eq!(content, "hello\nworld");
     }
 
     #[test]
     fn test_normalize_claude() {
         let input = r#"{"tool_name":"Bash","tool_input":{"command":"echo hello"},"tool_response":{"stdout":"hello"}}"#;
-        let norm = normalize(input).unwrap();
+        let norm = normalize(input).expect("Normalized successfully");
         assert_eq!(norm.agent_id, "claude_code");
         assert_eq!(norm.tool_name, "Bash");
         assert_eq!(norm.content, "hello");
@@ -471,7 +471,7 @@ mod tests {
     fn test_normalize_opencode() {
         let input =
             r#"{"type":"tool_result","tool":"shell","output":"hello","command":"echo hello"}"#;
-        let norm = normalize(input).unwrap();
+        let norm = normalize(input).expect("Normalized successfully");
         assert_eq!(norm.agent_id, "opencode");
         assert_eq!(norm.tool_name, "Bash");
         assert_eq!(norm.content, "hello");

src/hooks/post_tool.rs

@@ -682,7 +682,7 @@ mod tests {
         });
         let out = process_payload(&input.to_string(), None, None);
         assert!(out.is_some(), "Large ReadFile must be distilled");
-        let res = out.unwrap();
+        let res = out.expect("Output exists");
         assert!(
             res.contains("OMNI ReadFile"),
             "Must have OMNI ReadFile label"
@@ -708,7 +708,7 @@ mod tests {
         });
         let out = process_payload(&input.to_string(), None, None);
         assert!(out.is_some(), "Grep output must be distilled");
-        let res = out.unwrap();
+        let res = out.expect("Output exists");
         assert!(res.contains("OMNI Grep"), "Must have OMNI Grep label");
         assert!(res.contains("matches"), "Must show match count");
     }
@@ -784,7 +784,7 @@ mod tests {
         });
         let out = process_payload(&input.to_string(), None, None);
         assert!(out.is_some());
-        let res = out.unwrap();
+        let res = out.expect("Output exists");
         println!("PASSTHROUGH RES: {}", res);
         assert!(res.contains("OMNI: Passthrough"));
     }