diff --git a/build-scripts/vite.config.js b/build-scripts/vite.config.js index 7f5b964..29d781c 100644 --- a/build-scripts/vite.config.js +++ b/build-scripts/vite.config.js @@ -20,11 +20,14 @@ module.exports = defineConfig({ // splitVendorChunkPlugin(), ], - //root: './', - //base: './', - // optimizeDeps: { - // include: ['matrix-public-archive-shared'], - // }, + optimizeDeps: { + include: [ + // This doesn't seem to be necessary for the this package to work (ref + // https://vitejs.dev/guide/dep-pre-bundling.html#monorepos-and-linked-dependencies) + // + //'matrix-public-archive-shared' + ], + }, resolve: { alias: { // The `file:` packages don't seem resolve correctly so let's add an alias as well @@ -72,9 +75,17 @@ module.exports = defineConfig({ // Copy things like the version files from `public/` to `dist/` copyPublicDir: true, - // Fix `Error: 'default' is not exported by ...` when importing CommonJS files, see - // https://github.com/vitejs/vite/issues/2679 and docs: - // https://vitejs.dev/guide/dep-pre-bundling.html#monorepos-and-linked-dependencies - commonjsOptions: { include: [/shared/] }, + commonjsOptions: { + include: [ + // Fix `Error: 'default' is not exported by ...` when importing CommonJS files, see + // https://github.com/vitejs/vite/issues/2679 and docs: + // https://vitejs.dev/guide/dep-pre-bundling.html#monorepos-and-linked-dependencies + /shared\//, + + // Make all of our `require()` CommonJS calls compatible in the ESM client build. + // See https://vitejs.dev/guide/dep-pre-bundling.html#monorepos-and-linked-dependencies + /node_modules/, + ], + }, }, }); diff --git a/server/hydrogen-render/render-page-html.js b/server/hydrogen-render/render-page-html.js index 3a3a731..6826119 100644 --- a/server/hydrogen-render/render-page-html.js +++ b/server/hydrogen-render/render-page-html.js @@ -88,7 +88,7 @@ if (atEventId) { ${scripts .map( (scriptUrl) => - `` + `` ) .join('\n')}