@nx/webpack 23.1.0 → 23.2.0-beta.1

package/dist/src/executors/webpack/webpack.impl.js

@@ -89,6 +89,11 @@ async function* webpackExecutor(_options, context) {
     // Run build sequentially and bail when first one fails.
     (0, operators_1.mergeScan)((acc, config) => {
         if (!acc.hasErrors()) {
+            // Propagate watch mode from executor options to webpack config.
+            // Without this, NxAppWebpackPlugin-based configs run in single-run
+            // mode even when the executor is invoked with watch: true, causing
+            // @nx/js:node to restart the process after every build.
+            config.watch = options.watch ?? config.watch;
             return (0, run_webpack_1.runWebpack)(config).pipe((0, operators_1.tap)((stats) => {
                 console.info(stats.toString(config.stats));
             }));

package/dist/src/plugins/nx-webpack-plugin/lib/apply-base-config.js

@@ -150,6 +150,10 @@ function applyNxIndependentConfig(options, config) {
                 })
                 : new TerserPlugin({
                     minify: TerserPlugin.swcMinify,
+                    // terser-webpack-plugin 5.6+ forwards `extractComments` into swc's
+                    // minify options, which rejects it as an unknown field. Disable it
+                    // like the babel branch does above.
+                    extractComments: false,
                     // `terserOptions` options will be passed to `swc`
                     terserOptions: {
                         module: true,

package/dist/src/plugins/nx-webpack-plugin/lib/utils.js

@@ -16,12 +16,24 @@ function resolveConditionalExport(target) {
     if (typeof target === 'string') {
         return target;
     }
+    // Fallback arrays: take the first non-empty target. Node resolves them in
+    // order on disk; for the allowlist any valid target marks the subpath.
+    if (Array.isArray(target)) {
+        for (const candidate of target) {
+            const resolved = resolveConditionalExport(candidate);
+            if (resolved) {
+                return resolved;
+            }
+        }
+        return null;
+    }
     if (typeof target === 'object' && target !== null) {
         // Priority order for conditions
         const conditions = ['development', 'import', 'require', 'default'];
         for (const condition of conditions) {
-            if (target[condition] && typeof target[condition] === 'string') {
-                return target[condition];
+            const resolved = resolveConditionalExport(target[condition]);
+            if (resolved) {
+                return resolved;
             }
         }
     }

package/dist/src/utils/webpack/plugins/postcss-cli-resources.js

@@ -4,7 +4,6 @@ exports.PostcssCliResources = PostcssCliResources;
 const tslib_1 = require("tslib");
 const loader_utils_1 = require("loader-utils");
 const path = tslib_1.__importStar(require("path"));
-const url_1 = require("url");
 function wrapUrl(url) {
     let wrappedUrl;
     const hasSingleQuotes = url.indexOf("'") >= 0;
@@ -80,14 +79,19 @@ function PostcssCliResources(options) {
         // Separate URL query/hash from the file path before resolving
         const [, filePath, urlSuffix] = inputUrl.match(/^([^?#]*)(.*)$/);
         const resolvedPath = path.resolve(context, filePath.replace(/\\/g, '/'));
-        const { pathname } = (0, url_1.pathToFileURL)(resolvedPath);
+        // Resolve a relative filesystem path, not a file:// URL pathname: on Windows the
+        // pathname is `/C:/...` (spaces as %20), which the resolver cannot find. Relative
+        // (not absolute) lets `resolve()`'s `./`-prefixed lookup succeed on the first try.
+        const resolveRequest = path
+            .relative(context, resolvedPath)
+            .replace(/\\/g, '/');
         let hash = '';
         let search = '';
         if (urlSuffix) {
             ({ hash, search } = new URL(`file:///dummy${urlSuffix}`));
         }
         const resolver = (file, base) => new Promise((resolve, reject) => {
-            loader.resolve(base, decodeURI(file), (err, result) => {
+            loader.resolve(base, file, (err, result) => {
                 if (err) {
                     reject(err);
                     return;
@@ -95,7 +99,7 @@ function PostcssCliResources(options) {
                 resolve(result);
             });
         });
-        const result = await resolve(pathname, context, resolver);
+        const result = await resolve(resolveRequest, context, resolver);
         return new Promise((resolve, reject) => {
             loader.fs.readFile(result, (err, content) => {
                 if (err) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/webpack",
-  "version": "23.1.0",
+  "version": "23.2.0-beta.1",
   "private": false,
   "type": "commonjs",
   "files": [
@@ -118,8 +118,8 @@
     "tslib": "^2.3.0",
     "webpack-node-externals": "^3.0.0",
     "webpack-subresource-integrity": "^5.1.0",
-    "@nx/devkit": "23.1.0",
-    "@nx/js": "23.1.0"
+    "@nx/devkit": "23.2.0-beta.1",
+    "@nx/js": "23.2.0-beta.1"
   },
   "peerDependencies": {
     "webpack": "^5.0.0",
@@ -138,7 +138,7 @@
     }
   },
   "devDependencies": {
-    "nx": "23.1.0"
+    "nx": "23.2.0-beta.1"
   },
   "publishConfig": {
     "access": "public"