Merge pull request #293 from webpack/bugfix/imports-sub-path

Author: sokra

lib/ImportsFieldPlugin.js

@@ -54,15 +54,9 @@ module.exports = class ImportsFieldPlugin {
 			.getHook(this.source)
 			.tapAsync("ImportsFieldPlugin", (request, resolveContext, callback) => {
 				// When there is no description file, abort
-				if (!request.descriptionFilePath) return callback();
-
-				if (
-					// When the description file is inherited from parent, abort
-					// (There is no description file inside of this package)
-					request.relativePath !== "." ||
-					request.request === undefined
-				)
+				if (!request.descriptionFilePath || request.request === undefined) {
 					return callback();
+				}
 
 				const remainingRequest =
 					request.request + request.query + request.fragment;

test/fixtures/imports-field/dir/b.js

@@ -1196,6 +1196,21 @@ describe("ImportsFieldPlugin", () => {
 		});
 	});
 
+	it("should resolve using imports field instead of self-referencing for a subpath", done => {
+		resolver.resolve(
+			{},
+			path.resolve(fixture, "dir"),
+			"#imports-field",
+			{},
+			(err, result) => {
+				if (err) return done(err);
+				if (!result) throw new Error("No result");
+				result.should.equal(path.resolve(fixture, "b.js"));
+				done();
+			}
+		);
+	});
+
 	it("should resolve out of package scope", done => {
 		resolver.resolve({}, fixture, "#b", {}, (err, result) => {
 			if (err) return done(err);