@quiltt/react 4.2.0 → 4.2.1

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # @quiltt/react
 
+## 4.2.1
+
+### Patch Changes
+
+- [#350](https://github.com/quiltt/quiltt-js/pull/350) [`0233592`](https://github.com/quiltt/quiltt-js/commit/02335928bb872a6588c2ca81a1bd9a081053bd29) Thanks [@zubairaziz](https://github.com/zubairaziz)! - Bugfix: early return in effect prevents reconnect from being called
+
+- Updated dependencies [[`0233592`](https://github.com/quiltt/quiltt-js/commit/02335928bb872a6588c2ca81a1bd9a081053bd29)]:
+  - @quiltt/core@4.2.1
+
 ## 4.2.0
 
 ### Minor Changes

package/dist/index.js

@@ -4,7 +4,7 @@ export { u as useIsomorphicLayoutEffect } from './useIsomorphicLayoutEffect-clie
 import { Q as QuilttAuthProvider, i as isDeepEqual } from './QuilttAuthProvider-client-DLARZukU.js';
 export { b as useAuthenticateSession, a as useIdentifySession, u as useImportSession, c as useRevokeSession } from './QuilttAuthProvider-client-DLARZukU.js';
 export { u as useQuilttClient } from './useQuilttClient-client-CAAUait1.js';
-import { u as useQuilttConnector } from './useQuilttConnector-client-BK7ybRZe.js';
+import { u as useQuilttConnector } from './useQuilttConnector-client-CacEgWJN.js';
 export { u as useQuilttSession } from './useQuilttSession-client-Ddb55W0n.js';
 export { u as useQuilttSettings } from './useQuilttSettings-client-BOCBjFXe.js';
 export { u as useSession } from './useSession-client-CCAvnROP.js';

package/dist/{useQuilttConnector-client-BK7ybRZe.js → useQuilttConnector-client-CacEgWJN.js}

@@ -4,7 +4,7 @@ import { cdnBase } from '@quiltt/core';
 import { u as useQuilttSession } from './useQuilttSession-client-Ddb55W0n.js';
 import { u as useScript } from './useScript-client-JCgaTW9n.js';
 
-var version = "4.2.0";
+var version = "4.2.1";
 
 const useQuilttConnector = (connectorId, options)=>{
     const status = useScript(`${cdnBase}/v1/connector.js?agent=react-${version}`, {
@@ -20,50 +20,42 @@ const useQuilttConnector = (connectorId, options)=>{
     const prevConnectorIdRef = useRef(connectorId);
     const connectorCreatedRef = useRef(false);
     // Set Session
+    // biome-ignore lint/correctness/useExhaustiveDependencies: trigger effects when script status changes too
     useEffect(()=>{
         if (typeof Quiltt === 'undefined') return;
-        console.debug('[Quiltt] script status: ', status);
         Quiltt.authenticate(session?.token);
     }, [
         status,
         session?.token
     ]);
     // Set Connector
+    // biome-ignore lint/correctness/useExhaustiveDependencies: trigger effects when script status changes too
     useEffect(()=>{
         if (typeof Quiltt === 'undefined' || !connectorId) return;
-        console.debug('[Quiltt] script status: ', status);
         const currentConnectionId = options?.connectionId;
         const currentInstitution = options?.institution;
-        // Check if this is a connectionId change on the same connector
+        // Check for changes
         const connectionIdChanged = prevConnectionIdRef.current !== currentConnectionId;
         const connectorIdChanged = prevConnectorIdRef.current !== connectorId;
-        // If only connectionId changed (not the connectorId), the core SDK should handle this
-        // via the updated Handler.updateOptions method, so we don't need to recreate the connector
-        if (connectionIdChanged && !connectorIdChanged && connectorCreatedRef.current) {
-            // The SDK will automatically update the existing handler with new connectionId
-            // via the DocumentObserver -> Engine.onChange -> Handler.updateOptions flow
-            console.debug('[Quiltt] connectionId changed, SDK will handle update automatically');
-            // Update our refs
-            prevConnectionIdRef.current = currentConnectionId;
-            return;
-        }
-        // Only create new connector if we haven't created one yet or if connectorId changed
-        if (!connectorCreatedRef.current || connectorIdChanged) {
-            // Create new connector (initial mount or connectorId changed)
+        const hasChanges = connectionIdChanged || connectorIdChanged || !connectorCreatedRef.current;
+        // Update if there are changes, regardless of what the changes are
+        if (hasChanges) {
             if (currentConnectionId) {
+                // Always use reconnect when connectionId is available
                 setConnector(Quiltt.reconnect(connectorId, {
                     connectionId: currentConnectionId
                 }));
             } else {
+                // Use connect for new connections without connectionId
                 setConnector(Quiltt.connect(connectorId, {
                     institution: currentInstitution
                 }));
             }
+            // Update refs
             connectorCreatedRef.current = true;
+            prevConnectionIdRef.current = currentConnectionId;
+            prevConnectorIdRef.current = connectorId;
         }
-        // Update refs
-        prevConnectionIdRef.current = currentConnectionId;
-        prevConnectorIdRef.current = connectorId;
     }, [
         connectorId,
         options?.connectionId,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quiltt/react",
-  "version": "4.2.0",
+  "version": "4.2.1",
   "description": "React Components and Hooks for Quiltt Connector",
   "keywords": [
     "quiltt",
@@ -36,18 +36,18 @@
   "dependencies": {
     "@apollo/client": "^3.12.4",
     "use-debounce": "^10.0.4",
-    "@quiltt/core": "4.2.0"
+    "@quiltt/core": "4.2.1"
   },
   "devDependencies": {
     "@biomejs/biome": "1.9.4",
-    "@types/node": "22.14.1",
+    "@types/node": "22.15.31",
     "@types/react": "18.3.20",
     "@types/react-dom": "18.3.5",
     "bunchee": "6.3.4",
     "react": "18.3.1",
     "react-dom": "18.3.1",
     "rimraf": "6.0.1",
-    "typescript": "5.8.2"
+    "typescript": "5.8.3"
   },
   "peerDependencies": {
     "react": "^16.8.0 || ^17.0.0 || ^18.0.0 || ^19.0.0",

package/src/hooks/useQuilttConnector.ts

@@ -36,52 +36,41 @@ export const useQuilttConnector = (
   const connectorCreatedRef = useRef<boolean>(false)
 
   // Set Session
+  // biome-ignore lint/correctness/useExhaustiveDependencies: trigger effects when script status changes too
   useEffect(() => {
     if (typeof Quiltt === 'undefined') return
-    console.debug('[Quiltt] script status: ', status)
 
     Quiltt.authenticate(session?.token)
   }, [status, session?.token])
 
   // Set Connector
+  // biome-ignore lint/correctness/useExhaustiveDependencies: trigger effects when script status changes too
   useEffect(() => {
     if (typeof Quiltt === 'undefined' || !connectorId) return
-    console.debug('[Quiltt] script status: ', status)
 
     const currentConnectionId = options?.connectionId
     const currentInstitution = options?.institution
 
-    // Check if this is a connectionId change on the same connector
+    // Check for changes
     const connectionIdChanged = prevConnectionIdRef.current !== currentConnectionId
     const connectorIdChanged = prevConnectorIdRef.current !== connectorId
+    const hasChanges = connectionIdChanged || connectorIdChanged || !connectorCreatedRef.current
 
-    // If only connectionId changed (not the connectorId), the core SDK should handle this
-    // via the updated Handler.updateOptions method, so we don't need to recreate the connector
-    if (connectionIdChanged && !connectorIdChanged && connectorCreatedRef.current) {
-      // The SDK will automatically update the existing handler with new connectionId
-      // via the DocumentObserver -> Engine.onChange -> Handler.updateOptions flow
-      console.debug('[Quiltt] connectionId changed, SDK will handle update automatically')
-
-      // Update our refs
-      prevConnectionIdRef.current = currentConnectionId
-      return
-    }
-
-    // Only create new connector if we haven't created one yet or if connectorId changed
-    if (!connectorCreatedRef.current || connectorIdChanged) {
-      // Create new connector (initial mount or connectorId changed)
+    // Update if there are changes, regardless of what the changes are
+    if (hasChanges) {
       if (currentConnectionId) {
+        // Always use reconnect when connectionId is available
         setConnector(Quiltt.reconnect(connectorId, { connectionId: currentConnectionId }))
       } else {
+        // Use connect for new connections without connectionId
         setConnector(Quiltt.connect(connectorId, { institution: currentInstitution }))
       }
 
+      // Update refs
       connectorCreatedRef.current = true
+      prevConnectionIdRef.current = currentConnectionId
+      prevConnectorIdRef.current = connectorId
     }
-
-    // Update refs
-    prevConnectionIdRef.current = currentConnectionId
-    prevConnectorIdRef.current = connectorId
   }, [connectorId, options?.connectionId, options?.institution, status])
 
   // onEvent