@module-federation/bridge-react 0.0.0-next-20240923073245 → 0.0.0-next-20240924022552

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@module-federation/bridge-react",
-  "version": "0.0.0-next-20240923073245",
+  "version": "0.0.0-next-20240924022552",
   "publishConfig": {
     "access": "public"
   },
@@ -35,13 +35,12 @@
   "dependencies": {
     "@loadable/component": "^5.16.4",
     "react-error-boundary": "^4.0.13",
-    "@module-federation/bridge-shared": "0.0.0-next-20240923073245"
+    "@module-federation/bridge-shared": "0.0.0-next-20240924022552"
   },
   "peerDependencies": {
     "react": ">=16.9.0",
     "react-dom": ">=16.9.0",
-    "react-router-dom": ">=4",
-    "@module-federation/runtime": "0.0.0-next-20240923073245"
+    "react-router-dom": ">=4"
   },
   "devDependencies": {
     "@testing-library/react": "15.0.7",

package/src/create.tsx

@@ -1,11 +1,11 @@
 import React, { forwardRef } from 'react';
+import type { ProviderParams } from '@module-federation/bridge-shared';
+import { LoggerInstance } from './utils';
 import {
   ErrorBoundary,
   ErrorBoundaryPropsWithComponent,
 } from 'react-error-boundary';
-import { LoggerInstance } from './utils';
 import RemoteApp from './remote';
-import type { ProviderParams } from '@module-federation/bridge-shared';
 
 export interface RenderFnParams extends ProviderParams {
   dom?: any;

package/src/provider.tsx

@@ -2,31 +2,23 @@ import { useLayoutEffect, useRef, useState } from 'react';
 import * as React from 'react';
 import ReactDOM from 'react-dom';
 import ReactDOMClient from 'react-dom/client';
+import { RouterContext } from './context';
 import type {
   ProviderParams,
   RenderFnParams,
 } from '@module-federation/bridge-shared';
-import { ErrorBoundary } from 'react-error-boundary';
-import { RouterContext } from './context';
 import { LoggerInstance, atLeastReact18 } from './utils';
+import { ErrorBoundary } from 'react-error-boundary';
 
-type RenderParams = RenderFnParams & any;
-type DestroyParams = {
-  dom: HTMLElement;
-};
 type RootType = HTMLElement | ReactDOMClient.Root;
-
 type ProviderFnParams<T> = {
   rootComponent: React.ComponentType<T>;
   render?: (
     App: React.ReactElement,
     id?: HTMLElement | string,
   ) => RootType | Promise<RootType>;
-  hooks?: {
-    beforeBridgeRender?: (params: RenderFnParams) => void;
-    beforeBridgeDestroy?: (params: DestroyParams) => void;
-  };
 };
+
 export function createBridgeComponent<T>(bridgeInfo: ProviderFnParams<T>) {
   return () => {
     const rootMap = new Map<any, RootType>();
@@ -45,7 +37,7 @@ export function createBridgeComponent<T>(bridgeInfo: ProviderFnParams<T>) {
     };
 
     return {
-      async render(info: RenderParams) {
+      async render(info: RenderFnParams & any) {
         LoggerInstance.log(`createBridgeComponent render Info`, info);
         const {
           moduleName,
@@ -69,16 +61,6 @@ export function createBridgeComponent<T>(bridgeInfo: ProviderFnParams<T>) {
           </ErrorBoundary>
         );
 
-        // call beforeBridgeRender hook
-        if (
-          bridgeInfo?.hooks &&
-          bridgeInfo?.hooks.beforeBridgeRender &&
-          typeof bridgeInfo?.hooks.beforeBridgeRender === 'function'
-        ) {
-          bridgeInfo.hooks.beforeBridgeRender(info);
-        }
-
-        // call render function
         if (atLeastReact18(React)) {
           if (bridgeInfo?.render) {
             // in case bridgeInfo?.render is an async function, resolve this to promise
@@ -96,21 +78,10 @@ export function createBridgeComponent<T>(bridgeInfo: ProviderFnParams<T>) {
           renderFn?.(rootComponentWithErrorBoundary, info.dom);
         }
       },
-      async destroy(info: DestroyParams) {
+      async destroy(info: { dom: HTMLElement }) {
         LoggerInstance.log(`createBridgeComponent destroy Info`, {
           dom: info.dom,
         });
-
-        // call beforeBridgeDestroy hook
-        if (
-          bridgeInfo?.hooks &&
-          bridgeInfo?.hooks.beforeBridgeDestroy &&
-          typeof bridgeInfo?.hooks.beforeBridgeDestroy === 'function'
-        ) {
-          bridgeInfo.hooks.beforeBridgeDestroy(info);
-        }
-
-        // call destroy function
         if (atLeastReact18(React)) {
           const root = rootMap.get(info.dom);
           (root as ReactDOMClient.Root)?.unmount();

package/src/remote/index.tsx

@@ -7,10 +7,9 @@ import React, {
 } from 'react';
 import * as ReactRouterDOM from 'react-router-dom';
 import type { ProviderParams } from '@module-federation/bridge-shared';
+import { LoggerInstance, pathJoin } from '../utils';
 import { dispatchPopstateEnv } from '@module-federation/bridge-shared';
 import { ErrorBoundaryPropsWithComponent } from 'react-error-boundary';
-import { registerBridgeLifeCycle } from '../lifecycle';
-import { LoggerInstance, pathJoin } from '../utils';
 
 declare const __APP_VERSION__: string;
 export interface RenderFnParams extends ProviderParams {
@@ -40,7 +39,6 @@ const RemoteAppWrapper = forwardRef(function (
   props: RemoteAppParams & RenderFnParams,
   ref,
 ) {
-  const bridgeHook = registerBridgeLifeCycle();
   const RemoteApp = () => {
     LoggerInstance.log(`RemoteAppWrapper RemoteApp props >>>`, { props });
     const {
@@ -80,13 +78,6 @@ const RemoteAppWrapper = forwardRef(function (
           `createRemoteComponent LazyComponent render >>>`,
           renderProps,
         );
-
-        if (bridgeHook && bridgeHook?.lifecycle?.beforeBridgeRender) {
-          bridgeHook?.lifecycle?.beforeBridgeRender.emit({
-            ...renderProps,
-          });
-        }
-
         providerReturn.render(renderProps);
       });
 
@@ -98,16 +89,6 @@ const RemoteAppWrapper = forwardRef(function (
               `createRemoteComponent LazyComponent destroy >>>`,
               { moduleName, basename, dom: renderDom.current },
             );
-            if (bridgeHook && bridgeHook?.lifecycle?.beforeBridgeDestroy) {
-              bridgeHook?.lifecycle?.beforeBridgeDestroy.emit({
-                moduleName,
-                dom: renderDom.current,
-                basename,
-                memoryRoute,
-                fallback,
-                ...resProps,
-              });
-            }
             providerInfoRef.current?.destroy({
               dom: renderDom.current,
             });

package/src/lifecycle.ts

@@ -1,29 +0,0 @@
-import { getInstance } from '@module-federation/runtime';
-import helper from '@module-federation/runtime/helpers';
-
-function registerBridgeLifeCycle() {
-  const { registerPlugins, pluginHelper } = helper.global;
-  const host = getInstance();
-  const pluginSystem = new pluginHelper.PluginSystem({
-    beforeBridgeRender: new pluginHelper.SyncHook<
-      [Record<string, any>],
-      void
-    >(),
-    beforeBridgeDestroy: new pluginHelper.SyncHook<
-      [Record<string, any>],
-      void
-    >(),
-  });
-
-  if (host) {
-    registerPlugins<typeof pluginSystem.lifecycle, typeof pluginSystem>(
-      host?.options?.plugins,
-      [pluginSystem],
-    );
-    return pluginSystem;
-  }
-
-  return null;
-}
-
-export { registerBridgeLifeCycle };