Adjust hasErrorBoundary logic

packages/remix-react/browser.tsx

@@ -33,7 +33,8 @@ export function RemixBrowser(_props: RemixBrowserProps): ReactElement {
   if (!router) {
     let routes = createClientRoutes(
       window.__remixManifest.routes,
-      window.__remixRouteModules
+      window.__remixRouteModules,
+      window.__remixContext.future
     );
 
     let hydrationData = window.__remixContext.state;

packages/remix-react/routes.tsx

@@ -13,6 +13,7 @@ import { fetchData, isCatchResponse, isRedirectResponse } from "./data";
 import { prefetchStyleLinks } from "./links";
 import invariant from "./invariant";
 import { RemixRoute, RemixRouteError } from "./components";
+import { FutureConfig } from "./entry";
 
 export interface RouteManifest<Route> {
   [routeId: string]: Route;
@@ -41,13 +42,18 @@ export interface EntryRoute extends Route {
 export function createServerRoutes(
   manifest: RouteManifest<EntryRoute>,
   routeModules: RouteModules,
+  future: FutureConfig,
   parentId?: string
 ): DataRouteObject[] {
   return Object.values(manifest)
     .filter((route) => route.parentId === parentId)
     .map((route) => {
       let hasErrorBoundary =
-        route.id === "root" || route.hasErrorBoundary || route.hasCatchBoundary;
+        future.v2_errorBoundary === true
+          ? route.id === "root" || route.hasErrorBoundary
+          : route.id === "root" ||
+            route.hasCatchBoundary ||
+            route.hasErrorBoundary;
       let dataRoute: DataRouteObject = {
         caseSensitive: route.caseSensitive,
         element: <RemixRoute id={route.id} />,
@@ -71,13 +77,18 @@ export function createServerRoutes(
 export function createClientRoutes(
   manifest: RouteManifest<EntryRoute>,
   routeModulesCache: RouteModules,
+  future: FutureConfig,
   parentId?: string
 ): DataRouteObject[] {
   return Object.values(manifest)
     .filter((entryRoute) => entryRoute.parentId === parentId)
     .map((route) => {
       let hasErrorBoundary =
-        route.id === "root" || route.hasErrorBoundary || route.hasCatchBoundary;
+        future.v2_errorBoundary === true
+          ? route.id === "root" || route.hasErrorBoundary
+          : route.id === "root" ||
+            route.hasCatchBoundary ||
+            route.hasErrorBoundary;
 
       let dataRoute: DataRouteObject = {
         caseSensitive: route.caseSensitive,

packages/remix-react/server.tsx

@@ -25,7 +25,11 @@ export function RemixServer({ context, url }: RemixServerProps): ReactElement {
   }
 
   let { manifest, routeModules, serverHandoffString } = context;
-  let routes = createServerRoutes(manifest.routes, routeModules);
+  let routes = createServerRoutes(
+    manifest.routes,
+    routeModules,
+    context.future
+  );
   let router = createStaticRouter(routes, context.staticHandlerContext);
 
   return (

packages/remix-server-runtime/routes.ts

@@ -5,6 +5,7 @@ import type {
 } from "@remix-run/router";
 
 import { callRouteActionRR, callRouteLoaderRR } from "./data";
+import { FutureConfig } from "./entry";
 import type { ServerRouteModule } from "./routeModules";
 
 export interface RouteManifest<Route> {
@@ -54,17 +55,21 @@ export function createRoutes(
 // createStaticHandler
 export function createStaticHandlerDataRoutes(
   manifest: ServerRouteManifest,
+  future: FutureConfig,
   parentId?: string
 ): AgnosticDataRouteObject[] {
   return Object.values(manifest)
     .filter((route) => route.parentId === parentId)
     .map((route) => {
+      let hasErrorBoundary =
+        future.v2_errorBoundary === true
+          ? route.id === "root" || route.module.ErrorBoundary != null
+          : route.id === "root" ||
+            route.module.CatchBoundary != null ||
+            route.module.ErrorBoundary != null;
       let commonRoute = {
         // Always include root due to default boundaries
-        hasErrorBoundary:
-          route.id === "root" ||
-          route.module.CatchBoundary != null ||
-          route.module.ErrorBoundary != null,
+        hasErrorBoundary,
         id: route.id,
         path: route.path,
         loader: route.module.loader

packages/remix-server-runtime/server.ts

@@ -34,7 +34,7 @@ export const createRequestHandler: CreateRequestHandlerFunction = (
   mode
 ) => {
   let routes = createRoutes(build.routes);
-  let dataRoutes = createStaticHandlerDataRoutes(build.routes);
+  let dataRoutes = createStaticHandlerDataRoutes(build.routes, build.future);
   let serverMode = isServerMode(mode) ? mode : ServerMode.Production;
   let staticHandler = createStaticHandler(dataRoutes);