feat: implement language-specific distillation logic for non-Bash tools in post-tool hook

Author: fajarhide

src/distillers/build.rs

@@ -12,12 +12,39 @@ impl Distiller for BuildDistiller {
     ) -> String {
         let mut errors = Vec::new();
         let mut warnings = Vec::new();
+        let mut current_block = Vec::new();
+        let mut is_error_block = false;
 
         for seg in segments {
-            if seg.tier == SignalTier::Critical {
-                errors.push(seg.content.clone());
-            } else if seg.tier == SignalTier::Important {
-                warnings.push(seg.content.clone());
+            if seg.tier == SignalTier::Critical || seg.tier == SignalTier::Important {
+                if current_block.is_empty() {
+                    is_error_block = seg.tier == SignalTier::Critical;
+                }
+                // If we see a new critical and we're currently in a warning block,
+                // or if it's a clear new error boundary, flush it
+                if seg.tier == SignalTier::Critical && !current_block.is_empty() && !is_error_block
+                {
+                    warnings.push(current_block.join("\n"));
+                    current_block.clear();
+                    is_error_block = true;
+                }
+                current_block.push(seg.content.clone());
+            } else {
+                if !current_block.is_empty() {
+                    if is_error_block {
+                        errors.push(current_block.join("\n"));
+                    } else {
+                        warnings.push(current_block.join("\n"));
+                    }
+                    current_block.clear();
+                }
+            }
+        }
+        if !current_block.is_empty() {
+            if is_error_block {
+                errors.push(current_block.join("\n"));
+            } else {
+                warnings.push(current_block.join("\n"));
             }
         }
 

src/distillers/snapshots/omni__distillers__tests__cargo_build_distillation.snap

@@ -4,3 +4,7 @@ expression: output
 ---
 Build: 1 errors, 0 warnings
 error[E0432]: unresolved import `std::collections`
+ --> src/main.rs:1:5
+  |
+1 | use std::collections::Foo;
+  |     ^^^^^^^^^^^^^^^^^^^^^ no `Foo` in `collections`

src/distillers/test.rs

@@ -24,7 +24,15 @@ impl Distiller for TestDistiller {
                 if !seg.content.to_lowercase().contains("failed tests/")
                     && !seg.content.contains("===")
                 {
-                    failure_details.push(seg.content.clone());
+                    // Truncate to max 12 lines to keep just the assertion and stack trace
+                    let lines: Vec<&str> = seg.content.lines().collect();
+                    if lines.len() > 12 {
+                        let truncated =
+                            lines[..12].join("\n") + "\n       ... [stack trace truncated]";
+                        failure_details.push(truncated);
+                    } else {
+                        failure_details.push(seg.content.clone());
+                    }
                 }
             } else if seg.tier == SignalTier::Important
                 || seg.content.contains("PASS")
@@ -53,7 +61,7 @@ impl Distiller for TestDistiller {
 
         out.push_str(&format!("Tests: {} passed, {} failed\n", passed, failed));
 
-        let max_fails = 5;
+        let max_fails = 10;
         for (i, fail) in failure_details.iter().enumerate() {
             if i < max_fails {
                 out.push_str(fail);