From a643bd43b818aa5ff545f3f71fda8863edb8830d Mon Sep 17 00:00:00 2001
From: John O'Keefe <jrokeefe@protonmail.com>
Date: Wed, 25 Mar 2026 18:03:02 -0400
Subject: [PATCH] refactor: clarify database parameter validation in search
 service

Add comment to document that filter parameters use pgtype.Text
with explicit Valid=true flag to ensure proper SQL parameter handling.
This clarifies the intent behind the parameter building logic.

Improves code documentation for future maintenance.
---
 internal/services/search.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/internal/services/search.go b/internal/services/search.go
index 32b6585..6ec4f8a 100644
--- a/internal/services/search.go
+++ b/internal/services/search.go
@@ -63,7 +63,7 @@ func (s *SearchService) SearchMediaItemsUnified(ctx context.Context, params Sear
 		searchPattern = "%" + searchQuery + "%"
 	}
 
-	// Build database parameters
+	// Build database parameters - only set valid true if filter
 	dbParams := database.SearchMediaItemsUnifiedParams{
 		UserID:         params.UserID,
 		AuthorFilter:   pgtype.Text{String: params.AuthorFilter, Valid: true},