From e28aa0edf245e02cb7cdb380354cd62da2fce2c6 Mon Sep 17 00:00:00 2001 From: Aditi Chauhan Date: Sat, 4 Jul 2026 14:06:33 +0530 Subject: [PATCH] chore: format pr summary changes --- frontend/src/renderer/components/PRSummaryDisplay.tsx | 4 ++-- frontend/src/renderer/hooks/useSessionScmSummary.ts | 3 ++- frontend/src/renderer/lib/mock-data.ts | 6 +----- frontend/src/renderer/lib/pr-display.test.ts | 4 +++- frontend/src/renderer/lib/pr-display.ts | 5 +---- 5 files changed, 9 insertions(+), 13 deletions(-) diff --git a/frontend/src/renderer/components/PRSummaryDisplay.tsx b/frontend/src/renderer/components/PRSummaryDisplay.tsx index f1702c6ea..619100d5b 100644 --- a/frontend/src/renderer/components/PRSummaryDisplay.tsx +++ b/frontend/src/renderer/components/PRSummaryDisplay.tsx @@ -113,8 +113,8 @@ export function PRSummaryParts({ ) : null} - ); - })} + ); + })} ); } diff --git a/frontend/src/renderer/hooks/useSessionScmSummary.ts b/frontend/src/renderer/hooks/useSessionScmSummary.ts index 56250c023..cca1a6694 100644 --- a/frontend/src/renderer/hooks/useSessionScmSummary.ts +++ b/frontend/src/renderer/hooks/useSessionScmSummary.ts @@ -22,7 +22,8 @@ export function sessionScmSummaryQueryOptions(sessionId: string) { return { queryKey: sessionScmSummaryQueryKey(sessionId), enabled: Boolean(sessionId), - queryFn: () => (usePreviewData ? Promise.resolve(mockSessionScmSummaries[sessionId] ?? []) : fetchSessionScmSummary(sessionId)), + queryFn: () => + usePreviewData ? Promise.resolve(mockSessionScmSummaries[sessionId] ?? []) : fetchSessionScmSummary(sessionId), retry: 1, }; } diff --git a/frontend/src/renderer/lib/mock-data.ts b/frontend/src/renderer/lib/mock-data.ts index e8d583703..c354ab20d 100644 --- a/frontend/src/renderer/lib/mock-data.ts +++ b/frontend/src/renderer/lib/mock-data.ts @@ -286,11 +286,7 @@ export const mockWorkspaces: WorkspaceSummary[] = [ }, ]; -const prSummary = ( - sessionId: string, - number: number, - overrides: Partial = {}, -): SessionPRSummary => { +const prSummary = (sessionId: string, number: number, overrides: Partial = {}): SessionPRSummary => { const session = mockWorkspaces.flatMap((workspace) => workspace.sessions).find((item) => item.id === sessionId); const facts = session?.prs.find((item) => item.number === number); const url = facts?.url ?? `https://github.com/me/${session?.workspaceName ?? "preview"}/pull/${number}`; diff --git a/frontend/src/renderer/lib/pr-display.test.ts b/frontend/src/renderer/lib/pr-display.test.ts index fd4c9be6c..4bb8ca495 100644 --- a/frontend/src/renderer/lib/pr-display.test.ts +++ b/frontend/src/renderer/lib/pr-display.test.ts @@ -292,7 +292,9 @@ describe("prSummaryParts", () => { }); it("puts draft readiness under Review", () => { - const parts = prSummaryParts(summary({ state: "draft", review: { decision: "none", hasUnresolvedHumanComments: false, unresolvedBy: [] } })); + const parts = prSummaryParts( + summary({ state: "draft", review: { decision: "none", hasUnresolvedHumanComments: false, unresolvedBy: [] } }), + ); expect(parts.find((part) => part.key === "review")).toMatchObject({ status: "None", diff --git a/frontend/src/renderer/lib/pr-display.ts b/frontend/src/renderer/lib/pr-display.ts index d3ae654b1..31272232b 100644 --- a/frontend/src/renderer/lib/pr-display.ts +++ b/frontend/src/renderer/lib/pr-display.ts @@ -374,10 +374,7 @@ function formatLineDelta(additions: number, deletions: number): string | undefin return parts.length > 0 ? parts.join(" ") : undefined; } -function mergeAttentionLinks( - pr: SessionPRSummary, - kind: "merge_conflict" | "merge_blocked", -): PRSummaryLink[] { +function mergeAttentionLinks(pr: SessionPRSummary, kind: "merge_conflict" | "merge_blocked"): PRSummaryLink[] { const href = kind === "merge_conflict" ? mergeConflictUrl(pr) : pr.mergeability.prUrl || pr.htmlUrl || pr.url || undefined; const fileLinks = (pr.mergeability.conflictFiles ?? []).slice(0, 3).map((file) => ({