From 87c6a3d9f47d4602f6756e9673a0ebdc03ca66f1 Mon Sep 17 00:00:00 2001
From: i-trytoohard <ayetrytoohard@gmail.com>
Date: Mon, 18 May 2026 03:48:28 +0530
Subject: [PATCH] fix(web): hide unenriched PR diff stats (#1912)

Co-authored-by: i-trytoohard <193449657+i-trytoohard@users.noreply.github.com>
---
 .../src/components/SessionDetailPRCard.tsx    | 11 +++--
 .../SessionDetailPRCard.diffStats.test.tsx    | 40 +++++++++++++++++++
 2 files changed, 47 insertions(+), 4 deletions(-)
 create mode 100644 packages/web/src/components/__tests__/SessionDetailPRCard.diffStats.test.tsx

diff --git a/packages/web/src/components/SessionDetailPRCard.tsx b/packages/web/src/components/SessionDetailPRCard.tsx
index 55fc306fa..3f83d7f5a 100644
--- a/packages/web/src/components/SessionDetailPRCard.tsx
+++ b/packages/web/src/components/SessionDetailPRCard.tsx
@@ -178,6 +178,7 @@ export function SessionDetailPRCard({
   const allGreen = isPRMergeReady(pr);
   const blockerIssues = buildBlockerChips(pr, metadata, lifecyclePrReason);
   const fileCount = pr.changedFiles ?? 0;
+  const showDiffStats = !isPRUnenriched(pr);
   const showConflictActions = hasMergeConflicts(pr) && pr.state === "open";
   const compareUrl = showConflictActions ? buildGitHubCompareUrl(pr) : "";
 
@@ -213,10 +214,12 @@ export function SessionDetailPRCard({
         >
           PR #{pr.number}: {pr.title}
         </a>
-        <span className="session-detail-pr-card__diff-stats">
-          <span className="session-detail-diff--add">+{pr.additions}</span>{" "}
-          <span className="session-detail-diff--del">-{pr.deletions}</span>
-        </span>
+        {showDiffStats ? (
+          <span className="session-detail-pr-card__diff-stats">
+            <span className="session-detail-diff--add">+{pr.additions}</span>{" "}
+            <span className="session-detail-diff--del">-{pr.deletions}</span>
+          </span>
+        ) : null}
         {fileCount > 0 ? (
           <span className="session-detail-pr-card__diff-label">
             {fileCount} file{fileCount !== 1 ? "s" : ""}
diff --git a/packages/web/src/components/__tests__/SessionDetailPRCard.diffStats.test.tsx b/packages/web/src/components/__tests__/SessionDetailPRCard.diffStats.test.tsx
new file mode 100644
index 000000000..0f57e6b70
--- /dev/null
+++ b/packages/web/src/components/__tests__/SessionDetailPRCard.diffStats.test.tsx
@@ -0,0 +1,40 @@
+import { render, screen } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import { SessionDetailPRCard } from "../SessionDetailPRCard";
+import { makePR } from "../../__tests__/helpers";
+
+describe("SessionDetailPRCard diff stats", () => {
+  it("shows diff stats for enriched PRs", () => {
+    render(
+      <SessionDetailPRCard
+        pr={makePR({
+          enriched: true,
+          additions: 629,
+          deletions: 44,
+        })}
+        metadata={{}}
+        onAskAgentToFix={vi.fn()}
+      />,
+    );
+
+    expect(screen.getByText("+629")).toBeInTheDocument();
+    expect(screen.getByText("-44")).toBeInTheDocument();
+  });
+
+  it("hides diff stats for unenriched PRs", () => {
+    render(
+      <SessionDetailPRCard
+        pr={makePR({
+          enriched: false,
+          additions: 629,
+          deletions: 44,
+        })}
+        metadata={{}}
+        onAskAgentToFix={vi.fn()}
+      />,
+    );
+
+    expect(screen.queryByText("+629")).not.toBeInTheDocument();
+    expect(screen.queryByText("-44")).not.toBeInTheDocument();
+  });
+});