@fonderie/customers 6.1.3 → 6.1.5

package/brain/signatures.md

@@ -117,7 +117,7 @@ new CustomerAddressModel(store: IStoreAdapter): CustomerAddressModel
   .add(opts: { customerId: string; countryIso: string; subdivision1Iso?: string | null; subdivision2Iso?: string | null; zipPostalCode: string; unit?: string | null; line1?: string | null; line2?: string | null; labelId: string; isPrimary?: boolean; }): Promise<...>
   .updateLabel(addrId: string, customerId: string, labelId: string): Promise<ICustomerAddress>
   .setPrimary(addrId: string, customerId: string): Promise<void>
-  .remove(addrId: string, customerId: string): Promise<void>
+  .remove(addrId: string, customerId: string): Promise<boolean>
 
 new CustomerEmailModel(store: IStoreAdapter): CustomerEmailModel
   .list(customerId: string): Promise<ICustomerEmail[]>

package/dist/index.cjs

@@ -944,16 +944,18 @@ var CustomerAddressModel = class {
       );
     });
   }
+  /** Returns false when the address is not linked to THIS customer (no-op). */
   async remove(addrId, customerId) {
-    await this.store.transaction(async (tx) => {
+    return this.store.transaction(async (tx) => {
       const [deleted] = await tx.query(
         `DELETE FROM fonderie_customer_addresses
 				 WHERE addr_id = $1 AND customer_id = $2
 				 RETURNING is_primary AS "isPrimary"`,
         [addrId, customerId]
       );
+      if (!deleted) return false;
       await tx.query(`DELETE FROM fonderie_addresses WHERE id = $1`, [addrId]);
-      if (deleted?.isPrimary) {
+      if (deleted.isPrimary) {
         await tx.query(
           `UPDATE fonderie_customer_addresses
 					 SET is_primary = true
@@ -966,6 +968,7 @@ var CustomerAddressModel = class {
           [customerId]
         );
       }
+      return true;
     });
   }
 };
@@ -1075,11 +1078,23 @@ var CustomerLabelModel = class {
       [type]
     );
   }
-  remove(id) {
-    return this.store.query(
-      `DELETE FROM fonderie_customer_labels WHERE id = $1`,
+  /**
+   * Returns false when the label is still referenced. Labels are a SHARED
+   * table (no workspace column), so an unconditional delete would let one
+   * tenant destroy a label other tenants' emails/phones/addresses point at.
+   * Until labels are workspace-scoped, only an unreferenced label may go.
+   */
+  async remove(id) {
+    const rows = await this.store.query(
+      `DELETE FROM fonderie_customer_labels l
+			 WHERE l.id = $1
+			   AND NOT EXISTS (SELECT 1 FROM fonderie_customer_emails    WHERE label_id = l.id)
+			   AND NOT EXISTS (SELECT 1 FROM fonderie_customer_phones    WHERE label_id = l.id)
+			   AND NOT EXISTS (SELECT 1 FROM fonderie_customer_addresses WHERE label_id = l.id)
+			 RETURNING l.id`,
       [id]
-    ).then(() => void 0);
+    );
+    return rows.length > 0;
   }
   async findOrCreate(type, raw) {
     const value = raw.trim().toLowerCase();
@@ -1769,7 +1784,10 @@ function customerAddressController(store) {
       if (!isUuid(addrId)) {
         return (0, import_core3.setApiResponse)(import_core3.HTTP.UNPROCESSABLE, "INVALID_PARAMETER", "addrId must be a valid UUID");
       }
-      await addresses.remove(addrId, r.customer.id);
+      const removed = await addresses.remove(addrId, r.customer.id);
+      if (!removed) {
+        return (0, import_core3.setApiResponse)(import_core3.HTTP.NOT_FOUND, "NOT_FOUND", "Address not found");
+      }
       return (0, import_core3.setApiResponse)(import_core3.HTTP.OK, "ADDRESS_REMOVED", "Address removed successfully.");
     }
   };
@@ -2189,7 +2207,14 @@ function customerLabelController(store) {
       if (!isUuid(labelId)) {
         return (0, import_core8.setApiResponse)(import_core8.HTTP.UNPROCESSABLE, "INVALID_PARAMETER", "labelId must be a valid UUID");
       }
-      await labels.remove(labelId);
+      const removed = await labels.remove(labelId);
+      if (!removed) {
+        return (0, import_core8.setApiResponse)(
+          import_core8.HTTP.CONFLICT,
+          "LABEL_IN_USE",
+          "Label is still referenced by customer records and cannot be deleted."
+        );
+      }
       return (0, import_core8.setApiResponse)(import_core8.HTTP.OK, "LABEL_DELETED", "Label deleted successfully.");
     }
   };