fix(ui): align PR size thresholds with Prow/Kubernetes standard

Adopts the industry-standard Prow size schema (XS <10, S 10-29,
M 30-99, L 100-499, XL 500-999, XXL 1000+) replacing the previous
broader thresholds. Adds XXL category for mega-PRs.

Author: wgordon17

src/app/components/shared/SizeBadge.tsx

@@ -6,7 +6,7 @@ interface SizeBadgeProps {
   additions: number;
   deletions: number;
   changedFiles: number;
-  category?: "XS" | "S" | "M" | "L" | "XL";
+  category?: "XS" | "S" | "M" | "L" | "XL" | "XXL";
   filesUrl?: string;
 }
 
@@ -16,14 +16,16 @@ const SIZE_CONFIG = {
   M: "badge badge-warning badge-sm",
   L: "badge badge-error badge-sm",
   XL: "badge badge-error badge-sm",
+  XXL: "badge badge-error badge-sm",
 } as const;
 
-const SIZE_TOOLTIP: Record<"XS" | "S" | "M" | "L" | "XL", string> = {
+const SIZE_TOOLTIP: Record<"XS" | "S" | "M" | "L" | "XL" | "XXL", string> = {
   XS: "XS: <10 lines changed",
-  S: "S: 10–99 lines changed",
-  M: "M: 100–499 lines changed",
-  L: "L: 500–999 lines changed",
-  XL: "XL: 1000+ lines changed",
+  S: "S: 10–29 lines changed",
+  M: "M: 30–99 lines changed",
+  L: "L: 100–499 lines changed",
+  XL: "XL: 500–999 lines changed",
+  XXL: "XXL: 1000+ lines changed",
 };
 
 export default function SizeBadge(props: SizeBadgeProps) {

src/app/components/shared/filterTypes.ts

@@ -84,6 +84,7 @@ export const prFilterGroups: FilterChipGroupDef[] = [
       { value: "M", label: "M" },
       { value: "L", label: "L" },
       { value: "XL", label: "XL" },
+      { value: "XXL", label: "XXL" },
     ],
   },
 ];

src/app/stores/view.ts

@@ -32,7 +32,7 @@ export const PullRequestFiltersSchema = z.object({
   reviewDecision: z.enum(["all", "APPROVED", "CHANGES_REQUESTED", "REVIEW_REQUIRED", "mergeable"]).default("all"),
   draft: z.enum(["all", "draft", "ready"]).default("all"),
   checkStatus: z.enum(["all", "success", "failure", "pending", "conflict", "blocked", "none"]).default("all"),
-  sizeCategory: z.enum(["all", "XS", "S", "M", "L", "XL"]).default("all"),
+  sizeCategory: z.enum(["all", "XS", "S", "M", "L", "XL", "XXL"]).default("all"),
   user: z.enum(["all"]).or(z.string()).default("all"),
 });
 

src/shared/format.ts

@@ -83,13 +83,14 @@ export function formatDuration(startedAt: string, completedAt: string | null): s
 /**
  * Categorizes a PR by size based on total lines changed.
  */
