Change project to "type": "module" so we can use pure ESM dependencies #38
Conversation
| import { dirname, resolve } from 'path' | ||
| import { fileURLToPath } from 'url' | ||
| import { createServer as createViteServer } from 'vite' | ||
|
|
||
| const dir = dirname(fileURLToPath(import.meta.url)) |
There was a problem hiding this comment.
These changes are just to replace __dirname, see https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c#what-do-i-use-instead-of-__dirname-and-__filename
| async function prerender () { | ||
| const vite = await createViteServer({ | ||
| server: { middlewareMode: 'ssr' } | ||
| }) |
There was a problem hiding this comment.
The rest of these changes are mostly the same as before. We've just wrapped everything in an async function so that we can create a Vite dev server...
| }) | ||
|
|
||
| try { | ||
| const { Renderer } = await vite.ssrLoadModule('/src/entry-server.jsx') |
There was a problem hiding this comment.
... then use that dev server to access vite.ssrLoadModule.
This is now more consistent with what's happening in our server.js.
| import express from 'express' | ||
| import { createServer as createViteServer } from 'vite' | ||
|
|
||
| const dir = dirname(fileURLToPath(import.meta.url)) |
There was a problem hiding this comment.
As above, these changes are just to replace __dirname https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c#what-do-i-use-instead-of-__dirname-and-__filename
|
|
||
| export default defineConfig({ | ||
| plugins: config.plugins, | ||
| plugins: [react(), mdx({ rehypePlugins: [rehypeSlug] })], |
There was a problem hiding this comment.
We no longer need the extra vite.config.server.js due to the changes in prerender.js, so its config has moved in here.
Closes #28
Also allows upgrading from
vite-plugin-mdxto@mdx-js/rollupso we can get MDX 2