fix

Signed-off-by: pipiland2612 <[email protected]>

Author: pipiland2612

scripts/e2e/metrics_summary.py

@@ -79,9 +79,31 @@ def extract_metric_name(line):
         return line.split('{')[0].strip()
     return line.strip()
 
+def get_sample_lines(samples, max_lines=5):
+    """
+    Get sample lines from metric changes, limiting to max_lines.
+    Returns formatted sample lines for display.
+    """
+    sample_lines = []
+    line_count = 0
+
+    for sample in samples[:3]:  # Take first 3 samples to avoid too much output
+        lines = sample.split('\n')
+        for line in lines:
+            if line_count >= max_lines:
+                if len(samples) > 3 or any(len(s.split('\n')) > max_lines for s in samples):
+                    sample_lines.append("  ...")
+                break
+            sample_lines.append(f"  {line}")
+            line_count += 1
+        if line_count >= max_lines:
+            break
+
+    return sample_lines
+
 def generate_diff_summary(changes):
     """
-    Generates a markdown summary from the parsed diff changes.
+    Generates a markdown summary from the parsed diff changes with sample lines.
     """
     summary = ["## 📊 Metrics Diff Summary\n"]
 
@@ -100,12 +122,30 @@ def generate_diff_summary(changes):
         summary.append("\n### 🆕 Added Metrics")
         for metric, samples in changes['added'].items():
             summary.append(f"- `{metric}` ({len(samples)} variants)")
+            sample_lines = get_sample_lines(samples)
+            if sample_lines:
+                summary.append("<details>")
+                summary.append("<summary>View samples</summary>")
+                summary.append("")
+                summary.append("```")
+                summary.extend(sample_lines)
+                summary.append("```")
+                summary.append("</details>")
 
     # Removed metrics
     if changes['removed']:
         summary.append("\n### ❌ Removed Metrics")
         for metric, samples in changes['removed'].items():
             summary.append(f"- `{metric}` ({len(samples)} variants)")
+            sample_lines = get_sample_lines(samples)
+            if sample_lines:
+                summary.append("<details>")
+                summary.append("<summary>View samples</summary>")
+                summary.append("")
+                summary.append("```")
+                summary.extend(sample_lines)
+                summary.append("```")
+                summary.append("</details>")
 
     # Modified metrics
     if changes['modified']:
@@ -115,6 +155,32 @@ def generate_diff_summary(changes):
             summary.append(f"  - Added variants: {len(versions['added'])}")
             summary.append(f"  - Removed variants: {len(versions['removed'])}")
 
+            # Show samples of added variants
+            if versions['added']:
+                added_samples = get_sample_lines(versions['added'], max_lines=3)
+                if added_samples:
+                    summary.append("  - Sample added:")
+                    summary.append("    <details>")
+                    summary.append("    <summary>View added samples</summary>")
+                    summary.append("")
+                    summary.append("    ```")
+                    summary.extend([f"  {line}" for line in added_samples])
+                    summary.append("    ```")
+                    summary.append("    </details>")
+
+            # Show samples of removed variants
+            if versions['removed']:
+                removed_samples = get_sample_lines(versions['removed'], max_lines=3)
+                if removed_samples:
+                    summary.append("  - Sample removed:")
+                    summary.append("    <details>")
+                    summary.append("    <summary>View removed samples</summary>")
+                    summary.append("")
+                    summary.append("    ```")
+                    summary.extend([f"  {line}" for line in removed_samples])
+                    summary.append("    ```")
+                    summary.append("    </details>")
+
     return "\n".join(summary)
 
 def main():