Strip trailing newline from get_remote_url

git remote get-url emits the URL with a trailing newline. The decoded
result was returned verbatim, leaving the newline embedded in the URL
string used by submodule URL resolution and downstream string
operations.

https://claude.ai/code/session_01KKvrvnVvsBChohuxbRRmzA

Author: claude

CHANGELOG.rst

@@ -5,6 +5,7 @@ Release 0.14.1 (unreleased)
 * Fix ``Version`` comparison raising ``AttributeError`` when compared against a non-``Version`` object
 * Fix ``dfetch add`` matching a remote whose base URL is only a string prefix (not a path prefix) of the project URL
 * Fix git ref resolution spuriously matching the first reference for an empty revision
+* Strip the trailing newline from the git origin URL returned by ``get_remote_url``
 
 Release 0.14.0 (released 2026-06-14)
 ===========================

dfetch/vcs/git.py

@@ -682,7 +682,7 @@ def get_remote_url() -> str:
         """Get the url of the remote origin."""
         try:
             result = run_on_cmdline(logger, ["git", "remote", "get-url", "origin"])
-            decoded_result = str(result.stdout.decode())
+            decoded_result = str(result.stdout.decode()).strip()
         except SubprocessCommandError:
             decoded_result = ""
 

tests/test_git_vcs.py

@@ -318,6 +318,13 @@ def test_ls_remote():
         assert info == expected
 
 
+def test_get_remote_url_strips_trailing_newline():
+    """git remote get-url appends a newline that must not leak into the URL."""
+    with patch("dfetch.vcs.git.run_on_cmdline") as run_on_cmdline_mock:
+        run_on_cmdline_mock.return_value.stdout = b"https://github.com/org/repo.git\n"
+        assert GitLocalRepo.get_remote_url() == "https://github.com/org/repo.git"
+
+
 def test_find_branch_tip_or_tag_from_sha_empty_rev_matches_nothing():
     """An empty revision must not spuriously match the first reference."""
     info = {