From 9cfee5f86d08aae1cadbd2e98040d6dcb981df2f Mon Sep 17 00:00:00 2001 From: Rafael Audibert <32079912+rafaeelaudibert@users.noreply.github.com> Date: Fri, 29 May 2026 12:01:58 -0300 Subject: [PATCH 1/2] Highlight current user in signal suggested reviewers Sorts the current user to the top of the suggested reviewers list, and when a commit explains why the current user was picked, shows a small info icon next to the SHA with a "Why was I assigned?" tooltip that surfaces the rationale. Other reviewers keep the existing hover-on-SHA tooltip. Generated-By: PostHog Code Task-Id: bf5f3c64-2f8b-4ca2-8716-b91f7527b080 --- .../components/detail/ReportDetailPane.tsx | 60 +++++++++++++++---- 1 file changed, 48 insertions(+), 12 deletions(-) diff --git a/apps/code/src/renderer/features/inbox/components/detail/ReportDetailPane.tsx b/apps/code/src/renderer/features/inbox/components/detail/ReportDetailPane.tsx index bdc2be529..1ff69dea0 100644 --- a/apps/code/src/renderer/features/inbox/components/detail/ReportDetailPane.tsx +++ b/apps/code/src/renderer/features/inbox/components/detail/ReportDetailPane.tsx @@ -17,6 +17,7 @@ import { CaretRightIcon, ChatCircleIcon, EyeIcon, + InfoIcon, LinkSimpleIcon, Plus, ThumbsDownIcon, @@ -211,8 +212,12 @@ export function ReportDetailPane({ const reviewerArtefact = allArtefacts.find( (a): a is SuggestedReviewersArtefact => a.type === "suggested_reviewers", ); - return reviewerArtefact?.content ?? []; - }, [allArtefacts]); + const reviewers = reviewerArtefact?.content ?? []; + if (!me?.uuid) return reviewers; + const meIndex = reviewers.findIndex((r) => r.user?.uuid === me.uuid); + if (meIndex <= 0) return reviewers; + return [reviewers[meIndex], ...reviewers.filter((_, i) => i !== meIndex)]; + }, [allArtefacts, me?.uuid]); const signalFindings = useMemo(() => { const map = new Map(); @@ -818,16 +823,47 @@ export function ReportDetailPane({ {reviewer.relevant_commits.map((commit, i) => ( {i > 0 && ", "} - - - {commit.sha.slice(0, 7)} - - + {isMe ? ( + <> + + {commit.sha.slice(0, 7)} + + {commit.reason && ( + + + Why was I assigned? + + + {commit.reason} + + + } + > + + + + + )} + + ) : ( + + + {commit.sha.slice(0, 7)} + + + )} ))} From 862b414a338d723221965849e0bbbf117b3d41c0 Mon Sep 17 00:00:00 2001 From: Rafael Audibert <32079912+rafaeelaudibert@users.noreply.github.com> Date: Fri, 29 May 2026 12:14:06 -0300 Subject: [PATCH 2/2] Wrap commit SHA and info icon in a single tooltip Hovering either the commit link or the new info icon now opens the same tooltip, collapsing the per-isMe branching into one Tooltip with conditional content and a conditional trailing icon. Generated-By: PostHog Code Task-Id: bf5f3c64-2f8b-4ca2-8716-b91f7527b080 --- .../components/detail/ReportDetailPane.tsx | 56 ++++++++----------- 1 file changed, 24 insertions(+), 32 deletions(-) diff --git a/apps/code/src/renderer/features/inbox/components/detail/ReportDetailPane.tsx b/apps/code/src/renderer/features/inbox/components/detail/ReportDetailPane.tsx index 1ff69dea0..94e4c8c1f 100644 --- a/apps/code/src/renderer/features/inbox/components/detail/ReportDetailPane.tsx +++ b/apps/code/src/renderer/features/inbox/components/detail/ReportDetailPane.tsx @@ -823,8 +823,23 @@ export function ReportDetailPane({ {reviewer.relevant_commits.map((commit, i) => ( {i > 0 && ", "} - {isMe ? ( - <> + + + Why was I assigned? + + + {commit.reason} + + + ) : ( + commit.reason || undefined + ) + } + > + {commit.sha.slice(0, 7)} - {commit.reason && ( - - - Why was I assigned? - - - {commit.reason} - - - } - > - - - - + {isMe && commit.reason && ( + )} - - ) : ( - - - {commit.sha.slice(0, 7)} - - - )} + + ))}