upgraded dependencies

there were a couple of issues with the previous release, now fixed:
* Rollup's CommonJS plugin broke compatibility with certain libraries
* Rollup's Babel plugin now prompts us to make an explicit choice on how
  to deal with its helpers

on the latter, I'm somewhat uncomfortable prescribing that choice for
everyone, but for now it seems better than going down the route of
making our abstractions more leaky - doing so would require a concrete
scenario that helps us better understand what faucet's options are here

Author: FND

CHANGELOG.md

@@ -2,6 +2,21 @@ faucet-pipeline-js version history
 ==================================
 
 
+v2.1.5
+------
+
+_2020-05-22_
+
+notable changes for end users:
+
+* fixed a compatibility issues with CommonJS modules
+* silenced a warning when transpiling ESNext
+
+no significant changes for developers
+
+(skipping v2.1.4 due to a technical glitch in the release process)
+
+
 v2.1.3
 ------
 

lib/bundle/babel.js

@@ -3,7 +3,9 @@
 let { loadExtension } = require("faucet-pipeline-core/lib/util");
 
 module.exports = function generateTranspiler({ esnext, jsx, exclude }, { browsers }) {
-	let settings = {};
+	let settings = {
+		babelHelpers: "bundled"
+	};
 	let plugins = [];
 	let extensions = [];
 

lib/bundle/config.js

@@ -4,7 +4,7 @@
 let generateTranspiler = require("./babel");
 let { loadExtension, abort, repr } = require("faucet-pipeline-core/lib/util");
 let commonjs = require("@rollup/plugin-commonjs");
-let nodeResolve = require("@rollup/plugin-node-resolve");
+let { nodeResolve } = require("@rollup/plugin-node-resolve");
 
 let MODULE_FORMATS = { // maps faucet identifiers to Rollup identifiers
 	esm: true,

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "faucet-pipeline-js",
-	"version": "2.1.3",
+	"version": "2.1.5",
 	"description": "JavaScript module bundling for faucet-pipeline",
 	"author": "FND",
 	"contributors": [
@@ -27,20 +27,20 @@
 	},
 	"dependencies": {
 		"faucet-pipeline-core": "^1.4.0",
-		"rollup": "^2.10.2",
+		"rollup": "^2.10.7",
 		"rollup-plugin-cleanup": "~3.1.1",
-		"@rollup/plugin-commonjs": "~11.1.0",
-		"@rollup/plugin-node-resolve": "~7.1.3"
+		"@rollup/plugin-commonjs": "~12.0.0",
+		"@rollup/plugin-node-resolve": "~8.0.0"
 	},
 	"devDependencies": {
 		"eslint-config-fnd-jsx": "^1.8.0",
 		"faucet-pipeline-esnext": "file:pkg/faucet-pipeline-esnext",
+		"faucet-pipeline-jsmin": "file:pkg/faucet-pipeline-jsmin",
 		"faucet-pipeline-jsx": "file:pkg/faucet-pipeline-jsx",
 		"faucet-pipeline-typescript": "file:pkg/faucet-pipeline-typescript",
-		"faucet-pipeline-jsmin": "file:pkg/faucet-pipeline-jsmin",
 		"json-diff": "^0.5.4",
 		"mocha": "^7.1.2",
 		"npm-run-all": "^4.1.5",
-		"release-util-fnd": "^2.0.0"
+		"release-util-fnd": "^2.0.1"
 	}
 }

pkg/faucet-pipeline-esnext/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "faucet-pipeline-esnext",
-	"version": "2.1.3",
+	"version": "2.1.5",
 	"description": "ES6 and beyond for faucet-pipeline",
 	"author": "FND",
 	"license": "Apache-2.0",
@@ -15,7 +15,7 @@
 	"dependencies": {
 		"@babel/core": "~7.9.6",
 		"@babel/preset-env": "~7.9.6",
-		"@rollup/plugin-babel": "~5.0.0",
-		"faucet-pipeline-js": "2.1.3"
+		"@rollup/plugin-babel": "~5.0.2",
+		"faucet-pipeline-js": "2.1.5"
 	}
 }

pkg/faucet-pipeline-jsmin/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "faucet-pipeline-jsmin",
-	"version": "2.1.3",
+	"version": "2.1.5",
 	"description": "JavaScript minification for faucet-pipeline",
 	"author": "FND",
 	"license": "Apache-2.0",

pkg/faucet-pipeline-jsx/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "faucet-pipeline-jsx",
-	"version": "2.1.3",
+	"version": "2.1.5",
 	"description": "JSX for faucet-pipeline",
 	"author": "FND",
 	"license": "Apache-2.0",
@@ -14,6 +14,6 @@
 	},
 	"dependencies": {
 		"@babel/plugin-transform-react-jsx": "~7.9.4",
-		"faucet-pipeline-esnext": "2.1.3"
+		"faucet-pipeline-esnext": "2.1.5"
 	}
 }

pkg/faucet-pipeline-typescript/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "faucet-pipeline-typescript",
-	"version": "2.1.3",
+	"version": "2.1.5",
 	"description": "TypeScript for faucet-pipeline",
 	"author": "FND",
 	"contributors": [
@@ -16,8 +16,8 @@
 		"url": "https://github.com/faucet-pipeline/faucet-pipeline-js/issues"
 	},
 	"dependencies": {
-		"@rollup/plugin-typescript": "~4.1.1",
-		"faucet-pipeline-js": "2.1.3",
-		"typescript": "~3.9.2"
+		"@rollup/plugin-typescript": "~4.1.2",
+		"faucet-pipeline-js": "2.1.5",
+		"typescript": "~3.9.3"
 	}
 }

test/unit/test_bundling.js

@@ -93,8 +93,18 @@ console.log("[\\u2026] ".concat(util)); // eslint-disable-line no-console
 					filepath: path.resolve(FIXTURES_DIR, "./dist/bundle.js"),
 					/* eslint-disable max-len */
 					content: makeBundle(`
-function createCommonjsModule(fn, module) {
-	return module = { exports: {} }, fn(module, module.exports), module.exports;
+function createCommonjsModule(fn, basedir, module) {
+	return module = {
+	  path: basedir,
+	  exports: {},
+	  require: function (path, base) {
+      return commonjsRequire(path, (base === undefined || base === null) ? module.path : base);
+    }
+	}, fn(module, module.exports), module.exports;
+}
+
+function commonjsRequire () {
+	throw new Error('Dynamic requires are not currently supported by @rollup/plugin-commonjs');
 }
 
 var dist = createCommonjsModule(function (module) {