fix!: don't delete "main" in package.json

esbuild still doesn't support exports evanw/esbuild#187 it's uncertain as yet what impact this might have on other bundlers and loaders, hence this is a BREAKING CHANGE.
rvagg · Mar 5, 2021 · 8736b90 · 8736b90
1 parent 782fa5e
commit 8736b90
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 1 deletion.
diff --git a/src/package/index.js b/src/package/index.js
@@ -182,7 +182,6 @@ class Package {
     const json = copy(this.pkgjson)
 
     delete json.type
-    delete json.main
     json.browser = {}
     json.exports = {}
     const _join = (...args) => './' + join(...args)

diff --git a/test/fixtures/pkg-kitchensink/output-notests/package.json b/test/fixtures/pkg-kitchensink/output-notests/package.json
@@ -2,6 +2,7 @@
   "name": "pkg-kitchensink",
   "version": "0.0.0",
   "description": "",
+  "main": "index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"
   },

diff --git a/test/fixtures/pkg-kitchensink/output-tests/package.json b/test/fixtures/pkg-kitchensink/output-tests/package.json
@@ -2,6 +2,7 @@
   "name": "pkg-kitchensink",
   "version": "0.0.0",
   "description": "",
+  "main": "index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"
   },