feat(preferences): add scope columns migration and constants (#1361)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: whoAbhishekSah

core/preference/preference.go

@@ -44,6 +44,11 @@ const (
 	// user default traits
 	UserFirstName  = "first_name"
 	UserNewsletter = "newsletter"
+
+	// Zero values for global/unscoped preferences
+	// Used instead of NULL for PostgreSQL 14 compatibility
+	ScopeTypeGlobal = "app/platform"
+	ScopeIDGlobal   = "00000000-0000-0000-0000-000000000000"
 )
 
 type Trait struct {
@@ -84,6 +89,8 @@ type Preference struct {
 	Value        string    `json:"value"`
 	ResourceID   string    `json:"resource_id"`
 	ResourceType string    `json:"resource_type"`
+	ScopeType    string    `json:"scope_type"`
+	ScopeID      string    `json:"scope_id"`
 	CreatedAt    time.Time `json:"created_at"`
 	UpdatedAt    time.Time `json:"updated_at"`
 }

internal/store/postgres/migrations/20260129125246_add_preference_scope_columns.down.sql

@@ -0,0 +1,8 @@
+-- Revert scope columns addition
+ALTER TABLE preferences DROP CONSTRAINT IF EXISTS uq_preferences_resource_scope;
+
+ALTER TABLE preferences ADD CONSTRAINT resource_type_name_unique
+    UNIQUE (resource_type, resource_id, name);
+
+ALTER TABLE preferences DROP COLUMN IF EXISTS scope_id;
+ALTER TABLE preferences DROP COLUMN IF EXISTS scope_type;

internal/store/postgres/migrations/20260129125246_add_preference_scope_columns.up.sql

@@ -0,0 +1,14 @@
+-- Add scope columns for org-scoped user preferences
+-- scope_type references namespaces table (like resource_type)
+-- scope_id is text (like resource_id) to support different ID formats
+-- Using zero values instead of NULL for global preferences:
+-- - scope_type: 'app/platform' (global/unscoped)
+-- - scope_id: '00000000-0000-0000-0000-000000000000' (zero UUID)
+ALTER TABLE preferences ADD COLUMN scope_type text NOT NULL DEFAULT 'app/platform' REFERENCES namespaces (name) ON DELETE CASCADE;
+ALTER TABLE preferences ADD COLUMN scope_id text NOT NULL DEFAULT '00000000-0000-0000-0000-000000000000';
+
+-- Update unique constraint to include scope columns
+ALTER TABLE preferences DROP CONSTRAINT resource_type_name_unique;
+
+ALTER TABLE preferences ADD CONSTRAINT uq_preferences_resource_scope
+    UNIQUE (resource_type, resource_id, scope_type, scope_id, name);

internal/store/postgres/preference.go

@@ -13,12 +13,14 @@ type Preference struct {
 	Value        string    `db:"value"`
 	ResourceType string    `db:"resource_type"`
 	ResourceID   string    `db:"resource_id"`
+	ScopeType    string    `db:"scope_type"`
+	ScopeID      string    `db:"scope_id"`
 	CreatedAt    time.Time `db:"created_at"`
 	UpdatedAt    time.Time `db:"updated_at"`
 }
 
 func (from Preference) transformToPreference() preference.Preference {
-	return preference.Preference{
+	pref := preference.Preference{
 		ID:           from.ID.String(),
 		Name:         from.Name,
 		Value:        from.Value,
@@ -27,4 +29,12 @@ func (from Preference) transformToPreference() preference.Preference {
 		CreatedAt:    from.CreatedAt,
 		UpdatedAt:    from.UpdatedAt,
 	}
+	// Convert zero values back to empty strings for API layer
+	if from.ScopeType != preference.ScopeTypeGlobal {
+		pref.ScopeType = from.ScopeType
+	}
+	if from.ScopeID != preference.ScopeIDGlobal {
+		pref.ScopeID = from.ScopeID
+	}
+	return pref
 }

internal/store/postgres/preference_repository.go

@@ -29,15 +29,26 @@ func (s *PreferenceRepository) Set(ctx context.Context, pref preference.Preferen
 	if pref.ID == "" {
 		pref.ID = uuid.New().String()
 	}
+	// Use zero values for scope if not provided (global/unscoped preference)
+	scopeType := pref.ScopeType
+	if scopeType == "" {
+		scopeType = preference.ScopeTypeGlobal
+	}
+	scopeID := pref.ScopeID
+	if scopeID == "" {
+		scopeID = preference.ScopeIDGlobal
+	}
 	query, params, err := dialect.Insert(TABLE_PREFERENCES).Rows(
 		goqu.Record{
 			"id":            pref.ID,
 			"name":          pref.Name,
 			"value":         pref.Value,
 			"resource_type": pref.ResourceType,
 			"resource_id":   pref.ResourceID,
+			"scope_type":    scopeType,
+			"scope_id":      scopeID,
 			"updated_at":    goqu.L("NOW()"),
-		}).OnConflict(goqu.DoUpdate("resource_type, resource_id, name",
+		}).OnConflict(goqu.DoUpdate("resource_type, resource_id, scope_type, scope_id, name",
 		goqu.Record{
 			"value":      pref.Value,
 			"updated_at": time.Now().UTC(),