@drawbridge/drawbridge-utils 0.0.170 → 0.0.172

package/dist/providers.cjs

@@ -527,6 +527,7 @@ var row = ({ connection: connection2, data: data2, manifest, row: described }) =
   };
 };
 var segmentRowWrites = ({ connection: connection2, data: data2, manifest, row: described, segment }) => {
+  if ((described == null ? void 0 : described.id) === void 0 || (described == null ? void 0 : described.id) === null) return [];
   const built = row({ connection: connection2, data: data2, manifest, row: described });
   return [
     // PUSH IF ABSENT. The $ne guard is what makes a second concurrent register
@@ -774,10 +775,11 @@ var attentive_default2 = {
       sync: async ({ lead, settings, suppressed, token }, { fetcher } = {}) => {
         var _a, _b, _c, _d;
         const segment = settings == null ? void 0 : settings.segment;
-        if (!segment) return { message: "No Attentive segment is chosen for this connection.", skipped: true };
         const email = ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email);
         const phone = toE164(((_d = (_c = lead == null ? void 0 : lead.canonical) == null ? void 0 : _c.phone) == null ? void 0 : _d.value) || (lead == null ? void 0 : lead.phone));
-        if (!email && !phone) return { message: "That lead has no email address or phone number to sync.", skipped: true };
+        const request2 = { email: email || null, phone: phone || null, segment: (settings == null ? void 0 : settings.segment) || null };
+        if (!segment) return { message: "No Attentive segment is chosen for this connection.", request: request2, skipped: true };
+        if (!email && !phone) return { message: "That lead has no email address or phone number to sync.", request: request2, skipped: true };
         const user = {
           ...email && { email },
           ...phone && { phone }
@@ -799,6 +801,7 @@ var attentive_default2 = {
           });
           return {
             message: "Attentive accepted an unsubscribe for this contact \u2014 they have opted out.",
+            request: request2,
             response: { accepted: true, unsubscribed: true }
           };
         }
@@ -839,6 +842,7 @@ var attentive_default2 = {
           // queued them — a merchant who reads "synced" and looks for the person
           // in Attentive a second later has been told the wrong thing.
           message: "Attentive accepted this contact for the segment. Attentive processes these asynchronously, so it appears there shortly.",
+          request: request2,
           response: {
             accepted: true,
             ...(membership == null ? void 0 : membership.batchJobId) && { batchJobId: membership.batchJobId }
@@ -2855,7 +2859,7 @@ var drawbridge_default2 = {
           text: { max: 2e3, type: "string" }
         },
         triggers: ["schedule.day", "schedule.week", "schedule.month"],
-        usage: { actions: 0 }
+        usage: { actions: channels.email.actionsPerSend }
       }),
       // To organization MEMBERS. Never suppressed — an entrant's opt-out must
       // not silence an alert to staff — and not billed.
@@ -2880,10 +2884,14 @@ var drawbridge_default2 = {
           subject: { max: 150, type: "string" }
         },
         triggers: ["lead.insert"],
-        // Zero is a PRICE, and a deliberate one. Declared rather than omitted
-        // so "this is free" and "nobody decided" stay different statements;
-        // completeStep bills only when actions > 0.
-        usage: { actions: 0 }
+        // EVERY STEP IN A BILLABLE WORKFLOW COUNTS, team mail included. This
+        // leaves our SendGrid account and costs the same to send as any other
+        // email, so it carries the same weight. Whether anything is charged at
+        // all is the workflow's `billable` tag, not the step's audience — and
+        // this step triggers on lead.insert, so a merchant notifying
+        // themselves on every entry spends against their allowance twice per
+        // lead, once for the entry and once for the notification.
+        usage: { actions: channels.email.actionsPerSend }
       }),
       // To a LEAD. Suppression applies and the send is billed.
       send: () => ({
@@ -2992,7 +3000,7 @@ var api2 = async (path, { fetcher = fetch, method = "GET", payload, token }) =>
   }
   return response.status === 204 ? null : response.json();
 };
