Refactor validation code

Author: Avish Porwal

internal/api/handlers/v0/publish.go

@@ -12,6 +12,7 @@ import (
 	"github.com/modelcontextprotocol/registry/internal/database"
 	"github.com/modelcontextprotocol/registry/internal/model"
 	"github.com/modelcontextprotocol/registry/internal/service"
+	"github.com/modelcontextprotocol/registry/internal/validators"
 	"golang.org/x/net/html"
 )
 
@@ -48,15 +49,11 @@ func PublishHandler(registry service.RegistryService, authService auth.Service)
 			http.Error(w, "Invalid server detail payload: "+err.Error(), http.StatusBadRequest)
 			return
 		}
-		// Validate required fields
-		if serverDetail.Name == "" {
-			http.Error(w, "Name is required", http.StatusBadRequest)
-			return
-		}
 
-		// Version is required
-		if serverDetail.VersionDetail.Version == "" {
-			http.Error(w, "Version is required", http.StatusBadRequest)
+		// Validate the server detail
+		validator := validators.NewServerValidator()
+		if err := validator.Validate(&serverDetail.Server); err != nil {
+			http.Error(w, err.Error(), http.StatusBadRequest)
 			return
 		}
 

internal/validators/validators.go

@@ -0,0 +1,30 @@
+package validators
+
+import (
+	"fmt"
+
+	"github.com/modelcontextprotocol/registry/internal/model"
+)
+
+type Validator interface {
+	// Validate checks if the input is valid according to the validator's rules
+	Validate(obj *model.Server) error
+}
+
+type ServerValidator struct{}
+
+func (v *ServerValidator) Validate(obj *model.Server) error {
+	if obj.Name == "" {
+		return fmt.Errorf("Name is required")
+	}
+
+	// Version is required
+	if obj.VersionDetail.Version == "" {
+		return fmt.Errorf("Version is required")
+	}
+	return nil
+}
+
+func NewServerValidator() Validator {
+	return &ServerValidator{}
+}