feat!: add optional --main for build to include "main" (mikeal#8)

Don't include "main" in package.json by default, but include it if --main is
passed.

src/build.js

@@ -25,6 +25,9 @@ const run = async opts => {
   pkg.dist = dist
   return pkg
 }
-run.schema = { tests: false }
+run.schema = {
+  tests: false,
+  main: false
+}
 
 export default run

src/package/index.js

@@ -18,13 +18,14 @@ const { writeFile, mkdir, unlink, readdir, readFile, copyFile } = fs
 const plugins = [preserveShebangs.preserveShebangs()]
 
 class Package {
-  constructor ({ cwd, hooks, tests }) {
+  constructor ({ cwd, hooks, tests, main }) {
     this.cwd = cwd
     this.hooks = hooks || {}
     this.parsed = this.parse()
     this.files = new Map()
     this.testFiles = new Map()
     this.includeTests = tests
+    this.includeMain = main
   }
 
   file (url) {
@@ -181,7 +182,11 @@ class Package {
     const json = copy(this.pkgjson)
 
     delete json.type
-    json.main = `./${join('./cjs', json.main || './index.js')}`
+    if (this.includeMain) {
+      json.main = `./${join('./cjs', json.main || './index.js')}`
+    } else {
+      delete json.main
+    }
     json.browser = {}
     json.exports = {}
     const _join = (...args) => './' + join(...args)

test/fixtures/pkg-kitchensink/output-main/cjs/src/browser.js

@@ -0,0 +1,11 @@
+'use strict';
+
+Object.defineProperty(exports, '__esModule', { value: true });
+
+var sub = require('./sub.js');
+var browser = require('./sub/browser.js');
+
+
+
+exports.mod = sub;
+exports.sub = browser;

test/fixtures/pkg-kitchensink/output-main/cjs/src/deno.js

@@ -0,0 +1,11 @@
+'use strict';
+
+Object.defineProperty(exports, '__esModule', { value: true });
+
+var sub = require('./sub.js');
+var index = require('./sub/index.js');
+
+
+
+exports.mod = sub;
+exports.sub = index;

test/fixtures/pkg-kitchensink/output-main/cjs/src/index.js

@@ -0,0 +1,11 @@
+'use strict';
+
+Object.defineProperty(exports, '__esModule', { value: true });
+
+var sub = require('./sub.js');
+var index = require('./sub/index.js');
+
+
+
+exports.mod = sub;
+exports.sub = index;

test/fixtures/pkg-kitchensink/output-main/cjs/src/secondary.js

@@ -0,0 +1,5 @@
+'use strict';
+
+var secondary = 'secondary';
+
+module.exports = secondary;

test/fixtures/pkg-kitchensink/output-main/cjs/src/sub.js

@@ -0,0 +1,5 @@
+'use strict';
+
+var sub = 'sub';
+
+module.exports = sub;

test/fixtures/pkg-kitchensink/output-main/cjs/src/sub/browser.js

@@ -0,0 +1,5 @@
+'use strict';
+
+var browser = 'browser';
+
+module.exports = browser;

test/fixtures/pkg-kitchensink/output-main/cjs/src/sub/index.js

@@ -0,0 +1,5 @@
+'use strict';
+
+var index = 'import';
+
+module.exports = index;

test/fixtures/pkg-kitchensink/output-main/esm/package.json

@@ -0,0 +1 @@
+{ "type" : "module" }

test/fixtures/pkg-kitchensink/output-main/esm/src/browser.js

@@ -0,0 +1,6 @@
+import mod from './sub.js';
+import sub from './sub/browser.js';
+export {
+  mod,
+  sub
+};

test/fixtures/pkg-kitchensink/output-main/esm/src/deno.js

@@ -0,0 +1,6 @@
+import mod from './sub.js';
+import sub from './sub/index.js';
+export {
+  mod,
+  sub
+};

test/fixtures/pkg-kitchensink/output-main/esm/src/index.js

@@ -0,0 +1,6 @@
+import mod from './sub.js';
+import sub from './sub/index.js';
+export {
+  mod,
+  sub
+};

test/fixtures/pkg-kitchensink/output-main/esm/src/secondary.js

@@ -0,0 +1 @@
+export default 'secondary';

test/fixtures/pkg-kitchensink/output-main/esm/src/sub.js

@@ -0,0 +1 @@
+export default 'sub';

test/fixtures/pkg-kitchensink/output-main/esm/src/sub/browser.js

@@ -0,0 +1 @@
+export default 'browser';

test/fixtures/pkg-kitchensink/output-main/esm/src/sub/index.js

@@ -0,0 +1 @@
+export default 'import';

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

@@ -0,0 +1,28 @@
+{
+  "name": "pkg-kitchensink",
+  "version": "0.0.0",
+  "description": "",
+  "main": "./cjs/src/index.js",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "keywords": [],
+  "author": "Mikeal Rogers <[email protected]> (https://www.mikealrogers.com/)",
+  "license": "(Apache-2.0 AND MIT)",
+  "exports": {
+    ".": {
+      "browser": "./esm/src/browser.js",
+      "require": "./cjs/src/index.js",
+      "import": "./esm/src/index.js"
+    },
+    "./secondary": {
+      "browser": "./esm/src/secondary.js",
+      "require": "./cjs/src/secondary.js",
+      "import": "./esm/src/secondary.js"
+    }
+  },
+  "browser": {
+    ".": "./cjs/src/browser.js",
+    "./secondary": "./cjs/src/secondary.js"
+  }
+}

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

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

test/test-build.js

@@ -60,4 +60,13 @@ export default async test => {
     }
     */
   })
+
+  test('pkg-kitchensink w/ main', async test => {
+    const dist = pathToFileURL(await tempy.directory())
+    test.after(() => rmtree(fileURLToPath(dist)))
+    const opts = { cwd, dist, main: true }
+    await build(opts)
+    await verify(new URL('./output-main', url), dist)
+    await verify(dist, new URL('./output-main', url))
+  })
 }