Remove unused classic compiler HMR logic (#11594)

Author: markdalgleish

packages/react-router/lib/dom/ssr/browser.tsx

@@ -56,146 +56,6 @@ function initSsrInfo(): void {
   }
 }
 
-type HmrInfo = {
-  abortController: AbortController | undefined;
-  routerReadyResolve: (router: RemixRouter) => void;
-  routerReadyPromise: Promise<RemixRouter>;
-};
-
-let hmrInfo: HmrInfo | null = null;
-
-if (
-  import.meta &&
-  // @ts-expect-error
-  import.meta.hot &&
-  typeof window !== "undefined" &&
-  window.__remixManifest
-) {
-  let resolve: (router: RemixRouter) => void;
-  let routerReadyPromise = new Promise((r) => {
-    resolve = r;
-  }).catch(() => {
-    // This is a noop catch handler to avoid unhandled promise rejection warnings
-    // in the console. The promise is never rejected.
-    return undefined;
-  }) as Promise<RemixRouter>;
-
-  hmrInfo = {
-    abortController: undefined,
-    routerReadyResolve: resolve!,
-    // There's a race condition with HMR where the remix:manifest is signaled before
-    // the router is assigned in the RemixBrowser component. This promise gates the
-    // HMR handler until the router is ready
-    routerReadyPromise,
-  };
-
-  // @ts-expect-error
-  import.meta.hot.accept(
-    "remix:manifest",
-    async ({
-      assetsManifest,
-      needsRevalidation,
-    }: {
-      assetsManifest: AssetsManifest;
-      needsRevalidation: Set<string>;
-    }) => {
-      let router = await hmrInfo!.routerReadyPromise;
-      // This should never happen, but just in case...
-      if (!router || !ssrInfo || !hmrInfo) {
-        console.error(
-          "Failed to accept HMR update because the router/ssrInfo was not ready."
-        );
-        return;
-      }
-
-      let routeIds = [
-        ...new Set(
-          router.state.matches
-            .map((m) => m.route.id)
-            .concat(Object.keys(ssrInfo!.routeModules))
-        ),
-      ];
-
-      hmrInfo.abortController?.abort();
-      hmrInfo.abortController = new AbortController();
-      let signal = hmrInfo.abortController.signal;
-
-      // Load new route modules that we've seen.
-      let newRouteModules = Object.assign(
-        {},
-        ssrInfo.routeModules,
-        Object.fromEntries(
-          (
-            await Promise.all(
-              routeIds.map(async (id) => {
-                if (!assetsManifest.routes[id]) {
-                  return null;
-                }
-                let imported = await import(
-                  assetsManifest.routes[id].module +
-                    `?t=${assetsManifest.hmr?.timestamp}`
-                );
-                invariant(ssrInfo, "ssrInfo unavailable for HMR update");
-                return [
-                  id,
-                  {
-                    ...imported,
-                    // react-refresh takes care of updating these in-place,
-                    // if we don't preserve existing values we'll loose state.
-                    default: imported.default
-                      ? ssrInfo.routeModules[id]?.default || imported.default
-                      : imported.default,
-                    ErrorBoundary: imported.ErrorBoundary
-                      ? ssrInfo.routeModules[id]?.ErrorBoundary ||
-                        imported.ErrorBoundary
-                      : imported.ErrorBoundary,
-                    HydrateFallback: imported.HydrateFallback
-                      ? ssrInfo.routeModules[id]?.HydrateFallback ||
-                        imported.HydrateFallback
-                      : imported.HydrateFallback,
-                  },
-                ];
-              })
-            )
-          ).filter(Boolean) as [string, RouteModules[string]][]
-        )
-      );
-
-      Object.assign(ssrInfo.routeModules, newRouteModules);
-      // Create new routes
-      let routes = createClientRoutesWithHMRRevalidationOptOut(
-        needsRevalidation,
-        assetsManifest.routes,
-        ssrInfo.routeModules,
-        ssrInfo.context.state,
-        ssrInfo.context.future,
-        ssrInfo.context.isSpaMode
-      );
-
-      // This is temporary API and will be more granular before release
-      router._internalSetRoutes(routes);
-
-      // Wait for router to be idle before updating the manifest and route modules
-      // and triggering a react-refresh
-      let unsub = router.subscribe((state) => {
-        if (state.revalidation === "idle") {
-          unsub();
-          // Abort if a new update comes in while we're waiting for the
-          // router to be idle.
-          if (signal.aborted) return;
-          // Ensure RouterProvider setState has flushed before re-rendering
-          setTimeout(() => {
-            Object.assign(ssrInfo!.manifest, assetsManifest);
-            window.$RefreshRuntime$!.performReactRefresh();
-          }, 1);
-        }
-      });
-      window.__remixRevalidation = (window.__remixRevalidation || 0) + 1;
-      router.revalidate();
-    }
-  );
-}
-
 function createHydratedRouter(): RemixRouter {
   initSsrInfo();
 
@@ -330,9 +190,6 @@ function createHydratedRouter(): RemixRouter {
   router.createRoutesForHMR = createClientRoutesWithHMRRevalidationOptOut;
   window.__remixRouter = router;
 
-  // Notify that the router is ready for HMR
-  hmrInfo?.routerReadyResolve(router);
-
   return router;
 }