refactor(router-core,react-router,solid-router,vue-router): Remove long strings from production bundle (#6740)

Sheraff · web-flow · commit 343f8a8ff9ae · 2026-02-23T17:23:46.000+01:00
diff --git a/packages/react-router/src/Matches.tsx b/packages/react-router/src/Matches.tsx
@@ -98,13 +98,17 @@ function MatchesInner() {
         <CatchBoundary
           getResetKey={() => resetKey}
           errorComponent={ErrorComponent}
-          onCatch={(error) => {
-            warning(
-              false,
-              `The following error wasn't caught by any route! At the very least, consider setting an 'errorComponent' in your RootRoute!`,
-            )
-            warning(false, error.message || error.toString())
-          }}
+          onCatch={
+            process.env.NODE_ENV !== 'production'
+              ? (error) => {
+                  warning(
+                    false,
+                    `The following error wasn't caught by any route! At the very least, consider setting an 'errorComponent' in your RootRoute!`,
+                  )
+                  warning(false, error.message || error.toString())
+                }
+              : undefined
+          }
         >
           {matchComponent}
         </CatchBoundary>
diff --git a/packages/react-router/src/fileRoute.ts b/packages/react-router/src/fileRoute.ts
@@ -149,10 +149,12 @@ export class FileRoute<
     TMiddlewares,
     THandlers
   > => {
-    warning(
-      this.silent,
-      'FileRoute is deprecated and will be removed in the next major version. Use the createFileRoute(path)(options) function instead.',
-    )
+    if (process.env.NODE_ENV !== 'production') {
+      warning(
+        this.silent,
+        'FileRoute is deprecated and will be removed in the next major version. Use the createFileRoute(path)(options) function instead.',
+      )
+    }
     const route = createRoute(options as any)
     ;(route as any).isRoot = false
     return route as any
@@ -183,10 +185,12 @@ export function FileRouteLoader<
     >
   >,
 ) => TLoaderFn {
-  warning(
-    false,
-    `FileRouteLoader is deprecated and will be removed in the next major version. Please place the loader function in the the main route file, inside the \`createFileRoute('/path/to/file')(options)\` options`,
-  )
+  if (process.env.NODE_ENV !== 'production') {
+    warning(
+      false,
+      `FileRouteLoader is deprecated and will be removed in the next major version. Please place the loader function in the the main route file, inside the \`createFileRoute('/path/to/file')(options)\` options`,
+    )
+  }
   return (loaderFn) => loaderFn as any
 }
 
diff --git a/packages/router-core/src/router.ts b/packages/router-core/src/router.ts
@@ -1037,10 +1037,12 @@ export class RouterCore<
     TRouterHistory,
     TDehydrated
   > = (newOptions) => {
-    if (newOptions.notFoundRoute) {
-      console.warn(
-        'The notFoundRoute API is deprecated and will be removed in the next major version. See https://tanstack.com/router/v1/docs/framework/react/guide/not-found-errors#migrating-from-notfoundroute for more info.',
-      )
+    if (process.env.NODE_ENV !== 'production') {
+      if (newOptions.notFoundRoute) {
+        console.warn(
+          'The notFoundRoute API is deprecated and will be removed in the next major version. See https://tanstack.com/router/v1/docs/framework/react/guide/not-found-errors#migrating-from-notfoundroute for more info.',
+        )
+      }
     }
 
     const prevOptions = this.options
@@ -2702,7 +2704,9 @@ export class RouterCore<
       isDangerousProtocol(redirect.options.href, this.protocolAllowlist)
     ) {
       throw new Error(
-        `Redirect blocked: unsafe protocol in href "${redirect.options.href}". Allowed protocols: ${Array.from(this.protocolAllowlist).join(', ')}.`,
+        process.env.NODE_ENV !== 'production'
+          ? `Redirect blocked: unsafe protocol in href "${redirect.options.href}". Allowed protocols: ${Array.from(this.protocolAllowlist).join(', ')}.`
+          : 'Redirect blocked: unsafe protocol',
       )
     }
 
diff --git a/packages/solid-router/src/Matches.tsx b/packages/solid-router/src/Matches.tsx
@@ -94,14 +94,18 @@ function MatchesInner() {
         <CatchBoundary
           getResetKey={() => resetKey()}
           errorComponent={ErrorComponent}
-          onCatch={(error) => {
-            warning(
-              false,
-              `The following error wasn't caught by any route! At the very leas
+          onCatch={
+            process.env.NODE_ENV !== 'production'
+              ? (error) => {
+                  warning(
+                    false,
+                    `The following error wasn't caught by any route! At the very leas
     t, consider setting an 'errorComponent' in your RootRoute!`,
-            )
-            warning(false, error.message || error.toString())
-          }}
+                  )
+                  warning(false, error.message || error.toString())
+                }
+              : undefined
+          }
         >
           {matchComponent()}
         </CatchBoundary>
diff --git a/packages/solid-router/src/fileRoute.ts b/packages/solid-router/src/fileRoute.ts
@@ -138,10 +138,12 @@ export class FileRoute<
     TMiddlewares,
     THandlers
   > => {
-    warning(
-      this.silent,
-      'FileRoute is deprecated and will be removed in the next major version. Use the createFileRoute(path)(options) function instead.',
-    )
+    if (process.env.NODE_ENV !== 'production') {
+      warning(
+        this.silent,
+        'FileRoute is deprecated and will be removed in the next major version. Use the createFileRoute(path)(options) function instead.',
+      )
+    }
     const route = createRoute(options as any)
     ;(route as any).isRoot = false
     return route as any
@@ -173,10 +175,12 @@ export function FileRouteLoader<
     >
   >,
 ) => TLoaderFn {
-  warning(
-    false,
-    `FileRouteLoader is deprecated and will be removed in the next major version. Please place the loader function in the the main route file, inside the \`createFileRoute('/path/to/file')(options)\` options`,
-  )
+  if (process.env.NODE_ENV !== 'production') {
+    warning(
+      false,
+      `FileRouteLoader is deprecated and will be removed in the next major version. Please place the loader function in the the main route file, inside the \`createFileRoute('/path/to/file')(options)\` options`,
+    )
+  }
   return (loaderFn) => loaderFn as any
 }
 
diff --git a/packages/vue-router/src/Matches.tsx b/packages/vue-router/src/Matches.tsx
@@ -134,13 +134,16 @@ const MatchesInner = Vue.defineComponent({
       return Vue.h(CatchBoundary, {
         getResetKey: () => resetKey.value,
         errorComponent: errorComponentFn,
-        onCatch: (error: Error) => {
-          warning(
-            false,
-            `The following error wasn't caught by any route! At the very least, consider setting an 'errorComponent' in your RootRoute!`,
-          )
-          warning(false, error.message || error.toString())
-        },
+        onCatch:
+          process.env.NODE_ENV !== 'production'
+            ? (error: Error) => {
+                warning(
+                  false,
+                  `The following error wasn't caught by any route! At the very least, consider setting an 'errorComponent' in your RootRoute!`,
+                )
+                warning(false, error.message || error.toString())
+              }
+            : undefined,
         children: childElement,
       })
     }
diff --git a/packages/vue-router/src/fileRoute.ts b/packages/vue-router/src/fileRoute.ts
@@ -138,10 +138,12 @@ export class FileRoute<
     TMiddlewares,
     THandlers
   > => {
-    warning(
-      this.silent,
-      'FileRoute is deprecated and will be removed in the next major version. Use the createFileRoute(path)(options) function instead.',
-    )
+    if (process.env.NODE_ENV !== 'production') {
+      warning(
+        this.silent,
+        'FileRoute is deprecated and will be removed in the next major version. Use the createFileRoute(path)(options) function instead.',
+      )
+    }
     const route = createRoute(options as any)
     ;(route as any).isRoot = false
     return route as any
@@ -173,10 +175,12 @@ export function FileRouteLoader<
     >
   >,
 ) => TLoaderFn {
-  warning(
-    false,
-    `FileRouteLoader is deprecated and will be removed in the next major version. Please place the loader function in the the main route file, inside the \`createFileRoute('/path/to/file')(options)\` options`,
-  )
+  if (process.env.NODE_ENV !== 'production') {
+    warning(
+      false,
+      `FileRouteLoader is deprecated and will be removed in the next major version. Please place the loader function in the the main route file, inside the \`createFileRoute('/path/to/file')(options)\` options`,
+    )
+  }
   return (loaderFn) => loaderFn as any
 }
 
