@naturalcycles/firestore-lib 2.15.0 → 2.16.1

package/dist/firestore.db.js

@@ -157,7 +157,7 @@ export class FirestoreDB extends BaseCommonDB {
             }
             return;
         }
-        await pMap(_chunk(rows, MAX_ITEMS), async chunk => {
+        await pMap(_chunk(rows, MAX_ITEMS), async (chunk) => {
             // .batch is called "Atomic batch writer"
             // Executes multiple writes in a single atomic transaction-like commit — all succeed or all fail.
             // If any write in the batch fails (e.g., permission error, missing doc), the whole batch fails.
@@ -192,7 +192,7 @@ export class FirestoreDB extends BaseCommonDB {
                 tableRows.push([table, row]);
             }
         }
-        await pMap(_chunk(tableRows, MAX_ITEMS), async chunk => {
+        await pMap(_chunk(tableRows, MAX_ITEMS), async (chunk) => {
             const batch = firestore.batch();
             for (const [table, row] of chunk) {
                 _assert(row.id, `firestore-db doesn't support id auto-generation, but empty id was provided in multiSaveBatch`);
@@ -236,7 +236,7 @@ export class FirestoreDB extends BaseCommonDB {
             }
             return ids.length;
         }
-        await pMap(_chunk(ids, MAX_ITEMS), async chunk => {
+        await pMap(_chunk(ids, MAX_ITEMS), async (chunk) => {
             const batch = firestore.batch();
             for (const id of chunk) {
                 batch.delete(col.doc(escapeDocId(id)));
@@ -259,7 +259,7 @@ export class FirestoreDB extends BaseCommonDB {
             }
         }
         else {
-            await pMap(_chunk(refs, MAX_ITEMS), async chunk => {
+            await pMap(_chunk(refs, MAX_ITEMS), async (chunk) => {
                 const batch = firestore.batch();
                 for (const ref of chunk) {
                     batch.delete(ref);
@@ -278,7 +278,7 @@ export class FirestoreDB extends BaseCommonDB {
     async runInTransaction(fn, opt = {}) {
         const { readOnly } = opt;
         try {
-            await this.cfg.firestore.runTransaction(async firestoreTx => {
+            await this.cfg.firestore.runTransaction(async (firestoreTx) => {
                 const tx = new FirestoreDBTransaction(this, firestoreTx);
                 await fn(tx);
             }, {

package/dist/query.util.js

@@ -12,7 +12,10 @@ export function readAtToReadTime(opt) {
 }
 // Map DBQueryFilterOp to WhereFilterOp
 // Currently it's fully aligned!
-const OP_MAP = {};
+const OP_MAP = {
+// '=': '==',
+// in: 'array-contains',
+};
 export function dbQueryToFirestoreQuery(dbQuery, emptyQuery) {
     let q = emptyQuery;
     // filter
@@ -24,7 +27,11 @@ export function dbQueryToFirestoreQuery(dbQuery, emptyQuery) {
         q = q.orderBy(mapName(ord.name), ord.descending ? 'desc' : 'asc');
     }
     // limit
-    q = q.limit(dbQuery._limitValue);
+    // Careful here, Firestore just changed the behavior of limit(0) recently!
+    // Therefor we're guarding the limit(0) case and not sending the limit if it's zero.
+    if (dbQuery._limitValue) {
+        q = q.limit(dbQuery._limitValue);
+    }
     // selectedFields
     if (dbQuery._selectedFieldNames) {
         // id is filtered out, because in Firestore it's not a "property",

package/package.json

@@ -2,7 +2,7 @@
   "name": "@naturalcycles/firestore-lib",
   "type": "module",
   "dependencies": {
-    "@google-cloud/firestore": "^7",
+    "@google-cloud/firestore": "^8",
     "@naturalcycles/db-lib": "^10",
     "@naturalcycles/js-lib": "^15",
     "@naturalcycles/nodejs-lib": "^15",
@@ -10,8 +10,8 @@
   },
   "devDependencies": {
     "@types/node": "^25",
-    "@typescript/native-preview": "7.0.0-dev.20260301.1",
-    "firebase-admin": "^13",
+    "@typescript/native-preview": "beta",
+    "firebase-admin": "^14",
     "@naturalcycles/dev-lib": "18.4.2"
   },
   "exports": {
@@ -38,7 +38,7 @@
   "engines": {
     "node": ">=24.10.0"
   },
-  "version": "2.15.0",
+  "version": "2.16.1",
   "description": "Firestore implementation of CommonDB interface",
   "author": "Natural Cycles Team",
   "license": "MIT",

package/src/index.ts

@@ -1,4 +1,5 @@
 import { Firestore } from '@google-cloud/firestore'
+
 export * from './firestore.db.js'
 export * from './firestoreStreamReadable.js'
 export * from './query.util.js'

package/src/query.util.ts

@@ -40,7 +40,11 @@ export function dbQueryToFirestoreQuery<ROW extends ObjectWithId>(
   }
 
   // limit
-  q = q.limit(dbQuery._limitValue)
+  // Careful here, Firestore just changed the behavior of limit(0) recently!
+  // Therefor we're guarding the limit(0) case and not sending the limit if it's zero.
+  if (dbQuery._limitValue) {
+    q = q.limit(dbQuery._limitValue)
+  }
 
   // selectedFields
   if (dbQuery._selectedFieldNames) {