Refactor validation code

Author: Avish Porwal

internal/api/handlers/v0/publish.go

@@ -9,6 +9,7 @@ import (
 	"github.com/modelcontextprotocol/registry/internal/auth"
 	"github.com/modelcontextprotocol/registry/internal/model"
 	"github.com/modelcontextprotocol/registry/internal/service"
+	"github.com/modelcontextprotocol/registry/internal/validators"
 )
 
 // PublishServerInput represents the input for publishing a server
@@ -17,7 +18,6 @@ type PublishServerInput struct {
 	Body          model.PublishRequest
 }
 
-
 // RegisterPublishEndpoint registers the publish endpoint
 func RegisterPublishEndpoint(api huma.API, registry service.RegistryService, authService auth.Service) {
 	huma.Register(api, huma.Operation{
@@ -42,13 +42,10 @@ func RegisterPublishEndpoint(api huma.API, registry service.RegistryService, aut
 		// Convert PublishRequest body to ServerDetail
 		serverDetail := input.Body.ServerDetail
 
-		// Huma handles validation automatically based on struct tags
-		// But we can add custom validation if needed
-		if serverDetail.Name == "" {
-			return nil, huma.Error400BadRequest("Name is required")
-		}
-		if serverDetail.VersionDetail.Version == "" {
-			return nil, huma.Error400BadRequest("Version is required")
+		// Validate the server detail
+		validator := validators.NewServerValidator()
+		if err := validator.Validate(&serverDetail.Server); err != nil {
+			return nil, huma.Error400BadRequest(err.Error())
 		}
 
 		// Determine authentication method based on server name prefix

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{}
+}