@fonderie/customers 6.1.6 → 6.2.0

package/brain/outcomes.md

@@ -54,7 +54,9 @@ id                       UUID PRIMARY KEY DEFAULT gen_random_uuid()
 type                     fonderie_customer_label_type NOT NULL
 value                    TEXT NOT NULL
 created_at               TIMESTAMPTZ NOT NULL DEFAULT now()
+workspace_id             UUID
 -- CONSTRAINT uq_fcl_type_value UNIQUE (type, value)
+-- INDEX idx_fcl_workspace (workspace_id)
 ```
 
 ### `fonderie_customer_notes`

package/dist/index.cjs

@@ -1064,47 +1064,69 @@ var CustomerEmailModel = class {
 };
 
 // src/models/customer-label.model.ts
+var SELECT = `id, type, value, created_at AS "createdAt"`;
 var CustomerLabelModel = class {
   constructor(store) {
     this.store = store;
   }
   store;
-  list(type) {
+  /**
+   * List the labels a workspace may use: the shared/system defaults
+   * (workspace_id IS NULL) plus its OWN private labels. Never another
+   * workspace's — so listing can't enumerate other tenants' custom
+   * vocabulary.
+   */
+  list(type, workspaceId) {
     return this.store.query(
-      `SELECT id, type, value, created_at AS "createdAt"
+      `SELECT ${SELECT}
 			 FROM fonderie_customer_labels
-			 WHERE type = $1
+			 WHERE type = $1 AND (workspace_id IS NULL OR workspace_id = $2)
 			 ORDER BY value ASC`,
-      [type]
+      [type, workspaceId]
     );
   }
   /**
-   * 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.
+   * Delete a label. Only a label OWNED BY this workspace and UNREFERENCED can
+   * go: shared defaults (workspace_id IS NULL) and other workspaces' labels
+   * are untouchable, and an in-use label (still pointed at by an email/phone/
+   * address) is refused so it can't be destroyed out from under a record.
+   * Returns false when nothing was deleted.
    */
