@drawbridge/drawbridge-utils 0.0.171 → 0.0.173

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
@@ -2076,16 +2077,16 @@ var deliveryWrite = ({ at, code: code2, notification, permanent, provider, reaso
   };
 };
 var summariseEvents = (events) => {
-  const list = Array.isArray(events) ? events : [];
-  if (!list.length) return { message: "SendGrid delivered an empty event batch.", skipped: true };
+  const list2 = Array.isArray(events) ? events : [];
+  if (!list2.length) return { message: "SendGrid delivered an empty event batch.", skipped: true };
   const counts = {};
-  for (const { event } of list) {
+  for (const { event } of list2) {
     const name = event || "unknown";
     counts[name] = (counts[name] || 0) + 1;
   }
   const tally = Object.entries(counts).map(([name, count]) => count + " " + name).join(", ");
-  const refused = list.filter(({ event }) => REFUSALS.has(event)).slice(0, 5).map(({ email, reason, response, status }) => email + " \u2014 " + (reason || response || status || "no reason given"));
-  const writes = list.map((event) => {
+  const refused = list2.filter(({ event }) => REFUSALS.has(event)).slice(0, 5).map(({ email, reason, response, status }) => email + " \u2014 " + (reason || response || status || "no reason given"));
+  const writes = list2.map((event) => {
     var _a;
     return deliveryWrite({
       at: event.timestamp ? new Date(event.timestamp * 1e3) : void 0,
@@ -2105,7 +2106,7 @@ var summariseEvents = (events) => {
     });
   }).filter(Boolean);
   return {
-    message: [list.length + " SendGrid event(s): " + tally, ...refused].join(" | "),
+    message: [list2.length + " SendGrid event(s): " + tally, ...refused].join(" | "),
     ...writes.length ? { writes } : { skipped: true }
   };
 };
@@ -2972,6 +2973,14 @@ var klaviyo_default = `<svg width="500" height="500" viewBox="0 0 500 500" fill=
 <path d="M365.047 327.038H134.954V172.964H365.047L316.856 250.001L365.047 327.038Z" fill="#232121"/>
 </svg>`;
 
+// lib/connections/scopes.js
+var list = (value) => (Array.isArray(value) ? value.flatMap((entry) => String(entry).split(/\s+/)) : String(value ?? "").split(/\s+/)).map((entry) => entry.trim()).filter(Boolean);
+var missingScopes = ({ granted, required }) => {
+  if (granted === null || granted === void 0 || granted === "") return null;
+  const held = new Set(list(granted));
+  return list(required).filter((scope) => !held.has(scope));
+};
+
 // lib/connections/providers/klaviyo.js
 var api2 = async (path, { fetcher = fetch, method = "GET", payload, token }) => {
   const response = await fetcher("https://a.klaviyo.com/api" + path, {
@@ -2999,8 +3008,10 @@ var api2 = async (path, { fetcher = fetch, method = "GET", payload, token }) =>
   }
   return response.status === 204 ? null : response.json();
 };
-var segmentName = (title) => "Drawbridge: " + title;
-var canManageSegments = (settings) => String((settings == null ? void 0 : settings.scope) || "").split(/\s+/).includes("segments:write");
+var segmentName = (title, id) => "Drawbridge: " + title + " (" + String(id).slice(-6) + ")";
+var SCOPES = "accounts:read lists:read lists:write profiles:read profiles:write segments:read segments:write";
+var missing = (settings) => missingScopes({ granted: settings == null ? void 0 : settings.scope, required: SCOPES });
+var canManageSegments = (settings) => !(missing(settings) || []).includes("segments:write");
 var klaviyo_default2 = {
   // OAuth 2.1, and PKCE is REQUIRED rather than recommended: Klaviyo refuses an
   // exchange without a code_verifier matching the challenge the consent
@@ -3044,7 +3055,7 @@ var klaviyo_default2 = {
       // Update and Delete Segment each list `segments:write`
       // (raw.githubusercontent.com/klaviyo/openapi/main/openapi/stable.json,
       // revision 2026-07-15, fetched 2026-09-11).
-      scopes: "accounts:read lists:read lists:write profiles:read profiles:write segments:read segments:write",
+      scopes: SCOPES,
       // EVERY VENDOR URL, in one place. `revoke` used to be a literal inside
       // the disconnect hook — three vendor addresses, two of them declared,
       // which is exactly the kind of split that goes unnoticed.
@@ -3225,8 +3236,14 @@ var klaviyo_default2 = {
         });
         return { ok: Boolean(token) };
       },
-      // Klaviyo scopes are fixed at app level and re-consented, not drifted.
-      scopes: false,
+      // THEY DO DRIFT, and the comment here used to say they could not. Klaviyo
+      // scopes are fixed on the APP, so adding one re-consents every NEW grant
+      // and leaves every EXISTING one exactly as narrow as it was — with no
+      // error, no webhook, and nothing that notices. That is what left grants
+      // authenticating perfectly while silently unable to manage a segment.
+      //
+      // The same slot Shopify answers, so one caller can ask any vendor.
+      scopes: ({ scope }) => missingScopes({ granted: scope, required: SCOPES }),
       // KLAVIYO REQUIRES HTTP BASIC on the token endpoint and rejects the same
       // client_id/client_secret pair as body fields. Everything else about the
       // request is standard, so this is the shared implementation told the one
@@ -3247,9 +3264,9 @@ var klaviyo_default2 = {
       remove: false,
       sync: async ({ context, lead, segments, settings, suppressed, token }, { fetcher } = {}) => {
         var _a, _b, _c, _d, _e;
-        const list = settings == null ? void 0 : settings.list;
+        const list2 = settings == null ? void 0 : settings.list;
         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 };
+        if (!list2) 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;
@@ -3328,7 +3345,7 @@ var klaviyo_default2 = {
                   }]
                 }
               },
-              relationships: { list: { data: { id: list, type: "list" } } },
+              relationships: { list: { data: { id: list2, type: "list" } } },
               type: "profile-subscription-bulk-create-job"
             }
           },
@@ -3374,7 +3391,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) {
@@ -3501,7 +3518,7 @@ var klaviyo_default2 = {
       // The one call below proves the minted token is HONOURED — mint and
       // acceptance are different facts, and /accounts is already the call
       // the connect flow makes (auth.connect), so it needs no new scope.
-      health: async ({ connection: connection2, token }, { fetcher, read } = {}) => {
+      health: async ({ connection: connection2, settings, token }, { fetcher, read } = {}) => {
         const request2 = { connectionId: connection2.id };
         try {
           await api2("/accounts", { fetcher, token });
@@ -3553,8 +3570,8 @@ var klaviyo_default2 = {
         let pages = 0;
         while (next && audiences.length < limit && pages < 20) {
           const body = await api2(next, { fetcher, token });
-          for (const list of (body == null ? void 0 : body.data) || []) {
-            audiences.push({ id: list.id, title: ((_a = list == null ? void 0 : list.attributes) == null ? void 0 : _a.name) || list.id });
+          for (const list2 of (body == null ? void 0 : body.data) || []) {
+            audiences.push({ id: list2.id, title: ((_a = list2 == null ? void 0 : list2.attributes) == null ? void 0 : _a.name) || list2.id });
           }
           const link = (_b = body == null ? void 0 : body.links) == null ? void 0 : _b.next;
           next = link ? String(link).replace(/^https:\/\/a\.klaviyo\.com\/api/, "") : null;
@@ -3720,13 +3737,14 @@ var klaviyo_default2 = {
     var _a;
     if (!["active", "pending"].includes(data2 == null ? void 0 : data2.status)) return [];
     return [
-      // A connection made before segments were requested is authenticated and
-      // cannot manage them, and no error surfaces anywhere else — the register
-      // runs skip rather than fail.
-      ...canManageSegments(data2 == null ? void 0 : data2.settings) ? [] : [{
-        message: "Drawbridge now keeps a Klaviyo segment in step with each of your Drawbridge segments. Reconnect Klaviyo to allow it.",
-        title: "Reconnect Klaviyo"
-      }],
+      // THE MISSING SEGMENT SCOPE IS NOT HERE ANY MORE. It is an ERROR entry,
+      // written by the health check, because a task is quiet: it renders only in
+      // the body of this connection's own page, so a merchant who never opens it
+      // never learns that their segments stopped being published. An error entry
+      // reaches the card and the organization checklist too.
+      //
+      // Deliberately not a status change — see the health hook for why that
+      // would stop the contact sync that still works.
       ...((_a = data2 == null ? void 0 : data2.settings) == null ? void 0 : _a.list) ? [] : [{
         message: "Choose which Klaviyo list your contacts should sync into. Until you do, nothing is being synced.",
         title: "Choose a list"
@@ -3781,7 +3799,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
@@ -3962,14 +3983,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", {
@@ -4021,7 +4045,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) {
@@ -4127,7 +4151,7 @@ var mailchimp_default2 = {
           "/lists?count=" + count + "&offset=" + offset + "&fields=lists.id,lists.name,total_items",
           { dc: settings == null ? void 0 : settings.dc, fetcher, token }
         );
-        const audiences = ((body == null ? void 0 : body.lists) || []).map((list) => ({ id: list.id, title: (list == null ? void 0 : list.name) || list.id }));
+        const audiences = ((body == null ? void 0 : body.lists) || []).map((list2) => ({ id: list2.id, title: (list2 == null ? void 0 : list2.name) || list2.id }));
         const term = String((search == null ? void 0 : search.value) || "").trim().toLowerCase();
         const items = term ? audiences.filter((entry) => entry.title.toLowerCase().includes(term)) : audiences;
         const nextOffset = offset + count;
@@ -4593,10 +4617,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(),

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
@@ -2027,16 +2028,16 @@ var deliveryWrite = ({ at, code: code2, notification, permanent, provider, reaso
   };
 };
 var summariseEvents = (events) => {
-  const list = Array.isArray(events) ? events : [];
-  if (!list.length) return { message: "SendGrid delivered an empty event batch.", skipped: true };
+  const list2 = Array.isArray(events) ? events : [];
+  if (!list2.length) return { message: "SendGrid delivered an empty event batch.", skipped: true };
   const counts = {};
-  for (const { event } of list) {
+  for (const { event } of list2) {
     const name = event || "unknown";
     counts[name] = (counts[name] || 0) + 1;
   }
   const tally = Object.entries(counts).map(([name, count]) => count + " " + name).join(", ");
-  const refused = list.filter(({ event }) => REFUSALS.has(event)).slice(0, 5).map(({ email, reason, response, status }) => email + " \u2014 " + (reason || response || status || "no reason given"));
-  const writes = list.map((event) => {
+  const refused = list2.filter(({ event }) => REFUSALS.has(event)).slice(0, 5).map(({ email, reason, response, status }) => email + " \u2014 " + (reason || response || status || "no reason given"));
+  const writes = list2.map((event) => {
     var _a;
     return deliveryWrite({
       at: event.timestamp ? new Date(event.timestamp * 1e3) : void 0,
@@ -2056,7 +2057,7 @@ var summariseEvents = (events) => {
     });
   }).filter(Boolean);
   return {
-    message: [list.length + " SendGrid event(s): " + tally, ...refused].join(" | "),
+    message: [list2.length + " SendGrid event(s): " + tally, ...refused].join(" | "),
     ...writes.length ? { writes } : { skipped: true }
   };
 };
@@ -2923,6 +2924,14 @@ var klaviyo_default = `<svg width="500" height="500" viewBox="0 0 500 500" fill=
 <path d="M365.047 327.038H134.954V172.964H365.047L316.856 250.001L365.047 327.038Z" fill="#232121"/>
 </svg>`;
 
+// lib/connections/scopes.js
+var list = (value) => (Array.isArray(value) ? value.flatMap((entry) => String(entry).split(/\s+/)) : String(value ?? "").split(/\s+/)).map((entry) => entry.trim()).filter(Boolean);
+var missingScopes = ({ granted, required }) => {
+  if (granted === null || granted === void 0 || granted === "") return null;
+  const held = new Set(list(granted));
+  return list(required).filter((scope) => !held.has(scope));
+};
+
 // lib/connections/providers/klaviyo.js
 var api2 = async (path, { fetcher = fetch, method = "GET", payload, token }) => {
   const response = await fetcher("https://a.klaviyo.com/api" + path, {
@@ -2950,8 +2959,10 @@ var api2 = async (path, { fetcher = fetch, method = "GET", payload, token }) =>
   }
   return response.status === 204 ? null : response.json();
 };
-var segmentName = (title) => "Drawbridge: " + title;
-var canManageSegments = (settings) => String((settings == null ? void 0 : settings.scope) || "").split(/\s+/).includes("segments:write");
+var segmentName = (title, id) => "Drawbridge: " + title + " (" + String(id).slice(-6) + ")";
+var SCOPES = "accounts:read lists:read lists:write profiles:read profiles:write segments:read segments:write";
+var missing = (settings) => missingScopes({ granted: settings == null ? void 0 : settings.scope, required: SCOPES });
+var canManageSegments = (settings) => !(missing(settings) || []).includes("segments:write");
 var klaviyo_default2 = {
   // OAuth 2.1, and PKCE is REQUIRED rather than recommended: Klaviyo refuses an
   // exchange without a code_verifier matching the challenge the consent
@@ -2995,7 +3006,7 @@ var klaviyo_default2 = {
       // Update and Delete Segment each list `segments:write`
       // (raw.githubusercontent.com/klaviyo/openapi/main/openapi/stable.json,
       // revision 2026-07-15, fetched 2026-09-11).
-      scopes: "accounts:read lists:read lists:write profiles:read profiles:write segments:read segments:write",
+      scopes: SCOPES,
       // EVERY VENDOR URL, in one place. `revoke` used to be a literal inside
       // the disconnect hook — three vendor addresses, two of them declared,
       // which is exactly the kind of split that goes unnoticed.
@@ -3176,8 +3187,14 @@ var klaviyo_default2 = {
         });
         return { ok: Boolean(token) };
       },
-      // Klaviyo scopes are fixed at app level and re-consented, not drifted.
-      scopes: false,
+      // THEY DO DRIFT, and the comment here used to say they could not. Klaviyo
+      // scopes are fixed on the APP, so adding one re-consents every NEW grant
+      // and leaves every EXISTING one exactly as narrow as it was — with no
+      // error, no webhook, and nothing that notices. That is what left grants
+      // authenticating perfectly while silently unable to manage a segment.
+      //
+      // The same slot Shopify answers, so one caller can ask any vendor.
+      scopes: ({ scope }) => missingScopes({ granted: scope, required: SCOPES }),
       // KLAVIYO REQUIRES HTTP BASIC on the token endpoint and rejects the same
       // client_id/client_secret pair as body fields. Everything else about the
       // request is standard, so this is the shared implementation told the one
@@ -3198,9 +3215,9 @@ var klaviyo_default2 = {
       remove: false,
       sync: async ({ context, lead, segments, settings, suppressed, token }, { fetcher } = {}) => {
         var _a, _b, _c, _d, _e;
-        const list = settings == null ? void 0 : settings.list;
+        const list2 = settings == null ? void 0 : settings.list;
         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 };
+        if (!list2) 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;
@@ -3279,7 +3296,7 @@ var klaviyo_default2 = {
                   }]
                 }
               },
-              relationships: { list: { data: { id: list, type: "list" } } },
+              relationships: { list: { data: { id: list2, type: "list" } } },
               type: "profile-subscription-bulk-create-job"
             }
           },
@@ -3325,7 +3342,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) {
@@ -3452,7 +3469,7 @@ var klaviyo_default2 = {
       // The one call below proves the minted token is HONOURED — mint and
       // acceptance are different facts, and /accounts is already the call
       // the connect flow makes (auth.connect), so it needs no new scope.
-      health: async ({ connection: connection2, token }, { fetcher, read } = {}) => {
+      health: async ({ connection: connection2, settings, token }, { fetcher, read } = {}) => {
         const request2 = { connectionId: connection2.id };
         try {
           await api2("/accounts", { fetcher, token });
@@ -3504,8 +3521,8 @@ var klaviyo_default2 = {
         let pages = 0;
         while (next && audiences.length < limit && pages < 20) {
           const body = await api2(next, { fetcher, token });
-          for (const list of (body == null ? void 0 : body.data) || []) {
-            audiences.push({ id: list.id, title: ((_a = list == null ? void 0 : list.attributes) == null ? void 0 : _a.name) || list.id });
+          for (const list2 of (body == null ? void 0 : body.data) || []) {
+            audiences.push({ id: list2.id, title: ((_a = list2 == null ? void 0 : list2.attributes) == null ? void 0 : _a.name) || list2.id });
           }
           const link = (_b = body == null ? void 0 : body.links) == null ? void 0 : _b.next;
           next = link ? String(link).replace(/^https:\/\/a\.klaviyo\.com\/api/, "") : null;
@@ -3671,13 +3688,14 @@ var klaviyo_default2 = {
     var _a;
     if (!["active", "pending"].includes(data2 == null ? void 0 : data2.status)) return [];
     return [
-      // A connection made before segments were requested is authenticated and
-      // cannot manage them, and no error surfaces anywhere else — the register
-      // runs skip rather than fail.
-      ...canManageSegments(data2 == null ? void 0 : data2.settings) ? [] : [{
-        message: "Drawbridge now keeps a Klaviyo segment in step with each of your Drawbridge segments. Reconnect Klaviyo to allow it.",
-        title: "Reconnect Klaviyo"
-      }],
+      // THE MISSING SEGMENT SCOPE IS NOT HERE ANY MORE. It is an ERROR entry,
+      // written by the health check, because a task is quiet: it renders only in
+      // the body of this connection's own page, so a merchant who never opens it
+      // never learns that their segments stopped being published. An error entry
+      // reaches the card and the organization checklist too.
+      //
+      // Deliberately not a status change — see the health hook for why that
+      // would stop the contact sync that still works.
       ...((_a = data2 == null ? void 0 : data2.settings) == null ? void 0 : _a.list) ? [] : [{
         message: "Choose which Klaviyo list your contacts should sync into. Until you do, nothing is being synced.",
         title: "Choose a list"
@@ -3732,7 +3750,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
@@ -3913,14 +3934,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", {
@@ -3972,7 +3996,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) {
@@ -4078,7 +4102,7 @@ var mailchimp_default2 = {
           "/lists?count=" + count + "&offset=" + offset + "&fields=lists.id,lists.name,total_items",
           { dc: settings == null ? void 0 : settings.dc, fetcher, token }
         );
-        const audiences = ((body == null ? void 0 : body.lists) || []).map((list) => ({ id: list.id, title: (list == null ? void 0 : list.name) || list.id }));
+        const audiences = ((body == null ? void 0 : body.lists) || []).map((list2) => ({ id: list2.id, title: (list2 == null ? void 0 : list2.name) || list2.id }));
         const term = String((search == null ? void 0 : search.value) || "").trim().toLowerCase();
         const items = term ? audiences.filter((entry) => entry.title.toLowerCase().includes(term)) : audiences;
         const nextOffset = offset + count;
@@ -4544,10 +4568,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(),

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.171"
+  "version": "0.0.173"
 }