@naturalcycles/firestore-lib 2.3.0 → 2.4.0

package/dist/firestore.db.js

@@ -88,19 +88,19 @@ export class FirestoreDB extends BaseCommonDB {
         const method = methodMap[opt.saveMethod] || 'set';
         if (opt.tx) {
             const { tx } = opt.tx;
-            rows.forEach(row => {
+            for (const row of rows) {
                 _assert(row.id, `firestore-db doesn't support id auto-generation, but empty id was provided in saveBatch`);
                 tx[method](col.doc(escapeDocId(row.id)), _filterUndefinedValues(row));
-            });
+            }
             return;
         }
         // Firestore allows max 500 items in one batch
         await pMap(_chunk(rows, 500), async (chunk) => {
             const batch = firestore.batch();
-            chunk.forEach(row => {
+            for (const row of chunk) {
                 _assert(row.id, `firestore-db doesn't support id auto-generation, but empty id was provided in saveBatch`);
                 batch[method](col.doc(escapeDocId(row.id)), _filterUndefinedValues(row));
-            });
+            }
             await batch.commit();
         }, { concurrency: 1 });
     }
@@ -123,16 +123,16 @@ export class FirestoreDB extends BaseCommonDB {
         const col = firestore.collection(table);
         if (opt.tx) {
             const { tx } = opt.tx;
-            ids.forEach(id => {
+            for (const id of ids) {
                 tx.delete(col.doc(escapeDocId(id)));
-            });
+            }
             return ids.length;
         }
         await pMap(_chunk(ids, 500), async (chunk) => {
             const batch = firestore.batch();
-            chunk.forEach(id => {
+            for (const id of chunk) {
                 batch.delete(col.doc(escapeDocId(id)));
-            });
+            }
             await batch.commit();
         });
         return ids.length;

package/package.json

@@ -12,7 +12,7 @@
     "@types/node": "^24",
     "dotenv": "^17",
     "firebase-admin": "^13",
-    "@naturalcycles/dev-lib": "19.11.0"
+    "@naturalcycles/dev-lib": "18.4.2"
   },
   "exports": {
     ".": "./dist/index.js"
@@ -38,7 +38,7 @@
   "engines": {
     "node": ">=22.12.0"
   },
-  "version": "2.3.0",
+  "version": "2.4.0",
   "description": "Firestore implementation of CommonDB interface",
   "author": "Natural Cycles Team",
   "license": "MIT",

package/src/firestore.db.ts

@@ -159,14 +159,14 @@ export class FirestoreDB extends BaseCommonDB implements CommonDB {
     if (opt.tx) {
       const { tx } = opt.tx as FirestoreDBTransaction
 
-      rows.forEach(row => {
+      for (const row of rows) {
         _assert(
           row.id,
           `firestore-db doesn't support id auto-generation, but empty id was provided in saveBatch`,
         )
 
         tx[method as 'set' | 'create'](col.doc(escapeDocId(row.id)), _filterUndefinedValues(row))
-      })
+      }
       return
     }
 
@@ -176,7 +176,7 @@ export class FirestoreDB extends BaseCommonDB implements CommonDB {
       async chunk => {
         const batch = firestore.batch()
 
-        chunk.forEach(row => {
+        for (const row of chunk) {
           _assert(
             row.id,
             `firestore-db doesn't support id auto-generation, but empty id was provided in saveBatch`,
@@ -185,7 +185,7 @@ export class FirestoreDB extends BaseCommonDB implements CommonDB {
             col.doc(escapeDocId(row.id)),
             _filterUndefinedValues(row),
           )
-        })
+        }
 
         await batch.commit()
       },
@@ -227,18 +227,18 @@ export class FirestoreDB extends BaseCommonDB implements CommonDB {
     if (opt.tx) {
       const { tx } = opt.tx as FirestoreDBTransaction
 
-      ids.forEach(id => {
+      for (const id of ids) {
         tx.delete(col.doc(escapeDocId(id)))
-      })
+      }
       return ids.length
     }
 
     await pMap(_chunk(ids, 500), async chunk => {
       const batch = firestore.batch()
 
-      chunk.forEach(id => {
+      for (const id of chunk) {
         batch.delete(col.doc(escapeDocId(id)))
-      })
+      }
 
       await batch.commit()
     })