@squide/firefly 3.0.1 → 3.0.2

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # @squide/firefly
 
+## 3.0.2
+
+### Patch Changes
+
+- [#122](https://github.com/gsoft-inc/wl-squide/pull/122) [`cda7873`](https://github.com/gsoft-inc/wl-squide/commit/cda7873dcffbf424a625cf40c56a12eacbb2632e) Thanks [@patricklafrance](https://github.com/patricklafrance)! - Internal minor changes
+
+- Updated dependencies [[`cda7873`](https://github.com/gsoft-inc/wl-squide/commit/cda7873dcffbf424a625cf40c56a12eacbb2632e)]:
+  - @squide/msw@2.0.6
+
 ## 3.0.1
 
 ### Patch Changes

package/dist/AppRouter.js

@@ -1 +1 @@
-export { AppRouter, BootstrappingRoute } from './chunk-GSHD4LQA.js';
+export { AppRouter, BootstrappingRoute } from './chunk-MKFAFYJH.js';

package/dist/{chunk-GSHD4LQA.js → chunk-MKFAFYJH.js}

@@ -1,4 +1,4 @@
-import { useLogger } from '@squide/core';
+import { useLogger, isNil } from '@squide/core';
 import { useIsMswStarted } from '@squide/msw';
 import { useIsRouteMatchProtected, useRoutes } from '@squide/react-router';
 import { useAreModulesRegistered, useAreModulesReady } from '@squide/webpack-module-federation';
@@ -8,6 +8,9 @@ import { useLocation, Outlet, createBrowserRouter, RouterProvider } from 'react-
 import { jsx } from 'react/jsx-runtime';
 
 // src/AppRouter.tsx
+function isPromise(value) {
+  return !isNil(value) && !isNil(value.then) && !isNil(value.catch);
+}
 function BootstrappingRoute(props) {
   const {
     fallbackElement,
@@ -38,7 +41,11 @@ function BootstrappingRoute(props) {
       if ((areModulesRegistered || areModulesReady) && isMswStarted) {
         if (!isPublicDataLoaded) {
           logger.debug("[shell] Loading public data.");
-          onLoadPublicData().then(() => {
+          const result = onLoadPublicData();
+          if (!isPromise(result)) {
+            throw Error("[squide] An AppRouter onLoadPublicData handler must return a promise object.");
+          }
+          result.then(() => {
             setIsPublicDataLoaded(true);
             logger.debug("[shell] Public data has been loaded.");
           }).catch((error) => {
@@ -47,7 +54,7 @@ function BootstrappingRoute(props) {
         }
       }
     }
-  }, [logger, areModulesRegistered, areModulesReady, isMswStarted, isPublicDataLoaded, onLoadPublicData]);
+  }, [logger, areModulesRegistered, areModulesReady, isMswStarted, isPublicDataLoaded, showBoundary, onLoadPublicData]);
   const isActiveRouteProtected = useIsRouteMatchProtected(location, { throwWhenThereIsNoMatch: areModulesReady });
   useEffect(() => {
     if (onLoadProtectedData) {
@@ -55,7 +62,11 @@ function BootstrappingRoute(props) {
         if (isActiveRouteProtected) {
           if (!isProtectedDataLoaded) {
             logger.debug(`[shell] Loading protected data as "${location.pathname}" is a protected route.`);
-            onLoadProtectedData().then(() => {
+            const result = onLoadProtectedData();
+            if (!isPromise(result)) {
+              throw Error("[squide] An AppRouter onLoadProtectedData handler must return a promise object.");
+            }
+            result.then(() => {
               setIsProtectedDataLoaded(true);
               logger.debug("[shell] Protected data has been loaded.");
             }).catch((error) => {
@@ -67,7 +78,7 @@ function BootstrappingRoute(props) {
         }
       }
     }
-  }, [logger, location, areModulesRegistered, areModulesReady, isMswStarted, isActiveRouteProtected, isProtectedDataLoaded, onLoadProtectedData]);
+  }, [logger, location, areModulesRegistered, areModulesReady, isMswStarted, isActiveRouteProtected, isProtectedDataLoaded, showBoundary, onLoadProtectedData]);
   useEffect(() => {
     if (onCompleteRegistrations) {
       if (areModulesRegistered && isMswStarted && isPublicDataLoaded) {
@@ -118,7 +129,7 @@ function AppRouter(props) {
         children: routes
       }
     ]);
-  }, [areModulesRegistered, areModulesReady, routes, onLoadPublicData, onLoadProtectedData, onCompleteRegistrations, waitForMsw]);
+  }, [areModulesRegistered, areModulesReady, routes, onLoadPublicData, onLoadProtectedData, onCompleteRegistrations, waitForMsw, errorRenderer, fallbackElement]);
   return /* @__PURE__ */ jsx(RouterProvider, { ...routerProvidersProps, router });
 }
 

package/dist/index.js

@@ -1,4 +1,4 @@
-export { AppRouter, BootstrappingRoute } from './chunk-GSHD4LQA.js';
+export { AppRouter, BootstrappingRoute } from './chunk-MKFAFYJH.js';
 export { FireflyRuntime } from './chunk-6R4K3V6F.js';
 export * from '@squide/core';
 export * from '@squide/msw';

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@squide/firefly",
   "author": "Workleap",
-  "version": "3.0.1",
+  "version": "3.0.2",
   "description": "Helpers to facilitate the creation of a shell package with Squide firefly technology stack.",
   "license": "Apache-2.0",
   "repository": {
@@ -92,9 +92,9 @@
   },
   "dependencies": {
     "@squide/core": "3.1.0",
+    "@squide/msw": "2.0.6",
     "@squide/react-router": "3.0.1",
-    "@squide/webpack-module-federation": "3.0.1",
-    "@squide/msw": "2.0.5"
+    "@squide/webpack-module-federation": "3.0.1"
   },
   "sideEffects": false,
   "engines": {