-export function prSizeCategory(additions: number, deletions: number): "XS" | "S" | "M" | "L" | "XL" {
+export function prSizeCategory(additions: number, deletions: number): "XS" | "S" | "M" | "L" | "XL" | "XXL" {
   const total = (additions || 0) + (deletions || 0);
   if (total < 10) return "XS";
-  if (total < 100) return "S";
-  if (total < 500) return "M";
-  if (total < 1000) return "L";
-  return "XL";
+  if (total < 30) return "S";
+  if (total < 100) return "M";
+  if (total < 500) return "L";
+  if (total < 1000) return "XL";
+  return "XXL";
 }
 
 /**

tests/components/PullRequestsTab.test.tsx

@@ -166,10 +166,9 @@ describe("PullRequestsTab", () => {
     const pr = makePullRequest({ id: 1, title: "Big PR", additions: 300, deletions: 100, repoFullName: "org/repo-a" });
     setAllExpanded("pullRequests", ["org/repo-a"], true);
     render(() => <PullRequestsTab pullRequests={[pr]} userLogin="" />);
-    // prSizeCategory(300, 100) = 400 total -> M
-    // "M" appears as a size badge
-    const mEls = screen.getAllByText("M");
-    const badgeEl = mEls.find((el) => el.tagName.toLowerCase() === "span");
+    // prSizeCategory(300, 100) = 400 total -> L
+    const lEls = screen.getAllByText("L");
+    const badgeEl = lEls.find((el) => el.tagName.toLowerCase() === "span");
     expect(badgeEl).toBeDefined();
   });
 

tests/components/shared-badges.test.tsx

@@ -67,9 +67,9 @@ describe("SizeBadge", () => {
     screen.getByText("1 file");
   });
 
-  it("renders XL badge for large changes", () => {
+  it("renders XXL badge for large changes", () => {
     render(() => <SizeBadge additions={800} deletions={500} changedFiles={42} />);
-    screen.getByText("XL");
+    screen.getByText("XXL");
     screen.getByText("+800");
     screen.getByText("-500");
     screen.getByText("42 files");

tests/lib/format.test.ts

@@ -238,20 +238,24 @@ describe("prSizeCategory", () => {
     expect(prSizeCategory(3, 2)).toBe("XS");
   });
 
-  it("returns S for total 10-99", () => {
-    expect(prSizeCategory(50, 30)).toBe("S");
+  it("returns S for total 10-29", () => {
+    expect(prSizeCategory(10, 5)).toBe("S");
   });
 
-  it("returns M for total 100-499", () => {
-    expect(prSizeCategory(200, 100)).toBe("M");
+  it("returns M for total 30-99", () => {
+    expect(prSizeCategory(50, 30)).toBe("M");
   });
 
-  it("returns L for total 500-999", () => {
-    expect(prSizeCategory(600, 200)).toBe("L");
+  it("returns L for total 100-499", () => {
+    expect(prSizeCategory(200, 100)).toBe("L");
   });
 
-  it("returns XL for total >= 1000", () => {
-    expect(prSizeCategory(800, 500)).toBe("XL");
+  it("returns XL for total 500-999", () => {
+    expect(prSizeCategory(600, 200)).toBe("XL");
+  });
+
+  it("returns XXL for total >= 1000", () => {
+    expect(prSizeCategory(800, 500)).toBe("XXL");
   });
 
   it("returns XS for (0, 0)", () => {
@@ -266,12 +270,36 @@ describe("prSizeCategory", () => {
     expect(prSizeCategory(5, 5)).toBe("S");
   });
 
-  it("returns L for total 999 (boundary below 1000)", () => {
-    expect(prSizeCategory(500, 499)).toBe("L");
+  it("returns S for total 29 (boundary below 30)", () => {
+    expect(prSizeCategory(15, 14)).toBe("S");
+  });
+
+  it("returns M for total 30 (boundary at 30)", () => {
+    expect(prSizeCategory(15, 15)).toBe("M");
+  });
+
+  it("returns M for total 99 (boundary below 100)", () => {
+    expect(prSizeCategory(50, 49)).toBe("M");
+  });
+
+  it("returns L for total 100 (boundary at 100)", () => {
+    expect(prSizeCategory(50, 50)).toBe("L");
+  });
+
+  it("returns L for total 499 (boundary below 500)", () => {
+    expect(prSizeCategory(250, 249)).toBe("L");
+  });
+
+  it("returns XL for total 500 (boundary at 500)", () => {
+    expect(prSizeCategory(250, 250)).toBe("XL");
+  });
+
+  it("returns XL for total 999 (boundary below 1000)", () => {
+    expect(prSizeCategory(500, 499)).toBe("XL");
   });
 
-  it("returns XL for total 1000 (boundary at 1000)", () => {
-    expect(prSizeCategory(500, 500)).toBe("XL");
+  it("returns XXL for total 1000 (boundary at 1000)", () => {
+    expect(prSizeCategory(500, 500)).toBe("XXL");
   });
 
   it("handles NaN/undefined gracefully — defaults to XS", () => {