Merge pull request #77 from forkd-app/task/72-empty-recipe-revision-titles

Author: GV14982

api/services/recipe/recipe.go

@@ -24,6 +24,16 @@ const (
 	DEFAULT_LIST_RECIPE_SORT_FIELD = "publish_date"
 )
 
+func validateTitle(title string) error {
+	if len(title) < 3 {
+		return fmt.Errorf("title must be at least 3 characters long")
+	}
+	if len(title) > 150 {
+		return fmt.Errorf("title must be no more than 150 characters long")
+	}
+	return nil
+}
+
 type RecipeService interface {
 	GetRecipeByID(ctx context.Context, id uuid.UUID) (*model.Recipe, error)
 	GetRecipeBySlug(ctx context.Context, slug string, displayName string) (*model.Recipe, error)
@@ -142,6 +152,10 @@ func (r recipeService) AddRecipeRevision(ctx context.Context, input model.AddRev
 		return nil, nil
 	}
 
+	if err := validateTitle(input.Revision.Title); err != nil {
+		return nil, err
+	}
+
 	tx, err := r.conn.Begin(ctx)
 	if err != nil {
 		return nil, err
@@ -248,6 +262,10 @@ func (r recipeService) CreateRecipe(ctx context.Context, input model.CreateRecip
 		return nil, nil
 	}
 
+	if err := validateTitle(input.Revision.Title); err != nil {
+		return nil, err
+	}
+
 	user, _ := r.authService.GetUserSessionFromCtx(ctx)
 
 	tx, err := r.conn.Begin(ctx)

db/migrations/20250812230844_add_title_length_constraint.down.sql

@@ -0,0 +1,2 @@
+ALTER TABLE recipe_revisions
+DROP CONSTRAINT title_length_check;

db/migrations/20250812230844_add_title_length_constraint.up.sql

@@ -0,0 +1,3 @@
+ALTER TABLE recipe_revisions
+ADD CONSTRAINT title_length_check
+CHECK (char_length(trim(title)) >= 3 AND char_length(trim(title)) <= 150);