Replace redundant parsePullRequestList call with lightweight pullRequestSummaryToInfo conversion

In findOpenPr, pullRequests is already ReadonlyArray<GitHubPullRequestSummary>
(validated and normalized by GitHubCli.listOpenPullRequests). Passing these
through parsePullRequestList — which is designed for raw unknown JSON — was
redundant. Add a dedicated pullRequestSummaryToInfo function that directly
maps GitHubPullRequestSummary to PullRequestInfo without re-checking types.

Author: cursoragent

apps/server/src/git/Layers/GitManager.ts

@@ -22,7 +22,7 @@ import {
   type GitRunStackedActionOptions,
 } from "../Services/GitManager.ts";
 import { GitCore } from "../Services/GitCore.ts";
-import { GitHubCli } from "../Services/GitHubCli.ts";
+import { GitHubCli, type GitHubPullRequestSummary } from "../Services/GitHubCli.ts";
 import { TextGeneration } from "../Services/TextGeneration.ts";
 import { extractBranchNameFromRemoteRef } from "../remoteRefs.ts";
 import { ServerSettingsService } from "../../serverSettings.ts";
@@ -305,6 +305,25 @@ function parsePullRequestList(raw: unknown): PullRequestInfo[] {
   return parsed;
 }
 
+function pullRequestSummaryToInfo(pr: GitHubPullRequestSummary): PullRequestInfo {
+  return {
+    number: pr.number,
+    title: pr.title,
+    url: pr.url,
+    baseRefName: pr.baseRefName,
+    headRefName: pr.headRefName,
+    state: pr.state ?? "open",
+    updatedAt: null,
+    ...(pr.isCrossRepository != null ? { isCrossRepository: pr.isCrossRepository } : {}),
+    ...(pr.headRepositoryNameWithOwner
+      ? { headRepositoryNameWithOwner: pr.headRepositoryNameWithOwner }
+      : {}),
+    ...(pr.headRepositoryOwnerLogin
+      ? { headRepositoryOwnerLogin: pr.headRepositoryOwnerLogin }
+      : {}),
+  };
+}
+
 function gitManagerError(operation: string, detail: string, cause?: unknown): GitManagerError {
   return new GitManagerError({
     operation,
@@ -767,21 +786,13 @@ export const makeGitManager = Effect.fn("makeGitManager")(function* () {
         headSelector,
         limit: 1,
       });
-      const normalizedPullRequests = parsePullRequestList(pullRequests);
+      const normalizedPullRequests = pullRequests.map(pullRequestSummaryToInfo);
 
       const firstPullRequest = normalizedPullRequests.find((pullRequest) =>
         matchesBranchHeadContext(pullRequest, headContext),
       );
       if (firstPullRequest) {
-        return {
-          number: firstPullRequest.number,
-          title: firstPullRequest.title,
-          url: firstPullRequest.url,
-          baseRefName: firstPullRequest.baseRefName,
-          headRefName: firstPullRequest.headRefName,
-          state: "open",
-          updatedAt: null,
-        } satisfies PullRequestInfo;
+        return firstPullRequest;
       }
     }