Add comment directive to disable import parsing

ooflorent · ooflorent · commit f65e9dac98a0 · 2018-04-13T08:48:33.000+02:00
diff --git a/lib/dependencies/ImportParserPlugin.js b/lib/dependencies/ImportParserPlugin.js
@@ -32,8 +32,25 @@ class ImportParserPlugin {
 
 			const importOptions = parser.getCommentOptions(expr.range);
 			if (importOptions) {
+				if (typeof importOptions.webpackIgnore !== "undefined") {
+					if (typeof importOptions.webpackIgnore !== "boolean") {
+						parser.state.module.warnings.push(
+							new UnsupportedFeatureWarning(
+								parser.state.module,
+								`\`webpackIgnore\` expected a boolean, but received: ${
+									importOptions.webpackIgnore
+								}.`
+							)
+						);
+					} else {
+						// Do not instrument `import()` is `webpackIgnore` is `true`
+						if (importOptions.webpackIgnore) {
+							return false;
+						}
+					}
+				}
 				if (typeof importOptions.webpackChunkName !== "undefined") {
-					if (typeof importOptions.webpackChunkName !== "string")
+					if (typeof importOptions.webpackChunkName !== "string") {
 						parser.state.module.warnings.push(
 							new UnsupportedFeatureWarning(
 								parser.state.module,
@@ -42,10 +59,12 @@ class ImportParserPlugin {
 								}.`
 							)
 						);
-					else chunkName = importOptions.webpackChunkName;
+					} else {
+						chunkName = importOptions.webpackChunkName;
+					}
 				}
 				if (typeof importOptions.webpackMode !== "undefined") {
-					if (typeof importOptions.webpackMode !== "string")
+					if (typeof importOptions.webpackMode !== "string") {
 						parser.state.module.warnings.push(
 							new UnsupportedFeatureWarning(
 								parser.state.module,
@@ -54,7 +73,9 @@ class ImportParserPlugin {
 								}.`
 							)
 						);
-					else mode = importOptions.webpackMode;
+					} else {
+						mode = importOptions.webpackMode;
+					}
 				}
 				if (typeof importOptions.webpackInclude !== "undefined") {
 					if (
diff --git a/test/configCases/parsing/import-ignore/index.js b/test/configCases/parsing/import-ignore/index.js
@@ -0,0 +1,9 @@
+const fs = require("fs");
+const path = require("path");
+const should = require("should");
+
+it("should be able to ignore import()", () => {
+	const source = fs.readFileSync(path.join(__dirname, "bundle1.js"), "utf-8");
+	should(source).containEql(`import(/* webpackIgnore: true */ "./other2.js")`);
+	should(source).not.containEql(`import(/* webpackIgnore: false */ "./other3.js")`);
+});
diff --git a/test/configCases/parsing/import-ignore/other.js b/test/configCases/parsing/import-ignore/other.js
@@ -0,0 +1,2 @@
+import(/* webpackIgnore: true */ "./other2.js");
+import(/* webpackIgnore: false */ "./other3.js");
diff --git a/test/configCases/parsing/import-ignore/other2.js b/test/configCases/parsing/import-ignore/other2.js
@@ -0,0 +1 @@
+export default "other2";
diff --git a/test/configCases/parsing/import-ignore/other3.js b/test/configCases/parsing/import-ignore/other3.js
@@ -0,0 +1 @@
+export default "other3";
diff --git a/test/configCases/parsing/import-ignore/webpack.config.js b/test/configCases/parsing/import-ignore/webpack.config.js
@@ -0,0 +1,12 @@
+module.exports = {
+	entry: {
+		bundle0: "./index.js",
+		bundle1: "./other.js"
+	},
+	output: {
+		filename: "[name].js"
+	},
+	node: {
+		__dirname: false
+	}
+};

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+import(/* webpackIgnore: true */ "./other2.js");`
	`2`	`+import(/* webpackIgnore: false */ "./other3.js");`