cli: parse github repository url (close #307)

Author: devmatteini

CHANGELOG.md

@@ -7,6 +7,17 @@ to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+## [0.9.0] - 2025-08-14
+
+### Added
+
+- You can now also pass GitHub URL as repository argument to every
+  subcommand ([#307](https://github.com/devmatteini/dra/issues/307))
+
+```shell
+dra download https://github.com/devmatteini/dra-tests
+```
+
 ## [0.8.2] - 2025-05-28
 
 - Fix windows executable by including static crt ([#302](https://github.com/devmatteini/dra/issues/302))

Cargo.lock

@@ -34,6 +34,7 @@ xz2 = "0.1.7"
 bzip2 = "0.6.0"
 urlencoding = "2.1.3"
 itertools = "0.14.0"
+url = "2.5.4"
 
 [dev-dependencies]
 test-case = "3.3.1"

Cargo.toml

@@ -39,7 +39,7 @@ pub struct Cli {
 pub enum Command {
     /// Select and download an asset
     Download {
-        /// GitHub repository using format {owner}/{repo}
+        /// GitHub repository using format {owner}/{repo} or the repository URL https://github.com/{owner}/{repo}
         #[arg(value_parser = Repository::try_parse)]
         repo: Repository,
 
@@ -95,7 +95,7 @@ pub enum Command {
 
     /// Select an asset and generate an untagged version of it
     Untag {
-        /// GitHub repository using format {owner}/{repo}
+        /// GitHub repository using format {owner}/{repo} or the repository URL https://github.com/{owner}/{repo}
         #[arg(value_parser = Repository::try_parse)]
         repo: Repository,
     },

src/cli/root_command.rs

@@ -1,4 +1,5 @@
 use std::fmt::Formatter;
+use url::Url;
 
 #[derive(Debug, Eq, PartialEq, Clone)]
 pub struct Repository {
@@ -11,10 +12,19 @@ impl Repository {
         if src.is_empty() {
             return Err("Invalid repository. Cannot be empty".to_string());
         }
-        if !src.contains('/') {
+
+        if src.starts_with("http://github.com") || src.starts_with("https://github.com") {
+            Self::parse_url(src)
+        } else {
+            Self::parse(src)
+        }
+    }
+
+    fn parse(input: &str) -> Result<Repository, String> {
+        if !input.contains('/') {
             return Err("Invalid repository. Use {owner}/{repo} format".to_string());
         }
-        let parts = src
+        let parts = input
             .split('/')
             .filter(|x| !x.is_empty())
             .collect::<Vec<&str>>();
@@ -27,6 +37,23 @@ impl Repository {
             repo: parts[1].to_string(),
         })
     }
+
+    fn parse_url(input: &str) -> Result<Repository, String> {
+        let github_url = Url::parse(input).map_err(|x| format!("Invalid repository URL: {}", x))?;
+        let parts = github_url
+            .path()
+            .split('/')
+            .filter(|x| !x.is_empty())
+            .collect::<Vec<&str>>();
+        if parts.len() < 2 {
+            return Err("Invalid repository URL. Missing owner or repo".to_string());
+        }
+
+        Ok(Repository {
+            owner: parts[0].to_string(),
+            repo: parts[1].to_string(),
+        })
+    }
 }
 
 impl std::fmt::Display for Repository {
@@ -54,6 +81,36 @@ mod tests {
         );
     }
 
+    #[test]
+    fn valid_repository_from_url() {
+        let input = "https://github.com/foo/bar?tab=readme-ov-file";
+
+        let result = Repository::try_parse(input);
+
+        assert_eq!(
+            Ok(Repository {
+                owner: "foo".to_string(),
+                repo: "bar".to_string()
+            }),
+            result
+        );
+    }
+
+    #[test]
+    fn valid_repository_url_from_any_page() {
+        let input = "https://github.com/foo/bar/actions/runs/16966254957";
+
+        let result = Repository::try_parse(input);
+
+        assert_eq!(
+            Ok(Repository {
+                owner: "foo".to_string(),
+                repo: "bar".to_string()
+            }),
+            result
+        );
+    }
+
     #[test]
     fn missing_owner() {
         let input = "/bar";
@@ -72,6 +129,24 @@ mod tests {
         assert_error(|e| assert_contains("Missing owner or repo", e), result);
     }
 
+    #[test]
+    fn missing_owner_in_url() {
+        let input = "https://github.com";
+
+        let result = Repository::try_parse(input);
+
+        assert_error(|e| assert_contains("Missing owner or repo", e), result);
+    }
+
+    #[test]
+    fn missing_repo_in_url() {
+        let input = "https://github.com/foo/";
+
+        let result = Repository::try_parse(input);
+
+        assert_error(|e| assert_contains("Missing owner or repo", e), result);
+    }
+
     #[test]
     fn empty_repository() {
         let input = "";