@blackcode_sa/metaestetics-api 1.14.43 → 1.14.45

package/dist/admin/index.d.mts

@@ -1677,9 +1677,20 @@ interface BeforeAfterPerZone {
     before: MediaResource | null;
     /** URL for after photo or null if not available */
     after: MediaResource | null;
-    /** Optional note for the zone */
-    afterNote?: string | null;
+    /** Optional note for the before photo */
     beforeNote?: string | null;
+    /** Optional note for the after photo */
+    afterNote?: string | null;
+    /** Whether this photo pair (before AND after) should be visible to the patient */
+    showToPatient?: boolean;
+    /** Whether the before note should be visible to the patient */
+    beforeNoteVisibleToPatient?: boolean;
+    /** Whether the after note should be visible to the patient */
+    afterNoteVisibleToPatient?: boolean;
+    /** Timestamp when visibility was last updated */
+    visibilityUpdatedAt?: Timestamp;
+    /** ID of the doctor who last updated visibility */
+    visibilityUpdatedBy?: string;
 }
 /**
  * Interface for zone item data (products or notes per zone)
@@ -1786,7 +1797,14 @@ interface AppointmentMetadata {
     extendedProcedures?: ExtendedProcedureInfo[];
     recommendedProcedures: RecommendedProcedure[];
     finalbilling: FinalBilling | null;
-    finalizationNotes: string | null;
+    /** Final treatment notes shared with patient */
+    finalizationNotesShared?: string | null;
+    /** Final treatment notes for internal use only (not shared with patient) */
+    finalizationNotesInternal?: string | null;
+    /**
+     * @deprecated Use finalizationNotesShared instead. Kept for backward compatibility during migration.
+     */
+    finalizationNotes?: string | null;
     /**
      * @deprecated Use zonesData instead
      */

package/dist/admin/index.d.ts

@@ -1677,9 +1677,20 @@ interface BeforeAfterPerZone {
     before: MediaResource | null;
     /** URL for after photo or null if not available */
     after: MediaResource | null;
-    /** Optional note for the zone */
-    afterNote?: string | null;
+    /** Optional note for the before photo */
     beforeNote?: string | null;
+    /** Optional note for the after photo */
+    afterNote?: string | null;
+    /** Whether this photo pair (before AND after) should be visible to the patient */
+    showToPatient?: boolean;
+    /** Whether the before note should be visible to the patient */
+    beforeNoteVisibleToPatient?: boolean;
+    /** Whether the after note should be visible to the patient */
+    afterNoteVisibleToPatient?: boolean;
+    /** Timestamp when visibility was last updated */
+    visibilityUpdatedAt?: Timestamp;
+    /** ID of the doctor who last updated visibility */
+    visibilityUpdatedBy?: string;
 }
 /**
  * Interface for zone item data (products or notes per zone)
@@ -1786,7 +1797,14 @@ interface AppointmentMetadata {
     extendedProcedures?: ExtendedProcedureInfo[];
     recommendedProcedures: RecommendedProcedure[];
     finalbilling: FinalBilling | null;
-    finalizationNotes: string | null;
+    /** Final treatment notes shared with patient */
+    finalizationNotesShared?: string | null;
+    /** Final treatment notes for internal use only (not shared with patient) */
+    finalizationNotesInternal?: string | null;
+    /**
+     * @deprecated Use finalizationNotesShared instead. Kept for backward compatibility during migration.
+     */
+    finalizationNotes?: string | null;
     /**
      * @deprecated Use zonesData instead
      */

package/dist/index.d.mts

