feat(dev,server-runtime): detect built mode via build.mode

.changeset/curly-moons-cover.md

@@ -0,0 +1,9 @@
+---
+"@remix-run/dev": minor
+"@remix-run/server-runtime": minor
+---
+
+detect built mode via `build.mode`
+
+Prevents mode mismatch between built Remix server entry and user-land server.
+Additionally, all runtimes (including non-Node runtimes) can use `build.mode` to determine if HMR should be performed.

packages/remix-dev/__tests__/fixtures/cloudflare/server.ts

@@ -4,5 +4,5 @@ import * as build from "@remix-run/dev/server-build";
 export const onRequest = createPagesFunctionHandler({
   build,
   getLoadContext: (context) => context.env,
-  mode: process.env.NODE_ENV,
+  mode: build.mode,
 });

packages/remix-dev/__tests__/fixtures/deno/server.ts

@@ -5,7 +5,7 @@ import * as build from "@remix-run/dev/server-build";
 
 const remixHandler = createRequestHandlerWithStaticFiles({
   build,
-  mode: Deno.env.get("NODE_ENV"),
+  mode: build.mode,
   getLoadContext: () => ({}),
 });
 

packages/remix-dev/compiler/server/plugins/entry.ts

@@ -41,6 +41,7 @@ ${Object.keys(config.routes)
     )};`;
   })
   .join("\n")}
+  export const mode = ${JSON.stringify(options.mode)};
   export { default as assets } from ${JSON.stringify(
     assetsManifestVirtualModule.id
   )};

packages/remix-dev/server-build.ts

@@ -6,6 +6,7 @@ throw new Error(
     " by the Remix compiler at build time."
 );
 
+export const mode: ServerBuild["mode"] = undefined!;
 export const assets: ServerBuild["assets"] = undefined!;
 export const entry: ServerBuild["entry"] = undefined!;
 export const routes: ServerBuild["routes"] = undefined!;

packages/remix-server-runtime/build.ts

@@ -7,6 +7,7 @@ import type { AppLoadContext } from "./data";
  * The output of the compiler for the server build.
  */
 export interface ServerBuild {
+  mode: string;
   entry: {
     module: ServerEntryModule;
   };