@typicalday/firegraph 0.14.0 → 0.14.1

package/dist/cloudflare/index.js

@@ -6,7 +6,7 @@ import {
   dedupeIndexSpecs,
   isFirestoreSpecialType,
   validateJsonPathKey
-} from "../chunk-4MMQ5W74.js";
+} from "../chunk-NNBSUOOF.js";
 import {
   DEFAULT_CORE_INDEXES
 } from "../chunk-2DHMNTV6.js";
@@ -25,14 +25,14 @@ import {
   createMergedRegistry,
   createRegistry,
   generateId
-} from "../chunk-WRTFC5NG.js";
+} from "../chunk-3AHHXMWX.js";
 import {
   CapabilityNotSupportedError,
   FiregraphError,
   assertUpdatePayloadExclusive,
   deleteField,
   flattenPatch
-} from "../chunk-TK64DNVK.js";
+} from "../chunk-SIHE4UY4.js";
 import "../chunk-EQJUUVFG.js";
 
 // src/cloudflare/schema.ts

package/dist/firestore-enterprise/index.cjs

@@ -235,7 +235,6 @@ function isTerminalValue(value) {
   if (ctor && typeof ctor.name === "string" && FIRESTORE_TERMINAL_CTOR.has(ctor.name)) return true;
   return true;
 }
