Allow more ways to specify the repository

gh-get now accepts more repository definitions that are allowed by
GitHub CLI, see https://cli.github.com/manual/gh_repo_clone.

Fixes #3

Author: britter

README.md

@@ -16,6 +16,9 @@ Example:
 
 `gh get britter/gh-get` will clone this respository into `~/github/britter/gh-get`
 
+> [!TIP]
+> gh-get also accepts full github URLs like https://github.com/britter/gh-get.git as well as SSH protocol URLs like [email protected]:britter/gh-get.git.
+
 ## Configuration
 
 There are two environment variables that control the location gh-get clones repositories to:

main.go

@@ -5,18 +5,25 @@ import (
 	"log"
 	"os"
 
+	"github.com/britter/gh-get/pkg/github"
 	"github.com/cli/go-gh"
 )
 
 func main() {
 	ghFolder := getGhFolder()
-	repository, err := getRepository()
+	repositoryDefinition, err := getRepository()
 	if err != nil {
 		log.Fatal(err)
 		return
 	}
 
-	repoClone := []string{"repo", "clone", repository, ghFolder + "/" + repository}
+	repository, err := github.Parse(repositoryDefinition)
+	if err != nil {
+		log.Fatal(err)
+		return
+	}
+
+	repoClone := []string{"repo", "clone", repositoryDefinition, ghFolder + "/" + repository.Owner + "/" + repository.Name}
 	stdOut, stdErr, err := gh.Exec(repoClone...)
 	if err != nil {
 		log.Fatal(err)

pkg/github/repository.go

@@ -0,0 +1,29 @@
+package github
+
+import (
+	"fmt"
+	"strings"
+)
+
+type Repository struct {
+	Owner string
+	Name  string
+}
+
+func Parse(definition string) (Repository, error) {
+	definition = strings.TrimPrefix(definition, "https://github.com/")
+	definition = strings.TrimPrefix(definition, "[email protected]:")
+	definition = strings.TrimSuffix(definition, ".git")
+
+	parts := strings.Split(definition, "/")
+	if len(parts) != 2 {
+		return Repository{}, fmt.Errorf("Invalid repository definition: %s", definition)
+	}
+
+	owner, name := parts[0], parts[1]
+	if owner == "" || name == "" {
+		return Repository{}, fmt.Errorf("Invalid repository definition: %s", definition)
+	}
+
+	return Repository{parts[0], parts[1]}, nil
+}

pkg/github/repository_test.go

@@ -0,0 +1,47 @@
+package github
+
+import (
+	"testing"
+)
+
+func TestParse(t *testing.T) {
+	tests := []struct {
+		input         string
+		expectedOwner string
+		expectedName  string
+		expectError   bool
+	}{
+		// Valid cases
+		{"owner/name", "owner", "name", false},
+		{"https://github.com/owner/name", "owner", "name", false},
+		{"https://github.com/owner/name.git", "owner", "name", false},
+		{"[email protected]:owner/name.git", "owner", "name", false},
+
+		// Invalid cases
+		{"https://github.com/invalidinput", "", "", true}, // Missing /
+		{"invalidinput", "", "", true},                    // Missing /
+		{"https://github.com/", "", "", true},             // Empty path
+		{"owner/", "", "", true},                          // Missing name
+		{"/name", "", "", true},                           // Missing owner
+	}
+
+	for _, test := range tests {
+		repository, err := Parse(test.input)
+
+		if test.expectError {
+			if err == nil {
+				t.Errorf("Expected error for input %q, but got none", test.input)
+			}
+		} else {
+			if err != nil {
+				t.Errorf("Did not expect error fro input %q, but got: %v", test.input, err)
+			}
+			if repository.Owner != test.expectedOwner {
+				t.Errorf("For input %q, expected owner %q, but got %q", test.input, test.expectedOwner, repository.Owner)
+			}
+			if repository.Name != test.expectedName {
+				t.Errorf("For input %q, expected name %q, but got %q", test.input, test.expectedName, repository.Name)
+			}
+		}
+	}
+}