Merge pull request #12567 from mattcompiles/fix-broken-rebuild

lib/Compilation.js

@@ -1872,6 +1872,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 				});
 			}
 
+			this.processDependenciesQueue.invalidate(module);
 			this.processModuleDependencies(module, err => {
 				if (err) return callback(err);
 				this.removeReasonsOfDependencyBlock(module, {

test/configCases/rebuild/rebuildWithNewDependencies/a.js

@@ -0,0 +1 @@
+export default "a.js";

test/configCases/rebuild/rebuildWithNewDependencies/index.js

@@ -0,0 +1,7 @@
+import A from "./a";
+
+it("should compile", function (done) {
+	expect(A).toBe("other-file.js");
+
+	done();
+});

test/configCases/rebuild/rebuildWithNewDependencies/loader.js

@@ -0,0 +1,7 @@
+module.exports = function (source) {
+	if (this.shouldReplace)
+		return `import otherFile from './other-file.js';
+		export default otherFile;
+		`;
+	return source;
+};

test/configCases/rebuild/rebuildWithNewDependencies/other-file.js

@@ -0,0 +1 @@
+export default "other-file.js";

test/configCases/rebuild/rebuildWithNewDependencies/webpack.config.js

@@ -0,0 +1,60 @@
+const { resolve, join } = require("path");
+const { NormalModule } = require("../../../../");
+
+/**
+ * @param {import("../../../../").Compiler} compiler the compiler
+ */
+var testPlugin = compiler => {
+	compiler.hooks.compilation.tap("TestPlugin", compilation => {
+		let shouldReplace = false;
+		NormalModule.getCompilationHooks(compilation).loader.tap(
+			"TestPlugin",
+			loaderContext => {
+				/** @type {any} */ (loaderContext).shouldReplace = shouldReplace;
+			}
+		);
+		compilation.hooks.finishModules.tapAsync(
+			"TestPlugin",
+			function (modules, callback) {
+				const src = resolve(join(__dirname, "a.js"));
+
+				/**
+				 *
+				 * @param {any} m test
+				 * @returns {boolean} test
+				 */
+				function matcher(m) {
+					return m.resource && m.resource === src;
+				}
+
+				const module = Array.from(modules).find(matcher);
+
+				if (!module) {
+					throw new Error("something went wrong");
+				}
+
+				shouldReplace = true;
+				compilation.rebuildModule(module, err => {
+					shouldReplace = false;
+					callback(err);
+				});
+			}
+		);
+	});
+};
+
+/** @type {import("../../../../").Configuration} */
+module.exports = {
+	module: {
+		rules: [
+			{
+				test: /a.js/,
+				use: "./loader"
+			}
+		]
+	},
+	optimization: {
+		concatenateModules: false
+	},
+	plugins: [testPlugin]
+};