weboptimizer 4.0.46 → 4.0.48

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "weboptimizer",
-  "version": "4.0.46",
+  "version": "4.0.48",
   "description": "A generic web optimizer, (module) bundler and development environment.",
   "keywords": [
     "webpack",
@@ -94,7 +94,7 @@
     "babel-loader": "^10.1.1",
     "babel-plugin-polyfill-corejs3": "^1.0.0",
     "babel-plugin-transform-rewrite-imports": "^1.5.4",
-    "clientnode": "4.0.1480",
+    "clientnode": "4.0.1481",
     "core-js": "^3.49.0",
     "ejs": "^6.0.1",
     "exports-loader": "^5.0.0",

package/webpackConfigurator.js

@@ -779,19 +779,29 @@ if (configuration.path.configuration.json) try {
     treats (and, when async, awaits) as a whole. Re-exporting via "export *"
     preserves the (named) exports of all bundled modules while guaranteeing
     each one is evaluated in the given order.
+
+    NOTE: The barrel and every module it bundles are additionally forced to
+    "sideEffects: true" (see the corresponding module rule below). Otherwise a
+    consuming package declaring "sideEffects": false (e.g. a test bundle whose
+    exports are never imported) would have its bundled modules tree-shaken away
+    completely - resulting for example in a test bundle without any registered
+    test.
 */
 const generatedBarrelModuleFilePaths = [];
+const barrelledModuleFilePaths = [];
 const normalizedEntryInjection = {};
-for (const [chunkName, moduleIDs] of Object.entries(configuration.injection.entry.normalized)) if (Array.isArray(moduleIDs) && moduleIDs.length > 1) {
+for (const [chunkName, moduleIDs] of Object.entries(configuration.injection.entry.normalized)) if (configuration.givenCommandLineArguments[2] === 'test' && Array.isArray(moduleIDs) && moduleIDs.length > 1) {
   const barrelModuleFilePath = resolve(configuration.path.context, `.__${convertToValidVariableName(chunkName)}__.barrel.mjs`);
   await writeFile(barrelModuleFilePath, moduleIDs.map(moduleID => {
     const specifier = stripLoader(moduleID);
-    return `export * from ${JSON.stringify(specifier.startsWith('.') || specifier.startsWith('/') ? specifier : `./${specifier}`)}`;
+    const relativeOrAbsoluteSpecifier = specifier.startsWith('.') || specifier.startsWith('/') ? specifier : `./${specifier}`;
+    barrelledModuleFilePaths.push(resolve(configuration.path.context, relativeOrAbsoluteSpecifier));
+    return `export * from ${JSON.stringify(relativeOrAbsoluteSpecifier)}`;
   }).join('\n') + '\n', {
     encoding: configuration.encoding
   });
   generatedBarrelModuleFilePaths.push(barrelModuleFilePath);
-  normalizedEntryInjection[chunkName] = [`./${relative(configuration.path.context, barrelModuleFilePath)}`];
+  normalizedEntryInjection[chunkName] = [barrelModuleFilePath];
 } else normalizedEntryInjection[chunkName] = moduleIDs;
 if (generatedBarrelModuleFilePaths.length) pluginInstances.push({
   apply: compiler => {
@@ -898,7 +908,17 @@ export let webpackConfiguration = extend(true, {
   // endregion
   mode: configuration.debug ? 'development' : 'production',
   module: {
-    rules: module.additional.pre.map(evaluateAdditionalLoaderConfiguration).concat(loader.ejs, loader.script, loader.html.main, loader.html.ejs, loader.html.html, loader.style, loader.font.eot, loader.font.svg, loader.font.ttf, loader.font.woff, loader.image, loader.data, module.additional.post.map(evaluateAdditionalLoaderConfiguration))
+    rules:
+    /*
+        Force the generated barrel entry modules and every module
+        they bundle to be treated as side-effectful so a consuming
+        package declaring "sideEffects": false does not get them
+        tree-shaken away (which would empty e.g. a test bundle).
+    */
+    (generatedBarrelModuleFilePaths.length ? [{
+      include: generatedBarrelModuleFilePaths.concat(barrelledModuleFilePaths),
+      sideEffects: true
+    }] : []).concat(module.additional.pre.map(evaluateAdditionalLoaderConfiguration)).concat(loader.ejs, loader.script, loader.html.main, loader.html.ejs, loader.html.html, loader.style, loader.font.eot, loader.font.svg, loader.font.ttf, loader.font.woff, loader.image, loader.data, module.additional.post.map(evaluateAdditionalLoaderConfiguration))
   },
   node: configuration.nodeEnvironment,
   optimization: {