@@ -5604,9 +5604,20 @@ interface BeforeAfterPerZone {
     before: MediaResource | null;
     /** URL for after photo or null if not available */
     after: MediaResource | null;
-    /** Optional note for the zone */
-    afterNote?: string | null;
+    /** Optional note for the before photo */
     beforeNote?: string | null;
+    /** Optional note for the after photo */
+    afterNote?: string | null;
+    /** Whether this photo pair (before AND after) should be visible to the patient */
+    showToPatient?: boolean;
+    /** Whether the before note should be visible to the patient */
+    beforeNoteVisibleToPatient?: boolean;
+    /** Whether the after note should be visible to the patient */
+    afterNoteVisibleToPatient?: boolean;
+    /** Timestamp when visibility was last updated */
+    visibilityUpdatedAt?: Timestamp;
+    /** ID of the doctor who last updated visibility */
+    visibilityUpdatedBy?: string;
 }
 /**
  * Interface for zone photo upload data
@@ -5723,7 +5734,14 @@ interface AppointmentMetadata {
     extendedProcedures?: ExtendedProcedureInfo[];
     recommendedProcedures: RecommendedProcedure[];
     finalbilling: FinalBilling | null;
-    finalizationNotes: string | null;
+    /** Final treatment notes shared with patient */
+    finalizationNotesShared?: string | null;
+    /** Final treatment notes for internal use only (not shared with patient) */
+    finalizationNotesInternal?: string | null;
+    /**
+     * @deprecated Use finalizationNotesShared instead. Kept for backward compatibility during migration.
+     */
+    finalizationNotes?: string | null;
     /**
      * @deprecated Use zonesData instead
      */
@@ -7694,6 +7712,29 @@ declare class AppointmentService extends BaseService {
      * @returns The updated appointment
      */
     updateZonePhotoNotes(appointmentId: string, zoneId: string, photoIndex: number, beforeNote?: string, afterNote?: string): Promise<Appointment>;
+    /**
+     * Updates visibility of a photo pair (before AND after together)
+     *
+     * @param appointmentId ID of the appointment
+     * @param zoneId Zone ID
+     * @param photoIndex Index of the photo entry
+     * @param showToPatient Whether the photo pair should be visible to patient
+     * @param doctorId ID of the doctor making the change (for audit trail)
+     * @returns The updated appointment
+     */
+    updateZonePhotoVisibility(appointmentId: string, zoneId: string, photoIndex: number, showToPatient: boolean, doctorId: string): Promise<Appointment>;
+    /**
+     * Updates visibility of a photo note (before or after)
+     *
+     * @param appointmentId ID of the appointment
+     * @param zoneId Zone ID
+     * @param photoIndex Index of the photo entry
+     * @param noteType Type of note ('before' or 'after')
+     * @param visibleToPatient Whether the note should be visible to patient
+     * @param doctorId ID of the doctor making the change (for audit trail)
+     * @returns The updated appointment
+     */
+    updateZonePhotoNoteVisibility(appointmentId: string, zoneId: string, photoIndex: number, noteType: 'before' | 'after', visibleToPatient: boolean, doctorId: string): Promise<Appointment>;
     /**
      * Gets a specific photo entry from a zone
      *
@@ -7703,6 +7744,15 @@ declare class AppointmentService extends BaseService {
      * @returns Photo entry
      */
     getZonePhotoEntry(appointmentId: string, zoneId: string, photoIndex: number): Promise<BeforeAfterPerZone>;
+    /**
+     * Updates finalization notes (shared with patient and/or internal only)
+     *
+     * @param appointmentId ID of the appointment
+     * @param sharedNotes Notes to be shared with patient (optional)
+     * @param internalNotes Notes for internal use only (optional)
+     * @returns The updated appointment
+     */
+    updateFinalizationNotes(appointmentId: string, sharedNotes?: string | null, internalNotes?: string | null): Promise<Appointment>;
     /**
      * Gets all next steps recommendations for a patient from their past appointments.
      * Returns recommendations with context about which appointment, practitioner, and clinic suggested them.

package/dist/index.d.ts

@@ -5604,9 +5604,20 @@ interface BeforeAfterPerZone {
     before: MediaResource | null;
     /** URL for after photo or null if not available */
     after: MediaResource | null;
-    /** Optional note for the zone */
-    afterNote?: string | null;
+    /** Optional note for the before photo */
     beforeNote?: string | null;
+    /** Optional note for the after photo */
+    afterNote?: string | null;
+    /** Whether this photo pair (before AND after) should be visible to the patient */
+    showToPatient?: boolean;
+    /** Whether the before note should be visible to the patient */
+    beforeNoteVisibleToPatient?: boolean;
+    /** Whether the after note should be visible to the patient */
+    afterNoteVisibleToPatient?: boolean;
+    /** Timestamp when visibility was last updated */
+    visibilityUpdatedAt?: Timestamp;
+    /** ID of the doctor who last updated visibility */
+    visibilityUpdatedBy?: string;
 }
 /**
  * Interface for zone photo upload data
@@ -5723,7 +5734,14 @@ interface AppointmentMetadata {
     extendedProcedures?: ExtendedProcedureInfo[];
     recommendedProcedures: RecommendedProcedure[];
     finalbilling: FinalBilling | null;
-    finalizationNotes: string | null;
+    /** Final treatment notes shared with patient */
+    finalizationNotesShared?: string | null;
+    /** Final treatment notes for internal use only (not shared with patient) */
+    finalizationNotesInternal?: string | null;
+    /**
+     * @deprecated Use finalizationNotesShared instead. Kept for backward compatibility during migration.
+     */
+    finalizationNotes?: string | null;
     /**
      * @deprecated Use zonesData instead
      */
