feat(inbox): show live PR status on pull request list cards

Extend the batched diff-stats GraphQL request to also fetch each PR's
state/isDraft, so list cards render live status (open/draft/merged/closed)
with no extra API calls. ReportImplementationPrLink now reads status from
the batch context when present and only falls back to the per-PR usePrDetails
query on the standalone detail view.

Author: andrewm4894

packages/core/src/git/router-schemas.ts

@@ -352,10 +352,18 @@ export const getPrChangedFilesInput = z.object({
 export const getPrChangedFilesOutput = z.array(changedFileSchema);
 
 // getPrDiffStatsBatch schemas
+//
+// Beyond the diff numbers, the batch also carries the PR's live status
+// (`state`/`merged`/`draft`) so list cards can render it from the single
+// batched GraphQL request instead of firing one REST call per visible PR.
 export const prDiffStatsSchema = z.object({
   additions: z.number(),
   deletions: z.number(),
   changedFiles: z.number(),
+  /** Lowercased GitHub PR state: "open" | "closed" | "merged". */
+  state: z.string(),
+  merged: z.boolean(),
+  draft: z.boolean(),
 });
 export type PrDiffStats = z.infer<typeof prDiffStatsSchema>;
 

packages/ui/src/features/inbox/components/PullRequestCard.tsx

@@ -14,6 +14,7 @@ import { InboxCardTitle } from "@posthog/ui/features/inbox/components/InboxCardT
 import { PrDiffStats } from "@posthog/ui/features/inbox/components/PrDiffStats";
 import { PriorityMonogram } from "@posthog/ui/features/inbox/components/PriorityMonogram";
 import { SuggestedReviewerAvatarStack } from "@posthog/ui/features/inbox/components/SuggestedReviewerAvatarStack";
+import { ReportImplementationPrLink } from "@posthog/ui/features/inbox/components/utils/ReportImplementationPrLink";
 import { useInboxReportDetailPrefetch } from "@posthog/ui/features/inbox/hooks/useInboxReportDetailPrefetch";
 import { useInboxReportArtefacts } from "@posthog/ui/features/inbox/hooks/useInboxReports";
 import { Button as UiButton } from "@posthog/ui/primitives/Button";
@@ -121,10 +122,16 @@ export function PullRequestCard({
       >
         <Flex align="center" gap="2" className="shrink-0">
           {report.implementation_pr_url && (
-            <PrDiffStats
-              prUrl={report.implementation_pr_url}
-              hideWhileLoading
-            />
+            <>
+              <ReportImplementationPrLink
+                prUrl={report.implementation_pr_url}
+                size="sm"
+              />
+              <PrDiffStats
+                prUrl={report.implementation_pr_url}
+                hideWhileLoading
+              />
+            </>
           )}
           <SuggestedReviewerAvatarStack
             reportId={report.id}

packages/ui/src/features/inbox/components/utils/ReportImplementationPrLink.tsx

@@ -1,6 +1,7 @@
 import { GitMergeIcon, GitPullRequestIcon } from "@phosphor-icons/react";
 import { cn } from "@posthog/quill";
 import { usePrDetails } from "@posthog/ui/features/git-interaction/usePrDetails";
+import { usePrDiffStatsFromBatch } from "@posthog/ui/features/inbox/context/PrDiffStatsBatchContext";
 import { Tooltip } from "@radix-ui/themes";
 
 export type ImplementationPrLinkSize = "sm" | "md";
@@ -45,9 +46,29 @@ export function ReportImplementationPrLink({
   size = "sm",
   onLinkClick,
 }: ReportImplementationPrLinkProps) {
-  const {
-    meta: { state, merged, draft, isLoading },
-  } = usePrDetails(prUrl);
+  // On list surfaces the surrounding `PrDiffStatsBatchContext` already carries
+  // this PR's status from the single batched request, so read it from there and
+  // skip the per-PR query. Fall back to `usePrDetails` only on the standalone
+  // detail view, where no batch provider is mounted.
+  const batchEntry = usePrDiffStatsFromBatch(prUrl);
+  const fallback = usePrDetails(batchEntry.hasBatch ? null : prUrl);
+
+  const state = batchEntry.hasBatch
+    ? (batchEntry.stats?.state ?? null)
+    : fallback.meta.state;
+  const merged = batchEntry.hasBatch
+    ? (batchEntry.stats?.merged ?? false)
+    : fallback.meta.merged;
+  const draft = batchEntry.hasBatch
+    ? (batchEntry.stats?.draft ?? false)
+    : fallback.meta.draft;
+  const isLoading = batchEntry.hasBatch
+    ? batchEntry.isLoading && !batchEntry.stats
+    : fallback.meta.isLoading;
+
+  // If neither source could resolve a status (PR 404s, gh failed, or the batch
+  // had no entry for this PR), don't render a misleading "open" badge.
+  if (!isLoading && state === null) return null;
 
   const isSm = size === "sm";
 

packages/workspace-server/src/services/git/schemas.ts

@@ -308,10 +308,18 @@ export type PrDetailsByUrlOutput = z.infer<typeof getPrDetailsByUrlOutput>;
 
 export const getPrChangedFilesInput = z.object({ prUrl: z.string() });
 
+// Also carries the PR's live status (`state`/`merged`/`draft`) so list cards
+// can render it from the single batched GraphQL request instead of firing one
+// REST call per visible PR. Mirrors `prDiffStatsSchema` in
+// `@posthog/core/git/router-schemas`.
 export const prDiffStatsSchema = z.object({
   additions: z.number(),
   deletions: z.number(),
   changedFiles: z.number(),
+  /** Lowercased GitHub PR state: "open" | "closed" | "merged". */
+  state: z.string(),
+  merged: z.boolean(),
+  draft: z.boolean(),
 });
 export type PrDiffStats = z.infer<typeof prDiffStatsSchema>;
 

packages/workspace-server/src/services/git/service.ts

@@ -1054,7 +1054,7 @@ export class GitService extends TypedEventEmitter<GitCloneEvents> {
   ): Promise<Record<string, PrDiffStats>> {
     const aliasFragments = chunk
       .map(([, { parsed }], index) => {
-        return `pr${index}: repository(owner: "${escapeGraphqlString(parsed.owner)}", name: "${escapeGraphqlString(parsed.repo)}") { pullRequest(number: ${parsed.number}) { additions deletions changedFiles } }`;
+        return `pr${index}: repository(owner: "${escapeGraphqlString(parsed.owner)}", name: "${escapeGraphqlString(parsed.repo)}") { pullRequest(number: ${parsed.number}) { additions deletions changedFiles state isDraft } }`;
       })
       .join("\n");
     const query = `query InboxPrDiffStatsBatch {\n${aliasFragments}\n}`;
@@ -1075,6 +1075,8 @@ export class GitService extends TypedEventEmitter<GitCloneEvents> {
             additions: number;
             deletions: number;
             changedFiles: number;
+            state: string;
+            isDraft: boolean;
           } | null;
         } | null
       >;
@@ -1085,10 +1087,15 @@ export class GitService extends TypedEventEmitter<GitCloneEvents> {
       const [, { urls }] = chunk[i];
       const node = parsed.data?.[`pr${i}`]?.pullRequest;
       if (!node) continue;
+      // GraphQL `PullRequestState` is OPEN | CLOSED | MERGED; normalise to the
+      // lowercase state + merged boolean shape the badge expects.
       const stats: PrDiffStats = {
         additions: node.additions,
         deletions: node.deletions,
         changedFiles: node.changedFiles,
+        state: node.state.toLowerCase(),
+        merged: node.state === "MERGED",
+        draft: node.isDraft,
       };
       for (const url of urls) {
         out[url] = stats;