From 944a3f04cdbc5d3ff83912edb58a96fc726c496d Mon Sep 17 00:00:00 2001
From: Tobbe Lundberg <tobbe@tlundberg.com>
Date: Wed, 4 Sep 2024 15:56:27 +0200
Subject: [PATCH] fix(rsc): Don't run the Vite dev server in production

---
 packages/vite/src/rsc/rscWorker.ts | 49 +-----------------------------
 1 file changed, 1 insertion(+), 48 deletions(-)

diff --git a/packages/vite/src/rsc/rscWorker.ts b/packages/vite/src/rsc/rscWorker.ts
index 282d19484644..5867c29a0287 100644
--- a/packages/vite/src/rsc/rscWorker.ts
+++ b/packages/vite/src/rsc/rscWorker.ts
@@ -4,7 +4,6 @@
 // couldn't do SSR because it would be missing client-side React functions
 // like `useState` and `createContext`.
 import type { Buffer } from 'node:buffer'
-import { Server } from 'node:http'
 import path from 'node:path'
 import { Writable } from 'node:stream'
 import { parentPort } from 'node:worker_threads'
@@ -13,7 +12,7 @@ import { createElement } from 'react'
 
 import RSDWServer from 'react-server-dom-webpack/server'
 import type { ResolvedConfig } from 'vite'
-import { createServer, resolveConfig } from 'vite'
+import { resolveConfig } from 'vite'
 
 import { getPaths } from '@redwoodjs/project-config'
 import {
@@ -24,10 +23,6 @@ import {
 import { getEntriesFromDist } from '../lib/entries.js'
 import { registerFwGlobalsAndShims } from '../lib/registerFwGlobalsAndShims.js'
 import { StatusError } from '../lib/StatusError.js'
-import { rscReloadPlugin } from '../plugins/vite-plugin-rsc-reload.js'
-import { rscRoutesAutoLoader } from '../plugins/vite-plugin-rsc-routes-auto-loader.js'
-import { rscTransformUseClientPlugin } from '../plugins/vite-plugin-rsc-transform-client.js'
-import { rscTransformUseServerPlugin } from '../plugins/vite-plugin-rsc-transform-server.js'
 
 import type {
   MessageReq,
@@ -132,49 +127,7 @@ const handleRender = async ({ id, input }: MessageReq & { type: 'render' }) => {
 // server. So we have to register them here again.
 registerFwGlobalsAndShims()
 
-// TODO (RSC): this was copied from waku; they have a todo to remove it.
-// We need this to fix a WebSocket error in dev, `WebSocket server error: Port
-// is already in use`.
-const dummyServer = new Server()
-
-// TODO (RSC): `createServer` is mostly used to create a dev server. Is it OK
-// to use it like a production server like this?
-// TODO (RSC): Do we need to pass `define` here with RWJS_ENV etc? What about
-// `envFile: false`?
-// TODO (RSC): Do we need to care about index.html as it says in the docs
-// https://github.com/vitejs/vite-plugin-react/tree/main/packages/plugin-react#middleware-mode
-const vitePromise = createServer({
-  plugins: [
-    rscReloadPlugin((type) => {
-      if (!parentPort) {
-        throw new Error('parentPort is undefined')
-      }
-
-      const message: MessageRes = { type }
-      parentPort.postMessage(message)
-    }),
-    rscTransformUseClientPlugin({}),
-    rscTransformUseServerPlugin('', {}),
-    rscRoutesAutoLoader(),
-  ],
-  ssr: {
-    resolve: {
-      // TODO (RSC): Do we need `conditions` too?
-      // conditions: ['react-server'],
-      externalConditions: ['react-server'],
-    },
-  },
-  // Need to run in middlewareMode so that `buildStart` gets called for all
-  // plugins. Specifically vite's own vite:css plugin needs this to initialize
-  // the cssModulesCache WeakMap
-  // See https://github.com/vitejs/vite/issues/3798#issuecomment-862185554
-  server: { middlewareMode: true, hmr: { server: dummyServer } },
-  appType: 'custom',
-})
-
 const shutdown = async () => {
-  const vite = await vitePromise
-  await vite.close()
   if (!parentPort) {
     throw new Error('parentPort is undefined')
   }