fix(api): change Note GORM hook from BeforeSave to BeforeCreate

The Note model's BeforeSave hook validated name/content as non-empty,
but the Update path uses map-based GORM Updates() on an empty model
struct. GORM runs BeforeSave on that empty struct, not the update map,
causing false "name: cannot be empty" errors on every note update.

Moves required-field validation into the existing BeforeCreate hook in
models.go. Update-time validation is already handled by the API layer.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: ericfitz

.version

@@ -1,6 +1,6 @@
 {
   "major": 1,
   "minor": 2,
-  "patch": 0,
+  "patch": 1,
   "prerelease": ""
 }

api/models/hooks.go

@@ -92,17 +92,10 @@ func (d *Document) BeforeSave(tx *gorm.DB) error {
 }
 
 // --- Note Hooks ---
-
-// BeforeSave validates Note before create or update
-func (n *Note) BeforeSave(tx *gorm.DB) error {
-	if err := validation.ValidateNonEmpty("name", n.Name); err != nil {
-		return err
-	}
-	if err := validation.ValidateNonEmpty("content", string(n.Content)); err != nil {
-		return err
-	}
-	return nil
-}
+// Note: Required field validation (name, content) is in models.go BeforeCreate,
+// not here, because the Update path uses map-based GORM Updates() on an empty
+// model struct. A BeforeSave hook would validate the empty struct's zero-value
+// fields, causing false "cannot be empty" errors.
 
 // --- Repository Hooks ---
 

api/models/models.go

@@ -7,6 +7,7 @@ import (
 	"strings"
 	"time"
 
+	"github.com/ericfitz/tmi/api/validation"
 	"github.com/google/uuid"
 	"gorm.io/gorm"
 )
@@ -381,11 +382,21 @@ func (Note) TableName() string {
 	return tableName("notes")
 }
 
-// BeforeCreate generates a UUID if not set
+// BeforeCreate generates a UUID if not set and validates required fields.
+// Note: Required field validation is intentionally in BeforeCreate (not BeforeSave)
+// because the Update path uses map-based GORM Updates() on an empty model struct.
+// BeforeSave would validate the empty struct's zero-value fields, causing false
+// "cannot be empty" errors. Update-time validation is handled by the API layer.
 func (n *Note) BeforeCreate(tx *gorm.DB) error {
 	if n.ID == "" {
 		n.ID = uuid.New().String()
 	}
+	if err := validation.ValidateNonEmpty("name", n.Name); err != nil {
+		return err
+	}
+	if err := validation.ValidateNonEmpty("content", string(n.Content)); err != nil {
+		return err
+	}
 	return nil
 }
 

api/version.go

@@ -46,7 +46,7 @@ var (
 	// Minor version number
 	VersionMinor = "2"
 	// Patch version number
-	VersionPatch = "0"
+	VersionPatch = "1"
 	// VersionPreRelease is the pre-release label (e.g., "rc.0", "beta.1"), empty for stable releases
 	VersionPreRelease = ""
 	// GitCommit is the git commit hash from build