-var segmentName = (title) => "Drawbridge: " + title;
+var segmentName = (title, id) => "Drawbridge: " + title + " (" + String(id).slice(-6) + ")";
 var canManageSegments = (settings) => String((settings == null ? void 0 : settings.scope) || "").split(/\s+/).includes("segments:write");
 var klaviyo_default2 = {
   // OAuth 2.1, and PKCE is REQUIRED rather than recommended: Klaviyo refuses an
@@ -3239,11 +3247,12 @@ var klaviyo_default2 = {
       // different thing from deleting the profile.
       remove: false,
       sync: async ({ context, lead, segments, settings, suppressed, token }, { fetcher } = {}) => {
-        var _a, _b, _c;
+        var _a, _b, _c, _d, _e;
         const list = settings == null ? void 0 : settings.list;
-        if (!list) return { message: "No Klaviyo list is chosen for this connection.", skipped: true };
-        const email = ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email);
-        if (!email) return { message: "That lead has no email address to sync.", skipped: true };
+        const request2 = { email: ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email) || null, list: (settings == null ? void 0 : settings.list) || null };
+        if (!list) return { message: "No Klaviyo list is chosen for this connection.", request: request2, skipped: true };
+        const email = ((_d = (_c = lead == null ? void 0 : lead.canonical) == null ? void 0 : _c.email) == null ? void 0 : _d.value) || (lead == null ? void 0 : lead.email);
+        if (!email) return { message: "That lead has no email address to sync.", request: request2, skipped: true };
         const person = (context == null ? void 0 : context.contact) || null;
         const totals = (person == null ? void 0 : person.totals) || {};
         const count = (value) => typeof value === "number" ? value : (value == null ? void 0 : value.total) || 0;
@@ -3300,8 +3309,8 @@ var klaviyo_default2 = {
           },
           token
         });
-        const profileId = (_c = profile == null ? void 0 : profile.data) == null ? void 0 : _c.id;
-        if (!profileId) return { message: "Klaviyo returned no profile id.", skipped: true };
+        const profileId = (_e = profile == null ? void 0 : profile.data) == null ? void 0 : _e.id;
+        if (!profileId) return { message: "Klaviyo returned no profile id.", request: request2, skipped: true };
         await api2("/profile-subscription-bulk-create-jobs/", {
           fetcher,
           method: "POST",
@@ -3330,6 +3339,7 @@ var klaviyo_default2 = {
           // Merged into `context` for later steps in this run.
           context: { klaviyoProfileId: profileId },
           message: suppressed ? "Synced to Klaviyo as unsubscribed \u2014 this contact has opted out." : "Synced to the Klaviyo list.",
+          request: request2,
           // Recorded on the run for support to read back, not a write
           // instruction — the hook has already written what it needed to.
           response: { klaviyoProfileId: profileId }
@@ -3365,7 +3375,7 @@ var klaviyo_default2 = {
             skipped: true
           };
         }
-        const name = segmentName(segment.title);
+        const name = segmentName(segment.title, segment.id);
         const existing = segmentRowFor({ connection: connection2, segment });
         let id = null;
         if (existing == null ? void 0 : existing.id) {
@@ -3772,7 +3782,10 @@ var api3 = async (path, { dc, fetcher = fetch, method = "GET", payload, token })
 };
 var subscriberHash = (email) => (0, import_node_crypto5.createHash)("md5").update(String(email).trim().toLowerCase()).digest("hex");
 var TAG_NAME_LIMIT = 100;
-var tagName = (title) => ("Drawbridge: " + title).slice(0, TAG_NAME_LIMIT);
+var tagName = (title, id) => {
+  const suffix = " (" + String(id).slice(-6) + ")";
+  return ("Drawbridge: " + title).slice(0, TAG_NAME_LIMIT - suffix.length) + suffix;
+};
 var mailchimp_default2 = {
   // OAUTH 2, authorization code. Every url below is quoted from
   // mailchimp.com/developer/marketing/guides/access-user-data-oauth-2/ rather
@@ -3910,15 +3923,16 @@ var mailchimp_default2 = {
       // why there is no create-or-update branch here. Quoted from Mailchimp's
       // Marketing API reference for the list-members resource.
       sync: async ({ connection: connection2, lead, segments, settings, suppressed, token }, { fetcher, read } = {}) => {
-        var _a, _b, _c;
+        var _a, _b, _c, _d, _e;
         const audience = settings == null ? void 0 : settings.audience;
-        if (!audience) return { message: "No Mailchimp audience is chosen for this connection.", skipped: true };
-        const email = ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email);
-        if (!email) return { message: "That lead has no email address to sync.", skipped: true };
+        const request2 = { audience: (settings == null ? void 0 : settings.audience) || null, email: ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email) || null };
+        if (!audience) return { message: "No Mailchimp audience is chosen for this connection.", request: request2, skipped: true };
+        const email = ((_d = (_c = lead == null ? void 0 : lead.canonical) == null ? void 0 : _c.email) == null ? void 0 : _d.value) || (lead == null ? void 0 : lead.email);
+        if (!email) return { message: "That lead has no email address to sync.", request: request2, skipped: true };
         const hash = subscriberHash(email);
         const [firstName, ...restOfName] = String((lead == null ? void 0 : lead.name) || "").trim().split(/\s+/).filter(Boolean);
         const lastName = restOfName.join(" ");
-        const phone = ((_c = lead == null ? void 0 : lead.phone) == null ? void 0 : _c.number) || null;
+        const phone = ((_e = lead == null ? void 0 : lead.phone) == null ? void 0 : _e.number) || null;
         const mergeFields = {
           ...firstName && { FNAME: firstName },
           ...lastName && { LNAME: lastName },
@@ -3952,14 +3966,17 @@ var mailchimp_default2 = {
                 }
               },
               {
-                $project: { _id: 0, title: 1 }
+                // THE ID AS WELL AS THE TITLE, because the tag name carries it —
+                // see tagName. Without it every name here would end in the
+                // string 'undefined' and match nothing register wrote.
+                $project: { _id: 0, id: 1, title: 1 }
               }
             ]
           });
