Refactor widget and worker to use UserPreferencesRepository

Refactored DayAheadWidget and WidgetUpdateWorker to use UserPreferencesRepository instead of accessing SharedPreferences directly.
This improves architectural consistency and centralizes data access logic.

Changes:
- Updated UserPreferencesRepository to expose summary flow and hasLocation method.
- Updated DayAheadWidget to inject UserPreferencesRepository and collect summary flow.
- Updated WidgetUpdateWorker to inject UserPreferencesRepository and use it for location check, preference retrieval, and summary saving.

Author: google-labs-jules[bot]

app/src/main/java/com/example/theloop/DayAheadWidget.kt

@@ -4,6 +4,7 @@ import android.appwidget.AppWidgetManager
 import android.appwidget.AppWidgetProvider
 import android.content.Context
 import android.widget.RemoteViews
+import com.example.theloop.data.repository.UserPreferencesRepository
 import com.example.theloop.data.repository.WeatherRepository
 import com.example.theloop.models.WeatherResponse
 import com.example.theloop.utils.AppConstants
@@ -21,14 +22,16 @@ class DayAheadWidget : AppWidgetProvider() {
     @Inject
     lateinit var weatherRepository: WeatherRepository
 
+    @Inject
+    lateinit var userPreferencesRepository: UserPreferencesRepository
+
     override fun onUpdate(context: Context, appWidgetManager: AppWidgetManager, appWidgetIds: IntArray) {
         val pendingResult = goAsync()
 
         CoroutineScope(Dispatchers.IO).launch {
             try {
                 val weather = weatherRepository.weatherData.first()
-                val summary = context.getSharedPreferences(AppConstants.PREFS_NAME, Context.MODE_PRIVATE)
-                    .getString(AppConstants.KEY_SUMMARY_CACHE, context.getString(R.string.widget_default_summary))
+                val summary = userPreferencesRepository.summary.first()
 
                 for (appWidgetId in appWidgetIds) {
                     updateAppWidget(context, appWidgetManager, appWidgetId, weather, summary)

app/src/main/java/com/example/theloop/WidgetUpdateWorker.kt

@@ -6,6 +6,7 @@ import androidx.work.CoroutineWorker
 import androidx.work.WorkerParameters
 import com.example.theloop.data.repository.CalendarRepository
 import com.example.theloop.data.repository.NewsRepository
+import com.example.theloop.data.repository.UserPreferencesRepository
 import com.example.theloop.data.repository.WeatherRepository
 import com.example.theloop.utils.AppConstants
 import com.example.theloop.utils.SummaryUtils
@@ -24,23 +25,19 @@ class WidgetUpdateWorker @AssistedInject constructor(
     @Assisted workerParams: WorkerParameters,
     private val weatherRepo: WeatherRepository,
     private val newsRepo: NewsRepository,
-    private val calendarRepo: CalendarRepository
+    private val calendarRepo: CalendarRepository,
+    private val userPreferencesRepository: UserPreferencesRepository
 ) : CoroutineWorker(appContext, workerParams) {
 
     override suspend fun doWork(): Result {
-        val prefs = applicationContext.getSharedPreferences(AppConstants.PREFS_NAME, Context.MODE_PRIVATE)
-        val latStr = prefs.getString(AppConstants.KEY_LATITUDE, null)
-        val lonStr = prefs.getString(AppConstants.KEY_LONGITUDE, null)
-        val unit = prefs.getString(AppConstants.KEY_TEMP_UNIT, AppConstants.DEFAULT_TEMP_UNIT) ?: AppConstants.DEFAULT_TEMP_UNIT
-        val userName = prefs.getString(AppConstants.KEY_USER_NAME, "User") ?: "User"
-
-        if (latStr == null || lonStr == null) {
+        if (!userPreferencesRepository.hasLocation()) {
             return Result.success()
         }
 
         return try {
-            val lat = latStr.toDouble()
-            val lon = lonStr.toDouble()
+            val (lat, lon) = userPreferencesRepository.location.first()
+            val unit = userPreferencesRepository.tempUnit.first()
+            val userName = userPreferencesRepository.userName.first()
 
             // Update Data (saves to DB)
             val weatherSuccess = weatherRepo.refresh(lat, lon, unit)
@@ -66,7 +63,7 @@ class WidgetUpdateWorker @AssistedInject constructor(
             } else null
 
             if (summary != null) {
-                prefs.edit().putString(AppConstants.KEY_SUMMARY_CACHE, summary).apply()
+                userPreferencesRepository.saveSummary(summary)
             }
 
             // Trigger widget update

app/src/main/java/com/example/theloop/data/repository/UserPreferencesRepository.kt

@@ -2,6 +2,7 @@ package com.example.theloop.data.repository
 
 import android.content.Context
 import android.content.SharedPreferences
+import com.example.theloop.R
 import com.example.theloop.utils.AppConstants
 import dagger.hilt.android.qualifiers.ApplicationContext
 import kotlinx.coroutines.channels.awaitClose
@@ -36,6 +37,10 @@ class UserPreferencesRepository @Inject constructor(
         .map { prefs.getString(AppConstants.KEY_USER_NAME, "User") ?: "User" }
         .distinctUntilChanged()
 
+    val summary: Flow<String> = prefsChangeFlow
+        .map { prefs.getString(AppConstants.KEY_SUMMARY_CACHE, context.getString(R.string.widget_default_summary)) ?: context.getString(R.string.widget_default_summary) }
+        .distinctUntilChanged()
+
     val location: Flow<Pair<Double, Double>> = prefsChangeFlow
         .map {
             val latStr = prefs.getString(AppConstants.KEY_LATITUDE, AppConstants.DEFAULT_LATITUDE.toString())
@@ -57,6 +62,10 @@ class UserPreferencesRepository @Inject constructor(
         prefs.edit().putString(AppConstants.KEY_SUMMARY_CACHE, summary).apply()
     }
 
+    fun hasLocation(): Boolean {
+        return prefs.contains(AppConstants.KEY_LATITUDE) && prefs.contains(AppConstants.KEY_LONGITUDE)
+    }
+
     // Helper to get current location synchronously if needed (though Flow is better)
     fun getLocationSync(): Pair<Double, Double> {
         val latStr = prefs.getString(AppConstants.KEY_LATITUDE, AppConstants.DEFAULT_LATITUDE.toString())