harmony modules without exports have no exports instead of unknown

webpack · Jun 14, 2017 · d6a7594 · d6a7594
1 parent 7eb7529
commit d6a7594
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/lib/FlagDependencyExportsPlugin.js b/lib/FlagDependencyExportsPlugin.js
@@ -19,7 +19,7 @@ class FlagDependencyExportsPlugin {
 					module = queue[i];
 
 					if(module.providedExports !== true) {
-						moduleWithExports = false;
+						moduleWithExports = module.meta && module.meta.harmonyModule;
 						moduleProvidedExports = Array.isArray(module.providedExports) ? new Set(module.providedExports) : new Set();
 						processDependenciesBlock(module);
 						if(!moduleWithExports) {

diff --git a/lib/Stats.js b/lib/Stats.js
@@ -641,7 +641,10 @@ class Stats {
 		const processModuleContent = (module, prefix) => {
 			if(Array.isArray(module.providedExports)) {
 				colors.normal(prefix);
-				colors.cyan(`[exports: ${module.providedExports.join(", ")}]`);
+				if(module.providedExports.length === 0)
+					colors.cyan("[no exports]");
+				else
+					colors.cyan(`[exports: ${module.providedExports.join(", ")}]`);
 				newline();
 			}
 			if(module.usedExports !== undefined) {

diff --git a/test/statsCases/tree-shaking/expected.txt b/test/statsCases/tree-shaking/expected.txt
@@ -11,6 +11,7 @@ bundle.js  7.33 kB       0  [emitted]  main
    [2] (webpack)/test/statsCases/tree-shaking/unknown.js 0 bytes {0} [built]
        [only some exports used: c]
    [3] (webpack)/test/statsCases/tree-shaking/index.js 276 bytes {0} [built]
+       [no exports]
    [4] (webpack)/test/statsCases/tree-shaking/reexport-known.js 49 bytes {0} [built]
        [exports: a, b]
        [only some exports used: a]