@squide/firefly 3.0.0 → 3.0.2

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @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
+
+- [#118](https://github.com/gsoft-inc/wl-squide/pull/118) [`4864d30`](https://github.com/gsoft-inc/wl-squide/commit/4864d30764021a91d5827abb5b3ae7a4b4302c31) Thanks [@tjosepo](https://github.com/tjosepo)! - Omit "router" from routerProviderProps
+
 ## 3.0.0
 
 ### Minor Changes

package/dist/AppRouter.d.ts

@@ -22,8 +22,8 @@ interface AppRouterProps {
     onLoadProtectedData?: OnLoadProtectedDataFunction;
     onCompleteRegistrations?: OnCompleteRegistrationsFunction;
     waitForMsw: boolean;
-    routerProvidersProps?: RouterProviderProps;
+    routerProvidersProps?: Omit<RouterProviderProps, "router">;
 }
 declare function AppRouter(props: AppRouterProps): react_jsx_runtime.JSX.Element;
 
-export { AppRouter, AppRouterProps, BootstrappingRoute, OnCompleteRegistrationsFunction, OnLoadProtectedDataFunction, OnLoadPublicDataFunction };
+export { AppRouter, type AppRouterProps, BootstrappingRoute, type OnCompleteRegistrationsFunction, type OnLoadProtectedDataFunction, type OnLoadPublicDataFunction };

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/defineConfig.d.ts

@@ -21,4 +21,4 @@ interface FireflyDefineBuildRemoteModuleConfigOptions extends DefineBuildRemoteM
 }
 declare function defineBuildRemoteModuleConfig(swcConfig: SwcConfig, applicationName: string, { features, ...otherOptions }?: FireflyDefineBuildRemoteModuleConfigOptions): webpack.Configuration;
 
-export { FireflyDefineBuildHostConfigOptions, FireflyDefineBuildRemoteModuleConfigOptions, FireflyDefineDevHostConfigOptions, FireflyDefineDevRemoteModuleConfigOptions, FireflyFeatures, defineBuildHostConfig, defineBuildRemoteModuleConfig, defineDevHostConfig, defineDevRemoteModuleConfig };
+export { type FireflyDefineBuildHostConfigOptions, type FireflyDefineBuildRemoteModuleConfigOptions, type FireflyDefineDevHostConfigOptions, type FireflyDefineDevRemoteModuleConfigOptions, type FireflyFeatures, defineBuildHostConfig, defineBuildRemoteModuleConfig, defineDevHostConfig, defineDevRemoteModuleConfig };

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.0",
+  "version": "3.0.2",
   "description": "Helpers to facilitate the creation of a shell package with Squide firefly technology stack.",
   "license": "Apache-2.0",
   "repository": {
@@ -64,14 +64,14 @@
     }
   },
   "devDependencies": {
-    "@swc/core": "1.3.96",
+    "@swc/core": "1.3.99",
     "@swc/helpers": "0.5.3",
     "@swc/jest": "0.2.29",
     "@testing-library/jest-dom": "6.1.4",
-    "@testing-library/react": "14.0.0",
-    "@types/jest": "29.5.7",
-    "@types/react": "18.2.36",
-    "@types/react-dom": "18.2.14",
+    "@testing-library/react": "14.1.2",
+    "@types/jest": "29.5.10",
+    "@types/react": "18.2.39",
+    "@types/react-dom": "18.2.17",
     "@workleap/eslint-plugin": "3.0.0",
     "@workleap/swc-configs": "2.1.2",
     "@workleap/tsup-configs": "3.0.1",
@@ -79,20 +79,20 @@
     "@workleap/webpack-configs": "1.2.1",
     "jest": "29.7.0",
     "jest-environment-jsdom": "29.7.0",
-    "msw": "2.0.3",
+    "msw": "2.0.9",
     "react": "18.2.0",
     "react-dom": "18.2.0",
     "react-error-boundary": "4.0.11",
-    "react-router-dom": "6.18.0",
+    "react-router-dom": "6.20.0",
     "ts-jest": "29.1.1",
-    "tsup": "7.2.0",
+    "tsup": "8.0.1",
     "typescript": "5.2.2",
     "webpack": "5.89.0",
     "@squide/webpack-configs": "1.1.0"
   },
   "dependencies": {
     "@squide/core": "3.1.0",
-    "@squide/msw": "2.0.5",
+    "@squide/msw": "2.0.6",
     "@squide/react-router": "3.0.1",
     "@squide/webpack-module-federation": "3.0.1"
   },