test(compression): exact budget returns Full (#30)

Made-with: Cursor

Author: haasonsaas

src/parsing/llm_response.rs

@@ -400,6 +400,14 @@ fn repair_json_candidates(candidate: &str) -> Vec<String> {
         candidates.push(with_double_quotes);
     }
 
+    // Tab-to-space conversion: some LLMs emit tabs inside JSON (issue #28).
+    if trimmed.contains('\t') && (trimmed.starts_with('[') || trimmed.starts_with('{')) {
+        let tab_to_space = trimmed.replace('\t', " ");
+        if tab_to_space != trimmed {
+            candidates.push(tab_to_space);
+        }
+    }
+
     candidates
 }
 
@@ -1390,6 +1398,17 @@ let data = &input;
         assert!(comments[0].content.contains("don't"));
     }
 
+    #[test]
+    fn parse_json_with_tabs() {
+        // LLM sometimes emits tabs inside JSON; repair converts tab to space (issue #28).
+        let input = "[{\"line\":\t4,\t\"issue\":\t\"Tab-indented\"}]";
+        let file_path = PathBuf::from("src/lib.rs");
+        let comments = parse_llm_response(input, &file_path).unwrap();
+        assert_eq!(comments.len(), 1);
+        assert_eq!(comments[0].line_number, 4);
+        assert!(comments[0].content.contains("Tab-indented"));
+    }
+
     // ── Bug: find_json_array uses mismatched brackets ──────────────────
     //
     // `find_json_array` uses `find('[')` (first) + `rfind(']')` (last).

src/review/compression.rs

@@ -534,4 +534,19 @@ mod tests {
             + result.skipped_indices.len();
         assert_eq!(total_accounted, 2, "All files must be accounted for");
     }
+
+    /// #30: Adaptive compression — when total tokens exactly equals budget, strategy is Full.
+    #[test]
+    fn test_adaptive_compression_exact_budget_returns_full() {
+        let diff = make_simple_diff("single.rs", 200);
+        let budget = estimate_diff_tokens(&diff);
+        let result = compress_diffs(&[diff], budget, 5);
+        assert_eq!(
+            result.strategy,
+            CompressionStrategy::Full,
+            "Exact budget should use Full strategy"
+        );
+        assert_eq!(result.batches.len(), 1);
+        assert_eq!(result.skipped_indices.len(), 0);
+    }
 }