Merge pull request #1409 from athenacfr/fix/cjs-esm-compat

fix: bundle ESM-only deps into CJS output with Rollup

Author: remarkablemark

rollup.config.mjs

@@ -87,8 +87,35 @@ const esmConfigs = [
   },
 ];
 
+const cjsConfigs = [
+  {
+    input: 'src/index.ts',
+    output: {
+      file: 'lib/index.js',
+      format: 'cjs',
+      exports: 'named',
+      sourcemap: true,
+    },
+    plugins: getPlugins({ browser: false, outDir: 'lib' }),
+  },
+  {
+    input: 'src/client/html-to-dom.ts',
+    output: {
+      file: 'lib/client/html-to-dom.js',
+      format: 'cjs',
+      exports: 'named',
+      sourcemap: true,
+    },
+    plugins: getPlugins({ browser: true, outDir: 'lib/client' }),
+  },
+];
+
 const configs = [];
 
+if (process.env.CJS === 'true') {
+  configs.push(...cjsConfigs);
+}
+
 if (process.env.ESM === 'true') {
   configs.push(...esmConfigs);
 }

scripts/build.sh

@@ -1,8 +1,9 @@
 #!/usr/bin/env bash
 
 if [[ $1 == '--cjs' ]]; then
-  tsc --project tsconfig.build.json
+  tsc --project tsconfig.build.json --emitDeclarationOnly
   find lib -type f \( -name '*.mjs*' -or -name '*.mts*' \) -delete
+  rollup --config --failAfterWarnings --environment CJS:true
 
 elif [[ $1 == '--esm' ]]; then
   if [[ ! -z $(git status --porcelain src) ]]; then