Skip to content

Commit

Permalink
fix(webpack): use cjs for emitted webpack files (#395)
Browse files Browse the repository at this point in the history
  • Loading branch information
danielroe authored Jul 28, 2021
1 parent 3c2bc91 commit aed8092
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
6 changes: 4 additions & 2 deletions src/compat.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import fetch from 'node-fetch'
import { resolve } from 'upath'
import { readFile, writeFile } from 'fs-extra'
import { move, readFile, writeFile } from 'fs-extra'
import { build, generate, prepare } from './build'
import { getNitroContext, NitroContext } from './context'
import { createDevServer } from './server/dev'
Expand Down Expand Up @@ -78,7 +78,9 @@ export default function nuxt2CompatModule () {
// Generate mjs resources
nuxt.hook('build:compiled', async ({ name }) => {
if (name === 'server') {
await writeFile(resolve(nuxt.options.buildDir, 'dist/server/server.mjs'), 'export { default } from "./server.js"', 'utf8')
const jsServerEntry = resolve(nuxt.options.buildDir, 'dist/server/server.js')
await move(jsServerEntry, jsServerEntry.replace(/.js$/, '.cjs'))
await writeFile(jsServerEntry.replace(/.js$/, '.mjs'), 'export { default } from "./server.cjs"', 'utf8')
} else if (name === 'client') {
const manifest = await readFile(resolve(nuxt.options.buildDir, 'dist/server/client.manifest.json'), 'utf8')
await writeFile(resolve(nuxt.options.buildDir, 'dist/server/client.manifest.mjs'), 'export default ' + manifest, 'utf8')
Expand Down
5 changes: 4 additions & 1 deletion src/rollup/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,10 @@ export const getRollupConfig = (nitroContext: NitroContext) => {
inline: nitroContext.node === false || nitroContext.inlineDynamicImports,
globbyOptions: {
ignore: [
'server.js'
'client.manifest.mjs',
'server.cjs',
'server.mjs',
'server.manifest.mjs'
]
}
}))
Expand Down
2 changes: 1 addition & 1 deletion src/rollup/plugins/dynamic-require.ts
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ export function dynamicRequire ({ dir, globbyOptions, inline }: Options): Plugin
}

// Scan chunks
const files = await globby('**/*.js', { cwd: dir, absolute: false, ...globbyOptions })
const files = await globby('**/*.{cjs,mjs,js}', { cwd: dir, absolute: false, ...globbyOptions })
const chunks = files.map(id => ({
id,
src: resolve(dir, id).replace(/\\/g, '/'),
Expand Down

0 comments on commit aed8092

Please sign in to comment.