-          const joined = new Set(segments.map((entry) => entry.title));
-          const tags = (owned || []).map((entry) => entry.title).filter(Boolean).map((title) => ({
-            name: tagName(title),
-            status: joined.has(title) ? "active" : "inactive"
+          const joined = new Set(segments.map((entry) => entry.id));
+          const tags = (owned || []).filter((entry) => entry.id && entry.title).map((entry) => ({
+            name: tagName(entry.title, entry.id),
+            status: joined.has(entry.id) ? "active" : "inactive"
           }));
           if (tags.length > 0) {
             await api3("/lists/" + audience + "/members/" + hash + "/tags", {
@@ -3975,6 +3992,7 @@ var mailchimp_default2 = {
           // Merged into `context` for later steps in this run.
           context: { mailchimpMemberId: (member == null ? void 0 : member.id) || hash },
           message: suppressed ? "Synced to Mailchimp as unsubscribed \u2014 this contact has opted out." : "Synced to the Mailchimp audience.",
+          request: request2,
           // Recorded on the run for support to read back, not a write
           // instruction — the hook has already written what it needed to.
           response: { mailchimpMemberId: (member == null ? void 0 : member.id) || hash }
@@ -4010,7 +4028,7 @@ var mailchimp_default2 = {
         if (!audience) return { message: "No Mailchimp audience is chosen for this connection.", skipped: true };
         const segment = await currentSegment({ read, segment: context == null ? void 0 : context.segment });
         if (!(segment == null ? void 0 : segment.id) || segment.system) return { message: "That segment is not one this connection publishes.", skipped: true };
-        const name = tagName(segment.title);
+        const name = tagName(segment.title, segment.id);
         const existing = segmentRowFor({ connection: connection2, segment });
         let id = null;
         if (existing == null ? void 0 : existing.id) {
@@ -4582,10 +4600,6 @@ var shopify_default2 = {
         if (!(context == null ? void 0 : context.lead)) return { message: "Lead id is missing.", request: request2, response: { skipped: true }, skipped: true };
         if (!(discount == null ? void 0 : discount.id)) return { message: "Discount is not configured on this step.", request: request2, response: { skipped: true }, skipped: true };
         const adminAccessToken = await adminToken();
-        if (!context.shopifyCustomerId) {
-          const customer = await shopify.admin.getOrCreateCustomer({ adminAccessToken, domain: connection2.shop, email: context.email });
-          if (!(customer == null ? void 0 : customer.id)) return { message: "Shopify did not return a customer id \u2014 create/lookup failed.", request: request2, response: { skipped: true }, skipped: true };
-        }
         const discountCode = await shopify.admin.createDiscountCode({
           adminAccessToken,
           code: "DB-" + generateDiscountCode(),
@@ -5029,7 +5043,7 @@ var shopify_default2 = {
       //
       // The shell has already refused a missing or inactive Shopify connection,
       // so what is left is the two things only this hook can know are wrong.
-      product: async ({ connection: connection2, context, workflow }, { mintId } = {}) => {
+      product: async ({ connection: connection2, context, workflow }, { mintId, read } = {}) => {
         const request2 = {
           numericId: (context == null ? void 0 : context.id) || null,
           organizationId: workflow.organization,
@@ -5038,7 +5052,13 @@ var shopify_default2 = {
         if (!(context == null ? void 0 : context.id)) return { message: "Skipped \u2014 product webhook payload had no id.", request: request2, response: { skipped: true }, skipped: true };
         if (!connection2.shop) return { message: "Skipped \u2014 Shopify connection is missing shop domain.", request: request2, response: { skipped: true }, skipped: true };
         const providerId = "gid://shopify/Product/" + context.id;
-        const productId = mintId();
+        const query = {
+          "provider.id": providerId,
+          "provider.slug": "shopify",
+          "source.domain": connection2.shop
+        };
+        const existing = (read == null ? void 0 : read.get) ? await read.get({ collection: "product", query }) : null;
+        const productId = (existing == null ? void 0 : existing.id) || mintId();
         return {
           enqueues: [{
             data: { product: productId, providerId, shop: connection2.shop },
@@ -5068,11 +5088,7 @@ var shopify_default2 = {
             },
             operation: "update",
             options: { upsert: true },
-            query: {
-              "provider.id": providerId,
-              "provider.slug": "shopify",
-              "source.domain": connection2.shop
-            }
+            query
           }]
         };
       }
@@ -5342,7 +5358,7 @@ var shopify_default2 = {
               options: { jobId: "connection.update.register." + workflow.connection + "." + (0, import_node_crypto6.randomUUID)() },
               queue: "connection"
             }],
-            message: ((scopesMissing == null ? void 0 : scopesMissing.length) ? "Health check: ping ok, webhooks reconciled \u2014 connection errored, granted scopes are missing: " + scopesMissing.join(", ") + "." : refreshTokenRotated ? "Health check passed \u2014 refresh token rotated, ping ok, webhooks reconciled." : "Health check passed \u2014 ping ok, webhooks reconciled.") + (sourceRepaired ? " Billing source repaired from the live shop." : "") + (metered === null ? " No usage line on the store's approved plan \u2014 order billing needs the merchant to approve the updated plan." : ""),
+            message: ((scopesMissing == null ? void 0 : scopesMissing.length) ? "Health check: ping ok, scopes reconciled \u2014 connection errored, granted scopes are missing: " + scopesMissing.join(", ") + "." : refreshTokenRotated ? "Health check passed \u2014 refresh token rotated, ping ok, scopes reconciled." : "Health check passed \u2014 ping ok, scopes reconciled.") + (sourceRepaired ? " Billing source repaired from the live shop." : "") + (metered === null ? " No usage line on the store's approved plan \u2014 order billing needs the merchant to approve the updated plan." : ""),
             request: request2,
             response: {
               billingProbeFailed,

package/dist/providers.js

@@ -478,6 +478,7 @@ var row = ({ connection: connection2, data: data2, manifest, row: described }) =
   };
 };
 var segmentRowWrites = ({ connection: connection2, data: data2, manifest, row: described, segment }) => {
+  if ((described == null ? void 0 : described.id) === void 0 || (described == null ? void 0 : described.id) === null) return [];
   const built = row({ connection: connection2, data: data2, manifest, row: described });
   return [
     // PUSH IF ABSENT. The $ne guard is what makes a second concurrent register
@@ -725,10 +726,11 @@ var attentive_default2 = {
       sync: async ({ lead, settings, suppressed, token }, { fetcher } = {}) => {
         var _a, _b, _c, _d;
         const segment = settings == null ? void 0 : settings.segment;
-        if (!segment) return { message: "No Attentive segment is chosen for this connection.", skipped: true };
         const email = ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email);
         const phone = toE164(((_d = (_c = lead == null ? void 0 : lead.canonical) == null ? void 0 : _c.phone) == null ? void 0 : _d.value) || (lead == null ? void 0 : lead.phone));
-        if (!email && !phone) return { message: "That lead has no email address or phone number to sync.", skipped: true };
+        const request2 = { email: email || null, phone: phone || null, segment: (settings == null ? void 0 : settings.segment) || null };
+        if (!segment) return { message: "No Attentive segment is chosen for this connection.", request: request2, skipped: true };
+        if (!email && !phone) return { message: "That lead has no email address or phone number to sync.", request: request2, skipped: true };
         const user = {
           ...email && { email },
           ...phone && { phone }
@@ -750,6 +752,7 @@ var attentive_default2 = {
           });
           return {
             message: "Attentive accepted an unsubscribe for this contact \u2014 they have opted out.",
+            request: request2,
             response: { accepted: true, unsubscribed: true }
           };
         }
@@ -790,6 +793,7 @@ var attentive_default2 = {
           // queued them — a merchant who reads "synced" and looks for the person
           // in Attentive a second later has been told the wrong thing.
           message: "Attentive accepted this contact for the segment. Attentive processes these asynchronously, so it appears there shortly.",
+          request: request2,
           response: {
             accepted: true,
             ...(membership == null ? void 0 : membership.batchJobId) && { batchJobId: membership.batchJobId }
@@ -2806,7 +2810,7 @@ var drawbridge_default2 = {
           text: { max: 2e3, type: "string" }
         },
         triggers: ["schedule.day", "schedule.week", "schedule.month"],
-        usage: { actions: 0 }
+        usage: { actions: channels.email.actionsPerSend }
       }),
       // To organization MEMBERS. Never suppressed — an entrant's opt-out must
       // not silence an alert to staff — and not billed.
@@ -2831,10 +2835,14 @@ var drawbridge_default2 = {
           subject: { max: 150, type: "string" }
         },
         triggers: ["lead.insert"],
-        // Zero is a PRICE, and a deliberate one. Declared rather than omitted
-        // so "this is free" and "nobody decided" stay different statements;
-        // completeStep bills only when actions > 0.
-        usage: { actions: 0 }
+        // EVERY STEP IN A BILLABLE WORKFLOW COUNTS, team mail included. This
+        // leaves our SendGrid account and costs the same to send as any other
+        // email, so it carries the same weight. Whether anything is charged at
+        // all is the workflow's `billable` tag, not the step's audience — and
+        // this step triggers on lead.insert, so a merchant notifying
+        // themselves on every entry spends against their allowance twice per
+        // lead, once for the entry and once for the notification.
+        usage: { actions: channels.email.actionsPerSend }
       }),
       // To a LEAD. Suppression applies and the send is billed.
       send: () => ({
@@ -2943,7 +2951,7 @@ var api2 = async (path, { fetcher = fetch, method = "GET", payload, token }) =>
   }
   return response.status === 204 ? null : response.json();
 };
-var segmentName = (title) => "Drawbridge: " + title;
+var segmentName = (title, id) => "Drawbridge: " + title + " (" + String(id).slice(-6) + ")";
 var canManageSegments = (settings) => String((settings == null ? void 0 : settings.scope) || "").split(/\s+/).includes("segments:write");
 var klaviyo_default2 = {
   // OAuth 2.1, and PKCE is REQUIRED rather than recommended: Klaviyo refuses an
@@ -3190,11 +3198,12 @@ var klaviyo_default2 = {
       // different thing from deleting the profile.
       remove: false,
       sync: async ({ context, lead, segments, settings, suppressed, token }, { fetcher } = {}) => {
-        var _a, _b, _c;
+        var _a, _b, _c, _d, _e;
         const list = settings == null ? void 0 : settings.list;
-        if (!list) return { message: "No Klaviyo list is chosen for this connection.", skipped: true };
-        const email = ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email);
-        if (!email) return { message: "That lead has no email address to sync.", skipped: true };
+        const request2 = { email: ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email) || null, list: (settings == null ? void 0 : settings.list) || null };
+        if (!list) return { message: "No Klaviyo list is chosen for this connection.", request: request2, skipped: true };
+        const email = ((_d = (_c = lead == null ? void 0 : lead.canonical) == null ? void 0 : _c.email) == null ? void 0 : _d.value) || (lead == null ? void 0 : lead.email);
+        if (!email) return { message: "That lead has no email address to sync.", request: request2, skipped: true };
         const person = (context == null ? void 0 : context.contact) || null;
         const totals = (person == null ? void 0 : person.totals) || {};
         const count = (value) => typeof value === "number" ? value : (value == null ? void 0 : value.total) || 0;
@@ -3251,8 +3260,8 @@ var klaviyo_default2 = {
           },
           token
         });
-        const profileId = (_c = profile == null ? void 0 : profile.data) == null ? void 0 : _c.id;
-        if (!profileId) return { message: "Klaviyo returned no profile id.", skipped: true };
+        const profileId = (_e = profile == null ? void 0 : profile.data) == null ? void 0 : _e.id;
+        if (!profileId) return { message: "Klaviyo returned no profile id.", request: request2, skipped: true };
         await api2("/profile-subscription-bulk-create-jobs/", {
           fetcher,
           method: "POST",
@@ -3281,6 +3290,7 @@ var klaviyo_default2 = {
           // Merged into `context` for later steps in this run.
           context: { klaviyoProfileId: profileId },
           message: suppressed ? "Synced to Klaviyo as unsubscribed \u2014 this contact has opted out." : "Synced to the Klaviyo list.",
+          request: request2,
           // Recorded on the run for support to read back, not a write
           // instruction — the hook has already written what it needed to.
           response: { klaviyoProfileId: profileId }
@@ -3316,7 +3326,7 @@ var klaviyo_default2 = {
             skipped: true
           };
         }
-        const name = segmentName(segment.title);
+        const name = segmentName(segment.title, segment.id);
         const existing = segmentRowFor({ connection: connection2, segment });
         let id = null;
         if (existing == null ? void 0 : existing.id) {
@@ -3723,7 +3733,10 @@ var api3 = async (path, { dc, fetcher = fetch, method = "GET", payload, token })
 };
 var subscriberHash = (email) => createHash2("md5").update(String(email).trim().toLowerCase()).digest("hex");
 var TAG_NAME_LIMIT = 100;
-var tagName = (title) => ("Drawbridge: " + title).slice(0, TAG_NAME_LIMIT);
+var tagName = (title, id) => {
+  const suffix = " (" + String(id).slice(-6) + ")";
+  return ("Drawbridge: " + title).slice(0, TAG_NAME_LIMIT - suffix.length) + suffix;
+};
 var mailchimp_default2 = {
   // OAUTH 2, authorization code. Every url below is quoted from
   // mailchimp.com/developer/marketing/guides/access-user-data-oauth-2/ rather
@@ -3861,15 +3874,16 @@ var mailchimp_default2 = {
       // why there is no create-or-update branch here. Quoted from Mailchimp's
       // Marketing API reference for the list-members resource.
       sync: async ({ connection: connection2, lead, segments, settings, suppressed, token }, { fetcher, read } = {}) => {
-        var _a, _b, _c;
+        var _a, _b, _c, _d, _e;
         const audience = settings == null ? void 0 : settings.audience;
-        if (!audience) return { message: "No Mailchimp audience is chosen for this connection.", skipped: true };
-        const email = ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email);
-        if (!email) return { message: "That lead has no email address to sync.", skipped: true };
+        const request2 = { audience: (settings == null ? void 0 : settings.audience) || null, email: ((_b = (_a = lead == null ? void 0 : lead.canonical) == null ? void 0 : _a.email) == null ? void 0 : _b.value) || (lead == null ? void 0 : lead.email) || null };
+        if (!audience) return { message: "No Mailchimp audience is chosen for this connection.", request: request2, skipped: true };
+        const email = ((_d = (_c = lead == null ? void 0 : lead.canonical) == null ? void 0 : _c.email) == null ? void 0 : _d.value) || (lead == null ? void 0 : lead.email);
+        if (!email) return { message: "That lead has no email address to sync.", request: request2, skipped: true };
         const hash = subscriberHash(email);
         const [firstName, ...restOfName] = String((lead == null ? void 0 : lead.name) || "").trim().split(/\s+/).filter(Boolean);
         const lastName = restOfName.join(" ");
-        const phone = ((_c = lead == null ? void 0 : lead.phone) == null ? void 0 : _c.number) || null;
+        const phone = ((_e = lead == null ? void 0 : lead.phone) == null ? void 0 : _e.number) || null;
         const mergeFields = {
           ...firstName && { FNAME: firstName },
           ...lastName && { LNAME: lastName },
@@ -3903,14 +3917,17 @@ var mailchimp_default2 = {
                 }
               },
               {
-                $project: { _id: 0, title: 1 }
+                // THE ID AS WELL AS THE TITLE, because the tag name carries it —
+                // see tagName. Without it every name here would end in the
+                // string 'undefined' and match nothing register wrote.
+                $project: { _id: 0, id: 1, title: 1 }
               }
             ]
           });
-          const joined = new Set(segments.map((entry) => entry.title));
-          const tags = (owned || []).map((entry) => entry.title).filter(Boolean).map((title) => ({
-            name: tagName(title),
-            status: joined.has(title) ? "active" : "inactive"
+          const joined = new Set(segments.map((entry) => entry.id));
+          const tags = (owned || []).filter((entry) => entry.id && entry.title).map((entry) => ({
+            name: tagName(entry.title, entry.id),
+            status: joined.has(entry.id) ? "active" : "inactive"
           }));
           if (tags.length > 0) {
             await api3("/lists/" + audience + "/members/" + hash + "/tags", {
@@ -3926,6 +3943,7 @@ var mailchimp_default2 = {
           // Merged into `context` for later steps in this run.
           context: { mailchimpMemberId: (member == null ? void 0 : member.id) || hash },
           message: suppressed ? "Synced to Mailchimp as unsubscribed \u2014 this contact has opted out." : "Synced to the Mailchimp audience.",
+          request: request2,
           // Recorded on the run for support to read back, not a write
           // instruction — the hook has already written what it needed to.
           response: { mailchimpMemberId: (member == null ? void 0 : member.id) || hash }
@@ -3961,7 +3979,7 @@ var mailchimp_default2 = {
         if (!audience) return { message: "No Mailchimp audience is chosen for this connection.", skipped: true };
         const segment = await currentSegment({ read, segment: context == null ? void 0 : context.segment });
         if (!(segment == null ? void 0 : segment.id) || segment.system) return { message: "That segment is not one this connection publishes.", skipped: true };
-        const name = tagName(segment.title);
+        const name = tagName(segment.title, segment.id);
         const existing = segmentRowFor({ connection: connection2, segment });
         let id = null;
         if (existing == null ? void 0 : existing.id) {
@@ -4533,10 +4551,6 @@ var shopify_default2 = {
         if (!(context == null ? void 0 : context.lead)) return { message: "Lead id is missing.", request: request2, response: { skipped: true }, skipped: true };
         if (!(discount == null ? void 0 : discount.id)) return { message: "Discount is not configured on this step.", request: request2, response: { skipped: true }, skipped: true };
         const adminAccessToken = await adminToken();
-        if (!context.shopifyCustomerId) {
-          const customer = await shopify.admin.getOrCreateCustomer({ adminAccessToken, domain: connection2.shop, email: context.email });
-          if (!(customer == null ? void 0 : customer.id)) return { message: "Shopify did not return a customer id \u2014 create/lookup failed.", request: request2, response: { skipped: true }, skipped: true };
-        }
         const discountCode = await shopify.admin.createDiscountCode({
           adminAccessToken,
           code: "DB-" + generateDiscountCode(),
@@ -4980,7 +4994,7 @@ var shopify_default2 = {
       //
       // The shell has already refused a missing or inactive Shopify connection,
       // so what is left is the two things only this hook can know are wrong.
-      product: async ({ connection: connection2, context, workflow }, { mintId } = {}) => {
+      product: async ({ connection: connection2, context, workflow }, { mintId, read } = {}) => {
         const request2 = {
           numericId: (context == null ? void 0 : context.id) || null,
           organizationId: workflow.organization,
@@ -4989,7 +5003,13 @@ var shopify_default2 = {
         if (!(context == null ? void 0 : context.id)) return { message: "Skipped \u2014 product webhook payload had no id.", request: request2, response: { skipped: true }, skipped: true };
         if (!connection2.shop) return { message: "Skipped \u2014 Shopify connection is missing shop domain.", request: request2, response: { skipped: true }, skipped: true };
         const providerId = "gid://shopify/Product/" + context.id;
-        const productId = mintId();
+        const query = {
+          "provider.id": providerId,
+          "provider.slug": "shopify",
+          "source.domain": connection2.shop
+        };
+        const existing = (read == null ? void 0 : read.get) ? await read.get({ collection: "product", query }) : null;
+        const productId = (existing == null ? void 0 : existing.id) || mintId();
         return {
           enqueues: [{
             data: { product: productId, providerId, shop: connection2.shop },
@@ -5019,11 +5039,7 @@ var shopify_default2 = {
             },
             operation: "update",
             options: { upsert: true },
-            query: {
-              "provider.id": providerId,
-              "provider.slug": "shopify",
-              "source.domain": connection2.shop
-            }
+            query
           }]
         };
       }
@@ -5293,7 +5309,7 @@ var shopify_default2 = {
               options: { jobId: "connection.update.register." + workflow.connection + "." + randomUUID2() },
               queue: "connection"
             }],
-            message: ((scopesMissing == null ? void 0 : scopesMissing.length) ? "Health check: ping ok, webhooks reconciled \u2014 connection errored, granted scopes are missing: " + scopesMissing.join(", ") + "." : refreshTokenRotated ? "Health check passed \u2014 refresh token rotated, ping ok, webhooks reconciled." : "Health check passed \u2014 ping ok, webhooks reconciled.") + (sourceRepaired ? " Billing source repaired from the live shop." : "") + (metered === null ? " No usage line on the store's approved plan \u2014 order billing needs the merchant to approve the updated plan." : ""),
+            message: ((scopesMissing == null ? void 0 : scopesMissing.length) ? "Health check: ping ok, scopes reconciled \u2014 connection errored, granted scopes are missing: " + scopesMissing.join(", ") + "." : refreshTokenRotated ? "Health check passed \u2014 refresh token rotated, ping ok, scopes reconciled." : "Health check passed \u2014 ping ok, scopes reconciled.") + (sourceRepaired ? " Billing source repaired from the live shop." : "") + (metered === null ? " No usage line on the store's approved plan \u2014 order billing needs the merchant to approve the updated plan." : ""),
             request: request2,
             response: {
               billingProbeFailed,

package/package.json

@@ -13,7 +13,7 @@
     "tinycolor2": "1.6.0"
   },
   "devDependencies": {
-    "@drawbridge/drawbridge-agents": "0.1.47",
+    "@drawbridge/drawbridge-agents": "0.1.59",
     "@node-oauth/oauth2-server": "5.3.0",
     "tsup": "8.5.1",
     "typescript": "5.9.3",
@@ -226,5 +226,5 @@
     "prepublishOnly": ". \"$HOME/.nvm/nvm.sh\" && nvm use && tsup && node --test"
   },
   "types": "dist/index.d.ts",
-  "version": "0.0.170"
+  "version": "0.0.172"
 }