dexie-cloud-addon 4.0.4 → 4.0.6

package/dist/umd/dexie-cloud-addon.js

@@ -8,7 +8,7 @@
  *
  * ==========================================================================
  *
- * Version 4.0.4, Mon Apr 29 2024
+ * Version 4.0.5, Sat May 25 2024
  *
  * https://dexie.org
  *
@@ -4212,9 +4212,7 @@
                         // If the sync worker or service worker is syncing, wait 'til thei're done.
                         // It's no need to have two channels at the same time - even though it wouldnt
                         // be a problem - this is an optimization.
-                        yield db.cloud.syncState
-                            .pipe(filter(({ phase }) => phase === 'in-sync' || phase === 'error'), take(1))
-                            .toPromise();
+                        yield rxjs.firstValueFrom(db.cloud.syncState.pipe(filter(({ phase }) => phase === 'in-sync' || phase === 'error')));
                         console.debug('processing msg', msg);
                         const persistedSyncState = db.cloud.persistedSyncState.value;
                         //syncState.
@@ -4713,6 +4711,10 @@
                                 }
                                 const trans = req.trans;
                                 if ((_b = (_a = db.cloud.schema) === null || _a === void 0 ? void 0 : _a[tableName]) === null || _b === void 0 ? void 0 : _b.markedForSync) {
+                                    if (trans.mode === 'versionchange') {
+                                        // Don't mutate tables marked for sync in versionchange transactions.
+                                        return Promise.reject(new Dexie.UpgradeError(`Dexie Cloud Addon: Cannot upgrade or populate synced table "${tableName}". See https://dexie.org/cloud/docs/best-practices`));
+                                    }
                                     if (req.type === 'add' || req.type === 'put') {
                                         if (tableName === 'members') {
                                             for (const member of req.values) {
@@ -5417,7 +5419,7 @@
             yield sleep(3000);
             // Wait til user does something (move mouse, tap, scroll, click etc)
             console.debug('waiting for someone to do something');
-            yield userDoesSomething.pipe(take(1)).toPromise();
+            yield rxjs.firstValueFrom(userDoesSomething);
             console.debug('someone did something!');
         });
     }
@@ -6315,7 +6317,7 @@
         const syncComplete = new rxjs.Subject();
         dexie.cloud = {
             // @ts-ignore
-            version: "4.0.4",
+            version: "4.0.5",
             options: Object.assign({}, DEFAULT_OPTIONS),
             schema: null,
             get currentUserId() {
@@ -6373,10 +6375,8 @@
                         const syncState = db.cloud.persistedSyncState.value;
                         triggerSync(db, purpose);
                         if (wait) {
-                            const newSyncState = yield db.cloud.persistedSyncState
-                                .pipe(filter((newSyncState) => (newSyncState === null || newSyncState === void 0 ? void 0 : newSyncState.timestamp) != null &&
-                                (!syncState || newSyncState.timestamp > syncState.timestamp)), take(1))
-                                .toPromise();
+                            const newSyncState = yield rxjs.firstValueFrom(db.cloud.persistedSyncState.pipe(filter((newSyncState) => (newSyncState === null || newSyncState === void 0 ? void 0 : newSyncState.timestamp) != null &&
+                                (!syncState || newSyncState.timestamp > syncState.timestamp))));
                             if (newSyncState === null || newSyncState === void 0 ? void 0 : newSyncState.error) {
                                 throw new Error(`Sync error: ` + newSyncState.error);
                             }
@@ -6387,16 +6387,14 @@
                         triggerSync(db, purpose);
                         if (wait) {
                             console.debug('db.cloud.login() is waiting for sync completion...');
-                            yield rxjs.from(Dexie.liveQuery(() => __awaiter(this, void 0, void 0, function* () {
+                            yield rxjs.firstValueFrom(rxjs.from(Dexie.liveQuery(() => __awaiter(this, void 0, void 0, function* () {
                                 const syncNeeded = yield isSyncNeeded(db);
                                 const newSyncState = yield db.getPersistedSyncState();
                                 if ((newSyncState === null || newSyncState === void 0 ? void 0 : newSyncState.timestamp) !== (syncState === null || syncState === void 0 ? void 0 : syncState.timestamp) &&
                                     (newSyncState === null || newSyncState === void 0 ? void 0 : newSyncState.error))
                                     throw new Error(`Sync error: ` + newSyncState.error);
                                 return syncNeeded;
-                            })))
-                                .pipe(filter((isNeeded) => !isNeeded), take(1))
-                                .toPromise();
+                            }))).pipe(filter((isNeeded) => !isNeeded)));
                             console.debug('Done waiting for sync completion because we have nothing to push anymore');
                         }
                     }
@@ -6445,7 +6443,7 @@
                 const swRegistrations = 'serviceWorker' in navigator
                     ? yield navigator.serviceWorker.getRegistrations()
                     : [];
-                const initiallySynced = yield db.transaction('rw', db.$syncState, () => __awaiter(this, void 0, void 0, function* () {
+                const [initiallySynced, lastSyncedRealms] = yield db.transaction('rw', db.$syncState, () => __awaiter(this, void 0, void 0, function* () {
                     var _h, _j;
                     const { options, schema } = db.cloud;
                     const [persistedOptions, persistedSchema, persistedSyncState] = yield Promise.all([
@@ -6518,7 +6516,7 @@
                         // Let's assign all props as the newPersistedSchems should be what we should be working with.
                         Object.assign(schema, newPersistedSchema);
                     }
-                    return persistedSyncState === null || persistedSyncState === void 0 ? void 0 : persistedSyncState.initiallySynced;
+                    return [persistedSyncState === null || persistedSyncState === void 0 ? void 0 : persistedSyncState.initiallySynced, persistedSyncState === null || persistedSyncState === void 0 ? void 0 : persistedSyncState.realms];
                 }));
                 if (initiallySynced) {
                     db.setInitiallySynced(true);
@@ -6538,19 +6536,27 @@
                     // with things from the database and not just the default values.
                     // This is so that when db.open() completes, user should be safe
                     // to subscribe to these observables and get actual data.
-                    yield rxjs.combineLatest([
+                    yield rxjs.firstValueFrom(rxjs.combineLatest([
                         currentUserEmitter.pipe(skip(1), take(1)),
                         db.cloud.persistedSyncState.pipe(skip(1), take(1)),
-                    ]).toPromise();
+                    ]));
                 }
                 // HERE: If requireAuth, do athentication now.
                 let changedUser = false;
+                const user = yield db.getCurrentUser();
                 if ((_c = db.cloud.options) === null || _c === void 0 ? void 0 : _c.requireAuth) {
-                    const user = yield db.getCurrentUser();
                     if (!user.isLoggedIn) {
                         changedUser = yield login(db);
                     }
                 }
+                if (user.isLoggedIn && (!lastSyncedRealms || !lastSyncedRealms.includes(user.userId))) {
+                    // User has been logged in but this is not reflected in the sync state.
+                    // This can happen if page is reloaded after login but before the sync call following
+                    // the login was complete.
+                    // The user is to be viewed as changed becuase current syncState does not reflect the presence
+                    // of the logged-in user.
+                    changedUser = true; // Set changedUser to true to trigger a pull-sync later down.
+                }
                 if (localSyncWorker)
                     localSyncWorker.stop();
                 localSyncWorker = null;
@@ -6595,7 +6601,7 @@
         }
     }
     // @ts-ignore
-    dexieCloud.version = "4.0.4";
+    dexieCloud.version = "4.0.5";
     Dexie.Cloud = dexieCloud;
 
     exports.default = dexieCloud;