@nibssplc/cams-sdk-react 1.0.0-rc.131 → 1.0.0-rc.133

package/dist/index.esm.js

@@ -1756,27 +1756,44 @@ var MFAGate = function (_a) {
         }
         return parsed.data;
     }, [MFAEndpoints]);
-    // Extract the specific values we need from context for dependencies
+    // Extract primitives from context to avoid object reference churn in deps
     var contextRequiresMFA = "requiresMFA" in context ? context.requiresMFA : false;
     var contextAccessToken = "accessToken" in context ? context.accessToken : "";
-    var handleComplete = useCallback(function (success) {
-        var _a;
-        if (success) {
-            var accessToken = "accessToken" in context ? context.accessToken : "";
-            var idToken = "idToken" in context ? context.idToken : "";
-            Logger.info("MFA Authentication Successful");
-            setAuthState("Authenticated");
-            var tokens = { accessToken: accessToken, idToken: idToken };
-            console.log("[MFAGate] Calling onAuthSuccess callbacks", { tokens: tokens, hasLocalCallback: !!onAuthSuccess, hasContextCallback: "onAuthSuccess" in context && !!context.onAuthSuccess });
-            onAuthSuccess === null || onAuthSuccess === void 0 ? void 0 : onAuthSuccess(tokens);
-            if ("onAuthSuccess" in context) {
-                (_a = context.onAuthSuccess) === null || _a === void 0 ? void 0 : _a.call(context, tokens);
+    var contextIdToken = "idToken" in context ? context.idToken : "";
+    var contextHasUser = !!context.user;
+    var contextIsLoading = context.isLoading;
+    // Use refs so callbacks never cause effect/useCallback to re-run
+    var onAuthSuccessRef = useRef(onAuthSuccess);
+    var contextOnAuthSuccessRef = useRef("onAuthSuccess" in context ? context.onAuthSuccess : undefined);
+    useEffect(function () { onAuthSuccessRef.current = onAuthSuccess; }, [onAuthSuccess]);
+    useEffect(function () { contextOnAuthSuccessRef.current = "onAuthSuccess" in context ? context.onAuthSuccess : undefined; });
+    var fireOnAuthSuccess = useCallback(function (tokens) { return __awaiter$1(void 0, void 0, void 0, function () {
+        var _a, _b;
+        return __generator$1(this, function (_c) {
+            switch (_c.label) {
+                case 0: return [4 /*yield*/, Promise.all([
+                        (_a = onAuthSuccessRef.current) === null || _a === void 0 ? void 0 : _a.call(onAuthSuccessRef, tokens),
+                        (_b = contextOnAuthSuccessRef.current) === null || _b === void 0 ? void 0 : _b.call(contextOnAuthSuccessRef, tokens),
+                    ])];
+                case 1:
+                    _c.sent();
+                    return [2 /*return*/];
             }
-        }
-        else {
-            Logger.error("MFA Authentication Failed");
-        }
-    }, [context, onAuthSuccess]);
+        });
+    }); }, [] // stable — reads from refs
+    );
+    var handleComplete = useCallback(function (success) { return __awaiter$1(void 0, void 0, void 0, function () {
+        return __generator$1(this, function (_a) {
+            if (success) {
+                Logger.info("MFA Authentication Successful");
+                setAuthState("Completing");
+            }
+            else {
+                Logger.error("MFA Authentication Failed");
+            }
+            return [2 /*return*/];
+        });
+    }); }, []);
     var handleAuthFailed = useCallback(function () { return __awaiter$1(void 0, void 0, void 0, function () {
         var _a;
         return __generator$1(this, function (_b) {
@@ -1796,11 +1813,11 @@ var MFAGate = function (_a) {
         });
     }); }, [context, onAuthError]);
     useEffect(function () {
-        if (context.isLoading) {
+        if (contextIsLoading) {
             setAuthState("Loading");
             return;
         }
-        if (context.user) {
+        if (contextHasUser) {
             setAuthState("Authenticated");
             return;
         }
@@ -1809,14 +1826,34 @@ var MFAGate = function (_a) {
             return;
         }
         var shouldRequireMFA = requiresMFA !== null && requiresMFA !== void 0 ? requiresMFA : contextRequiresMFA;
-        setAuthState(shouldRequireMFA ? "MFA_Required" : "Authenticated");
+        if (shouldRequireMFA) {
+            setAuthState("MFA_Required");
+            return;
+        }
+        // Only fire once — guard against re-running while already completing/authenticated
+        setAuthState(function (prev) {
+            if (prev === "Completing" || prev === "Authenticated")
+                return prev;
+            return "Completing";
+        });
     }, [
-        context.isLoading,
-        context.user,
+        contextIsLoading,
+        contextHasUser,
         contextRequiresMFA,
         contextAccessToken,
         requiresMFA,
     ]);
+    // Separate effect: fire callback and advance to Authenticated only when Completing
+    useEffect(function () {
+        if (authState !== "Completing")
+            return;
+        var cancelled = false;
+        fireOnAuthSuccess({ accessToken: contextAccessToken, idToken: contextIdToken }).then(function () {
+            if (!cancelled)
+                setAuthState("Authenticated");
+        });
+        return function () { cancelled = true; };
+    }, [authState, contextAccessToken, contextIdToken, fireOnAuthSuccess]);
     if (useADLogin && !CredentialsAuthEndpoint)
         return jsx(ErrorFallback, { message: "Invalid AD Login Configuration." });
     if (!ValidatedMFAEndpoints)
@@ -1832,6 +1869,8 @@ var MFAGate = function (_a) {
             return fallback;
         case "MFA_Required":
             return (jsx(MFAOptions, { MFAEndpoints: ValidatedMFAEndpoints, onComplete: handleComplete, onAuthFailed: handleAuthFailed }));
+        case "Completing":
+            return fallback;
         case "Authenticated":
             return jsx(Fragment, { children: children });
         case "Unauthenticated":