@data-weave/backend-firestore 0.7.1 → 0.7.2

package/dist/FirestoreDataManager.d.ts

@@ -24,6 +24,7 @@ export declare class FirestoreDataManager<T extends FirestoreTypes.DocumentData,
     private readonly converter;
     private readonly opts?;
     private mergedConverter;
+    private metadataConverter;
     private collection;
     private collectionQuery;
     private managerOptions;
@@ -35,6 +36,15 @@ export declare class FirestoreDataManager<T extends FirestoreTypes.DocumentData,
     read(id: string, options?: FirestoreReadOptions): Promise<WithMetadata<T> | undefined>;
     create(data: WithFieldValue<WithoutId<T>>, options?: FirebaseCreateOptions): Promise<IdentifiableReference<WithMetadata<T>>>;
     private _update;
+    /**
+     * Update only the internal metadata fields of a document.
+     *
+     * Unlike {@link _update}, this bypasses the user-provided converter so it never re-serializes
+     * (and therefore never clobbers) user data. This matters because the user converter rebuilds the
+     * full document shape, which turns absent fields into `undefined` and can overwrite existing map
+     * fields with empty objects when Firestore strips the undefined values.
+     */
+    private _updateMetadata;
     update(id: string, data: WithoutId<Partial<WithFieldValue<T>>>, options?: FirestoreWriteOptions): Promise<void>;
     upsert(id: string, data: WithFieldValue<WithoutId<T>>, options?: FirestoreWriteOptions): Promise<void>;
     count(params?: QueryParams<SerializedT>): Promise<number>;

package/dist/FirestoreDataManager.js

@@ -18,6 +18,7 @@ class FirestoreDataManager {
     converter;
     opts;
     mergedConverter;
+    metadataConverter;
     collection;
     collectionQuery;
     managerOptions;
@@ -29,8 +30,9 @@ class FirestoreDataManager {
         this.collectionPath = collectionPath;
         this.converter = converter;
         this.opts = opts;
+        this.metadataConverter = new FirestoreMetadata_1.FirestoreMetadataConverter();
         // @ts-expect-error - Force merge FirestoreDataConverter and InternalFirestoreDataConverter
-        this.mergedConverter = new utils_1.MergeConverters(this.converter, new FirestoreMetadata_1.FirestoreMetadataConverter());
+        this.mergedConverter = new utils_1.MergeConverters(this.converter, this.metadataConverter);
         this.managerOptions = this.validateOptions({ ...defaultFirebaseDataManagerOptions, ...this.opts });
         this.refCache = this.managerOptions.refCache || new datamanager_1.MapCache(100);
         this.listCache = this.managerOptions.listCache || new datamanager_1.MapCache(100);
@@ -109,6 +111,25 @@ class FirestoreDataManager {
         }
         return this.firestore.updateDoc(ref, serializedData);
     }
+    /**
+     * Update only the internal metadata fields of a document.
+     *
+     * Unlike {@link _update}, this bypasses the user-provided converter so it never re-serializes
+     * (and therefore never clobbers) user data. This matters because the user converter rebuilds the
+     * full document shape, which turns absent fields into `undefined` and can overwrite existing map
+     * fields with empty objects when Firestore strips the undefined values.
+     */
+    async _updateMetadata(id, metadata, options) {
+        const serializedData = this.metadataConverter.toFirestore({
+            ...metadata,
+            [FirestoreMetadata_1.FIRESTORE_KEYS.UPDATED_AT]: this.firestore.serverTimestamp(),
+        });
+        const ref = this.firestore.doc(this.collection, id);
+        if (options?.transaction) {
+            return options.transaction.update(ref, serializedData);
+        }
+        return this.firestore.updateDoc(ref, serializedData);
+    }
     async update(id, data, options) {
         await this._update(id, data, options);
     }
@@ -183,10 +204,7 @@ class FirestoreDataManager {
     }
     async delete(id, options) {
         if (this.managerOptions.deleteMode === 'soft') {
-            await this._update(id, {
-                ...{},
-                [FirestoreMetadata_1.FIRESTORE_KEYS.DELETED]: true,
-            }, options);
+            await this._updateMetadata(id, { [FirestoreMetadata_1.FIRESTORE_KEYS.DELETED]: true }, options);
             return;
         }
         if (options?.transaction) {

package/dist/FirestoreMetadata.d.ts

@@ -16,8 +16,8 @@ export type FirestoreSerializedMetadata = {
     [FIRESTORE_INTERAL_KEYS.UPDATED_AT]: FirestoreTypes.Timestamp;
 };
 export declare class FirestoreMetadataConverter implements InternalFirestoreDataConverter<Metadata, FirestoreSerializedMetadata> {
-    toFirestore(data: WithFieldValue<WithoutId<Metadata>>): {
-        __deleted: FirestoreTypes.FieldValue | FirestoreTypes.WithFieldValue<boolean>;
+    toFirestore(data: Partial<WithFieldValue<WithoutId<Metadata>>>): {
+        __deleted: boolean;
         __createdAt: FirestoreTypes.Timestamp;
         __updatedAt: FirestoreTypes.Timestamp;
     };

package/dist/FirestoreMetadata.js

@@ -16,6 +16,7 @@ var FIRESTORE_KEYS;
 class FirestoreMetadataConverter {
     toFirestore(data) {
         return {
+            // Cast: value may be absent on partial metadata writes; Firestore strips undefined fields
             [FIRESTORE_INTERAL_KEYS.DELETED]: data[FIRESTORE_KEYS.DELETED],
             // Cast: Firestore handles conversion from Date to Timestamp internally
             [FIRESTORE_INTERAL_KEYS.CREATED_AT]: data[FIRESTORE_KEYS.CREATED_AT],

package/dist/FirestoreReference.js

@@ -70,7 +70,8 @@ class FirestoreReference extends datamanager_1.LiveReference {
         if (!(0, utils_1.checkIfReferenceExists)(docSnapshot))
             throw new Error(`Document does not exist ${this.docRef.path}`);
         const data = docSnapshot.data(this.options?.snapshotOptions);
-        if (this.options?.filterDeleted && data?.deleted === true) {
+        const deletedFlag = data?.[FirestoreMetadata_1.FIRESTORE_KEYS.DELETED];
+        if (this.options?.filterDeleted && deletedFlag === true) {
             return undefined;
         }
         return data;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@data-weave/backend-firestore",
-    "version": "0.7.1",
+    "version": "0.7.2",
     "author": "",
     "main": "dist/index.js",
     "types": "dist/index.d.ts",
@@ -23,7 +23,7 @@
         "registry": "https://registry.npmjs.org/"
     },
     "devDependencies": {
-        "@data-weave/datamanager": "~0.7.1",
+        "@data-weave/datamanager": "~0.7.2",
         "@firebase/firestore": "^4.7.8",
         "@firebase/functions-types": "^0.6.3",
         "@google-cloud/firestore": "^7.11.1",