fix(commonjs): use correct version and add package exports (#1038)

lukastaegert · lukastaegert · commit 1424249b02ad · 2022-04-24T07:58:20.000+02:00
diff --git a/packages/commonjs/package.json b/packages/commonjs/package.json
@@ -13,8 +13,12 @@
   "author": "Rich Harris <richard.a.harris@gmail.com>",
   "homepage": "https://github.com/rollup/plugins/tree/master/packages/commonjs/#readme",
   "bugs": "https://github.com/rollup/plugins/issues",
-  "main": "dist/index.js",
-  "module": "dist/index.es.js",
+  "main": "./dist/cjs/index.js",
+  "module": "./dist/es/index.js",
+  "exports": {
+    "require": "./dist/cjs/index.js",
+    "import": "./dist/es/index.js"
+  },
   "engines": {
     "node": ">= 12.0.0"
   },
diff --git a/packages/commonjs/rollup.config.js b/packages/commonjs/rollup.config.js
@@ -1,5 +1,7 @@
 import json from '@rollup/plugin-json';
 
+import { emitModulePackageFile } from '../../shared/rollup.config';
+
 import pkg from './package.json';
 
 export default {
@@ -10,6 +12,7 @@ export default {
     {
       file: pkg.module,
       format: 'es',
+      plugins: [emitModulePackageFile()],
       sourcemap: true
     },
     {
diff --git a/packages/commonjs/test/fixtures/samples/caching/rollupWorker.js b/packages/commonjs/test/fixtures/samples/caching/rollupWorker.js
@@ -2,7 +2,7 @@ const { workerData, parentPort } = require('worker_threads');
 
 const { rollup } = require('rollup');
 
-const commonjs = require('../../../../dist/index');
+const commonjs = require('../../../..');
 const { getCodeFromBundle } = require('../../../helpers/util');
 
 generateCode(workerData);
diff --git a/packages/commonjs/test/helpers/util.js b/packages/commonjs/test/helpers/util.js
@@ -1,6 +1,6 @@
 const path = require('path');
 
-const commonjsPlugin = require('../../dist/index');
+const commonjsPlugin = require('../..');
 
 function commonjs(options) {
   delete require.cache[require.resolve('../..')];
diff --git a/packages/commonjs/test/test.js b/packages/commonjs/test/test.js
@@ -451,7 +451,7 @@ test('does not warn even if the ES module does not export "default"', async (t)
 test('compiles with cache', async (t) => {
   // specific commonjs require() to ensure same instance is used
   // eslint-disable-next-line global-require
-  const commonjsInstance = require('../dist/index');
+  const commonjsInstance = require('..');
 
   const bundle = await rollup({
     input: 'fixtures/function/index/main.js',

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,7 @@`
`1`	`1`	`import json from '@rollup/plugin-json';`
`2`	`2`
	`3`	`+import { emitModulePackageFile } from '../../shared/rollup.config';`
	`4`	`+`
`3`	`5`	`import pkg from './package.json';`
`4`	`6`
`5`	`7`	`export default {`
`@@ -10,6 +12,7 @@ export default {`
`10`	`12`	`{`
`11`	`13`	`file: pkg.module,`
`12`	`14`	`format: 'es',`
	`15`	`+ plugins: [emitModulePackageFile()],`
`13`	`16`	`sourcemap: true`
`14`	`17`	`},`
`15`	`18`	`{`