@exotel-npm-dev/exotel-ai-assist 1.0.4 → 1.0.6

package/dist/controller/index.js

@@ -343,6 +343,7 @@ class BroadcastChannelTransport {
     this.pendingUrl = null;
     this.pendingAuthToken = null;
     this.lockReleaser = null;
+    this._destroyed = false;
     const channelName = `exotel-ai-assist:${sessionHash}`;
     const lockName = `exotel-ai-assist-leader:${sessionHash}`;
     this.channel = new BroadcastChannel(channelName);
@@ -378,20 +379,27 @@ class BroadcastChannelTransport {
     }, lock => {
       if (lock) {
         this.isLeader = true;
-        if (this.pendingUrl && this.pendingAuthToken) {
-          this._openSocket(this.pendingUrl, this.pendingAuthToken);
-          this.pendingUrl = null;
-          this.pendingAuthToken = null;
-        }
         // Hold the lock for the lifetime of this transport instance.
         return new Promise(resolve => {
           this.lockReleaser = resolve;
+          // The lock can arrive asynchronously AFTER destroy() was called
+          // (e.g. React StrictMode runs effect cleanup before the microtask
+          // queue drains). Release the lock immediately so the next transport
+          // in line can acquire it, and skip opening any socket.
+          if (this._destroyed) {
+            resolve();
+            return;
+          }
+          if (this.pendingUrl && this.pendingAuthToken) {
+            this._openSocket(this.pendingUrl, this.pendingAuthToken);
+            this.pendingUrl = null;
+            this.pendingAuthToken = null;
+          }
         });
       }
       return Promise.resolve();
     }).catch(() => {
-      // Navigator Locks not supported — every tab owns its own WebSocket.
-      this.isLeader = true;
+      if (!this._destroyed) this.isLeader = true;
     });
   }
   _openSocket(url, authToken) {
@@ -401,6 +409,7 @@ class BroadcastChannelTransport {
     this.socket = new WebSocket(url);
     this.socket.onopen = () => {
       var _a;
+      if (this._destroyed) return;
       this.socketConnected = true;
       this.socket.send(JSON.stringify({
         type: "auth",
@@ -409,36 +418,39 @@ class BroadcastChannelTransport {
       const msg = {
         type: "CONNECTED"
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onmessage = ev => {
       var _a;
+      if (this._destroyed) return;
       const msg = {
         type: "MESSAGE",
         payload: ev.data
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onerror = () => {
       var _a;
+      if (this._destroyed) return;
       const msg = {
         type: "ERROR",
         message: "WebSocket error"
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onclose = ev => {
       var _a;
+      if (this._destroyed) return;
       this.socketConnected = false;
       const msg = {
         type: "DISCONNECTED",
         code: ev.code
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.socket = null;
     };
   }
@@ -467,6 +479,7 @@ class BroadcastChannelTransport {
   }
   destroy() {
     var _a;
+    this._destroyed = true;
     this.disconnect();
     this.channel.close();
     (_a = this.lockReleaser) === null || _a === void 0 ? void 0 : _a.call(this);

package/dist/index.js

@@ -19761,6 +19761,7 @@ class BroadcastChannelTransport {
     this.pendingUrl = null;
     this.pendingAuthToken = null;
     this.lockReleaser = null;
+    this._destroyed = false;
     const channelName = `exotel-ai-assist:${sessionHash}`;
     const lockName = `exotel-ai-assist-leader:${sessionHash}`;
     this.channel = new BroadcastChannel(channelName);
@@ -19796,20 +19797,27 @@ class BroadcastChannelTransport {
     }, lock => {
       if (lock) {
         this.isLeader = true;
-        if (this.pendingUrl && this.pendingAuthToken) {
-          this._openSocket(this.pendingUrl, this.pendingAuthToken);
-          this.pendingUrl = null;
-          this.pendingAuthToken = null;
-        }
         // Hold the lock for the lifetime of this transport instance.
         return new Promise(resolve => {
           this.lockReleaser = resolve;
+          // The lock can arrive asynchronously AFTER destroy() was called
+          // (e.g. React StrictMode runs effect cleanup before the microtask
+          // queue drains). Release the lock immediately so the next transport
+          // in line can acquire it, and skip opening any socket.
+          if (this._destroyed) {
+            resolve();
+            return;
+          }
+          if (this.pendingUrl && this.pendingAuthToken) {
+            this._openSocket(this.pendingUrl, this.pendingAuthToken);
+            this.pendingUrl = null;
+            this.pendingAuthToken = null;
+          }
         });
       }
       return Promise.resolve();
     }).catch(() => {
-      // Navigator Locks not supported — every tab owns its own WebSocket.
-      this.isLeader = true;
+      if (!this._destroyed) this.isLeader = true;
     });
   }
   _openSocket(url, authToken) {
@@ -19819,6 +19827,7 @@ class BroadcastChannelTransport {
     this.socket = new WebSocket(url);
     this.socket.onopen = () => {
       var _a;
+      if (this._destroyed) return;
       this.socketConnected = true;
       this.socket.send(JSON.stringify({
         type: "auth",
@@ -19827,36 +19836,39 @@ class BroadcastChannelTransport {
       const msg = {
         type: "CONNECTED"
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onmessage = ev => {
       var _a;
+      if (this._destroyed) return;
       const msg = {
         type: "MESSAGE",
         payload: ev.data
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onerror = () => {
       var _a;
+      if (this._destroyed) return;
       const msg = {
         type: "ERROR",
         message: "WebSocket error"
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onclose = ev => {
       var _a;
+      if (this._destroyed) return;
       this.socketConnected = false;
       const msg = {
         type: "DISCONNECTED",
         code: ev.code
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.socket = null;
     };
   }
@@ -19885,6 +19897,7 @@ class BroadcastChannelTransport {
   }
   destroy() {
     var _a;
+    this._destroyed = true;
     this.disconnect();
     this.channel.close();
     (_a = this.lockReleaser) === null || _a === void 0 ? void 0 : _a.call(this);
@@ -21189,7 +21202,6 @@ ExotelAIAssistController.RECONNECT_BASE_MS = 3000;
 
 function useExotelAIAssist(params) {
   const controllerRef = reactExports.useRef(null);
-  const paramsRef = reactExports.useRef(params);
   const [status, setStatus] = reactExports.useState("idle");
   const [suggestions, setSuggestions] = reactExports.useState([]);
   const [transcripts, setTranscripts] = reactExports.useState([]);
@@ -21198,10 +21210,14 @@ function useExotelAIAssist(params) {
   const [lastError, setLastError] = reactExports.useState(null);
   const paramHash = reactExports.useMemo(() => Utils.hash(params), [params]);
   reactExports.useEffect(() => {
-    paramsRef.current = params;
-  });
-  reactExports.useEffect(() => {
-    const ctrl = new ExotelAIAssistController(paramsRef.current);
+    // Clear all stale state from the previous session before connecting.
+    setStatus("idle");
+    setSuggestions([]);
+    setTranscripts([]);
+    setSentiment(null);
+    setBotConfig(null);
+    setLastError(null);
+    const ctrl = new ExotelAIAssistController(params);
     controllerRef.current = ctrl;
     ctrl.on("statusChange", setStatus);
     ctrl.on("botConfig", setBotConfig);
@@ -21219,22 +21235,6 @@ function useExotelAIAssist(params) {
       controllerRef.current = null;
     };
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, []);
-  const prevParamHashRef = reactExports.useRef(paramHash);
-  reactExports.useEffect(() => {
-    if (!controllerRef.current) return;
-    const prevParamHash = prevParamHashRef.current;
-    const paramHashChanged = paramHash !== prevParamHash;
-    if (paramHashChanged) {
-      prevParamHashRef.current = paramHash;
-      setSuggestions([]);
-      setTranscripts([]);
-      setSentiment(null);
-      setBotConfig(null);
-      setLastError(null);
-    }
-    controllerRef.current.setParams(params);
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [paramHash]);
   const connect = reactExports.useCallback(() => {
     var _a;
@@ -21771,7 +21771,8 @@ function TranscriptTab({
           alignItems: "flex-start",
           gap: "12px",
           padding: "10px",
-          borderBottom: "1px solid #e5e7eb"
+          borderBottom: "1px solid #e5e7eb",
+          textAlign: "left"
         },
         children: [jsxRuntimeExports.jsx("span", {
           style: {

package/dist/react/index.js

@@ -5047,6 +5047,7 @@ class BroadcastChannelTransport {
     this.pendingUrl = null;
     this.pendingAuthToken = null;
     this.lockReleaser = null;
+    this._destroyed = false;
     const channelName = `exotel-ai-assist:${sessionHash}`;
     const lockName = `exotel-ai-assist-leader:${sessionHash}`;
     this.channel = new BroadcastChannel(channelName);
@@ -5082,20 +5083,27 @@ class BroadcastChannelTransport {
     }, lock => {
       if (lock) {
         this.isLeader = true;
-        if (this.pendingUrl && this.pendingAuthToken) {
-          this._openSocket(this.pendingUrl, this.pendingAuthToken);
-          this.pendingUrl = null;
-          this.pendingAuthToken = null;
-        }
         // Hold the lock for the lifetime of this transport instance.
         return new Promise(resolve => {
           this.lockReleaser = resolve;
+          // The lock can arrive asynchronously AFTER destroy() was called
+          // (e.g. React StrictMode runs effect cleanup before the microtask
+          // queue drains). Release the lock immediately so the next transport
+          // in line can acquire it, and skip opening any socket.
+          if (this._destroyed) {
+            resolve();
+            return;
+          }
+          if (this.pendingUrl && this.pendingAuthToken) {
+            this._openSocket(this.pendingUrl, this.pendingAuthToken);
+            this.pendingUrl = null;
+            this.pendingAuthToken = null;
+          }
         });
       }
       return Promise.resolve();
     }).catch(() => {
-      // Navigator Locks not supported — every tab owns its own WebSocket.
-      this.isLeader = true;
+      if (!this._destroyed) this.isLeader = true;
     });
   }
   _openSocket(url, authToken) {
@@ -5105,6 +5113,7 @@ class BroadcastChannelTransport {
     this.socket = new WebSocket(url);
     this.socket.onopen = () => {
       var _a;
+      if (this._destroyed) return;
       this.socketConnected = true;
       this.socket.send(JSON.stringify({
         type: "auth",
@@ -5113,36 +5122,39 @@ class BroadcastChannelTransport {
       const msg = {
         type: "CONNECTED"
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onmessage = ev => {
       var _a;
+      if (this._destroyed) return;
       const msg = {
         type: "MESSAGE",
         payload: ev.data
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onerror = () => {
       var _a;
+      if (this._destroyed) return;
       const msg = {
         type: "ERROR",
         message: "WebSocket error"
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
     };
     this.socket.onclose = ev => {
       var _a;
+      if (this._destroyed) return;
       this.socketConnected = false;
       const msg = {
         type: "DISCONNECTED",
         code: ev.code
       };
-      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.channel.postMessage(msg);
+      (_a = this.handler) === null || _a === void 0 ? void 0 : _a.call(this, msg);
       this.socket = null;
     };
   }
@@ -5171,6 +5183,7 @@ class BroadcastChannelTransport {
   }
   destroy() {
     var _a;
+    this._destroyed = true;
     this.disconnect();
     this.channel.close();
     (_a = this.lockReleaser) === null || _a === void 0 ? void 0 : _a.call(this);
@@ -6475,7 +6488,6 @@ ExotelAIAssistController.RECONNECT_BASE_MS = 3000;
 
 function useExotelAIAssist(params) {
   const controllerRef = React.useRef(null);
-  const paramsRef = React.useRef(params);
   const [status, setStatus] = React.useState("idle");
   const [suggestions, setSuggestions] = React.useState([]);
   const [transcripts, setTranscripts] = React.useState([]);
@@ -6484,10 +6496,14 @@ function useExotelAIAssist(params) {
   const [lastError, setLastError] = React.useState(null);
   const paramHash = React.useMemo(() => Utils.hash(params), [params]);
   React.useEffect(() => {
-    paramsRef.current = params;
-  });
-  React.useEffect(() => {
-    const ctrl = new ExotelAIAssistController(paramsRef.current);
+    // Clear all stale state from the previous session before connecting.
+    setStatus("idle");
+    setSuggestions([]);
+    setTranscripts([]);
+    setSentiment(null);
+    setBotConfig(null);
+    setLastError(null);
+    const ctrl = new ExotelAIAssistController(params);
     controllerRef.current = ctrl;
     ctrl.on("statusChange", setStatus);
     ctrl.on("botConfig", setBotConfig);
@@ -6505,22 +6521,6 @@ function useExotelAIAssist(params) {
       controllerRef.current = null;
     };
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, []);
-  const prevParamHashRef = React.useRef(paramHash);
-  React.useEffect(() => {
-    if (!controllerRef.current) return;
-    const prevParamHash = prevParamHashRef.current;
-    const paramHashChanged = paramHash !== prevParamHash;
-    if (paramHashChanged) {
-      prevParamHashRef.current = paramHash;
-      setSuggestions([]);
-      setTranscripts([]);
-      setSentiment(null);
-      setBotConfig(null);
-      setLastError(null);
-    }
-    controllerRef.current.setParams(params);
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [paramHash]);
   const connect = React.useCallback(() => {
     var _a;
@@ -7057,7 +7057,8 @@ function TranscriptTab({
           alignItems: "flex-start",
           gap: "12px",
           padding: "10px",
-          borderBottom: "1px solid #e5e7eb"
+          borderBottom: "1px solid #e5e7eb",
+          textAlign: "left"
         },
         children: [jsxRuntime.jsx("span", {
           style: {

package/dist/transport/index.d.ts

@@ -16,6 +16,7 @@ export declare class BroadcastChannelTransport implements ITransport {
     private pendingUrl;
     private pendingAuthToken;
     private lockReleaser;
+    private _destroyed;
     /**
      * @param sessionHash - A hash of all connection-relevant params (see
      *   Utils.hashParams). Tabs with identical params share one WebSocket;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exotel-npm-dev/exotel-ai-assist",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "description": "AI-powered real-time suggestions, transcript, and sentiment for agent calls",
   "license": "Apache-2.0",
   "main": "dist/index.js",