@@ -7694,6 +7712,29 @@ declare class AppointmentService extends BaseService {
      * @returns The updated appointment
      */
     updateZonePhotoNotes(appointmentId: string, zoneId: string, photoIndex: number, beforeNote?: string, afterNote?: string): Promise<Appointment>;
+    /**
+     * Updates visibility of a photo pair (before AND after together)
+     *
+     * @param appointmentId ID of the appointment
+     * @param zoneId Zone ID
+     * @param photoIndex Index of the photo entry
+     * @param showToPatient Whether the photo pair should be visible to patient
+     * @param doctorId ID of the doctor making the change (for audit trail)
+     * @returns The updated appointment
+     */
+    updateZonePhotoVisibility(appointmentId: string, zoneId: string, photoIndex: number, showToPatient: boolean, doctorId: string): Promise<Appointment>;
+    /**
+     * Updates visibility of a photo note (before or after)
+     *
+     * @param appointmentId ID of the appointment
+     * @param zoneId Zone ID
+     * @param photoIndex Index of the photo entry
+     * @param noteType Type of note ('before' or 'after')
+     * @param visibleToPatient Whether the note should be visible to patient
+     * @param doctorId ID of the doctor making the change (for audit trail)
+     * @returns The updated appointment
+     */
+    updateZonePhotoNoteVisibility(appointmentId: string, zoneId: string, photoIndex: number, noteType: 'before' | 'after', visibleToPatient: boolean, doctorId: string): Promise<Appointment>;
     /**
      * Gets a specific photo entry from a zone
      *
@@ -7703,6 +7744,15 @@ declare class AppointmentService extends BaseService {
      * @returns Photo entry
      */
     getZonePhotoEntry(appointmentId: string, zoneId: string, photoIndex: number): Promise<BeforeAfterPerZone>;
+    /**
+     * Updates finalization notes (shared with patient and/or internal only)
+     *
+     * @param appointmentId ID of the appointment
+     * @param sharedNotes Notes to be shared with patient (optional)
+     * @param internalNotes Notes for internal use only (optional)
+     * @returns The updated appointment
+     */
+    updateFinalizationNotes(appointmentId: string, sharedNotes?: string | null, internalNotes?: string | null): Promise<Appointment>;
     /**
      * Gets all next steps recommendations for a patient from their past appointments.
      * Returns recommendations with context about which appointment, practitioner, and clinic suggested them.

package/dist/index.js

@@ -3903,8 +3903,14 @@ var finalizedDetailsSchema = import_zod3.z.object({
 var beforeAfterPerZoneSchema = import_zod3.z.object({
   before: mediaResourceSchema.nullable(),
   after: mediaResourceSchema.nullable(),
+  beforeNote: import_zod3.z.string().nullable().optional(),
   afterNote: import_zod3.z.string().nullable().optional(),
-  beforeNote: import_zod3.z.string().nullable().optional()
+  showToPatient: import_zod3.z.boolean().optional().default(false),
+  beforeNoteVisibleToPatient: import_zod3.z.boolean().optional().default(false),
+  afterNoteVisibleToPatient: import_zod3.z.boolean().optional().default(false),
+  visibilityUpdatedAt: import_zod3.z.any().optional(),
+  // Timestamp
+  visibilityUpdatedBy: import_zod3.z.string().optional()
 });
 var billingPerZoneSchema = import_zod3.z.object({
   Product: import_zod3.z.string().min(MIN_STRING_LENGTH, "Product name is required"),
@@ -3962,6 +3968,7 @@ var zoneItemDataSchema = import_zod3.z.object({
     }
   ),
   notes: import_zod3.z.string().max(MAX_STRING_LENGTH_LONG, "Notes too long").optional(),
+  notesVisibleToPatient: import_zod3.z.boolean().optional().default(false),
   subtotal: import_zod3.z.number().min(0, "Subtotal must be non-negative").optional(),
   ionNumber: import_zod3.z.string().optional(),
   createdAt: import_zod3.z.string().optional(),
@@ -4025,7 +4032,10 @@ var appointmentMetadataSchema = import_zod3.z.object({
   recommendedProcedures: import_zod3.z.array(recommendedProcedureSchema).optional().default([]),
   zoneBilling: import_zod3.z.record(import_zod3.z.string(), billingPerZoneSchema).nullable().optional(),
   finalbilling: finalBillingSchema.nullable(),
-  finalizationNotes: import_zod3.z.string().nullable()
+  finalizationNotesShared: import_zod3.z.string().nullable().optional(),
+  finalizationNotesInternal: import_zod3.z.string().nullable().optional(),
+  finalizationNotes: import_zod3.z.string().nullable().optional()
+  // @deprecated - kept for backward compatibility
 });
 var createAppointmentSchema = import_zod3.z.object({
   clinicBranchId: import_zod3.z.string().min(MIN_STRING_LENGTH, "Clinic branch ID is required"),
@@ -4826,10 +4836,14 @@ function initializeMetadata(appointment) {
     extendedProcedures: [],
     recommendedProcedures: [],
     finalbilling: null,
+    finalizationNotesShared: null,
+    finalizationNotesInternal: null,
     finalizationNotes: null
+    // @deprecated - kept for backward compatibility
   };
 }
 async function addItemToZoneUtil(db, appointmentId, zoneId, item) {
+  var _a;
   validateZoneKeyFormat(zoneId);
   const appointment = await getAppointmentOrThrow(db, appointmentId);
   const metadata = initializeMetadata(appointment);
@@ -4843,6 +4857,8 @@ async function addItemToZoneUtil(db, appointmentId, zoneId, item) {
     parentZone: zoneId,
     // Set parentZone to the zone key
     subtotal: calculateItemSubtotal(item),
+    // Set default visibility to false (privacy-first) if notes exist and visibility not explicitly set
+    notesVisibleToPatient: (_a = item.notesVisibleToPatient) != null ? _a : item.notes ? false : void 0,
     createdAt: now,
     updatedAt: now
   };
@@ -5342,7 +5358,17 @@ async function getRecommendedProceduresUtil(db, appointmentId) {
 
 // src/services/appointment/utils/zone-photo.utils.ts
 var import_firestore13 = require("firebase/firestore");
-async function updateZonePhotoEntryUtil(db, appointmentId, zoneId, photoIndex, updates) {
+function addVisibilityAudit(updates, doctorId) {
+  if (updates.showToPatient !== void 0 || updates.beforeNoteVisibleToPatient !== void 0 || updates.afterNoteVisibleToPatient !== void 0) {
+    return {
+      ...updates,
+      visibilityUpdatedAt: (0, import_firestore13.serverTimestamp)(),
+      visibilityUpdatedBy: doctorId
+    };
+  }
+  return updates;
+}
+async function updateZonePhotoEntryUtil(db, appointmentId, zoneId, photoIndex, updates, doctorId) {
   var _a;
   const appointment = await getAppointmentOrThrow(db, appointmentId);
   const zonePhotos = (_a = appointment.metadata) == null ? void 0 : _a.zonePhotos;
@@ -5353,11 +5379,12 @@ async function updateZonePhotoEntryUtil(db, appointmentId, zoneId, photoIndex, u
   if (photoIndex < 0 || photoIndex >= zoneArray.length) {
     throw new Error(`Invalid photo index ${photoIndex} for zone ${zoneId}. Must be between 0 and ${zoneArray.length - 1}`);
   }
+  const updatesWithAudit = doctorId ? addVisibilityAudit(updates, doctorId) : updates;
   const updatedZonePhotos = { ...zonePhotos };
   updatedZonePhotos[zoneId] = [...zoneArray];
   updatedZonePhotos[zoneId][photoIndex] = {
     ...zoneArray[photoIndex],
-    ...updates
+    ...updatesWithAudit
   };
   const appointmentRef = (0, import_firestore13.doc)(db, APPOINTMENTS_COLLECTION, appointmentId);
   await (0, import_firestore13.updateDoc)(appointmentRef, {
@@ -5401,6 +5428,20 @@ async function getZonePhotoEntryUtil(db, appointmentId, zoneId, photoIndex) {
   }
   return zoneArray[photoIndex];
 }
+async function updateZonePhotoVisibilityUtil(db, appointmentId, zoneId, photoIndex, showToPatient, doctorId) {
+  return updateZonePhotoEntryUtil(
+    db,
+    appointmentId,
+    zoneId,
+    photoIndex,
+    { showToPatient },
+    doctorId
+  );
+}
+async function updateZonePhotoNoteVisibilityUtil(db, appointmentId, zoneId, photoIndex, noteType, visibleToPatient, doctorId) {
+  const updates = noteType === "before" ? { beforeNoteVisibleToPatient: visibleToPatient } : { afterNoteVisibleToPatient: visibleToPatient };
+  return updateZonePhotoEntryUtil(db, appointmentId, zoneId, photoIndex, updates, doctorId);
+}
 
 // src/services/appointment/appointment.service.ts
 var AppointmentService = class extends BaseService {
@@ -6267,6 +6308,7 @@ var AppointmentService = class extends BaseService {
    * @returns The updated appointment
    */
   async updateAppointmentZonePhoto(appointmentId, zoneId, photoType, mediaMetadata, notes) {
+    var _a, _b, _c, _d;
     try {
       console.log(
         `[APPOINTMENT_SERVICE] Updating appointment metadata for ${photoType} photo in zone ${zoneId}`
@@ -6284,7 +6326,10 @@ var AppointmentService = class extends BaseService {
         recommendedProcedures: [],
         zoneBilling: null,
         finalbilling: null,
+        finalizationNotesShared: null,
+        finalizationNotesInternal: null,
         finalizationNotes: null
+        // @deprecated - kept for backward compatibility
       };
       let currentZonePhotos = {};
       if (currentMetadata.zonePhotos) {
@@ -6299,7 +6344,11 @@ var AppointmentService = class extends BaseService {
                 before: oldData.before || null,
                 after: oldData.after || null,
                 beforeNote: null,
-                afterNote: null
+                afterNote: null,
+                showToPatient: false,
+                // Default: not visible to patient (privacy-first)
+                beforeNoteVisibleToPatient: false,
+                afterNoteVisibleToPatient: false
               }
             ];
           }
@@ -6312,7 +6361,13 @@ var AppointmentService = class extends BaseService {
         before: photoType === "before" ? mediaMetadata.url : null,
         after: photoType === "after" ? mediaMetadata.url : null,
         beforeNote: photoType === "before" ? notes || null : null,
-        afterNote: photoType === "after" ? notes || null : null
+        afterNote: photoType === "after" ? notes || null : null,
+        showToPatient: false,
+        // Default: not visible to patient
+        beforeNoteVisibleToPatient: false,
+        // Default: not visible to patient
+        afterNoteVisibleToPatient: false
+        // Default: not visible to patient
       };
       currentZonePhotos[zoneId] = [...currentZonePhotos[zoneId], newEntry];
       if (currentZonePhotos[zoneId].length > 10) {
@@ -6331,7 +6386,10 @@ var AppointmentService = class extends BaseService {
             zoneBilling: currentMetadata.zoneBilling
           },
           finalbilling: currentMetadata.finalbilling,
-          finalizationNotes: currentMetadata.finalizationNotes
+          finalizationNotesShared: (_a = currentMetadata.finalizationNotesShared) != null ? _a : null,
+          finalizationNotesInternal: (_b = currentMetadata.finalizationNotesInternal) != null ? _b : null,
+          finalizationNotes: (_d = (_c = currentMetadata.finalizationNotes) != null ? _c : currentMetadata.finalizationNotesShared) != null ? _d : null
+          // @deprecated - migrate from old field if exists
         },
         updatedAt: (0, import_firestore14.serverTimestamp)()
       };
@@ -6385,7 +6443,7 @@ var AppointmentService = class extends BaseService {
    * @returns The updated appointment
    */
   async deleteZonePhoto(appointmentId, zoneId, photoIndex) {
-    var _a, _b, _c, _d, _e, _f, _g, _h, _i;
+    var _a, _b, _c, _d, _e, _f, _g, _h, _i, _j, _k, _l, _m, _n, _o, _p;
     try {
       console.log(
         `[APPOINTMENT_SERVICE] Deleting zone photo index ${photoIndex} for zone ${zoneId} in appointment ${appointmentId}`
@@ -6442,7 +6500,10 @@ var AppointmentService = class extends BaseService {
             zoneBilling: appointment.metadata.zoneBilling
           },
           finalbilling: ((_h = appointment.metadata) == null ? void 0 : _h.finalbilling) || null,
-          finalizationNotes: ((_i = appointment.metadata) == null ? void 0 : _i.finalizationNotes) || null
+          finalizationNotesShared: (_l = (_k = (_i = appointment.metadata) == null ? void 0 : _i.finalizationNotesShared) != null ? _k : (_j = appointment.metadata) == null ? void 0 : _j.finalizationNotes) != null ? _l : null,
+          finalizationNotesInternal: (_n = (_m = appointment.metadata) == null ? void 0 : _m.finalizationNotesInternal) != null ? _n : null,
+          finalizationNotes: (_p = (_o = appointment.metadata) == null ? void 0 : _o.finalizationNotes) != null ? _p : null
+          // @deprecated
         },
         updatedAt: (0, import_firestore14.serverTimestamp)()
       };
@@ -6641,7 +6702,7 @@ var AppointmentService = class extends BaseService {
    * @returns The updated appointment with recalculated billing
    */
   async recalculateFinalBilling(appointmentId, taxRate) {
-    var _a;
+    var _a, _b, _c, _d, _e;
     try {
       console.log(
         `[APPOINTMENT_SERVICE] Recalculating final billing for appointment ${appointmentId}`
@@ -6663,7 +6724,10 @@ var AppointmentService = class extends BaseService {
         extendedProcedures: [],
         recommendedProcedures: [],
         finalbilling: null,
+        finalizationNotesShared: null,
+        finalizationNotesInternal: null,
         finalizationNotes: null
+        // @deprecated
       };
       const shouldUpdatePaymentStatus = finalbilling.finalPrice > 0 && appointment.paymentStatus === "not_applicable" /* NOT_APPLICABLE */;
       const updateData = {
@@ -6679,7 +6743,10 @@ var AppointmentService = class extends BaseService {
             zoneBilling: currentMetadata.zoneBilling
           },
           finalbilling,
-          finalizationNotes: currentMetadata.finalizationNotes
+          finalizationNotesShared: (_b = currentMetadata.finalizationNotesShared) != null ? _b : null,
+          finalizationNotesInternal: (_c = currentMetadata.finalizationNotesInternal) != null ? _c : null,
+          finalizationNotes: (_e = (_d = currentMetadata.finalizationNotes) != null ? _d : currentMetadata.finalizationNotesShared) != null ? _e : null
+          // @deprecated
         },
         ...shouldUpdatePaymentStatus && {
           paymentStatus: "unpaid" /* UNPAID */
@@ -6879,6 +6946,64 @@ var AppointmentService = class extends BaseService {
       throw error;
     }
   }
+  /**
+   * Updates visibility of a photo pair (before AND after together)
+   *
+   * @param appointmentId ID of the appointment
+   * @param zoneId Zone ID
+   * @param photoIndex Index of the photo entry
+   * @param showToPatient Whether the photo pair should be visible to patient
+   * @param doctorId ID of the doctor making the change (for audit trail)
+   * @returns The updated appointment
+   */
+  async updateZonePhotoVisibility(appointmentId, zoneId, photoIndex, showToPatient, doctorId) {
+    try {
+      console.log(
+        `[APPOINTMENT_SERVICE] Updating photo visibility at index ${photoIndex} for zone ${zoneId} to ${showToPatient}`
+      );
+      return await updateZonePhotoVisibilityUtil(
+        this.db,
+        appointmentId,
+        zoneId,
+        photoIndex,
+        showToPatient,
+        doctorId
+      );
+    } catch (error) {
+      console.error(`[APPOINTMENT_SERVICE] Error updating zone photo visibility:`, error);
+      throw error;
+    }
+  }
+  /**
+   * Updates visibility of a photo note (before or after)
+   *
+   * @param appointmentId ID of the appointment
+   * @param zoneId Zone ID
+   * @param photoIndex Index of the photo entry
+   * @param noteType Type of note ('before' or 'after')
+   * @param visibleToPatient Whether the note should be visible to patient
+   * @param doctorId ID of the doctor making the change (for audit trail)
+   * @returns The updated appointment
+   */
+  async updateZonePhotoNoteVisibility(appointmentId, zoneId, photoIndex, noteType, visibleToPatient, doctorId) {
+    try {
+      console.log(
+        `[APPOINTMENT_SERVICE] Updating ${noteType} note visibility at index ${photoIndex} for zone ${zoneId} to ${visibleToPatient}`
+      );
+      return await updateZonePhotoNoteVisibilityUtil(
+        this.db,
+        appointmentId,
+        zoneId,
+        photoIndex,
+        noteType,
+        visibleToPatient,
+        doctorId
+      );
+    } catch (error) {
+      console.error(`[APPOINTMENT_SERVICE] Error updating zone photo note visibility:`, error);
+      throw error;
+    }
+  }
   /**
    * Gets a specific photo entry from a zone
    *
@@ -6898,6 +7023,54 @@ var AppointmentService = class extends BaseService {
       throw error;
     }
   }
+  /**
+   * Updates finalization notes (shared with patient and/or internal only)
+   *
+   * @param appointmentId ID of the appointment
+   * @param sharedNotes Notes to be shared with patient (optional)
+   * @param internalNotes Notes for internal use only (optional)
+   * @returns The updated appointment
+   */
+  async updateFinalizationNotes(appointmentId, sharedNotes, internalNotes) {
+    try {
+      const appointment = await this.getAppointmentById(appointmentId);
+      if (!appointment) {
+        throw new Error(`Appointment ${appointmentId} not found`);
+      }
+      const currentMetadata = appointment.metadata || {
+        selectedZones: null,
+        zonePhotos: null,
+        zonesData: null,
+        appointmentProducts: [],
+        extendedProcedures: [],
+        recommendedProcedures: [],
+        finalbilling: null,
+        finalizationNotesShared: null,
+        finalizationNotesInternal: null,
+        finalizationNotes: null
+      };
+      const updateData = {
+        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 !== void 0 ? sharedNotes : currentMetadata.finalizationNotesShared,
+          finalizationNotesInternal: internalNotes !== void 0 ? internalNotes : currentMetadata.finalizationNotesInternal,
+          // Keep deprecated field for backward compatibility during migration
+          finalizationNotes: sharedNotes !== void 0 ? sharedNotes : currentMetadata.finalizationNotes || currentMetadata.finalizationNotesShared
+        },
+        updatedAt: (0, import_firestore14.serverTimestamp)()
+      };
+      return await this.updateAppointment(appointmentId, updateData);
+    } catch (error) {
+      console.error(`[APPOINTMENT_SERVICE] Error updating finalization notes:`, error);
+      throw error;
+    }
+  }
   /**
    * Gets all next steps recommendations for a patient from their past appointments.
    * Returns recommendations with context about which appointment, practitioner, and clinic suggested them.