From 8d0a11b4ee8784ffcd1a6ecab6573505fe0c64a2 Mon Sep 17 00:00:00 2001 From: Jinwoo-H Date: Wed, 13 May 2026 02:33:34 -0400 Subject: [PATCH] Remove sidebar PR refresh adornments Co-authored-by: Orca --- .../src/components/sidebar/WorktreeCard.tsx | 38 ++++--------------- 1 file changed, 8 insertions(+), 30 deletions(-) diff --git a/src/renderer/src/components/sidebar/WorktreeCard.tsx b/src/renderer/src/components/sidebar/WorktreeCard.tsx index 02494227812..39cacc44466 100644 --- a/src/renderer/src/components/sidebar/WorktreeCard.tsx +++ b/src/renderer/src/components/sidebar/WorktreeCard.tsx @@ -7,7 +7,6 @@ import { Tooltip, TooltipTrigger, TooltipContent } from '@/components/ui/tooltip import { AlertTriangle, Bell, - Clock3, GitMerge, LoaderCircle, CircleCheck, @@ -286,7 +285,6 @@ const WorktreeCard = React.memo(function WorktreeCard({ ) const showIssue = cardProps.includes('issue') - const showPRMeta = cardProps.includes('pr') && Boolean(pr) // Same rationale for issues: once that section is hidden, polling only burns // GitHub calls and keeps stale-but-invisible data warm for no user benefit. @@ -523,39 +521,19 @@ const WorktreeCard = React.memo(function WorktreeCard({ - {!showPRMeta && prRefreshState?.status === 'queued' && ( - + {pr.checksStatus === 'success' && ( + )} - {!showPRMeta && prRefreshState?.status === 'in-flight' && ( - + {pr.checksStatus === 'failure' && ( + + )} + {pr.checksStatus === 'pending' && ( + )} - {(showPRMeta || - !['queued', 'in-flight'].includes(prRefreshState?.status ?? '')) && - pr.checksStatus === 'success' && ( - - )} - {(showPRMeta || - !['queued', 'in-flight'].includes(prRefreshState?.status ?? '')) && - pr.checksStatus === 'failure' && ( - - )} - {(showPRMeta || - !['queued', 'in-flight'].includes(prRefreshState?.status ?? '')) && - pr.checksStatus === 'pending' && ( - - )} - - {!showPRMeta && prRefreshState?.status === 'queued' - ? 'CI refresh queued' - : !showPRMeta && prRefreshState?.status === 'in-flight' - ? 'Refreshing CI checks' - : prRefreshState?.status === 'paused' - ? `CI checks ${checksLabel(pr.checksStatus).toLowerCase()} (refresh paused)` - : `CI checks ${checksLabel(pr.checksStatus).toLowerCase()}`} - + {`CI checks ${checksLabel(pr.checksStatus).toLowerCase()}`}