Remove USER_PRESENT handler for consistent widget behavior

USER_PRESENT broadcast only works when the app process is running
(runtime registration required since Android 8.0), causing inconsistent
behavior depending on whether the app was killed by the OS.

Widget updates now rely solely on:
- updatePeriodMillis (30 min) for system-triggered updates
- WorkManager periodic task for background weather fetches
- CONNECTIVITY_CHANGE for faster response when network returns

This provides consistent behavior regardless of app state.

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

Author: timbortnik

android/app/src/main/kotlin/org/bortnik/meteogram/MeteogramApplication.kt

@@ -36,7 +36,7 @@ class MeteogramApplication : Application() {
     /**
      * Register ContentObserver for immediate Material You color change detection.
      * This fires instantly when colors change (while app process is alive).
-     * WorkManager and USER_PRESENT provide fallback when app is killed.
+     * WorkManager provides fallback when app is killed.
      */
     private fun registerThemeObserver() {
         if (Build.VERSION.SDK_INT < Build.VERSION_CODES.S) return
@@ -75,15 +75,15 @@ class MeteogramApplication : Application() {
 
         // LOCALE_CHANGED and TIMEZONE_CHANGED are handled by manifest-declared receiver
         // (required because app process is killed on locale change)
-        // USER_PRESENT and CONNECTIVITY_CHANGE require runtime registration
+        // CONNECTIVITY_CHANGE requires runtime registration
+        // Note: USER_PRESENT removed - it only works when app is running, causing inconsistent behavior
         val filter = IntentFilter().apply {
-            addAction(Intent.ACTION_USER_PRESENT)
             addAction("android.net.conn.CONNECTIVITY_CHANGE")
         }
 
-        // Use RECEIVER_EXPORTED to receive system broadcasts (USER_PRESENT, etc.)
+        // Use RECEIVER_EXPORTED to receive system broadcasts
         registerReceiver(widgetEventReceiver, filter, RECEIVER_EXPORTED)
         receiverRegistered = true
-        Log.d(TAG, "Widget event receiver registered (runtime: USER_PRESENT, CONNECTIVITY_CHANGE)")
+        Log.d(TAG, "Widget event receiver registered (runtime: CONNECTIVITY_CHANGE)")
     }
 }

android/app/src/main/kotlin/org/bortnik/meteogram/WidgetEventReceiver.kt

@@ -5,21 +5,21 @@ import android.content.Context
 import android.content.Intent
 import android.net.ConnectivityManager
 import android.net.NetworkCapabilities
-import android.os.Build
 import android.util.Log
 
 /**
  * BroadcastReceiver for system events that trigger widget updates.
  *
  * Registration:
- * - USER_PRESENT, CONNECTIVITY_CHANGE: Runtime (in MeteogramApplication)
+ * - CONNECTIVITY_CHANGE: Runtime (in MeteogramApplication)
  * - LOCALE_CHANGED, TIMEZONE_CHANGED: Manifest (app killed on change)
  *
+ * Note: USER_PRESENT was removed because it only works when app process is running,
+ * causing inconsistent behavior. Widget updates rely on updatePeriodMillis (30 min)
+ * and WorkManager for consistent behavior regardless of app state.
+ *
  * Note: CONNECTIVITY_CHANGE is deprecated since Android 7.0 (API 24).
  * On newer devices, connectivity changes may be delayed or batched.
- * For more reliable network monitoring, consider using:
- * - ConnectivityManager.registerNetworkCallback() for API 21+
- * - WorkManager with NetworkType.CONNECTED constraint
  * Current implementation uses WorkManager periodic task as primary mechanism,
  * with CONNECTIVITY_CHANGE as supplementary for faster response on older devices.
  */
@@ -32,15 +32,6 @@ class WidgetEventReceiver : BroadcastReceiver() {
         Log.d(TAG, "Received broadcast: ${intent.action}")
 
         when (intent.action) {
-            Intent.ACTION_USER_PRESENT -> {
-                // Screen unlocked - re-render only if needed:
-                // 1. Crossed 30-min boundary (now indicator moved)
-                // 2. Weather data updated while locked (background fetch)
-                Log.d(TAG, "User present - checking if re-render needed")
-                updateMaterialYouColors(context)
-                WidgetUtils.rerenderAllWidgetsIfNeeded(context)
-                fetchWeatherIfStale(context)
-            }
             Intent.ACTION_LOCALE_CHANGED -> {
                 // Locale changed - re-render with new units/format
                 Log.d(TAG, "Locale changed - triggering re-render")
@@ -61,16 +52,6 @@ class WidgetEventReceiver : BroadcastReceiver() {
         }
     }
 
-    private fun updateMaterialYouColors(context: Context) {
-        // Update Material You colors if changed (Android 12+)
-        // Does NOT trigger re-render - caller handles that
-        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
-            if (MaterialYouColorExtractor.updateColorsIfChanged(context)) {
-                Log.d(TAG, "Material You colors updated")
-            }
-        }
-    }
-
     private fun fetchWeatherIfStale(context: Context) {
         WidgetUtils.fetchWeatherIfStale(context)
     }