@drawbridge/drawbridge-utils 0.0.171 → 0.0.172

package/dist/connections/index.cjs

@@ -618,6 +618,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
@@ -3090,7 +3091,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
@@ -3465,7 +3466,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) {
@@ -3872,7 +3873,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
@@ -4053,14 +4057,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", {
@@ -4112,7 +4119,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) {
@@ -4665,10 +4672,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/connections/index.d.cts

@@ -854,6 +854,13 @@ const row = ({ connection, data, manifest, row : described }) => ({
 
 const segmentRowWrites = ({ connection, data, manifest, row : described, segment }) => {
 
+	// NO ID, NO ROW. `String( undefined )` is the string 'undefined', which is a
+	// perfectly valid string as far as the api's schema is concerned — so a row
+	// built without a vendor id would store and then render as a live-looking
+	// link to an object that does not exist. The three hooks all guard before
+	// they get here; this is the guard that does not depend on them remembering.
+	if( described?.id === undefined || described?.id === null ) return [];
+
 	const built = row({ connection, data, manifest, row : described });
 
 	return [
@@ -3531,7 +3538,30 @@ const api$1 = async ( path, { fetcher = fetch, method = 'GET', payload, token }
 
 // The segment's name at Klaviyo, spelled once. It is a LABEL — the definition
 // below keys on the id — so a rename never has to touch a profile.
-const segmentName = ( title ) => 'Drawbridge: ' + title;
+//
+// THE ID IS IN THE LABEL TOO, and that part is not cosmetic. Nothing makes a
+// THE ID IS IN THE NAME BECAUSE THE NAME IS NOT AN IDENTITY. Keyed on title
+// alone, a second segment called "VIP" found the first one's Klaviyo segment by
+// name and adopted it: both rows then pointed at one segment whose definition
+// computes the FIRST one's membership, so the merchant sent to the wrong people,
+// and deleting either segment deleted that object out from under the other with
+// nothing to repair it.
+//
+// The api now carries a unique index on (organization, title) and a 409 on both
+// create and rename, so that pair can no longer be created. The suffix stays
+// anyway: it is what makes the damage impossible rather than merely unlikely,
+// and it is the only part of this that holds if a title ever reaches Klaviyo
+// from somewhere that did not go through the route.
+//
+// KLAVIYO IS NOT DOING THIS FOR US. Create Segment describes `name` only as "A
+// helpful name to label the segment" and documents no uniqueness constraint and
+// no duplicate-name error (developers.klaviyo.com/en/reference/create_segment,
+// fetched 2026-09-12), so the search cannot lean on the vendor. What IS
+// documented is that filter comparisons are case-sensitive
+// (developers.klaviyo.com/en/docs/filtering_, same date), which is why the
+// find-by-name below re-checks with === and agrees with the vendor rather than
+// guessing.
+const segmentName = ( title, id ) => 'Drawbridge: ' + title + ' (' + String( id ).slice( -6 ) + ')';
 
 // THE GRANT THIS NEEDS. Klaviyo's scopes are set on the app and a token carries
 // only what the merchant consented to, so a connection made before segments were
@@ -4047,7 +4077,7 @@ var klaviyo = {
 
 				}
 
-				const name = segmentName( segment.title );
+				const name = segmentName( segment.title, segment.id );
 				const existing = segmentRowFor({ connection, segment });
 
 				let id = null;
@@ -4657,13 +4687,29 @@ const subscriberHash = ( email ) => createHash( 'md5' )
 // server-side truncation is the worse outcome because register would then look
 // for a name Mailchimp had silently changed.
 //
-// ponytail: two segments whose first 88 characters match would share one tag.
-// The upgrade is to append a short hash of the segment id, which costs the name
-// its readability in the merchant's own audience — not worth it until somebody
-// actually collides.
+// THE SEGMENT ID IS IN THE NAME, and it has to be. Nothing makes a Drawbridge
+// segment title unique — no index on the collection, no check in the route, no
+// check in the form — so two segments called "VIP" are ordinary. Keyed on title
+// alone, the second one's register finds the first one's tag, adopts it, and
+// both rows point at one object: the merchant's two segments are one tag, and
+// deleting either segment deletes the tag out from under the other, leaving a
+// row pointing at an id that no longer exists and nothing to repair it.
+//
+// The last six characters of the id are enough to separate them and short
+// enough to leave the title readable in the merchant's own audience.
+//
+// A rename now changes the name and keeps the suffix, so register's find-by-name
+// still matches after one — and the truncation below cuts the TITLE rather than
+// the suffix for the same reason.
 const TAG_NAME_LIMIT = 100;
 
-const tagName = ( title ) => ( 'Drawbridge: ' + title ).slice( 0, TAG_NAME_LIMIT );
+const tagName = ( title, id ) => {
+
+	const suffix = ' (' + String( id ).slice( -6 ) + ')';
+
+	return ( 'Drawbridge: ' + title ).slice( 0, TAG_NAME_LIMIT - suffix.length ) + suffix;
+
+};
 
 // Mailchimp — contact sync, not a sender.
 var mailchimp = {
@@ -4958,19 +5004,21 @@ var mailchimp = {
 								}
 							},
 							{
-								$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 joined = new Set( segments.map( ( entry ) => entry.id ) );
 
 					const tags = ( owned || [] )
-						.map( ( entry ) => entry.title )
-						.filter( Boolean )
-						.map( ( title ) => ({
-							name : tagName( title ),
-							status : joined.has( title ) ? 'active' : 'inactive'
+						.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 ){
@@ -5040,7 +5088,7 @@ var mailchimp = {
 
 				if( ! 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, segment });
 
 				let id = null;
@@ -5768,23 +5816,33 @@ var shopify = {
 
 				const request = { email : context?.email || null, lead : context?.lead || null, shop : connection.shop };
 
+				// KEPT even though nothing below sends the address anywhere. A code
+				// that reaches nobody is a wasted action and a merchant's wasted
+				// allowance: issuing is only half a workflow, and the email step after
+				// it needs somewhere to send the code.
 				if( ! context?.email ) return { message : 'Lead email is missing.', request, response : { skipped : true }, skipped : true };
 				if( ! context?.lead ) return { message : 'Lead id is missing.', request, response : { skipped : true }, skipped : true };
 				if( ! discount?.id ) return { message : 'Discount is not configured on this step.', request, response : { skipped : true }, skipped : true };
 
 				const adminAccessToken = await adminToken();
 
-				// The customer must exist before a code is mapped to them. An earlier
-				// commerce.customer step usually did this and left the id on the
-				// context; when this step runs alone, it does it here.
-				if( ! context.shopifyCustomerId ){
-
-					const customer = await shopify.admin.getOrCreateCustomer({ adminAccessToken, domain : connection.shop, email : context.email });
-
-					if( ! customer?.id ) return { message : 'Shopify did not return a customer id — create/lookup failed.', request, response : { skipped : true }, skipped : true };
-
-				}
-
+				// NOTHING IS SENT TO SHOPIFY TO LINK THE CODE TO THE PERSON, because
+				// there is nothing to send: discountRedeemCodeBulkAdd takes a discount
+				// and a list of codes, and no customer
+				// (shopify.dev/docs/api/admin-graphql/2026-04/mutations/discountRedeemCodeBulkAdd).
+				// The link lives entirely on our side — `lead.shopifyDiscountCode`
+				// below — and attribution happens when the order comes back carrying
+				// the code, matched org-scoped against that field by commerce.order.
+				//
+				// This step used to call getOrCreateCustomer first, on the stated
+				// reasoning that "the customer must exist before a code is mapped to
+				// them". No code was ever mapped to them: the result was assigned and
+				// discarded, createDiscountCode never received it, and redemption
+				// attribution never consulted it. All it did was create a customer in
+				// the merchant's store as a side effect of issuing a discount — which
+				// QA reported as surprising, and was right to. A merchant who wants the
+				// buyer to exist at the store adds the Create customer step, which is
+				// what that step is for.
 				const discountCode = await shopify.admin.createDiscountCode({
 					adminAccessToken,
 					code : 'DB-' + generateDiscountCode(),

package/dist/connections/index.d.ts

@@ -854,6 +854,13 @@ const row = ({ connection, data, manifest, row : described }) => ({
 
 const segmentRowWrites = ({ connection, data, manifest, row : described, segment }) => {
 
+	// NO ID, NO ROW. `String( undefined )` is the string 'undefined', which is a
+	// perfectly valid string as far as the api's schema is concerned — so a row
+	// built without a vendor id would store and then render as a live-looking
+	// link to an object that does not exist. The three hooks all guard before
+	// they get here; this is the guard that does not depend on them remembering.
+	if( described?.id === undefined || described?.id === null ) return [];
+
 	const built = row({ connection, data, manifest, row : described });
 
 	return [
@@ -3531,7 +3538,30 @@ const api$1 = async ( path, { fetcher = fetch, method = 'GET', payload, token }
 
 // The segment's name at Klaviyo, spelled once. It is a LABEL — the definition
 // below keys on the id — so a rename never has to touch a profile.
-const segmentName = ( title ) => 'Drawbridge: ' + title;
+//
+// THE ID IS IN THE LABEL TOO, and that part is not cosmetic. Nothing makes a
+// THE ID IS IN THE NAME BECAUSE THE NAME IS NOT AN IDENTITY. Keyed on title
+// alone, a second segment called "VIP" found the first one's Klaviyo segment by
+// name and adopted it: both rows then pointed at one segment whose definition
+// computes the FIRST one's membership, so the merchant sent to the wrong people,
+// and deleting either segment deleted that object out from under the other with
+// nothing to repair it.
+//
+// The api now carries a unique index on (organization, title) and a 409 on both
+// create and rename, so that pair can no longer be created. The suffix stays
+// anyway: it is what makes the damage impossible rather than merely unlikely,
+// and it is the only part of this that holds if a title ever reaches Klaviyo
+// from somewhere that did not go through the route.
+//
+// KLAVIYO IS NOT DOING THIS FOR US. Create Segment describes `name` only as "A
+// helpful name to label the segment" and documents no uniqueness constraint and
+// no duplicate-name error (developers.klaviyo.com/en/reference/create_segment,
+// fetched 2026-09-12), so the search cannot lean on the vendor. What IS
+// documented is that filter comparisons are case-sensitive
+// (developers.klaviyo.com/en/docs/filtering_, same date), which is why the
+// find-by-name below re-checks with === and agrees with the vendor rather than
+// guessing.
+const segmentName = ( title, id ) => 'Drawbridge: ' + title + ' (' + String( id ).slice( -6 ) + ')';
 
 // THE GRANT THIS NEEDS. Klaviyo's scopes are set on the app and a token carries
 // only what the merchant consented to, so a connection made before segments were
@@ -4047,7 +4077,7 @@ var klaviyo = {
 
 				}
 
-				const name = segmentName( segment.title );
+				const name = segmentName( segment.title, segment.id );
 				const existing = segmentRowFor({ connection, segment });
 
 				let id = null;
@@ -4657,13 +4687,29 @@ const subscriberHash = ( email ) => createHash( 'md5' )
 // server-side truncation is the worse outcome because register would then look
 // for a name Mailchimp had silently changed.
 //
-// ponytail: two segments whose first 88 characters match would share one tag.
-// The upgrade is to append a short hash of the segment id, which costs the name
-// its readability in the merchant's own audience — not worth it until somebody
-// actually collides.
+// THE SEGMENT ID IS IN THE NAME, and it has to be. Nothing makes a Drawbridge
+// segment title unique — no index on the collection, no check in the route, no
+// check in the form — so two segments called "VIP" are ordinary. Keyed on title
+// alone, the second one's register finds the first one's tag, adopts it, and
+// both rows point at one object: the merchant's two segments are one tag, and
+// deleting either segment deletes the tag out from under the other, leaving a
+// row pointing at an id that no longer exists and nothing to repair it.
+//
+// The last six characters of the id are enough to separate them and short
+// enough to leave the title readable in the merchant's own audience.
+//
+// A rename now changes the name and keeps the suffix, so register's find-by-name
+// still matches after one — and the truncation below cuts the TITLE rather than
+// the suffix for the same reason.
 const TAG_NAME_LIMIT = 100;
 
-const tagName = ( title ) => ( 'Drawbridge: ' + title ).slice( 0, TAG_NAME_LIMIT );
+const tagName = ( title, id ) => {
+
+	const suffix = ' (' + String( id ).slice( -6 ) + ')';
+
+	return ( 'Drawbridge: ' + title ).slice( 0, TAG_NAME_LIMIT - suffix.length ) + suffix;
+
+};
 
 // Mailchimp — contact sync, not a sender.
 var mailchimp = {
@@ -4958,19 +5004,21 @@ var mailchimp = {
 								}
 							},
 							{
-								$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 joined = new Set( segments.map( ( entry ) => entry.id ) );
 
 					const tags = ( owned || [] )
-						.map( ( entry ) => entry.title )
-						.filter( Boolean )
-						.map( ( title ) => ({
-							name : tagName( title ),
-							status : joined.has( title ) ? 'active' : 'inactive'
+						.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 ){
@@ -5040,7 +5088,7 @@ var mailchimp = {
 
 				if( ! 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, segment });
 
 				let id = null;
@@ -5768,23 +5816,33 @@ var shopify = {
 
 				const request = { email : context?.email || null, lead : context?.lead || null, shop : connection.shop };
 
+				// KEPT even though nothing below sends the address anywhere. A code
+				// that reaches nobody is a wasted action and a merchant's wasted
+				// allowance: issuing is only half a workflow, and the email step after
+				// it needs somewhere to send the code.
 				if( ! context?.email ) return { message : 'Lead email is missing.', request, response : { skipped : true }, skipped : true };
 				if( ! context?.lead ) return { message : 'Lead id is missing.', request, response : { skipped : true }, skipped : true };
 				if( ! discount?.id ) return { message : 'Discount is not configured on this step.', request, response : { skipped : true }, skipped : true };
 
 				const adminAccessToken = await adminToken();
 
-				// The customer must exist before a code is mapped to them. An earlier
-				// commerce.customer step usually did this and left the id on the
-				// context; when this step runs alone, it does it here.
-				if( ! context.shopifyCustomerId ){
-
-					const customer = await shopify.admin.getOrCreateCustomer({ adminAccessToken, domain : connection.shop, email : context.email });
-
-					if( ! customer?.id ) return { message : 'Shopify did not return a customer id — create/lookup failed.', request, response : { skipped : true }, skipped : true };
-
-				}
-
+				// NOTHING IS SENT TO SHOPIFY TO LINK THE CODE TO THE PERSON, because
+				// there is nothing to send: discountRedeemCodeBulkAdd takes a discount
+				// and a list of codes, and no customer
+				// (shopify.dev/docs/api/admin-graphql/2026-04/mutations/discountRedeemCodeBulkAdd).
+				// The link lives entirely on our side — `lead.shopifyDiscountCode`
+				// below — and attribution happens when the order comes back carrying
+				// the code, matched org-scoped against that field by commerce.order.
+				//
+				// This step used to call getOrCreateCustomer first, on the stated
+				// reasoning that "the customer must exist before a code is mapped to
+				// them". No code was ever mapped to them: the result was assigned and
+				// discarded, createDiscountCode never received it, and redemption
+				// attribution never consulted it. All it did was create a customer in
+				// the merchant's store as a side effect of issuing a discount — which
+				// QA reported as surprising, and was right to. A merchant who wants the
+				// buyer to exist at the store adds the Create customer step, which is
+				// what that step is for.
 				const discountCode = await shopify.admin.createDiscountCode({
 					adminAccessToken,
 					code : 'DB-' + generateDiscountCode(),

package/dist/connections/index.js

@@ -542,6 +542,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
@@ -3014,7 +3015,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
@@ -3389,7 +3390,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) {
@@ -3796,7 +3797,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
@@ -3977,14 +3981,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", {
@@ -4036,7 +4043,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) {
@@ -4589,10 +4596,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.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
@@ -2999,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
@@ -3374,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) {
@@ -3781,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
@@ -3962,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", {
@@ -4021,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) {
@@ -4593,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(),

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
@@ -2950,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
@@ -3325,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) {
@@ -3732,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
@@ -3913,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", {
@@ -3972,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) {
@@ -4544,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(),

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.172"
 }