future flag: unstable_optimizeDeps (#12292)

* future flag: unstable_optimizeDeps

* refactor: remove unused code

Author: pcattori

packages/react-router-dev/config/config.ts

@@ -83,7 +83,9 @@ export type ServerBundlesBuildManifest = BaseBuildManifest & {
 
 type ServerModuleFormat = "esm" | "cjs";
 
-interface FutureConfig {}
+interface FutureConfig {
+  unstable_optimizeDeps: boolean;
+}
 
 export type BuildManifest = DefaultBuildManifest | ServerBundlesBuildManifest;
 
@@ -475,7 +477,10 @@ async function resolveConfig({
     );
   }
 
-  let future: FutureConfig = {};
+  let future: FutureConfig = {
+    unstable_optimizeDeps:
+      reactRouterUserConfig.future?.unstable_optimizeDeps ?? false,
+  };
 
   let reactRouterConfig: ResolvedReactRouterConfig = deepFreeze({
     appDirectory,

packages/react-router-dev/vite/plugin.ts

@@ -395,27 +395,6 @@ export let setReactRouterDevLoadContext = (
   reactRouterDevLoadContext = loadContext;
 };
 
-// Inlined from https://github.com/jsdf/deep-freeze
-let deepFreeze = (o: any) => {
-  Object.freeze(o);
-  let oIsFunction = typeof o === "function";
-  let hasOwnProp = Object.prototype.hasOwnProperty;
-  Object.getOwnPropertyNames(o).forEach(function (prop) {
-    if (
-      hasOwnProp.call(o, prop) &&
-      (oIsFunction
-        ? prop !== "caller" && prop !== "callee" && prop !== "arguments"
-        : true) &&
-      o[prop] !== null &&
-      (typeof o[prop] === "object" || typeof o[prop] === "function") &&
-      !Object.isFrozen(o[prop])
-    ) {
-      deepFreeze(o[prop]);
-    }
-  });
-  return o;
-};
-
 type ReactRouterVitePlugin = () => Vite.Plugin[];
 /**
  * React Router [Vite plugin.](https://vitejs.dev/guide/using-plugins.html)
@@ -825,6 +804,14 @@ export const reactRouterVitePlugin: ReactRouterVitePlugin = () => {
             },
           },
           optimizeDeps: {
+            entries: ctx.reactRouterConfig.future.unstable_optimizeDeps
+              ? [
+                  ctx.entryClientFilePath,
+                  ...Object.values(ctx.reactRouterConfig.routes).map((route) =>
+                    path.join(ctx.reactRouterConfig.appDirectory, route.file)
+                  ),
+                ]
+              : [],
             include: [
               // Pre-bundle React dependencies to avoid React duplicates,
               // even if React dependencies are not direct dependencies.