-  async remove(id) {
+  async remove(id, workspaceId) {
     const rows = await this.store.query(
       `DELETE FROM fonderie_customer_labels l
 			 WHERE l.id = $1
+			   AND l.workspace_id = $2
 			   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]
+      [id, workspaceId]
     );
     return rows.length > 0;
   }
-  async findOrCreate(type, raw) {
+  /**
+   * Resolve a (type, value) to a label id for this workspace: reuse a shared
+   * default when one exists (so common labels like 'work' stay canonical),
+   * otherwise create/reuse a label PRIVATE to this workspace.
+   */
+  async findOrCreate(type, raw, workspaceId) {
     const value = raw.trim().toLowerCase();
-    const [row] = await this.store.query(
-      `INSERT INTO fonderie_customer_labels (type, value)
-			 VALUES ($1, $2)
-			 ON CONFLICT (type, value) DO UPDATE SET value = EXCLUDED.value
-			 RETURNING id, type, value, created_at AS "createdAt"`,
+    const [shared] = await this.store.query(
+      `SELECT ${SELECT} FROM fonderie_customer_labels
+			 WHERE type = $1 AND value = $2 AND workspace_id IS NULL
+			 LIMIT 1`,
       [type, value]
     );
+    if (shared) return shared;
+    const [row] = await this.store.query(
+      `INSERT INTO fonderie_customer_labels (type, value, workspace_id)
+			 VALUES ($1, $2, $3)
+			 ON CONFLICT (type, value, workspace_id) WHERE workspace_id IS NOT NULL
+			 DO UPDATE SET value = EXCLUDED.value
+			 RETURNING ${SELECT}`,
+      [type, value, workspaceId]
+    );
     if (!row) throw new Error("Failed to find or create label");
     return row;
   }
@@ -1711,7 +1733,7 @@ function customerAddressController(store) {
         return (0, import_core3.setApiResponse)(import_core3.HTTP.UNPROCESSABLE, "INVALID_PARAMETER", "zipPostalCode is required");
       }
       const rawLabel = typeof body?.["label"] === "string" ? body["label"] : "service";
-      const { id: labelId } = await labels.findOrCreate("address", rawLabel);
+      const { id: labelId } = await labels.findOrCreate("address", rawLabel, r.workspaceId);
       let created;
       try {
         created = await addresses.add({
@@ -1749,7 +1771,7 @@ function customerAddressController(store) {
         return (0, import_core3.setApiResponse)(import_core3.HTTP.UNPROCESSABLE, "INVALID_PARAMETER", "label is required");
       }
       try {
-        const { id: labelId } = await labels.findOrCreate("address", body["label"].trim());
+        const { id: labelId } = await labels.findOrCreate("address", body["label"].trim(), r.workspaceId);
         const updated = await addresses.updateLabel(addrId, r.customer.id, labelId);
         return (0, import_core3.setApiResponse)(import_core3.HTTP.OK, "ADDRESS_UPDATED", "Address updated successfully.", {
           address: toCustomerAddressDTO(updated)
@@ -1844,7 +1866,7 @@ function customerEmailController(store) {
       }
       const rawLabel = typeof body?.["label"] === "string" ? body["label"] : "work";
       const isPrimary2 = body?.["isPrimary"] === true;
-      const { id: labelId } = await labels.findOrCreate("email", rawLabel);
+      const { id: labelId } = await labels.findOrCreate("email", rawLabel, r.workspaceId);
       let created;
       try {
         created = await emails.add({
@@ -1876,7 +1898,7 @@ function customerEmailController(store) {
         return (0, import_core4.setApiResponse)(import_core4.HTTP.UNPROCESSABLE, "INVALID_PARAMETER", "label is required");
       }
       try {
-        const { id: labelId } = await labels.findOrCreate("email", body["label"].trim());
+        const { id: labelId } = await labels.findOrCreate("email", body["label"].trim(), r.workspaceId);
         const updated = await emails.updateLabel(emailId, r.customer.id, labelId);
         return (0, import_core4.setApiResponse)(import_core4.HTTP.OK, "EMAIL_UPDATED", "Email updated successfully.", {
           email: toCustomerEmailDTO(updated)
@@ -2053,7 +2075,7 @@ function customerPhoneController(store) {
       }
       const rawLabel = typeof body?.["label"] === "string" ? body["label"] : "mobile";
       const isPrimary2 = body?.["isPrimary"] === true;
-      const { id: labelId } = await labels.findOrCreate("phone", rawLabel);
+      const { id: labelId } = await labels.findOrCreate("phone", rawLabel, r.workspaceId);
       let created;
       try {
         created = await phones.add({
@@ -2085,7 +2107,7 @@ function customerPhoneController(store) {
         return (0, import_core6.setApiResponse)(import_core6.HTTP.UNPROCESSABLE, "INVALID_PARAMETER", "label is required");
       }
       try {
-        const { id: labelId } = await labels.findOrCreate("phone", body["label"].trim());
+        const { id: labelId } = await labels.findOrCreate("phone", body["label"].trim(), r.workspaceId);
         const updated = await phones.updateLabel(phoneId, r.customer.id, labelId);
         return (0, import_core6.setApiResponse)(import_core6.HTTP.OK, "PHONE_UPDATED", "Phone updated successfully.", {
           phone: toCustomerPhoneDTO(updated)
@@ -2196,7 +2218,8 @@ function customerLabelController(store) {
       if (!type || !VALID_TYPES.includes(type)) {
         return (0, import_core8.setApiResponse)(import_core8.HTTP.UNPROCESSABLE, "INVALID_PARAMETER", "type must be phone, email, or address");
       }
-      const rows = await labels.list(type);
+      if (!ctx.workspace) return (0, import_core8.setApiResponse)(import_core8.HTTP.NOT_FOUND, "NOT_FOUND", "Workspace not found");
+      const rows = await labels.list(type, ctx.workspace.id);
       return (0, import_core8.setApiResponse)(import_core8.HTTP.OK, "LABELS_FETCHED", "Labels retrieved successfully.", {
         labels: rows.map(toCustomerLabelDTO)
       });
@@ -2207,12 +2230,13 @@ function customerLabelController(store) {
       if (!isUuid(labelId)) {
         return (0, import_core8.setApiResponse)(import_core8.HTTP.UNPROCESSABLE, "INVALID_PARAMETER", "labelId must be a valid UUID");
       }
-      const removed = await labels.remove(labelId);
+      if (!ctx.workspace) return (0, import_core8.setApiResponse)(import_core8.HTTP.NOT_FOUND, "NOT_FOUND", "Workspace not found");
+      const removed = await labels.remove(labelId, ctx.workspace.id);
       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."
+          "Label cannot be deleted \u2014 it is a shared default, not owned by this workspace, or still referenced."
         );
       }
       return (0, import_core8.setApiResponse)(import_core8.HTTP.OK, "LABEL_DELETED", "Label deleted successfully.");