From ca21e22147f071b091bf5736882f3a455dd2ff58 Mon Sep 17 00:00:00 2001
From: M57 <41645758+Hy4ri@users.noreply.github.com>
Date: Sat, 20 Jun 2026 14:07:46 +0300
Subject: [PATCH] ci(#71): change FGS type from dataSync to remoteMessaging for
 indefinite WebSocket listener (#162)

Co-authored-by: M57 <hy4ri@users.noreply.github.com>
---
 app/src/main/AndroidManifest.xml                       |  4 ++--
 .../notification/ChatNotificationService.kt            | 10 ++++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index c8b60117..9b4e1785 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -5,7 +5,7 @@
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
     <uses-permission android:name="android.permission.POST_NOTIFICATIONS" />
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
-    <uses-permission android:name="android.permission.FOREGROUND_SERVICE_DATA_SYNC" />
+    <uses-permission android:name="android.permission.FOREGROUND_SERVICE_REMOTE_MESSAGING" />
 
     <application
         android:allowBackup="true"
@@ -27,7 +27,7 @@
 
         <service
             android:name=".notification.ChatNotificationService"
-            android:foregroundServiceType="dataSync"
+            android:foregroundServiceType="remoteMessaging"
             android:exported="false" />
 
         <receiver
diff --git a/app/src/main/java/com/m57/hermescontrol/notification/ChatNotificationService.kt b/app/src/main/java/com/m57/hermescontrol/notification/ChatNotificationService.kt
index 8669cfa3..c4a0d8f8 100644
--- a/app/src/main/java/com/m57/hermescontrol/notification/ChatNotificationService.kt
+++ b/app/src/main/java/com/m57/hermescontrol/notification/ChatNotificationService.kt
@@ -29,6 +29,16 @@ import kotlinx.coroutines.launch
  * is backgrounded and posts a notification when a new assistant reply
  * completes (MessageComplete event) while the app is not in the foreground.
  *
+ * FGS type: `remoteMessaging` — indefinite listener that maintains a
+ * long-lived connection to a remote server for receiving messages. This is
+ * the correct type per Android 14+ policy (dataSync has a time budget and
+ * is intended for finite sync operations).
+ *
+ * Channel importance: IMPORTANCE_MIN — the ongoing notification is a
+ * persistent indicator, not an alert. PRIORITY_MIN matches.
+ * The separate `hermes_chat` channel uses IMPORTANCE_HIGH for actual
+ * message notifications, which is correct.
+ *
  * Lifecycle:
  * - Started by [NotificationHelper.start] when the user sends a message and
  *   the app is about to go to the background (called from ChatScreen's