fix(remix-dev): ensure devServerPort config value is the same across reloads

contributors.yml

@@ -306,6 +306,7 @@
 - SufianBabri
 - supachaidev
 - tascord
+- tessellator
 - TheRealAstoo
 - therealflyingcoder
 - thomasheyenbrock

packages/remix-dev/__tests__/readConfig-test.ts

@@ -52,4 +52,9 @@ describe("readConfig", () => {
     `
     );
   });
+
+  it("returns the same devServerPort value across reloads", async () => {
+    let newConfig = await readConfig(remixRoot);
+    expect(newConfig.devServerPort).toBe(config.devServerPort);
+  });
 });

packages/remix-dev/config.ts

@@ -8,6 +8,11 @@ import { defineConventionalRoutes } from "./config/routesConvention";
 import { ServerMode, isValidServerMode } from "./config/serverModes";
 import { serverBuildVirtualModule } from "./compiler/virtualModules";
 
+/* eslint-disable prefer-let/prefer-let */
+declare global {
+  var __devServerPort: number | undefined;
+}
+
 export interface RemixMdxConfig {
   rehypePlugins?: any[];
   remarkPlugins?: any[];
@@ -358,7 +363,10 @@ export async function readConfig(
       process.env.REMIX_DEV_SERVER_WS_PORT || 8002
     );
   }
-  let devServerPort = await getPort({ port: appConfig.devServerPort });
+  if (!global.__devServerPort) {
+    global.__devServerPort = await getPort({ port: appConfig.devServerPort });
+  }
+  let devServerPort = global.__devServerPort;
   // set env variable so un-bundled servers can use it
   process.env.REMIX_DEV_SERVER_WS_PORT = `${devServerPort}`;
   let devServerBroadcastDelay = appConfig.devServerBroadcastDelay || 0;