fix(cli): keep every url of a precedence remote in owner resolution

listRemoteOwnerUrls used find() to pull origin/upstream, taking only the
first remote.<name>.url. Git allows several urls per remote (e.g. `git
remote set-url --add`), so a second origin/upstream url was dropped and
could miss a matching org owner. Collect all urls per precedence remote
with flatMap; dedup already handles repeats.

Reported by Copilot review on #55.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: theCodeDrift

packages/cli/src/util/git-remote.ts

@@ -142,9 +142,12 @@ const REMOTE_PRECEDENCE = ["origin", "upstream"];
 export async function listRemoteOwnerUrls(cwd: string): Promise<string[]> {
   const remotes = await listRemoteConfig(cwd);
   const ordered = [
-    ...REMOTE_PRECEDENCE.map((name) =>
-      remotes.find((remote) => remote.name === name)
-    ).filter((remote) => remote !== undefined),
+    // A remote can carry several `remote.<name>.url` entries (e.g. `git remote
+    // set-url --add`), so collect ALL of a precedence remote's urls, not just
+    // the first — otherwise a second origin/upstream url is dropped entirely.
+    ...REMOTE_PRECEDENCE.flatMap((name) =>
+      remotes.filter((remote) => remote.name === name)
+    ),
     ...remotes.filter((remote) => !REMOTE_PRECEDENCE.includes(remote.name)),
   ];
 

packages/cli/test/git-remote.test.ts

@@ -128,6 +128,31 @@ describe("listRemoteOwnerUrls", () => {
     expect(await listRemoteOwnerUrls(directory)).toEqual([]);
   });
 
+  it("keeps every url of a precedence remote (multi-url origin)", async () => {
+    // `git remote set-url --add` gives origin a second url; both owners must be
+    // considered, in order, rather than the first winning outright.
+    await execFileAsync(
+      "git",
+      ["remote", "add", "origin", "git@github.com:primary/widgets.git"],
+      { cwd: directory }
+    );
+    await execFileAsync(
+      "git",
+      [
+        "remote",
+        "set-url",
+        "--add",
+        "origin",
+        "https://github.com/secondary/widgets.git",
+      ],
+      { cwd: directory }
+    );
+    expect(await listRemoteOwnerUrls(directory)).toEqual([
+      "https://github.com/primary",
+      "https://github.com/secondary",
+    ]);
+  });
+
   it("matches a github remote given in ssh:// url form", async () => {
     // Regression guard: the old regex canonicalizer threw on ssh:// remotes, so
     // this repo would have resolved to no current-org context.