fix: set foreign key constraints for every connection

Author: Bajahaw

backend/cmd/data/datasource.go

@@ -19,7 +19,10 @@ func InitDataSource(dataSourceName string) error {
 	if err = os.MkdirAll(dir, 0o755); err != nil {
 		return err
 	}
-	DB, err = sql.Open("sqlite", dataSourceName)
+	// Add _pragma=foreign_keys(1) to ensure foreign keys are enabled on every connection
+	// This is critical for modernc.org/sqlite with connection pooling
+	dsn := dataSourceName + "?_pragma=foreign_keys(1)"
+	DB, err = sql.Open("sqlite", dsn)
 	if err != nil {
 		return err
 	}
@@ -30,11 +33,17 @@ func InitDataSource(dataSourceName string) error {
 		return err
 	}
 
+	// Set connection pool settings first
+	DB.SetMaxOpenConns(10)
+	DB.SetMaxIdleConns(5)
+	DB.SetConnMaxLifetime(0)
+
+	// Enable foreign keys - CRITICAL: Must be executed to take effect
 	if _, err = DB.Exec(`PRAGMA foreign_keys = ON;`); err != nil {
 		return err
 	}
 
-	// standard optimizations
+	// Standard optimizations (these are persistent per database file)
 	if _, err = DB.Exec(`PRAGMA journal_mode = WAL;`); err != nil {
 		return err
 	}
@@ -43,10 +52,6 @@ func InitDataSource(dataSourceName string) error {
 		return err
 	}
 
-	DB.SetMaxOpenConns(10)
-	DB.SetMaxIdleConns(5)
-	DB.SetConnMaxLifetime(0)
-
 	schema := `
 	CREATE TABLE IF NOT EXISTS Users (
 		id INTEGER PRIMARY KEY AUTOINCREMENT,

backend/cmd/provider/repo.go

@@ -109,7 +109,7 @@ func (repo *Repo) saveModels(models []Model) error {
 		args = append(args, m.ID, m.ProviderID, m.Name, m.IsEnabled)
 	}
 
-	// on conflict, replace
+	// on conflict, only update the enabled status
 	sb.WriteString(" ON CONFLICT(id) DO UPDATE SET is_enabled=excluded.is_enabled")
 
 	_, err := repo.db.Exec(sb.String(), args...)