style: cargo fmt

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: haasonsaas

src/adapters/anthropic.rs

@@ -124,10 +124,15 @@ impl LLMAdapter for AnthropicAdapter {
                 if c.content_type == "text" {
                     Ok(c.text.clone())
                 } else {
-                    Err(anyhow::anyhow!("Unsupported content type: {}", c.content_type))
+                    Err(anyhow::anyhow!(
+                        "Unsupported content type: {}",
+                        c.content_type
+                    ))
                 }
             })
-            .ok_or_else(|| anyhow::anyhow!("Anthropic returned empty content array — no content generated"))??;
+            .ok_or_else(|| {
+                anyhow::anyhow!("Anthropic returned empty content array — no content generated")
+            })??;
 
         Ok(LLMResponse {
             content,
@@ -310,7 +315,10 @@ mod tests {
         let adapter = AnthropicAdapter::new(test_config(&server.url())).unwrap();
         let result = adapter.complete(test_request()).await;
 
-        assert!(result.is_err(), "Unsupported content type should return an error");
+        assert!(
+            result.is_err(),
+            "Unsupported content type should return an error"
+        );
         let err = result.unwrap_err().to_string();
         assert!(
             err.contains("Unsupported content type"),
@@ -339,9 +347,16 @@ mod tests {
         let adapter = AnthropicAdapter::new(test_config(&server.url())).unwrap();
         let result = adapter.complete(test_request()).await;
 
-        assert!(result.is_err(), "Empty content array should return an error, not silently succeed");
+        assert!(
+            result.is_err(),
+            "Empty content array should return an error, not silently succeed"
+        );
         let err = result.unwrap_err().to_string();
-        assert!(err.contains("empty content"), "Error should mention empty content: {}", err);
+        assert!(
+            err.contains("empty content"),
+            "Error should mention empty content: {}",
+            err
+        );
     }
 
     #[test]

src/adapters/openai.rs

@@ -196,7 +196,9 @@ impl OpenAIAdapter {
             .choices
             .first()
             .map(|c| c.message.content.clone())
-            .ok_or_else(|| anyhow::anyhow!("OpenAI returned empty choices array — no content generated"))?;
+            .ok_or_else(|| {
+                anyhow::anyhow!("OpenAI returned empty choices array — no content generated")
+            })?;
 
         Ok(LLMResponse {
             content,
@@ -435,9 +437,16 @@ mod tests {
         let adapter = OpenAIAdapter::new(test_config(&server.url())).unwrap();
         let result = adapter.complete(test_request()).await;
 
-        assert!(result.is_err(), "Empty choices should return an error, not silently succeed");
+        assert!(
+            result.is_err(),
+            "Empty choices should return an error, not silently succeed"
+        );
         let err = result.unwrap_err().to_string();
-        assert!(err.contains("empty choices"), "Error should mention empty choices: {}", err);
+        assert!(
+            err.contains("empty choices"),
+            "Error should mention empty choices: {}",
+            err
+        );
     }
 
     #[test]

src/core/diff_parser.rs

@@ -372,12 +372,14 @@ impl DiffParser {
                 // Some diff tools emit truly empty lines for empty context lines
                 // (omitting the leading space). Treat as context to keep line numbers in sync.
                 (ChangeType::Context, "")
-            } else { match line.chars().next() {
-                Some('+') => (ChangeType::Added, &line[1..]),
-                Some('-') => (ChangeType::Removed, &line[1..]),
-                Some(' ') => (ChangeType::Context, &line[1..]),
-                _ => (ChangeType::Context, line),
-            }};
+            } else {
+                match line.chars().next() {
+                    Some('+') => (ChangeType::Added, &line[1..]),
+                    Some('-') => (ChangeType::Removed, &line[1..]),
+                    Some(' ') => (ChangeType::Context, &line[1..]),
+                    _ => (ChangeType::Context, line),
+                }
+            };
 
             let diff_line = match change_type {
                 ChangeType::Added => {
@@ -579,8 +581,7 @@ index abc..def 100644\n\
     fn test_parse_text_diff_sets_is_new_for_new_file() {
         // BUG: parse_text_diff always sets is_new=false even when old_content is empty
         let diff =
-            DiffParser::parse_text_diff("", "new content\n", PathBuf::from("new_file.rs"))
-                .unwrap();
+            DiffParser::parse_text_diff("", "new content\n", PathBuf::from("new_file.rs")).unwrap();
         assert!(
             diff.is_new,
             "parse_text_diff with empty old content should set is_new=true"

src/core/function_chunker.rs

@@ -918,14 +918,14 @@ fn next_func() {
     fn test_find_function_end_single_quoted_string_with_brace() {
         // BUG: single-quoted strings aren't tracked, so `'{'` is counted as a real brace
         let lines: Vec<&str> = vec![
-            "function foo() {",         // line 0: depth = 1
-            "    let s = '{';",         // line 1: depth should stay 1, but goes to 2
-            "    console.log(s);",      // line 2
-            "}",                        // line 3: depth goes to 1 (not 0!), so foo "never ends"
-            "",                         // line 4
-            "function bar() {",         // line 5: depth goes to 2
-            "    return 1;",            // line 6
-            "}",                        // line 7: depth goes to 1, and foo is "found" here (wrong!)
+            "function foo() {",    // line 0: depth = 1
+            "    let s = '{';",    // line 1: depth should stay 1, but goes to 2
+            "    console.log(s);", // line 2
+            "}",                   // line 3: depth goes to 1 (not 0!), so foo "never ends"
+            "",                    // line 4
+            "function bar() {",    // line 5: depth goes to 2
+            "    return 1;",       // line 6
+            "}",                   // line 7: depth goes to 1, and foo is "found" here (wrong!)
         ];
         let end = find_function_end(&lines, 0, "js");
         assert_eq!(
@@ -943,7 +943,7 @@ fn next_func() {
             "    Self { name }",                        // line 1
             "}",                                        // line 2
             "",
-            "fn other() {",                             // line 4
+            "fn other() {", // line 4
         ];
         let end = find_function_end(&lines, 0, "rs");
         assert_eq!(

src/server/github.rs

@@ -1177,5 +1177,4 @@ mod tests {
 
         assert!(verify_webhook_signature(secret, body, &signature).is_ok());
     }
-
 }

src/server/storage_json.rs

@@ -1386,19 +1386,37 @@ mod tests {
         // 1 completed, 1 failed, 1 timeout (not a failure per se)
         backend
             .save_review(&make_session_with_event(
-                "r1", now, ReviewStatus::Complete, "review.completed", "gpt-4o", "github", 100,
+                "r1",
+                now,
+                ReviewStatus::Complete,
+                "review.completed",
+                "gpt-4o",
+                "github",
+                100,
             ))
             .await
             .unwrap();
         backend
             .save_review(&make_session_with_event(
-                "r2", now, ReviewStatus::Failed, "review.failed", "gpt-4o", "github", 200,
+                "r2",
+                now,
+                ReviewStatus::Failed,
+                "review.failed",
+                "gpt-4o",
+                "github",
+                200,
             ))
             .await
             .unwrap();
         backend
             .save_review(&make_session_with_event(
-                "r3", now, ReviewStatus::Failed, "review.timeout", "gpt-4o", "github", 300,
+                "r3",
+                now,
+                ReviewStatus::Failed,
+                "review.timeout",
+                "gpt-4o",
+                "github",
+                300,
             ))
             .await
             .unwrap();