PR Page - The modern PR page

24
import React, { useMemo } from "react";
25
26
type Role = "author" | "reviewer" | "observer";
27
28
interface Action {
Alyssa Wright
4h ago
Could you add a short comment here explaining why we’re handling the error this way? I think future readers might not immediately understand the reasoning.
😀 1
🎉 2
Show 2 replies
Dana Hiera
4h ago
Nice catch. I added a guard clause for the empty case and wrote a test to cover it. Thanks for pointing that out!
29
id: string;
30
label: string;
31
primary?: string;
32
action: () => void;
33-+
}
34-+
35-+
interface Props {
36-+
title: string;
37-+
subtitle?: string;
38-+
role?: Role;
39
ciStatus?: "passing" | "failing" | "unknown";
40
actions?: Action[];
41
}
42
43
/**
44
* ActionCard
45
* A small, context-aware card used on PR pages to
46
* surface the most useful next step.
47
* - prioritizes primary actions
48
* - adapts to role & CI status
49
*/
50
export const

24
import React, { useMemo } from "react";
25
26
type Role = "author" | "reviewer" | "observer";
27
28
interface Action {
Alyssa Wright
4h ago
Could you add a short comment here explaining why we’re handling the error this way? I think future readers might not immediately understand the reasoning.
😀 1
🎉 2
Show 2 replies
Dana Hiera
4h ago
Nice catch. I added a guard clause for the empty case and wrote a test to cover it. Thanks for pointing that out!
29
id: string;
30
label: string;
31
primary?: string;
32
action: () => void;
33-+
}
34-+
35-+
interface Props {
36-+
title: string;
37-+
subtitle?: string;
38-+
role?: Role;
39
ciStatus?: "passing" | "failing" | "unknown";
40
actions?: Action[];
41
}
42
43
/**
44
* ActionCard
45
* A small, context-aware card used on PR pages to
46
* surface the most useful next step.
47
* - prioritizes primary actions
48
* - adapts to role & CI status
49
*/
50
export const