@blackcode_sa/metaestetics-api 1.14.44 → 1.14.45

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

@@ -7,6 +7,30 @@ import {
 import { getAppointmentOrThrow } from './zone-management.utils';
 import { MediaResource } from '../../media/media.service';
 
+/**
+ * Updates visibility fields with audit trail
+ * @param updates Partial updates object
+ * @param doctorId ID of the doctor making the change
+ */
+function addVisibilityAudit(
+  updates: Partial<BeforeAfterPerZone>,
+  doctorId: string,
+): Partial<BeforeAfterPerZone> {
+  // Only add audit fields if visibility-related fields are being updated
+  if (
+    updates.showToPatient !== undefined ||
+    updates.beforeNoteVisibleToPatient !== undefined ||
+    updates.afterNoteVisibleToPatient !== undefined
+  ) {
+    return {
+      ...updates,
+      visibilityUpdatedAt: serverTimestamp() as any,
+      visibilityUpdatedBy: doctorId,
+    };
+  }
+  return updates;
+}
+
 /**
  * Updates a specific photo entry in a zone by index
  * Can update before/after photos and their notes
@@ -16,6 +40,7 @@ import { MediaResource } from '../../media/media.service';
  * @param zoneId Zone ID
  * @param photoIndex Index of the photo entry to update
  * @param updates Partial updates to apply
+ * @param doctorId Optional doctor ID for audit trail (required if updating visibility)
  * @returns Updated appointment
  */
 export async function updateZonePhotoEntryUtil(
@@ -23,7 +48,8 @@ export async function updateZonePhotoEntryUtil(
   appointmentId: string,
   zoneId: string,
   photoIndex: number,
-  updates: Partial<BeforeAfterPerZone>
+  updates: Partial<BeforeAfterPerZone>,
+  doctorId?: string,
 ): Promise<Appointment> {
   const appointment = await getAppointmentOrThrow(db, appointmentId);
   
@@ -41,12 +67,16 @@ export async function updateZonePhotoEntryUtil(
     throw new Error(`Invalid photo index ${photoIndex} for zone ${zoneId}. Must be between 0 and ${zoneArray.length - 1}`);
   }
 
-  // Update the entry
+  // Update the entry with audit trail if visibility is being changed
+  const updatesWithAudit = doctorId
+    ? addVisibilityAudit(updates, doctorId)
+    : updates;
+
   const updatedZonePhotos = { ...zonePhotos };
   updatedZonePhotos[zoneId] = [...zoneArray];
   updatedZonePhotos[zoneId][photoIndex] = {
     ...zoneArray[photoIndex],
-    ...updates,
+    ...updatesWithAudit,
   };
 
   // Update appointment
@@ -171,3 +201,61 @@ export async function getZonePhotoEntryUtil(
   return zoneArray[photoIndex];
 }
 
+/**
+ * Updates visibility of a photo pair (before AND after together)
+ * 
+ * @param db Firestore instance
+ * @param appointmentId Appointment ID
+ * @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 Updated appointment
+ */
+export async function updateZonePhotoVisibilityUtil(
+  db: Firestore,
+  appointmentId: string,
+  zoneId: string,
+  photoIndex: number,
+  showToPatient: boolean,
+  doctorId: string,
+): Promise<Appointment> {
+  return updateZonePhotoEntryUtil(
+    db,
+    appointmentId,
+    zoneId,
+    photoIndex,
+    { showToPatient },
+    doctorId,
+  );
+}
+
+/**
+ * Updates visibility of a photo note (before or after)
+ * 
+ * @param db Firestore instance
+ * @param appointmentId Appointment ID
+ * @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 Updated appointment
+ */
+export async function updateZonePhotoNoteVisibilityUtil(
+  db: Firestore,
+  appointmentId: string,
+  zoneId: string,
+  photoIndex: number,
+  noteType: 'before' | 'after',
+  visibleToPatient: boolean,
+  doctorId: string,
+): Promise<Appointment> {
+  const updates: Partial<BeforeAfterPerZone> =
+    noteType === 'before'
+      ? { beforeNoteVisibleToPatient: visibleToPatient }
+      : { afterNoteVisibleToPatient: visibleToPatient };
+
+  return updateZonePhotoEntryUtil(db, appointmentId, zoneId, photoIndex, updates, doctorId);
+}
+

package/src/types/appointment/index.ts

@@ -120,9 +120,20 @@ export 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;
 }
 
 /**
@@ -157,8 +168,6 @@ export interface ZoneItemData {
   notes?: string;
   subtotal?: number;
   ionNumber?: string;
-  lotNumber?: string; // Optional lot/batch number for the product
-  expiryDate?: string; // Optional expiry date (ISO date string)
   createdAt?: string; // ISO timestamp
   updatedAt?: string; // ISO timestamp
 }
@@ -249,7 +258,14 @@ export 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/src/validations/appointment.schema.ts

@@ -143,8 +143,13 @@ export const finalizedDetailsSchema = z.object({
 export const beforeAfterPerZoneSchema = z.object({
   before: mediaResourceSchema.nullable(),
   after: mediaResourceSchema.nullable(),
-  afterNote: z.string().nullable().optional(),
   beforeNote: z.string().nullable().optional(),
+  afterNote: z.string().nullable().optional(),
+  showToPatient: z.boolean().optional().default(false),
+  beforeNoteVisibleToPatient: z.boolean().optional().default(false),
+  afterNoteVisibleToPatient: z.boolean().optional().default(false),
+  visibilityUpdatedAt: z.any().optional(), // Timestamp
+  visibilityUpdatedBy: z.string().optional(),
 });
 
 /**
@@ -220,10 +225,9 @@ export const zoneItemDataSchema = z
       },
     ),
     notes: z.string().max(MAX_STRING_LENGTH_LONG, 'Notes too long').optional(),
+    notesVisibleToPatient: z.boolean().optional().default(false),
     subtotal: z.number().min(0, 'Subtotal must be non-negative').optional(),
     ionNumber: z.string().optional(),
-    lotNumber: z.string().max(MAX_STRING_LENGTH, 'Lot number too long').optional(),
-    expiryDate: z.string().optional(), // ISO date string (YYYY-MM-DD format)
     createdAt: z.string().optional(),
     updatedAt: z.string().optional(),
   })
@@ -302,7 +306,9 @@ export const appointmentMetadataSchema = z.object({
   recommendedProcedures: z.array(recommendedProcedureSchema).optional().default([]),
   zoneBilling: z.record(z.string(), billingPerZoneSchema).nullable().optional(),
   finalbilling: finalBillingSchema.nullable(),
-  finalizationNotes: z.string().nullable(),
+  finalizationNotesShared: z.string().nullable().optional(),
+  finalizationNotesInternal: z.string().nullable().optional(),
+  finalizationNotes: z.string().nullable().optional(), // @deprecated - kept for backward compatibility
 });
 
 // --- Main Appointment Schemas ---