@blackcode_sa/metaestetics-api 1.14.46 → 1.14.49

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@blackcode_sa/metaestetics-api",
   "private": false,
-  "version": "1.14.46",
+  "version": "1.14.49",
   "description": "Firebase authentication service with anonymous upgrade support",
   "main": "dist/index.js",
   "module": "dist/index.mjs",

package/src/services/appointment/appointment.service.ts

@@ -2200,6 +2200,34 @@ export class AppointmentService extends BaseService {
     }
   }
 
+  /**
+   * Updates visibility of notes for a zone item (product or standalone note)
+   *
+   * @param appointmentId ID of the appointment
+   * @param zoneId Zone ID
+   * @param itemIndex Index of the item in the zone
+   * @param notesVisibleToPatient Whether the notes should be visible to patient
+   * @returns The updated appointment
+   */
+  async updateZoneItemNoteVisibility(
+    appointmentId: string,
+    zoneId: string,
+    itemIndex: number,
+    notesVisibleToPatient: boolean,
+  ): Promise<Appointment> {
+    try {
+      console.log(
+        `[APPOINTMENT_SERVICE] Updating zone item note visibility at index ${itemIndex} for zone ${zoneId} to ${notesVisibleToPatient}`,
+      );
+      return await updateZoneItemUtil(this.db, appointmentId, zoneId, itemIndex, {
+        notesVisibleToPatient,
+      });
+    } catch (error) {
+      console.error(`[APPOINTMENT_SERVICE] Error updating zone item note visibility:`, error);
+      throw error;
+    }
+  }
+
   /**
    * Gets a specific photo entry from a zone
    *
@@ -2238,6 +2266,12 @@ export class AppointmentService extends BaseService {
     internalNotes?: string | null,
   ): Promise<Appointment> {
     try {
+      console.log('🔍 [APPOINTMENT_SERVICE] updateFinalizationNotes called:', {
+        appointmentId,
+        sharedNotes,
+        internalNotes,
+      });
+
       const appointment = await this.getAppointmentById(appointmentId);
       if (!appointment) {
         throw new Error(`Appointment ${appointmentId} not found`);
@@ -2256,29 +2290,51 @@ export class AppointmentService extends BaseService {
         finalizationNotes: null,
       };
 
+      console.log('🔍 [APPOINTMENT_SERVICE] Current metadata:', {
+        finalizationNotesShared: currentMetadata.finalizationNotesShared,
+        finalizationNotesInternal: currentMetadata.finalizationNotesInternal,
+        finalizationNotes: currentMetadata.finalizationNotes,
+      });
+
+      const metadataUpdate = {
+        selectedZones: currentMetadata.selectedZones,
+        zonePhotos: currentMetadata.zonePhotos,
+        zonesData: currentMetadata.zonesData || null,
+        appointmentProducts: currentMetadata.appointmentProducts || [],
+        extendedProcedures: currentMetadata.extendedProcedures || [],
+        recommendedProcedures: currentMetadata.recommendedProcedures || [],
+        finalbilling: currentMetadata.finalbilling,
+        finalizationNotesShared:
+          sharedNotes !== undefined ? sharedNotes : currentMetadata.finalizationNotesShared,
+        finalizationNotesInternal:
+          internalNotes !== undefined ? internalNotes : currentMetadata.finalizationNotesInternal,
+        // Keep deprecated field for backward compatibility during migration
+        finalizationNotes:
+          sharedNotes !== undefined
+            ? sharedNotes
+            : currentMetadata.finalizationNotes || currentMetadata.finalizationNotesShared,
+      };
+
+      console.log('🔍 [APPOINTMENT_SERVICE] Update data metadata:', {
+        finalizationNotesShared: metadataUpdate.finalizationNotesShared,
+        finalizationNotesInternal: metadataUpdate.finalizationNotesInternal,
+        finalizationNotes: metadataUpdate.finalizationNotes,
+      });
+
       const updateData: UpdateAppointmentData = {
-        metadata: {
-          selectedZones: currentMetadata.selectedZones,
-          zonePhotos: currentMetadata.zonePhotos,
-          zonesData: currentMetadata.zonesData || null,
-          appointmentProducts: currentMetadata.appointmentProducts || [],
-          extendedProcedures: currentMetadata.extendedProcedures || [],
-          recommendedProcedures: currentMetadata.recommendedProcedures || [],
-          finalbilling: currentMetadata.finalbilling,
-          finalizationNotesShared:
-            sharedNotes !== undefined ? sharedNotes : currentMetadata.finalizationNotesShared,
-          finalizationNotesInternal:
-            internalNotes !== undefined ? internalNotes : currentMetadata.finalizationNotesInternal,
-          // Keep deprecated field for backward compatibility during migration
-          finalizationNotes:
-            sharedNotes !== undefined
-              ? sharedNotes
-              : currentMetadata.finalizationNotes || currentMetadata.finalizationNotesShared,
-        },
+        metadata: metadataUpdate,
         updatedAt: serverTimestamp(),
       };
 
-      return await this.updateAppointment(appointmentId, updateData);
+      const result = await this.updateAppointment(appointmentId, updateData);
+      
+      console.log('🔍 [APPOINTMENT_SERVICE] After update, result metadata:', {
+        finalizationNotesShared: result.metadata?.finalizationNotesShared,
+        finalizationNotesInternal: result.metadata?.finalizationNotesInternal,
+        finalizationNotes: result.metadata?.finalizationNotes,
+      });
+
+      return result;
     } catch (error) {
       console.error(`[APPOINTMENT_SERVICE] Error updating finalization notes:`, error);
       throw error;

package/src/services/appointment/utils/zone-photo.utils.ts

@@ -1,4 +1,4 @@
-import { Firestore, updateDoc, serverTimestamp, doc } from 'firebase/firestore';
+import { Firestore, updateDoc, serverTimestamp, doc, Timestamp } from 'firebase/firestore';
 import {
   Appointment,
   BeforeAfterPerZone,
@@ -9,6 +9,9 @@ import { MediaResource } from '../../media/media.service';
 
 /**
  * Updates visibility fields with audit trail
+ * Note: We use Timestamp.now() instead of serverTimestamp() because Firestore
+ * doesn't support serverTimestamp() inside arrays. The timestamp will be set
+ * on the client side, which is acceptable for audit purposes.
  * @param updates Partial updates object
  * @param doctorId ID of the doctor making the change
  */
@@ -24,7 +27,7 @@ function addVisibilityAudit(
   ) {
     return {
       ...updates,
-      visibilityUpdatedAt: serverTimestamp() as any,
+      visibilityUpdatedAt: Timestamp.now(), // Use client-side timestamp (Firestore limitation)
       visibilityUpdatedBy: doctorId,
     };
   }

package/src/types/appointment/index.ts

@@ -166,6 +166,7 @@ export interface ZoneItemData {
   parentZone: string; // Zone key in format "category.zone" (e.g., "face.forehead")
   subzones: string[];
   notes?: string;
+  notesVisibleToPatient?: boolean; // Whether notes are visible to patient (privacy-first, default false)
   subtotal?: number;
   ionNumber?: string;
   createdAt?: string; // ISO timestamp