@harperfast/harper 5.0.7 → 5.0.8

package/dist/resources/Table.js

@@ -325,6 +325,10 @@ function makeTable(options) {
                         // we listen for events by iterating through the async iterator provided by the subscription
                         for await (const event of subscription) {
                             try {
+                                if (!event || typeof event !== 'object') {
+                                    logger_ts_1.logger.error?.('Bad subscription event', event);
+                                    continue;
+                                }
                                 const firstWrite = event.type === 'transaction' ? event.writes[0] : event;
                                 if (!firstWrite) {
                                     logger_ts_1.logger.error?.('Bad subscription event', event);
@@ -1321,7 +1325,8 @@ function makeTable(options) {
          */
         static evict(id, existingRecord, existingVersion) {
             let entry;
-            let transaction = txnForContext({ transaction: new DatabaseTransaction_ts_1.DatabaseTransaction() }).getReadTxn();
+            const lmdbTransaction = txnForContext({ transaction: new DatabaseTransaction_ts_1.DatabaseTransaction() });
+            let transaction = lmdbTransaction.getReadTxn();
             let options = { transaction };
             try {
                 if (hasSourceGet || audit) {
@@ -1354,7 +1359,13 @@ function makeTable(options) {
                 }
             }
             finally {
-                return transaction.commit();
+                if (primaryStore.ifVersion) {
+                    // LMDB: committing the wrapper calls doneReadTxn(), removing it from trackedTxns
+                    return lmdbTransaction.commit();
+                }
+                // RocksDB: eviction writes went directly into the raw transaction via options;
+                // commit it directly, as DatabaseTransaction.commit() would abort it (no tracked writes)
+                return transaction?.commit();
             }
         }
         /**