fix(download): correctly parse remote URL

This changes the "zmk download" command to use giturlparse when
determining the URL for the GitHub actions page. This correctly handles
URLs that end in ".git" as well as SSH "user@host:path" URLS.

Author: joelspadin

pyproject.toml

@@ -17,6 +17,7 @@ classifiers = [
 dependencies = [
   "backports.strenum; python_version < '3.11'",
   "dacite >= 1.9.2, < 2.0.0",
+  "giturlparse >= 0.14.0, < 0.15.0",
   "mako >= 1.3.10, < 2.0.0",
   "rich >= 14.2.0, < 15.0.0",
   "ruamel.yaml >= 0.18.17, < 0.19.0",

zmk/commands/download.py

@@ -4,9 +4,11 @@
 
 import webbrowser
 
+import giturlparse
 import typer
 
 from ..config import get_config
+from ..exceptions import FatalError
 from ..repo import Repo
 
 
@@ -22,7 +24,13 @@ def download(ctx: typer.Context) -> None:
 
 
 def _get_actions_url(repo: Repo):
-    remote = repo.git("remote", capture_output=True).strip()
-    remote_url = repo.git("remote", "get-url", remote, capture_output=True).strip()
+    remote_url = repo.get_remote_url()
 
-    return f"{remote_url}/actions/workflows/build.yml"
+    p = giturlparse.parse(remote_url)
+
+    match p.platform:
+        case "github":
+            return f"https://github.com/{p.owner}/{p.repo}/actions/workflows/build.yml"
+
+        case _:
+            raise FatalError(f"Unsupported remote URL: {remote_url}")

zmk/repo.py

@@ -116,6 +116,11 @@ def get_modules(self) -> Generator[Module, None, None]:
         for line in modules.splitlines():
             yield Module(self.west_path / line)
 
+    def get_remote_url(self) -> str:
+        """Get the remote URL for the checked out Git branch."""
+        remote = self.git("remote", capture_output=True).strip()
+        return self.git("remote", "get-url", remote, capture_output=True).strip()
+
     @property
     def build_matrix_path(self) -> Path:
         """Path to the "build.yaml" file."""