-var SAFE_KEY_RE = /^[A-Za-z_][A-Za-z0-9_-]*$/;
 function assertUpdatePayloadExclusive(update) {
   if (update.replaceData !== void 0 && update.dataOps !== void 0) {
     throw new Error(
@@ -274,9 +273,9 @@ function walkForDeleteSentinels(node, path, parent, visit) {
 }
 function assertSafePath(path) {
   for (const seg of path) {
-    if (!SAFE_KEY_RE.test(seg)) {
+    if (seg === "") {
       throw new Error(
-        `firegraph: unsafe object key ${JSON.stringify(seg)} at path ${path.map((p) => JSON.stringify(p)).join(" > ")}. Keys used inside update payloads must match /^[A-Za-z_][A-Za-z0-9_-]*$/ so they can be embedded safely in SQLite JSON paths.`
+        `firegraph: empty object key at path ${path.map((p) => JSON.stringify(p)).join(" > ")}. Object keys in update payloads must be non-empty.`
       );
     }
   }
@@ -2302,7 +2301,7 @@ function generateId() {
 }
 
 // src/firestore-enterprise/backend.ts
-var import_firestore4 = require("@google-cloud/firestore");
+var import_firestore5 = require("@google-cloud/firestore");
 
 // src/bulk.ts
 var MAX_BATCH_SIZE = 500;
@@ -2527,6 +2526,13 @@ async function getFirestoreSurface() {
   }
   return { P: _Pipelines, Ts: _Timestamp };
 }
+var UNESCAPED_FIELD_NAME_RE = /^[_A-Za-z][_A-Za-z0-9]*$/;
+function buildDataPathAlias(segments) {
+  const encoded = segments.map(
+    (seg) => UNESCAPED_FIELD_NAME_RE.test(seg) ? seg : "`" + seg.replace(/\\/g, "\\\\").replace(/`/g, "\\`") + "`"
+  );
+  return ["data", ...encoded].join(".");
+}
 function buildFilterExpression(P, filter) {
   const { field: fieldName, op, value } = filter;
   switch (op) {
@@ -2604,7 +2610,7 @@ async function runFirestorePipelineUpdate(db, collectionPath, filters, patch, _o
   }
   const { P, Ts } = await getFirestoreSurface();
   const transforms = ops.map((op) => {
-    const alias = `data.${op.path.join(".")}`;
+    const alias = buildDataPathAlias(op.path);
     return P.constant(op.value).as(alias);
   });
   transforms.push(P.constant(Ts.now()).as("updatedAt"));
@@ -2635,8 +2641,8 @@ function createFirestoreAdapter(db, collectionPath) {
         await collectionRef.doc(docId).set(data);
       }
     },
-    async updateDoc(docId, data) {
-      await collectionRef.doc(docId).update(data);
+    async updateDoc(docId, args) {
+      await collectionRef.doc(docId).update(...args);
     },
     async deleteDoc(docId) {
       await collectionRef.doc(docId).delete();
@@ -2672,8 +2678,8 @@ function createTransactionAdapter(db, collectionPath, tx) {
         tx.set(collectionRef.doc(docId), data);
       }
     },
-    updateDoc(docId, data) {
-      tx.update(collectionRef.doc(docId), data);
+    updateDoc(docId, args) {
+      tx.update(collectionRef.doc(docId), ...args);
     },
     deleteDoc(docId) {
       tx.delete(collectionRef.doc(docId));
@@ -2705,8 +2711,8 @@ function createBatchAdapter(db, collectionPath) {
         batch.set(collectionRef.doc(docId), data);
       }
     },
-    updateDoc(docId, data) {
-      batch.update(collectionRef.doc(docId), data);
+    updateDoc(docId, args) {
+      batch.update(collectionRef.doc(docId), ...args);
     },
     deleteDoc(docId) {
       batch.delete(collectionRef.doc(docId));
@@ -3325,6 +3331,27 @@ async function runFirestoreEngineTraversal(db, collectionPath, params) {
   return result;
 }
 
+// src/internal/firestore-update.ts
+var import_firestore4 = require("@google-cloud/firestore");
+init_serialization();
+function buildFirestoreUpdateArgs(update, db) {
+  assertUpdatePayloadExclusive(update);
+  const args = [];
+  if (update.replaceData) {
+    args.push("data", deserializeFirestoreTypes(update.replaceData, db));
+  } else if (update.dataOps) {
+    for (const op of update.dataOps) {
+      assertSafePath(op.path);
+      args.push(new import_firestore4.FieldPath("data", ...op.path), op.delete ? import_firestore4.FieldValue.delete() : op.value);
+    }
+  }
+  args.push("updatedAt", import_firestore4.FieldValue.serverTimestamp());
+  if (update.v !== void 0) {
+    args.push("v", update.v);
+  }
+  return args;
+}
+
 // src/internal/firestore-vector.ts
 var ENVELOPE_FIELDS3 = /* @__PURE__ */ new Set([
   "aType",
@@ -3393,9 +3420,6 @@ async function runFirestoreFindNearest(base, params) {
   return snap.docs.map((doc) => doc.data());
 }
 
-// src/firestore-enterprise/backend.ts
-init_serialization();
-
 // src/firestore-enterprise/pipeline-adapter.ts
 var _Pipelines6 = null;
 async function getPipelines5() {
@@ -3476,30 +3500,8 @@ var ENTERPRISE_BASE_CAPS = /* @__PURE__ */ new Set([
 var _emulatorFallbackWarned = false;
 var _classicInProductionWarned = false;
 var _previewDmlWarned = false;
-function dottedDataPath(op) {
-  assertSafePath(op.path);
-  return `data.${op.path.join(".")}`;
-}
-function buildFirestoreUpdate(update, db) {
-  assertUpdatePayloadExclusive(update);
-  const out = {
-    updatedAt: import_firestore4.FieldValue.serverTimestamp()
-  };
-  if (update.replaceData) {
-    out.data = deserializeFirestoreTypes(update.replaceData, db);
-  } else if (update.dataOps) {
-    for (const op of update.dataOps) {
-      const key = dottedDataPath(op);
-      out[key] = op.delete ? import_firestore4.FieldValue.delete() : op.value;
-    }
-  }
-  if (update.v !== void 0) {
-    out.v = update.v;
-  }
-  return out;
-}
 function stampWritableRecord(record) {
-  const now = import_firestore4.FieldValue.serverTimestamp();
+  const now = import_firestore5.FieldValue.serverTimestamp();
   const out = {
     aType: record.aType,
     aUid: record.aUid,
@@ -3532,7 +3534,7 @@ var FirestoreEnterpriseTransactionBackend = class {
     );
   }
   async updateDoc(docId, update) {
-    this.adapter.updateDoc(docId, buildFirestoreUpdate(update, this.db));
+    this.adapter.updateDoc(docId, buildFirestoreUpdateArgs(update, this.db));
   }
   async deleteDoc(docId) {
     this.adapter.deleteDoc(docId);
@@ -3551,7 +3553,7 @@ var FirestoreEnterpriseBatchBackend = class {
     );
   }
   updateDoc(docId, update) {
-    this.adapter.updateDoc(docId, buildFirestoreUpdate(update, this.db));
+    this.adapter.updateDoc(docId, buildFirestoreUpdateArgs(update, this.db));
   }
   deleteDoc(docId) {
     this.adapter.deleteDoc(docId);
@@ -3599,7 +3601,7 @@ var FirestoreEnterpriseBackendImpl = class _FirestoreEnterpriseBackendImpl {
     );
   }
   updateDoc(docId, update) {
-    return this.adapter.updateDoc(docId, buildFirestoreUpdate(update, this.db));
+    return this.adapter.updateDoc(docId, buildFirestoreUpdateArgs(update, this.db));
   }
   deleteDoc(docId) {
     return this.adapter.deleteDoc(docId);