Fix remote URL parsing for paths with spaces.

Parse `git remote -v` output without truncating whitespace-containing URLs and add a regression test to lock in behavior for local remotes with spaced paths.

Made-with: Cursor

Author: bschellenberger2600

fixtures.go

@@ -200,14 +200,27 @@ func GetRemotes(t *testing.T, repoPath string) map[string]string {
 	}
 
 	for _, line := range strings.Split(lines, "\n") {
+		line = strings.TrimSpace(line)
 		if line == "" {
 			continue
 		}
-		parts := strings.Fields(line)
-		if len(parts) >= 2 {
-			name := parts[0]
-			url := parts[1]
-			remotes[name] = url
+
+		name, remainder, ok := strings.Cut(line, "\t")
+		if !ok {
+			// Fallback for unusual formatting that does not use tabs.
+			idx := strings.IndexAny(line, " \t")
+			if idx == -1 {
+				continue
+			}
+			name = line[:idx]
+			remainder = strings.TrimSpace(line[idx+1:])
+		}
+
+		remainder = strings.TrimSuffix(remainder, " (fetch)")
+		remainder = strings.TrimSuffix(remainder, " (push)")
+
+		if name != "" && remainder != "" {
+			remotes[name] = remainder
 		}
 	}
 

fixtures_test.go

@@ -79,6 +79,26 @@ func TestCreateTestRepo_WithRemotes(t *testing.T) {
 	}
 }
 
+func TestCreateTestRepo_WithRemotePathContainingSpaces(t *testing.T) {
+	remotePath := testutil.CreateBareRemote(t, "origin with space")
+
+	repoPath := testutil.CreateTestRepo(t, testutil.RepoOptions{
+		Name: "remote-space-repo",
+		Remotes: map[string]string{
+			"origin": remotePath,
+		},
+	})
+
+	remotes := testutil.GetRemotes(t, repoPath)
+	originURL, exists := remotes["origin"]
+	if !exists {
+		t.Fatal("Expected 'origin' remote to be configured")
+	}
+	if originURL != remotePath {
+		t.Fatalf("Expected origin URL %q, got %q", remotePath, originURL)
+	}
+}
+
 func TestCreateBareRemote(t *testing.T) {
 	remotePath := testutil.CreateBareRemote(t, "test-remote")