Fix reversed remote fallback and missing PR item filter guard

cursoragent · juliusmarminge · cursoragent · commit 5e2a3e91a717 · 2026-03-09T18:10:23.000Z
- Remove .toReversed() from listRemoteNames so resolvePrimaryRemoteName
  falls back to the most-significant remote (consistent with parseRemoteNames
  usage elsewhere)
- Add explicit checkoutPullRequestItemValue guard in filteredBranchPickerItems
  filter and include it in the useMemo dependency array

Co-authored-by: Julius Marminge &lt;juliusmarminge@users.noreply.github.com&gt;
diff --git a/apps/server/src/git/Layers/GitCore.ts b/apps/server/src/git/Layers/GitCore.ts
@@ -420,7 +420,7 @@ const makeGitCore = Effect.gen(function* () {
 
   const listRemoteNames = (cwd: string): Effect.Effect<ReadonlyArray<string>, GitCommandError> =>
     runGitStdout("GitCore.listRemoteNames", cwd, ["remote"]).pipe(
-      Effect.map((stdout) => parseRemoteNames(stdout).toReversed()),
+      Effect.map((stdout) => parseRemoteNames(stdout)),
     );
 
   const resolvePrimaryRemoteName = (cwd: string): Effect.Effect<string, GitCommandError> =>
diff --git a/apps/web/src/components/BranchToolbarBranchSelector.tsx b/apps/web/src/components/BranchToolbarBranchSelector.tsx
@@ -126,9 +126,10 @@ export function BranchToolbarBranchSelector({
         ? branchPickerItems
         : branchPickerItems.filter((itemValue) => {
             if (createBranchItemValue && itemValue === createBranchItemValue) return true;
+            if (checkoutPullRequestItemValue && itemValue === checkoutPullRequestItemValue) return true;
             return itemValue.toLowerCase().includes(normalizedBranchQuery);
           }),
-    [branchPickerItems, createBranchItemValue, normalizedBranchQuery],
+    [branchPickerItems, checkoutPullRequestItemValue, createBranchItemValue, normalizedBranchQuery],
   );
   const [resolvedActiveBranch, setOptimisticBranch] = useOptimistic(
     canonicalActiveBranch,
