fix(greader): stop sync from undoing mark all as read

markAsRead pushed only locally-known IDs via edit-tag, so unread items only on the server stayed unread and got flipped back on the next sync. Use the bulk mark-all-as-read endpoint instead.

Author: amerkay

app/src/main/java/me/ash/reader/domain/service/GoogleReaderRssService.kt

@@ -773,6 +773,26 @@ constructor(
     ) {
         val accountId = accountService.getCurrentAccountId()
         val googleReaderAPI = getGoogleReaderAPI()
+
+        // The server can have unread items we don't have locally yet. Marking by ID
+        // misses those, and the next sync flips them back to unread. Mark the whole
+        // stream on the server instead using markAllAsRead.
+        if (!isUnread && articleId == null) {
+            val streamTag =
+                when {
+                    groupId != null ->
+                        GoogleReaderAPI.Stream.Category(groupId.dollarLast()).tag
+                    feedId != null -> GoogleReaderAPI.Stream.Feed(feedId.dollarLast()).tag
+                    else -> GoogleReaderAPI.Stream.AllItems.tag
+                }
+            super.markAsRead(groupId, feedId, articleId, before, isUnread)
+            googleReaderAPI.markAllAsRead(
+                streamId = streamTag,
+                sinceTimestamp = before?.time?.times(1000L),
+            )
+            return
+        }
+
         val markList: List<String> =
             when {
                 groupId != null -> {