From 863eded6887b363c7b6709488488ad8aaea09bdc Mon Sep 17 00:00:00 2001 From: iamnbutler Date: Thu, 22 Oct 2026 12:24:19 -0400 Subject: [PATCH] fix(ui): key checks by name and URL so shared check URLs don't collide --- packages/ui/src/components/checks/checks.tsx | 2 +- .../ui/src/components/session-details/session-details.tsx | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/packages/ui/src/components/checks/checks.tsx b/packages/ui/src/components/checks/checks.tsx index 82d52525..ad078355 100644 --- a/packages/ui/src/components/checks/checks.tsx +++ b/packages/ui/src/components/checks/checks.tsx @@ -72,7 +72,7 @@ export function ChecksNotice({ pull, failed, onFix, onDismiss, className }: Chec {failed.map((check, index) => ( - + {index > 0 && ", "} {check.url ? ( diff --git a/packages/ui/src/components/session-details/session-details.tsx b/packages/ui/src/components/session-details/session-details.tsx index a5718a8b..6b961588 100644 --- a/packages/ui/src/components/session-details/session-details.tsx +++ b/packages/ui/src/components/session-details/session-details.tsx @@ -188,13 +188,15 @@ function Checks({ checks, summary: [Icon, color, label] }: { ); const name = `${check.name}: ${check.state}`; + // GitHub can list several checks under one URL, such as codecov/patch and codecov/project. + const key = `${check.name}\0${check.url}`; return check.url ? ( - + {content} ) - :
{content}
; + :
{content}
; })} )}