Skip to content

Commit 9a207c2

Browse files
fix: remove review session label
1 parent a564d6a commit 9a207c2

3 files changed

Lines changed: 8 additions & 15 deletions

File tree

frontend/src/renderer/components/SessionInspector.test.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -252,16 +252,16 @@ describe("SessionInspector reviews tab", () => {
252252
expect(onOpenReviewerTerminal).toHaveBeenCalledWith({ handleId: "reviewer-pane", harness: "codex" });
253253
});
254254

255-
it("shows the session reviewer identity, worker session name, and aggregate verdict", async () => {
255+
it("shows the reviewer identity and aggregate verdict", async () => {
256256
mockCommonGets([approvedReview], "reviewer-pane", [reviewState(3, "changes_requested", "abc123")]);
257257

258258
renderWithQuery(<SessionInspector session={session([pr(3, "open")])} />);
259259
await openReviewsTab();
260260

261261
expect(await screen.findByText("codex")).toBeInTheDocument();
262262
expect(screen.getByText("reviewer")).toBeInTheDocument();
263-
expect(screen.getByText("sess-1")).toBeInTheDocument();
264-
expect(screen.getByText("review session")).toBeInTheDocument();
263+
expect(screen.queryByText("sess-1")).not.toBeInTheDocument();
264+
expect(screen.queryByText("review session")).not.toBeInTheDocument();
265265
expect(screen.getAllByText("Changes requested")).not.toHaveLength(0);
266266
});
267267

frontend/src/renderer/components/SessionInspector.tsx

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -504,13 +504,7 @@ function ReviewPanel({
504504
</div>
505505
<div className="reviewer-card">
506506
<div className="reviewer-card__top">
507-
<div className="reviewer-card__identity">
508-
<div className="reviewer-card__name">
509-
<GitPullRequest aria-hidden="true" />
510-
<span>{session.id}</span>
511-
</div>
512-
<div className="reviewer-card__session">review session</div>
513-
</div>
507+
<span className="reviewer-card__label">Pull requests</span>
514508
<span className={cn("reviewer-status", `reviewer-status--${aggregateVerdict.tone}`)}>
515509
{aggregateVerdict.label}
516510
</span>

frontend/src/renderer/styles.css

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1060,15 +1060,14 @@ body.is-resizing-x [data-slot="sidebar-container"] {
10601060
gap: 4px;
10611061
}
10621062

1063-
.reviewer-card__session {
1063+
.reviewer-card__label {
10641064
min-width: 0;
10651065
overflow: hidden;
10661066
text-overflow: ellipsis;
10671067
white-space: nowrap;
1068-
font-size: 10.5px;
1069-
text-transform: uppercase;
1070-
letter-spacing: 0.04em;
1071-
color: var(--fg-passive);
1068+
font-size: 12px;
1069+
font-weight: 650;
1070+
color: var(--fg-muted);
10721071
}
10731072

10741073
.reviewer-status {

0 commit comments

Comments
 (0)