@bayonai/offline 0.5.0 → 0.5.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.5.1 - 2026-09-12
+
+- Prevented timed-out conditional IndexedDB replacements and deletions from
+  committing after the operation has already reported failure.
+- Preserved strict chronological replay when an older record has an active
+  claim by leaving it and all later records for a future ordered run.
+
 ## 0.5.0 - 2026-09-12
 
 - Added `createBrowserOfflineFallbackRecovery`, a framework-free controller

package/README.md

@@ -202,6 +202,11 @@ Interrupted sync claims expire after one minute so recovery can resume. Records
 created before snapshot identities were introduced remain readable; they receive
 a stable compatibility identity and are upgraded when next updated.
 
+Conditional IndexedDB mutations are cancelled when their storage deadline is
+exceeded, so an operation cannot report failure and then commit later. Ordered
+replay also stops at an active older claim; it never delivers a later record
+whose remote effect could subsequently be overwritten by that predecessor.
+
 Host apps that change their pending-write payload shape can normalize current
 records while keeping domain logic outside the package:
 

package/dist/idbDriver.js

@@ -1,4 +1,4 @@
-import { wrap } from "idb";
+import { wrap, } from "idb";
 export function createIdbKeyValueDriver({ databaseName, databaseVersion = 1, indexedDB = globalThis.indexedDB, operationTimeoutMs, storeName, }) {
     if (!indexedDB) {
         return null;
@@ -44,25 +44,18 @@ export function createIdbKeyValueDriver({ databaseName, databaseVersion = 1, ind
             });
         },
         async deleteIf(key, predicate) {
-            return runBrowserStorageOperation({
-                fallback: false,
+            return runConditionalStorageOperation({
+                databasePromise,
                 operationName: "conditionally delete key",
-                operation: async () => {
-                    const database = await databasePromise;
-                    // The ownership check and deletion must share a transaction; splitting
-                    // them can delete a newer offline snapshot written by another trigger.
-                    const transaction = database.transaction(storeName, "readwrite");
-                    const objectStore = transaction.objectStore(storeName);
+                storeName,
+                timeoutMs: operationTimeoutMs,
+                apply: async (objectStore) => {
                     const current = (await objectStore.get(key)) ?? null;
-                    if (!predicate(current)) {
-                        await transaction.done;
+                    if (!predicate(current))
                         return false;
-                    }
                     await objectStore.delete(key);
-                    await transaction.done;
                     return true;
                 },
-                timeoutMs: operationTimeoutMs,
             });
         },
         async get(key) {
@@ -123,23 +116,18 @@ export function createIdbKeyValueDriver({ databaseName, databaseVersion = 1, ind
             });
         },
         async setIf(key, value, predicate) {
-            return runBrowserStorageOperation({
-                fallback: false,
+            return runConditionalStorageOperation({
+                databasePromise,
                 operationName: "conditionally replace key",
-                operation: async () => {
-                    const database = await databasePromise;
-                    const transaction = database.transaction(storeName, "readwrite");
-                    const objectStore = transaction.objectStore(storeName);
+                storeName,
+                timeoutMs: operationTimeoutMs,
+                apply: async (objectStore) => {
                     const current = (await objectStore.get(key)) ?? null;
-                    if (!predicate(current)) {
-                        await transaction.done;
+                    if (!predicate(current))
                         return false;
-                    }
                     await objectStore.put(value, key);
-                    await transaction.done;
                     return true;
                 },
-                timeoutMs: operationTimeoutMs,
             });
         },
     };
@@ -158,7 +146,7 @@ async function openKeyValueDatabase({ databaseName, databaseVersion, indexedDB,
     }
     return database;
 }
-async function runBrowserStorageOperation({ fallback, operation, operationName, timeoutMs, }) {
+async function runBrowserStorageOperation({ cancelOnTimeout, fallback, operation, operationName, timeoutMs, }) {
     let timeoutId = null;
     try {
         return await Promise.race([
@@ -168,7 +156,9 @@ async function runBrowserStorageOperation({ fallback, operation, operationName,
                     console.error(`IndexedDB timed out while trying to ${operationName}.`, {
                         timeoutMs,
                     });
-                    resolve(fallback);
+                    if (!cancelOnTimeout || cancelOnTimeout()) {
+                        resolve(fallback);
+                    }
                 }, timeoutMs);
             }),
         ]);
@@ -183,3 +173,39 @@ async function runBrowserStorageOperation({ fallback, operation, operationName,
         }
     }
 }
+async function runConditionalStorageOperation({ apply, databasePromise, operationName, storeName, timeoutMs, }) {
+    let transaction;
+    let cancelled = false;
+    return runBrowserStorageOperation({
+        fallback: false,
+        operationName,
+        timeoutMs,
+        cancelOnTimeout() {
+            cancelled = true;
+            if (!transaction)
+                return true;
+            try {
+                transaction.abort();
+                return true;
+            }
+            catch (error) {
+                // Product decision: when commit has already started, report its actual
+                // result instead of returning a false failure that can later mutate data.
+                console.warn("[Bayon offline] Conditional transaction already settling at timeout.", { error, operationName });
+                return false;
+            }
+        },
+        operation: async () => {
+            const database = await databasePromise;
+            if (cancelled)
+                return false;
+            transaction = database.transaction(storeName, "readwrite");
+            // Aborting can reject both a request and transaction.done. Observe the
+            // latter immediately so cancellation never creates an unhandled rejection.
+            void transaction.done.catch(() => undefined);
+            const result = await apply(transaction.objectStore(storeName));
+            await transaction.done;
+            return result;
+        },
+    });
+}

package/dist/pendingWrites.js

@@ -277,7 +277,10 @@ async function performPendingWriteSync({ select, stopOnFailure, store, sync, })
         });
         summary.attempted += 1;
         summary[result.status] += 1;
-        if (stopOnFailure && result.status === "failed") {
+        // Product decision: a superseded result can represent an older live claim.
+        // Preserve strict ordering by leaving every later record for a future run.
+        if (stopOnFailure &&
+            (result.status === "failed" || result.status === "superseded")) {
             break;
         }
     }

package/dist-cjs/idbDriver.js

@@ -47,25 +47,18 @@ function createIdbKeyValueDriver({ databaseName, databaseVersion = 1, indexedDB
             });
         },
         async deleteIf(key, predicate) {
-            return runBrowserStorageOperation({
-                fallback: false,
+            return runConditionalStorageOperation({
+                databasePromise,
                 operationName: "conditionally delete key",
-                operation: async () => {
-                    const database = await databasePromise;
-                    // The ownership check and deletion must share a transaction; splitting
-                    // them can delete a newer offline snapshot written by another trigger.
-                    const transaction = database.transaction(storeName, "readwrite");
-                    const objectStore = transaction.objectStore(storeName);
+                storeName,
+                timeoutMs: operationTimeoutMs,
+                apply: async (objectStore) => {
                     const current = (await objectStore.get(key)) ?? null;
-                    if (!predicate(current)) {
-                        await transaction.done;
+                    if (!predicate(current))
                         return false;
-                    }
                     await objectStore.delete(key);
-                    await transaction.done;
                     return true;
                 },
-                timeoutMs: operationTimeoutMs,
             });
         },
         async get(key) {
@@ -126,23 +119,18 @@ function createIdbKeyValueDriver({ databaseName, databaseVersion = 1, indexedDB
             });
         },
         async setIf(key, value, predicate) {
-            return runBrowserStorageOperation({
-                fallback: false,
+            return runConditionalStorageOperation({
+                databasePromise,
                 operationName: "conditionally replace key",
-                operation: async () => {
-                    const database = await databasePromise;
-                    const transaction = database.transaction(storeName, "readwrite");
-                    const objectStore = transaction.objectStore(storeName);
+                storeName,
+                timeoutMs: operationTimeoutMs,
+                apply: async (objectStore) => {
                     const current = (await objectStore.get(key)) ?? null;
-                    if (!predicate(current)) {
-                        await transaction.done;
+                    if (!predicate(current))
                         return false;
-                    }
                     await objectStore.put(value, key);
-                    await transaction.done;
                     return true;
                 },
-                timeoutMs: operationTimeoutMs,
             });
         },
     };
@@ -161,7 +149,7 @@ async function openKeyValueDatabase({ databaseName, databaseVersion, indexedDB,
     }
     return database;
 }
-async function runBrowserStorageOperation({ fallback, operation, operationName, timeoutMs, }) {
+async function runBrowserStorageOperation({ cancelOnTimeout, fallback, operation, operationName, timeoutMs, }) {
     let timeoutId = null;
     try {
         return await Promise.race([
@@ -171,7 +159,9 @@ async function runBrowserStorageOperation({ fallback, operation, operationName,
                     console.error(`IndexedDB timed out while trying to ${operationName}.`, {
                         timeoutMs,
                     });
-                    resolve(fallback);
+                    if (!cancelOnTimeout || cancelOnTimeout()) {
+                        resolve(fallback);
+                    }
                 }, timeoutMs);
             }),
         ]);
@@ -186,3 +176,39 @@ async function runBrowserStorageOperation({ fallback, operation, operationName,
         }
     }
 }
+async function runConditionalStorageOperation({ apply, databasePromise, operationName, storeName, timeoutMs, }) {
+    let transaction;
+    let cancelled = false;
+    return runBrowserStorageOperation({
+        fallback: false,
+        operationName,
+        timeoutMs,
+        cancelOnTimeout() {
+            cancelled = true;
+            if (!transaction)
+                return true;
+            try {
+                transaction.abort();
+                return true;
+            }
+            catch (error) {
+                // Product decision: when commit has already started, report its actual
+                // result instead of returning a false failure that can later mutate data.
+                console.warn("[Bayon offline] Conditional transaction already settling at timeout.", { error, operationName });
+                return false;
+            }
+        },
+        operation: async () => {
+            const database = await databasePromise;
+            if (cancelled)
+                return false;
+            transaction = database.transaction(storeName, "readwrite");
+            // Aborting can reject both a request and transaction.done. Observe the
+            // latter immediately so cancellation never creates an unhandled rejection.
+            void transaction.done.catch(() => undefined);
+            const result = await apply(transaction.objectStore(storeName));
+            await transaction.done;
+            return result;
+        },
+    });
+}

package/dist-cjs/pendingWrites.js

@@ -291,7 +291,10 @@ async function performPendingWriteSync({ select, stopOnFailure, store, sync, })
         });
         summary.attempted += 1;
         summary[result.status] += 1;
-        if (stopOnFailure && result.status === "failed") {
+        // Product decision: a superseded result can represent an older live claim.
+        // Preserve strict ordering by leaving every later record for a future run.
+        if (stopOnFailure &&
+            (result.status === "failed" || result.status === "superseded")) {
             break;
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bayonai/offline",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Reusable offline read-cache, pending-write, and silent recovery primitives.",
   "license": "MIT",
   "author": "Bayon AI",
@@ -55,6 +55,7 @@
     "idb": "^8.0.3"
   },
   "devDependencies": {
+    "fake-indexeddb": "^6.2.5",
     "typescript": "^5",
     "vitest": "^4.1.6"
   }