Fix issue-743 by sanitizing the URL

Author: Avish Porwal

cmd/publisher/commands/init.go

@@ -11,6 +11,7 @@ import (
 	"strings"
 	"time"
 
+	publisherutils "github.com/modelcontextprotocol/registry/cmd/publisher/utils"
 	apiv0 "github.com/modelcontextprotocol/registry/pkg/api/v0"
 	"github.com/modelcontextprotocol/registry/pkg/model"
 )
@@ -390,7 +391,7 @@ func createServerJSON(
 	var repo *model.Repository
 	if repoURL != "" && repoSource != "" {
 		repo = &model.Repository{
-			URL:    repoURL,
+			URL:    publisherutils.SanitizeRepoURL(repoURL, packageType),
 			Source: repoSource,
 		}
 

cmd/publisher/publisher.exe

@@ -0,0 +1,20 @@
+package publisherutils
+
+import (
+	"strings"
+
+	"github.com/modelcontextprotocol/registry/pkg/model"
+)
+
+// SanitizeRepoURL normalizes repository URLs based on source/package type.
+// - For git sources, it strips the optional "git+" prefix (e.g., "git+https://..." -> "https://...")
+// - For other types, returns the URL unchanged unless specific normalization is required
+func SanitizeRepoURL(url string, packageType string) string {
+	switch packageType {
+	// Some package ecosystems (like npm) may embed git repo URLs; be defensive and strip git+ if present
+	case model.RegistryTypeNPM:
+		return strings.TrimPrefix(url, "git+")
+	default:
+		return url
+	}
+}

cmd/publisher/utils/publisherutils.go

@@ -0,0 +1,45 @@
+package publisherutils_test
+
+import (
+	"testing"
+
+	publisherutils "github.com/modelcontextprotocol/registry/cmd/publisher/utils"
+	"github.com/modelcontextprotocol/registry/pkg/model"
+)
+
+func TestSanitizeRepoURL(t *testing.T) {
+	tests := []struct {
+		name    string
+		inURL   string
+		pkgType string
+		wantURL string
+	}{
+		{
+			name:    "git url without prefix stays same",
+			inURL:   "https://github.com/jonathanhefner/my-package.git",
+			pkgType: model.RegistryTypePyPI,
+			wantURL: "https://github.com/jonathanhefner/my-package.git",
+		},
+		{
+			name:    "npm repo url with git+ prefix sanitized",
+			inURL:   "git+https://github.com/org/repo.git",
+			pkgType: model.RegistryTypeNPM,
+			wantURL: "https://github.com/org/repo.git",
+		},
+		{
+			name:    "npm repo url without prefix unchanged",
+			inURL:   "https://github.com/org/repo.git",
+			pkgType: model.RegistryTypeNPM,
+			wantURL: "https://github.com/org/repo.git",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			got := publisherutils.SanitizeRepoURL(tt.inURL, tt.pkgType)
+			if got != tt.wantURL {
+				t.Fatalf("SanitizeRepoURL(%q,%q)=%q; want %q", tt.inURL, tt.pkgType, got, tt.wantURL)
+			}
+		})
+	}
+}