@chordcommerce/analytics 1.22.1-beta.1 → 1.22.2

package/dist/cjs/client/index.d.ts

@@ -31,7 +31,7 @@ export declare class ChordAnalytics<T extends ObjectTypes = ObjectTypes> {
      */
     private cdpReadyPromise;
     private consentConfig;
-    private resetInProgress;
+    private resetCompletePromise;
     /**
      * Queue for events waiting for consent to be ready
      */
@@ -73,6 +73,7 @@ export declare class ChordAnalytics<T extends ObjectTypes = ObjectTypes> {
     ccdp: () => AnyAnalytics;
     logger: (message?: any, ...optionalParams: any[]) => void;
     isResetting: () => boolean;
+    private waitForReadyState;
     /**
      * Wait for consent manager to be ready (no timeout - queues indefinitely).
      * Uses a shared promise so multiple events share the same watcher.
@@ -151,6 +152,7 @@ export declare class ChordAnalytics<T extends ObjectTypes = ObjectTypes> {
      */
     private sendPageEvent;
     reset: () => Promise<void>;
+    private performReset;
     /**
      * Handler for postMessage events from the Shopify Web Pixel sandbox.
      * Bound to the instance for proper `this` context and cleanup.

package/dist/cjs/client/index.js

@@ -87,7 +87,9 @@ var ChordAnalytics = /** @class */ (function () {
          */
         this.cdpReadyPromise = null;
         this.consentConfig = null;
-        this.resetInProgress = false;
+        // Non-null while reset() is in flight. Acts as both the coalescing handle and
+        // the gate that track/identify/page await before executing.
+        this.resetCompletePromise = null;
         /**
          * Queue for events waiting for consent to be ready
          */
@@ -260,7 +262,21 @@ var ChordAnalytics = /** @class */ (function () {
             console.log.apply(console, __spreadArray([message], optionalParams, false));
         };
         this.isResetting = function () {
-            return _this.resetInProgress;
+            return _this.resetCompletePromise !== null;
+        };
+        // Single gate for all outgoing operations. Returns a Promise only when there
+        // is actually something to wait for, preserving synchronous execution in the
+        // common case where CDP is ready and no reset is in flight.
+        this.waitForReadyState = function () {
+            var cdp = _this.cdpReadyPromise;
+            var reset = _this.resetCompletePromise;
+            if (!cdp && !reset)
+                return undefined;
+            // eslint-disable-next-line @typescript-eslint/no-empty-function
+            if (cdp && reset)
+                return Promise.all([cdp, reset]).then(function () { });
+            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+            return cdp !== null && cdp !== void 0 ? cdp : reset;
         };
         /**
          * Wait for consent manager to be ready (no timeout - queues indefinitely).
@@ -453,6 +469,7 @@ var ChordAnalytics = /** @class */ (function () {
          * If consent is not ready, the event is queued until consent is available.
          */
         this.track = function (event, props, options) { return __awaiter(_this, void 0, void 0, function () {
+            var gate;
             return __generator(this, function (_a) {
                 switch (_a.label) {
                     case 0:
@@ -462,18 +479,13 @@ var ChordAnalytics = /** @class */ (function () {
                         }
                         if (!event)
                             this.logger('No event name provided');
-                        if (!this.cdpReadyPromise) return [3 /*break*/, 2];
-                        if (this.options.enableLogging && this.options.debug) {
-                            // eslint-disable-next-line no-console
-                            console.log("[Chord Analytics DEBUG]: track(\"".concat(event, "\") waiting for cdpReadyPromise"));
-                        }
-                        return [4 /*yield*/, this.cdpReadyPromise];
+                        gate = this.waitForReadyState();
+                        if (!gate) return [3 /*break*/, 2];
+                        return [4 /*yield*/, gate
+                            // If consent is configured but not ready, queue the event
+                        ];
                     case 1:
                         _a.sent();
-                        if (this.options.enableLogging && this.options.debug) {
-                            // eslint-disable-next-line no-console
-                            console.log("[Chord Analytics DEBUG]: track(\"".concat(event, "\") cdpReadyPromise resolved"));
-                        }
                         _a.label = 2;
                     case 2:
                         // If consent is configured but not ready, queue the event
@@ -583,11 +595,15 @@ var ChordAnalytics = /** @class */ (function () {
          * If consent is not ready, the event is queued until consent is available.
          */
         this.identify = function (userIdOrTraits, traitsOrOptions, options) { return __awaiter(_this, void 0, void 0, function () {
+            var gate;
             return __generator(this, function (_a) {
                 switch (_a.label) {
                     case 0:
-                        if (!this.cdpReadyPromise) return [3 /*break*/, 2];
-                        return [4 /*yield*/, this.cdpReadyPromise];
+                        gate = this.waitForReadyState();
+                        if (!gate) return [3 /*break*/, 2];
+                        return [4 /*yield*/, gate
+                            // If consent is configured but not ready, queue the event
+                        ];
                     case 1:
                         _a.sent();
                         _a.label = 2;
@@ -670,11 +686,15 @@ var ChordAnalytics = /** @class */ (function () {
         this.page = function (options, props) {
             if (props === void 0) { props = {}; }
             return __awaiter(_this, void 0, void 0, function () {
+                var gate;
                 return __generator(this, function (_a) {
                     switch (_a.label) {
                         case 0:
-                            if (!this.cdpReadyPromise) return [3 /*break*/, 2];
-                            return [4 /*yield*/, this.cdpReadyPromise];
+                            gate = this.waitForReadyState();
+                            if (!gate) return [3 /*break*/, 2];
+                            return [4 /*yield*/, gate
+                                // If consent is configured but not ready, queue the event
+                            ];
                         case 1:
                             _a.sent();
                             _a.label = 2;
@@ -742,14 +762,21 @@ var ChordAnalytics = /** @class */ (function () {
                 });
             });
         };
-        this.reset = function () { return __awaiter(_this, void 0, void 0, function () {
+        // Coalesces concurrent reset() calls — a second caller gets the same in-flight
+        // Promise rather than triggering a duplicate reset.
+        this.reset = function () {
+            if (!_this.resetCompletePromise) {
+                _this.resetCompletePromise = _this.performReset();
+            }
+            return _this.resetCompletePromise;
+        };
+        this.performReset = function () { return __awaiter(_this, void 0, void 0, function () {
             var a, b, maxAttempts, attempts, userState, error_3;
             var _a;
             return __generator(this, function (_b) {
                 switch (_b.label) {
                     case 0:
                         _b.trys.push([0, 4, 5, 6]);
-                        this.resetInProgress = true;
                         a = this.ccdp();
                         if (a && typeof a.reset === 'function') {
                             a.reset();
@@ -785,7 +812,7 @@ var ChordAnalytics = /** @class */ (function () {
                         this.logger(error_3);
                         return [3 /*break*/, 6];
                     case 5:
-                        this.resetInProgress = false;
+                        this.resetCompletePromise = null;
                         return [7 /*endfinally*/];
                     case 6: return [2 /*return*/];
                 }

package/dist/client/index.d.ts

@@ -31,7 +31,7 @@ export declare class ChordAnalytics<T extends ObjectTypes = ObjectTypes> {
      */
     private cdpReadyPromise;
     private consentConfig;
-    private resetInProgress;
+    private resetCompletePromise;
     /**
      * Queue for events waiting for consent to be ready
      */
@@ -73,6 +73,7 @@ export declare class ChordAnalytics<T extends ObjectTypes = ObjectTypes> {
     ccdp: () => AnyAnalytics;
     logger: (message?: any, ...optionalParams: any[]) => void;
     isResetting: () => boolean;
+    private waitForReadyState;
     /**
      * Wait for consent manager to be ready (no timeout - queues indefinitely).
      * Uses a shared promise so multiple events share the same watcher.
@@ -151,6 +152,7 @@ export declare class ChordAnalytics<T extends ObjectTypes = ObjectTypes> {
      */
     private sendPageEvent;
     reset: () => Promise<void>;
+    private performReset;
     /**
      * Handler for postMessage events from the Shopify Web Pixel sandbox.
      * Bound to the instance for proper `this` context and cleanup.

package/dist/client/index.js

@@ -36,7 +36,9 @@ export class ChordAnalytics {
          */
         this.cdpReadyPromise = null;
         this.consentConfig = null;
-        this.resetInProgress = false;
+        // Non-null while reset() is in flight. Acts as both the coalescing handle and
+        // the gate that track/identify/page await before executing.
+        this.resetCompletePromise = null;
         /**
          * Queue for events waiting for consent to be ready
          */
@@ -205,7 +207,21 @@ export class ChordAnalytics {
             console.log(message, ...optionalParams);
         };
         this.isResetting = () => {
-            return this.resetInProgress;
+            return this.resetCompletePromise !== null;
+        };
+        // Single gate for all outgoing operations. Returns a Promise only when there
+        // is actually something to wait for, preserving synchronous execution in the
+        // common case where CDP is ready and no reset is in flight.
+        this.waitForReadyState = () => {
+            const cdp = this.cdpReadyPromise;
+            const reset = this.resetCompletePromise;
+            if (!cdp && !reset)
+                return undefined;
+            // eslint-disable-next-line @typescript-eslint/no-empty-function
+            if (cdp && reset)
+                return Promise.all([cdp, reset]).then(() => { });
+            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+            return cdp !== null && cdp !== void 0 ? cdp : reset;
         };
         /**
          * Wait for consent manager to be ready (no timeout - queues indefinitely).
@@ -359,18 +375,9 @@ export class ChordAnalytics {
             }
             if (!event)
                 this.logger('No event name provided');
-            // Wait for CDP to be ready before sending track events
-            if (this.cdpReadyPromise) {
-                if (this.options.enableLogging && this.options.debug) {
-                    // eslint-disable-next-line no-console
-                    console.log(`[Chord Analytics DEBUG]: track("${event}") waiting for cdpReadyPromise`);
-                }
-                yield this.cdpReadyPromise;
-                if (this.options.enableLogging && this.options.debug) {
-                    // eslint-disable-next-line no-console
-                    console.log(`[Chord Analytics DEBUG]: track("${event}") cdpReadyPromise resolved`);
-                }
-            }
+            const gate = this.waitForReadyState();
+            if (gate)
+                yield gate;
             // If consent is configured but not ready, queue the event
             if (this.consentConfig && !this.consentConfig.isConsentReady()) {
                 if (this.options.enableLogging && this.options.debug) {
@@ -466,10 +473,9 @@ export class ChordAnalytics {
          * If consent is not ready, the event is queued until consent is available.
          */
         this.identify = (userIdOrTraits, traitsOrOptions, options) => __awaiter(this, void 0, void 0, function* () {
-            // Wait for CDP to be ready before sending identify events
-            if (this.cdpReadyPromise) {
-                yield this.cdpReadyPromise;
-            }
+            const gate = this.waitForReadyState();
+            if (gate)
+                yield gate;
             // If consent is configured but not ready, queue the event
             if (this.consentConfig && !this.consentConfig.isConsentReady()) {
                 return this.queueEventForConsent('identify', [
@@ -536,10 +542,9 @@ export class ChordAnalytics {
          * If consent is not ready, the event is queued until consent is available.
          */
         this.page = (options, props = {}) => __awaiter(this, void 0, void 0, function* () {
-            // Wait for CDP to be ready before sending page events
-            if (this.cdpReadyPromise) {
-                yield this.cdpReadyPromise;
-            }
+            const gate = this.waitForReadyState();
+            if (gate)
+                yield gate;
             // If consent is configured but not ready, queue the event
             if (this.consentConfig && !this.consentConfig.isConsentReady()) {
                 return this.queueEventForConsent('page', [options, props]);
@@ -584,10 +589,17 @@ export class ChordAnalytics {
                 }
             });
         });
-        this.reset = () => __awaiter(this, void 0, void 0, function* () {
+        // Coalesces concurrent reset() calls — a second caller gets the same in-flight
+        // Promise rather than triggering a duplicate reset.
+        this.reset = () => {
+            if (!this.resetCompletePromise) {
+                this.resetCompletePromise = this.performReset();
+            }
+            return this.resetCompletePromise;
+        };
+        this.performReset = () => __awaiter(this, void 0, void 0, function* () {
             var _e;
             try {
-                this.resetInProgress = true;
                 const a = this.ccdp();
                 if (a && typeof a.reset === 'function') {
                     a.reset();
@@ -617,7 +629,7 @@ export class ChordAnalytics {
                 this.logger(error);
             }
             finally {
-                this.resetInProgress = false;
+                this.resetCompletePromise = null;
             }
         });
         /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@chordcommerce/analytics",
   "description": "Chord Commerce event tracking",
-  "version": "1.22.1-beta.1",
+  "version": "1.22.2",
   "author": "Chord Commerce <help@chord.co> (https://www.chord.co)",
   "type": "module",
   "main": "dist/index.js",