feat: Open PR 滞留日数分布チャート

app/components/layout/nav-config.ts

@@ -1,4 +1,5 @@
 import {
+  ChartAreaIcon,
   CircleDotIcon,
   FunnelIcon,
   GitMergeIcon,
@@ -50,6 +51,11 @@ export function getNavConfig(orgSlug: string): NavGroupProps[] {
           url: href('/:orgSlug/analysis/reviews', { orgSlug }),
           icon: FunnelIcon,
         },
+        {
+          title: 'Inventory',
+          url: href('/:orgSlug/analysis/inventory', { orgSlug }),
+          icon: ChartAreaIcon,
+        },
         {
           title: 'Feedbacks',
           url: href('/:orgSlug/analysis/feedbacks', { orgSlug }),

app/routes/$orgSlug/analysis/inventory/+components/open-pr-inventory-chart.tsx

@@ -0,0 +1,105 @@
+import { Area, AreaChart, CartesianGrid, XAxis, YAxis } from 'recharts'
+import {
+  Card,
+  CardContent,
+  CardDescription,
+  CardHeader,
+  CardTitle,
+} from '~/app/components/ui/card'
+import {
+  type ChartConfig,
+  ChartContainer,
+  ChartLegend,
+  ChartLegendContent,
+  ChartTooltip,
+  ChartTooltipContent,
+} from '~/app/components/ui/chart'
+import { useTimezone } from '~/app/hooks/use-timezone'
+import dayjs from '~/app/libs/dayjs'
+
+import type { OpenPRInventoryAggregation } from '../+functions/aggregate'
+
+// Colors aligned with Review Stacks age thresholds
+const chartConfig = {
+  daysUnder1: { label: '< 1d', color: '#3b82f6' }, // blue-500
+  days1to3: { label: '1-3d', color: '#10b981' }, // emerald-500
+  days3to7: { label: '3-7d', color: '#f59e0b' }, // amber-500
+  days7to14: { label: '7-14d', color: '#ef4444' }, // red-500
+  days14to30: { label: '14-30d', color: '#a855f7' }, // purple-500
+  days31Plus: { label: '31d+', color: '#262626' }, // neutral-800
+} satisfies ChartConfig
+
+export function OpenPRInventoryChart({
+  data,
+}: {
+  data: OpenPRInventoryAggregation
+}) {
+  const timezone = useTimezone()
+  const { weeks } = data
+
+  const hasAnyTotal = weeks.some((w) => w.total > 0)
+
+  if (weeks.length === 0 || !hasAnyTotal) {
+    return (
+      <Card>
+        <CardHeader>
+          <CardTitle>Open PR inventory</CardTitle>
+          <CardDescription>
+            No open pull requests match the selected filters for this period.
+          </CardDescription>
+        </CardHeader>
+      </Card>
+    )
+  }
+
+  return (
+    <Card>
+      <CardHeader>
+        <CardTitle>Open PR inventory</CardTitle>
+        <CardDescription>
+          Stacked open PR counts by age (days open), measured at each week end
+          (or now for the current week). Shows where review backlog builds up.
+        </CardDescription>
+      </CardHeader>
+      <CardContent>
+        <ChartContainer
+          config={chartConfig}
+          style={{ height: 320, width: '100%' }}
+        >
+          <AreaChart data={weeks}>
+            <CartesianGrid vertical={false} />
+            <XAxis dataKey="weekLabel" tick={{ fontSize: 12 }} />
+            <YAxis allowDecimals={false} />
+            <ChartTooltip
+              content={
+                <ChartTooltipContent
+                  labelFormatter={(_, payload) => {
+                    const snapshotAt = payload?.[0]?.payload?.snapshotAt as
+                      | string
+                      | undefined
+                    return snapshotAt
+                      ? `Week ending ${dayjs.utc(snapshotAt).tz(timezone).format('YYYY-MM-DD')}`
+                      : ''
+                  }}
+                />
+              }
+            />
+            <ChartLegend
+              content={<ChartLegendContent className="flex-wrap" />}
+            />
+            {Object.keys(chartConfig).map((key) => (
+              <Area
+                key={key}
+                type="linear"
+                dataKey={key}
+                stackId="inventory"
+                fill={`var(--color-${key})`}
+                stroke={`var(--color-${key})`}
+              />
+            ))}
+          </AreaChart>
+        </ChartContainer>
+      </CardContent>
+    </Card>
+  )
+}

app/routes/$orgSlug/analysis/inventory/+functions/aggregate.test.ts

@@ -0,0 +1,199 @@
+import { describe, expect, test } from 'vitest'
+
+import dayjs from '~/app/libs/dayjs'
+
+import {
+  aggregateWeeklyOpenPRInventory,
+  isOpenAtSnapshot,
+  type OpenPRInventoryRawRow,
+} from './aggregate'
+
+const baseRow = (
+  overrides: Partial<OpenPRInventoryRawRow>,
+): OpenPRInventoryRawRow => ({
+  repositoryId: 'repo-1',
+  number: 1,
+  pullRequestCreatedAt: '2024-06-01T00:00:00.000Z',
+  mergedAt: null,
+  closedAt: null,
+  firstReviewedAt: null,
+  ...overrides,
+})
+
+describe('isOpenAtSnapshot', () => {
+  test('mergedAt/closedAt exactly at snapshotAt are not open', () => {
+    const snap = '2024-06-09T12:00:00.000Z'
+    expect(
+      isOpenAtSnapshot(
+        baseRow({
+          pullRequestCreatedAt: '2024-06-01T00:00:00.000Z',
+          mergedAt: snap,
+          closedAt: null,
+        }),
+        snap,
+      ),
+    ).toBe(false)
+    expect(
+      isOpenAtSnapshot(
+        baseRow({
+          pullRequestCreatedAt: '2024-06-01T00:00:00.000Z',
+          mergedAt: null,
+          closedAt: snap,
+        }),
+        snap,
+      ),
+    ).toBe(false)
+  })
+})
+
+describe('aggregateWeeklyOpenPRInventory', () => {
+  const tz = 'UTC'
+
+  const allBucketKeys = [
+    'daysUnder1',
+    'days1to3',
+    'days3to7',
+    'days7to14',
+    'days14to30',
+    'days31Plus',
+  ] as const
+
+  test('counts only PRs open at each week snapshot', () => {
+    const snapshotSunday = '2024-06-09T23:59:59.999Z'
+    const sinceDate = '2024-06-01T00:00:00.000Z'
+    const now = snapshotSunday
+
+    const rows: OpenPRInventoryRawRow[] = [
+      baseRow({
+        number: 1,
+        pullRequestCreatedAt: '2024-06-08T12:00:00.000Z',
+        mergedAt: null,
+        closedAt: null,
+      }),
+      baseRow({
+        number: 2,
+        pullRequestCreatedAt: '2024-06-01T00:00:00.000Z',
+        mergedAt: '2024-06-05T00:00:00.000Z',
+        closedAt: null,
+      }),
+    ]
+
+    const { weeks } = aggregateWeeklyOpenPRInventory(rows, sinceDate, now, tz)
+    const w = weeks.find((x) => x.snapshotAt === snapshotSunday)
+    expect(w?.total).toBe(1)
+    expect(w?.days1to3).toBe(1)
+  })
+
+  test('bucket boundaries (age days)', () => {
+    const snapshotSunday = '2024-06-09T23:59:59.999Z'
+    const sinceDate = '2024-05-01T00:00:00.000Z'
+    const now = snapshotSunday
+
+    const cases: {
+      age: number
+      key: (typeof allBucketKeys)[number]
+    }[] = [
+      { age: 0, key: 'daysUnder1' },
+      { age: 1, key: 'days1to3' },
+      { age: 2, key: 'days1to3' },
+      { age: 3, key: 'days3to7' },
+      { age: 6, key: 'days3to7' },
+      { age: 7, key: 'days7to14' },
+      { age: 13, key: 'days7to14' },
+      { age: 14, key: 'days14to30' },
+      { age: 29, key: 'days14to30' },
+      { age: 30, key: 'days31Plus' },
+      { age: 31, key: 'days31Plus' },
+    ]
+
+    for (const { age, key } of cases) {
+      const created = dayjs
+        .utc(snapshotSunday)
+        .tz(tz)
+        .subtract(age, 'day')
+        .toISOString()
+      const { weeks } = aggregateWeeklyOpenPRInventory(
+        [baseRow({ number: age, pullRequestCreatedAt: created })],
+        sinceDate,
+        now,
+        tz,
+      )
+      const w = weeks.find((x) => x.snapshotAt === snapshotSunday)
+      expect(w, `age ${age}`).toBeDefined()
+      expect(w?.total, `age ${age}`).toBe(1)
+      expect(w?.[key], `age ${age} → ${key}`).toBe(1)
+      const bucketSum = allBucketKeys.reduce((sum, k) => sum + (w?.[k] ?? 0), 0)
+      expect(bucketSum, `age ${age} single bucket`).toBe(1)
+    }
+  })
+
+  test('current incomplete week uses now as snapshot and counts at now', () => {
+    const sinceDate = '2024-06-03T00:00:00.000Z'
+    const now = '2024-06-05T15:00:00.000Z'
+    const rows: OpenPRInventoryRawRow[] = [
+      baseRow({
+        pullRequestCreatedAt: '2024-06-03T10:00:00.000Z',
+        mergedAt: null,
+        closedAt: null,
+      }),
+    ]
+
+    const { weeks } = aggregateWeeklyOpenPRInventory(rows, sinceDate, now, tz)
+    expect(weeks).toHaveLength(1)
+    expect(weeks[0]?.snapshotAt).toBe(now)
+    expect(weeks[0]?.total).toBe(1)
+  })
+
+  test('empty rows still yields weeks with zeros', () => {
+    const sinceDate = '2024-06-01T00:00:00.000Z'
+    const now = '2024-06-15T12:00:00.000Z'
+    const { weeks } = aggregateWeeklyOpenPRInventory([], sinceDate, now, tz)
+    expect(weeks.length).toBeGreaterThan(0)
+    for (const w of weeks) {
+      expect(w.total).toBe(0)
+    }
+  })
+
+  test('unreviewedOnly filters by snapshot-time review status', () => {
+    const sinceDate = '2024-06-01T00:00:00.000Z'
+    const now = '2024-06-16T23:59:59.999Z'
+    const rows: OpenPRInventoryRawRow[] = [
+      baseRow({
+        number: 1,
+        pullRequestCreatedAt: '2024-06-03T10:00:00.000Z',
+        firstReviewedAt: '2024-06-10T12:00:00.000Z',
+      }),
+      baseRow({
+        number: 2,
+        pullRequestCreatedAt: '2024-06-05T10:00:00.000Z',
+        firstReviewedAt: null,
+      }),
+    ]
+
+    const result = aggregateWeeklyOpenPRInventory(
+      rows,
+      sinceDate,
+      now,
+      tz,
+      true,
+    )
+
+    // Week of Jun 3 (snapshot Jun 9): PR#1 not yet reviewed → counted, PR#2 counted → total 2
+    const week1 = result.weeks.find((w) => w.weekLabel === '06/03')
+    expect(week1?.total).toBe(2)
+
+    // Week of Jun 10 (snapshot Jun 16): PR#1 reviewed on Jun 10 ≤ snapshot → excluded, PR#2 counted → total 1
+    const week2 = result.weeks.find((w) => w.weekLabel === '06/10')
+    expect(week2?.total).toBe(1)
+  })
+
+  test('sinceDate after now yields empty weeks', () => {
+    const { weeks } = aggregateWeeklyOpenPRInventory(
+      [],
+      '2025-01-01T00:00:00.000Z',
+      '2024-01-01T00:00:00.000Z',
+      tz,
+    )
+    expect(weeks).toEqual([])
+  })
+})