fix!: don't delete "main" in package.json (mikeal#7)

* 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.

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

Author: rvagg

src/package/index.js

@@ -53,8 +53,7 @@ class Package {
 
     const exports = {}
     if (!json.exports) {
-      if (!json.main) exports['.'] = { import: this.file(toURL('./index.js')) }
-      exports['.'] = { import: this.file(toURL(json.main)) }
+      exports['.'] = { import: this.file(toURL(json.main || './index.js')) }
     } else {
       for (const [key, value] of Object.entries(json.exports)) {
         if (typeof value === 'string') exports[key] = { import: this.file(toURL(value)) }
@@ -182,7 +181,7 @@ class Package {
     const json = copy(this.pkgjson)
 
     delete json.type
-    delete json.main
+    json.main = `./${join('./cjs', json.main || './index.js')}`
     json.browser = {}
     json.exports = {}
     const _join = (...args) => './' + join(...args)

test/fixtures/pkg-kitchensink/input/package.json

@@ -2,7 +2,7 @@
   "name": "pkg-kitchensink",
   "version": "0.0.0",
   "description": "",
-  "main": "index.js",
+  "main": "src/index.js",
   "type": "module",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"

test/fixtures/pkg-kitchensink/output-notests/package.json

@@ -2,6 +2,7 @@
   "name": "pkg-kitchensink",
   "version": "0.0.0",
   "description": "",
+  "main": "./cjs/src/index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"
   },

test/fixtures/pkg-kitchensink/output-tests/package.json

@@ -2,6 +2,7 @@
   "name": "pkg-kitchensink",
   "version": "0.0.0",
   "description": "",
+  "main": "./cjs/src/index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"
   },