fix(webapp): pass isProduction into StaleAssetRecovery as a prop

kathiekiwi · kathiekiwi · commit 52ff02910786 · 2026-07-14T14:11:48.000Z
diff --git a/apps/webapp/app/components/StaleAssetRecovery.tsx b/apps/webapp/app/components/StaleAssetRecovery.tsx
@@ -47,8 +47,8 @@ const script = `(function () {
   });
 })();`;
 
-export function StaleAssetRecovery() {
-  if (process.env.NODE_ENV !== "production") {
+export function StaleAssetRecovery({ isProduction }: { isProduction: boolean }) {
+  if (!isProduction) {
     return null;
   }
 
diff --git a/apps/webapp/app/root.tsx b/apps/webapp/app/root.tsx
@@ -19,6 +19,11 @@ import { getUser } from "./services/session.server";
 import { getTimezonePreference } from "./services/preferences/uiPreferences.server";
 import { appEnvTitleTag } from "./utils";
 
+// Derived here (not inside StaleAssetRecovery) so the shared component takes
+// the flag as a prop. NODE_ENV is statically replaced in browser bundles, and
+// the ErrorBoundary can't rely on loader data.
+const isProduction = process.env.NODE_ENV === "production";
+
 export const links: LinksFunction = () => {
   return [{ rel: "stylesheet", href: tailwindStylesheetUrl }];
 };
@@ -98,7 +103,7 @@ export function ErrorBoundary() {
         <head>
           <meta charSet="utf-8" />
 
-          <StaleAssetRecovery />
+          <StaleAssetRecovery isProduction={isProduction} />
           <Meta />
           <Links />
         </head>
@@ -125,7 +130,7 @@ export default function App() {
     <>
       <html lang="en" className="h-full">
         <head>
-          <StaleAssetRecovery />
+          <StaleAssetRecovery isProduction={isProduction} />
           <Meta />
           <Links />
         </head>

Original file line number	Diff line number	Diff line change
@@ -47,8 +47,8 @@ const script = `(function () {
`47`	`47`	`});`
`48`	`48`	})();`;
`49`	`49`
`50`		`-export function StaleAssetRecovery() {`
`51`		`- if (process.env.NODE_ENV !== "production") {`
	`50`	`+export function StaleAssetRecovery({ isProduction }: { isProduction: boolean }) {`
	`51`	`+ if (!isProduction) {`
`52`	`52`	`return null;`
`53`	`53`	`}`
`54`	`54`