Fix issue-743 by sanitizing the URL

Author: Avish Porwal

cmd/publisher/commands/init.go

@@ -23,12 +23,14 @@ func InitCommand() error {
 
 	// Detect if we're in a subdirectory of the git repository
 	subfolder := detectSubfolder()
+	packageType := detectPackageType()
 
 	// Try to detect values from environment
-	name := detectServerName(subfolder)
+	name := detectServerName(subfolder, packageType)
 	description := detectDescription()
 	version := "1.0.0"
-	repoURL := detectRepoURL()
+	packageIdentifier := detectPackageIdentifier(name, packageType)
+	repoURL := detectRepoURL(packageType)
 	repoSource := MethodGitHub
 	if repoURL != "" && !strings.Contains(repoURL, "github.com") {
 		if strings.Contains(repoURL, "gitlab.com") {
@@ -38,9 +40,6 @@ func InitCommand() error {
 		}
 	}
 
-	packageType := detectPackageType()
-	packageIdentifier := detectPackageIdentifier(name, packageType)
-
 	// Create example environment variables
 	envVars := []model.KeyValueInput{
 		{
@@ -156,9 +155,9 @@ func getNameFromPackageJSON() string {
 	return fmt.Sprintf("io.github.<your-username>/%s", name)
 }
 
-func detectServerName(subfolder string) string {
+func detectServerName(subfolder string, packageType string) string {
 	// Try to get from git remote
-	repoURL := detectRepoURL()
+	repoURL := detectRepoURL(packageType)
 	if repoURL != "" && strings.Contains(repoURL, "github.com") {
 		name := buildGitHubServerName(repoURL, subfolder)
 		if name != "" {
@@ -212,7 +211,17 @@ func detectDescription() string {
 	return "An MCP server that provides [describe what your server does]"
 }
 
-func detectRepoURL() string {
+func detectRepoURL(packageType string) string {
+	sanitizeURL := func(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
+		}
+	}
+
 	// Try git remote
 	ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second)
 	defer cancel()
@@ -233,11 +242,11 @@ func detectRepoURL() string {
 		if json.Unmarshal(data, &pkg) == nil {
 			if repo, ok := pkg["repository"].(map[string]any); ok {
 				if url, ok := repo["url"].(string); ok {
-					return strings.TrimSuffix(url, ".git")
+					return sanitizeURL(strings.TrimSuffix(url, ".git"), packageType)
 				}
 			}
 			if repo, ok := pkg["repository"].(string); ok {
-				return strings.TrimSuffix(repo, ".git")
+				return sanitizeURL(strings.TrimSuffix(repo, ".git"), packageType)
 			}
 		}
 	}