ignoreErrors option

Author: medikoo

README.md

@@ -113,6 +113,11 @@ See [extensions](#extensions) section for more info.
 
 Include [source maps][], for easier debugging. Source maps work very well in WebKit and Chrome's web inspector. Firefox's Firebug however has some [issues][firebug issue].
 
+##### ignoreErrors `boolean`
+
+Ignore not parsable require paths (e.g. `require('./lang/' + lang)`) if any.
+Dynamic paths in require calls are considered a bad practice and won't be possible with upcoming _ES6 modules_ standard. Still if we deal with modules that do that, we can workaround it by turning this option on, and including missing modules with [`include`](https://github.com/medikoo/modules-webmake/edit/master/README.md#include-string) option.
+
 ##### cache `boolean` _programmatical usage only_
 
 Cache files content and its calculated dependencies. On repeated request only modified files are re-read and parsed.  

bin/webmake

@@ -23,6 +23,10 @@ var count     = require('es5-ext/lib/Object/count')
 			string: true,
 			description: "Optional extensions"
 		},
+		'ignore-errors': {
+			boolean: true,
+			description: "Ignore unparsable require calls"
+		},
 		include: {
 			string: true,
 			description: "Additional module(s) that should be included (and are" +
@@ -57,6 +61,7 @@ if (argv.include) {
 
 options.ext = argv.ext;
 options.sourceMap = argv.sourcemap;
+options.ignoreErrors = argv['ignore-errors'];
 options.output = argv._[1];
 
 var time = now();

lib/parser.js

@@ -1,6 +1,7 @@
 'use strict';
 
 var commonLeft   = require('es5-ext/lib/Array/prototype/common-left')
+  , compact      = require('es5-ext/lib/Array/prototype/compact')
   , copy         = require('es5-ext/lib/Array/prototype/copy')
   , last         = require('es5-ext/lib/Array/prototype/last')
   , CustomError  = require('es5-ext/lib/Error/custom')
@@ -36,12 +37,18 @@ if (sep == null) {
 }
 dirEndMatch = new RegExp('(?:^|\\' + sep + ')\\.*$');
 
-parseDependencies = function (text, filename) {
+parseDependencies = function (text, filename, ignoreErrors) {
 	return findRequires(text, { raw: true }).map(function (node) {
 		var path = node.value;
 		if (!path) {
-			throw new TypeError("Not supported require call: " + node.raw +
-				" at " + filename + ":" + node.line);
+			if (!ignoreErrors) {
+				throw new CustomError("Not parsable require call: `" + node.raw +
+					"` at " + filename + ":" + node.line + "\n             You may" +
+					" ignore such errors with ignoreErrors option ('ignore-errors'" +
+					" when running from command line)", 'DYNAMIC_REQUIRE');
+			}
+			console.warn("Not parsable require call (ignored): `" + node.raw +
+				"` at " + filename + ":" + node.line);
 		}
 		return path;
 	});
@@ -96,7 +103,8 @@ readFileData = function (filename, parser, localFilename) {
 		}
 
 		if (sLast.call(code) !== '\n') code += '\n';
-		deps = noDeps ? [] : parseDependencies(code, filename);
+		deps = noDeps ? [] :
+				compact.call(parseDependencies(code, filename, parser.ignoreErrors));
 
 		if (parser.sourceMap) {
 			code = 'eval(' + stringify(code + '//@ sourceURL=' +
@@ -426,6 +434,7 @@ module.exports = exports = function (/* options */) {
 		packages: d({}),
 		modulesFiles: d([]),
 		sourceMap: d(Boolean(options.sourceMap)),
+		ignoreErrors: d(Boolean(options.ignoreErrors)),
 		cache: d(Boolean(options.cache)),
 		ext: d(ext),
 		extNames: d(keys(ext))

test/__playground/lib/dynamic.js

@@ -0,0 +1,9 @@
+'use strict';
+
+var lang = 'pl';
+
+try {
+	require('./raz/dwa/' + lang);
+} catch (e) {}
+
+exports.foo = 'bar';

test/webmake.js

@@ -85,6 +85,20 @@ module.exports = {
 			a(program.getZ().name, 'z', "External name");
 		}).end(d);
 	},
+	"Dynamic": {
+		"Error": function (t, a, d) {
+			var input = pg + '/lib/dynamic.js';
+			t(input)(a.never, function (e) { a(e.code, 'DYNAMIC_REQUIRE'); })
+				.done(d, d);
+		},
+		"Ignored": function (t, a, d) {
+			var input = pg + '/lib/dynamic.js';
+			t(input, { ignoreErrors: true })(function (result) {
+				var program = runInNewContext(result, {}, input);
+				a(program.foo, 'bar');
+			}, a.never).done(d, d);
+		}
+	},
 	"Error on native": function (t, a, d) {
 		t(pg + '/require-native.js', function (err) {
 			a.ok(startsWith.call(err.message, "Cannot require"));