@tanstack/db 0.0.33 → 0.1.1

package/dist/esm/collection.d.ts

@@ -289,6 +289,11 @@ export declare class CollectionImpl<T extends object = Record<string, unknown>,
      * This method processes operations from pending transactions and applies them to the synced data
      */
     commitPendingTransactions: () => void;
+    /**
+     * Schedule cleanup of a transaction when it completes
+     * @private
+     */
+    private scheduleTransactionCleanup;
     private ensureStandardSchema;
     getKeyFromItem(item: T): TKey;
     generateGlobalKey(key: any, item: any): string;
@@ -343,7 +348,7 @@ export declare class CollectionImpl<T extends object = Record<string, unknown>,
      */
     private updateIndexes;
     private deepEqual;
-    private validateData;
+    validateData(data: unknown, type: `insert` | `update`, key?: TKey): T | never;
     /**
      * Inserts one or more items into the collection
      * @param items - Single item or array of items to insert

package/dist/esm/collection.js

@@ -252,6 +252,7 @@ class CollectionImpl {
       if (ambientTransaction) {
         ambientTransaction.applyMutations(mutations);
         this.transactions.set(ambientTransaction.id, ambientTransaction);
+        this.scheduleTransactionCleanup(ambientTransaction);
         this.recomputeOptimisticState(true);
         return ambientTransaction;
       } else {
@@ -266,6 +267,7 @@ class CollectionImpl {
         directOpTransaction.applyMutations(mutations);
         directOpTransaction.commit();
         this.transactions.set(directOpTransaction.id, directOpTransaction);
+        this.scheduleTransactionCleanup(directOpTransaction);
         this.recomputeOptimisticState(true);
         return directOpTransaction;
       }
@@ -306,6 +308,7 @@ class CollectionImpl {
       if (ambientTransaction) {
         ambientTransaction.applyMutations(mutations);
         this.transactions.set(ambientTransaction.id, ambientTransaction);
+        this.scheduleTransactionCleanup(ambientTransaction);
         this.recomputeOptimisticState(true);
         return ambientTransaction;
       }
@@ -321,6 +324,7 @@ class CollectionImpl {
       directOpTransaction.applyMutations(mutations);
       directOpTransaction.commit();
       this.transactions.set(directOpTransaction.id, directOpTransaction);
+      this.scheduleTransactionCleanup(directOpTransaction);
       this.recomputeOptimisticState(true);
       return directOpTransaction;
     };
@@ -661,11 +665,8 @@ class CollectionImpl {
     this.optimisticUpserts.clear();
     this.optimisticDeletes.clear();
     const activeTransactions = [];
-    const completedTransactions = [];
     for (const transaction of this.transactions.values()) {
-      if (transaction.state === `completed`) {
-        completedTransactions.push(transaction);
-      } else if (![`completed`, `failed`].includes(transaction.state)) {
+      if (![`completed`, `failed`].includes(transaction.state)) {
         activeTransactions.push(transaction);
       }
     }
@@ -700,19 +701,11 @@ class CollectionImpl {
     });
     if (this.pendingSyncedTransactions.length > 0 && !triggeredByUserAction) {
       const pendingSyncKeys = /* @__PURE__ */ new Set();
-      const completedTransactionMutations = /* @__PURE__ */ new Set();
       for (const transaction of this.pendingSyncedTransactions) {
         for (const operation of transaction.operations) {
           pendingSyncKeys.add(operation.key);
         }
       }
-      for (const tx of completedTransactions) {
-        for (const mutation of tx.mutations) {
-          if (mutation.collection === this) {
-            completedTransactionMutations.add(mutation.mutationId);
-          }
-        }
-      }
       const filteredEvents = filteredEventsBySyncStatus.filter((event) => {
         if (event.type === `delete` && pendingSyncKeys.has(event.key)) {
           const hasActiveOptimisticMutation = activeTransactions.some(
@@ -933,6 +926,20 @@ class CollectionImpl {
     }
     return result;
   }
+  /**
+   * Schedule cleanup of a transaction when it completes
+   * @private
+   */
+  scheduleTransactionCleanup(transaction) {
+    if (transaction.state === `completed`) {
+      this.transactions.delete(transaction.id);
+      return;
+    }
+    transaction.isPersisted.promise.then(() => {
+      this.transactions.delete(transaction.id);
+    }).catch(() => {
+    });
+  }
   ensureStandardSchema(schema) {
     if (schema && `~standard` in schema) {
       return schema;
@@ -1213,11 +1220,13 @@ class CollectionImpl {
         }
       });
       emptyTransaction.commit();
+      this.scheduleTransactionCleanup(emptyTransaction);
       return emptyTransaction;
     }
     if (ambientTransaction) {
       ambientTransaction.applyMutations(mutations);
       this.transactions.set(ambientTransaction.id, ambientTransaction);
+      this.scheduleTransactionCleanup(ambientTransaction);
       this.recomputeOptimisticState(true);
       return ambientTransaction;
     }
@@ -1232,6 +1241,7 @@ class CollectionImpl {
     directOpTransaction.applyMutations(mutations);
     directOpTransaction.commit();
     this.transactions.set(directOpTransaction.id, directOpTransaction);
+    this.scheduleTransactionCleanup(directOpTransaction);
     this.recomputeOptimisticState(true);
     return directOpTransaction;
   }