cry-synced-db-client 0.1.147 → 0.1.148

package/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 ## Unreleased
 
+### `uploadDirtyItems` follow-up pass — drain in-sync writes immediately
+
+Writes that land **during** a sync iteration had their
+`scheduleRestUpload()` guarded out by `isSyncing()` (silent drop, no
+re-schedule), so they sat in `_dirty_changes` until the next 60s
+auto-sync tick. Particularly visible for high-frequency writeOnly
+collections (e.g. `prehodi`, written on every route change) on tablets
+with intensive navigation: dirty items piled up until the next tick.
+
+`SyncEngine.sync()` now performs a single follow-up pass right after the
+primary `uploadDirtyItems()` call:
+
+1. `flushAllPendingChanges()` — forces any in-flight 500ms Dexie
+   debounces to land in `_dirty_changes` before the second snapshot.
+2. `uploadDirtyItems(calledFrom + ":followUp")` — drains entries that
+   accumulated during the first pass's server roundtrip.
+
+Single pass (not a loop) — bounded work; later writes after sync
+completes will trigger their own `scheduleRestUpload()` once the
+`isSyncing` flag clears. Sequential `await` ordering means no concurrent
+server roundtrips and no new race conditions vs. the existing
+snapshot-then-clear pattern in `_dirty_changes`. Stats from the
+follow-up pass are merged into `uploadStats` so `onSyncEnd` /
+`collectionStats.sentCount` reflect both passes.
+
+Errors in the follow-up are caught by the same outer `try/catch` as the
+first pass — a follow-up failure does not roll back the first pass's
+already-cleared dirty entries; affected items are caught at the next
+sync tick (same retry semantics as before).
+
 ### Auto-eviction co-located with sync — one round-trip total
 
 When `evictStaleRecordsEveryHrs > 0` and the interval has elapsed, the

package/dist/index.js

@@ -2553,6 +2553,17 @@ var _SyncEngine = class _SyncEngine {
       let uploadStats = { sentCount: 0 };
       try {
         uploadStats = await this.uploadDirtyItems(calledFrom);
+        await this.deps.flushAllPendingChanges();
+        const followUp = await this.uploadDirtyItems(`${calledFrom != null ? calledFrom : "sync"}:followUp`);
+        if (followUp.sentCount > 0) {
+          uploadStats.sentCount += followUp.sentCount;
+          if (followUp.collectionSentCounts) {
+            uploadStats.collectionSentCounts = uploadStats.collectionSentCounts || {};
+            for (const [c, n] of Object.entries(followUp.collectionSentCounts)) {
+              uploadStats.collectionSentCounts[c] = (uploadStats.collectionSentCounts[c] || 0) + n;
+            }
+          }
+        }
       } catch (err) {
         console.error(
           "uploadDirtyItems failed (download succeeded, staying online):",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cry-synced-db-client",
-  "version": "0.1.147",
+  "version": "0.1.148",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",