Upgrade Classic Remix Compiler to Remix Vite

[uday-rana]

References

Remix Vite: Migrating
How to Fix CJS/ESM Bugs in Remix
remix-run/remix#8978
remix-run/remix#6004 (reply in thread)

[Copilot]

Copilot reviewed 13 out of 17 changed files in this pull request and generated no comments.

Files not reviewed (4)

• .dockerignore: Language not supported
• .oxlintrc.json: Language not supported
• Dockerfile: Language not supported
• package.json: Language not supported

Comments suppressed due to low confidence (2)

eslint.config.mjs:16

• Consider using a standard approach to obtain the directory name in ES modules, for example by using fileURLToPath(new URL('.', import.meta.url)) instead of relying on import.meta.dirname.

const gitignorePath = path.resolve(import.meta.dirname, '.gitignore');

server.ts:15

• Ensure that the server file is executed as an ES module to support top-level await; if not, consider refactoring the initialization of viteDevServer into an async context.

const viteDevServer = process.env.NODE_ENV === 'production' ? undefined : await import('vite').then((vite) => vite.createServer({ server: { middlewareMode: true } }));

[uday-rana]

CI is failing on some runs. I'm going to try using https://github.com/kiliman/remix-express-vite-plugin instead of esbuild

[uday-rana]

I can't figure out how to get the server.ts build working together with Vite, and when trying to use the Remix Express Vite plugin (so we don't need to compile the TypeScript for the server in a separate step), the server doesn't start at all.

If we really want to do this I see two ways forward:

• Use tsx to run server.ts in production: This method is suggested in an official blog post and in GitHub discussions: Using custom typescript express server remix-run/remix#6004, Remix + Vite + Express - no root js files remix-run/remix#8978
• Convert server.ts to server.js - All of the official Remix examples use JavaScript for the server

cc @humphd

[theoforger]

@uday-rana I'd say let's not rush this (and #895) into the upcoming release. Seems like there are still some decisions to be made