@angular-devkit/build-angular 13.0.3 → 13.0.4

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "@angular-devkit/build-angular",
-  "version": "13.0.3",
+  "version": "13.0.4",
   "description": "Angular Webpack Build Facade",
   "main": "src/index.js",
   "typings": "src/index.d.ts",
   "builders": "builders.json",
   "dependencies": {
     "@ampproject/remapping": "1.0.1",
-    "@angular-devkit/architect": "0.1300.3",
-    "@angular-devkit/build-webpack": "0.1300.3",
-    "@angular-devkit/core": "13.0.3",
+    "@angular-devkit/architect": "0.1300.4",
+    "@angular-devkit/build-webpack": "0.1300.4",
+    "@angular-devkit/core": "13.0.4",
     "@babel/core": "7.15.8",
     "@babel/generator": "7.15.8",
     "@babel/helper-annotate-as-pure": "7.15.4",
@@ -20,7 +20,7 @@
     "@babel/runtime": "7.15.4",
     "@babel/template": "7.15.4",
     "@discoveryjs/json-ext": "0.5.5",
-    "@ngtools/webpack": "13.0.3",
+    "@ngtools/webpack": "13.0.4",
     "ansi-colors": "4.1.1",
     "babel-loader": "8.2.3",
     "babel-plugin-istanbul": "6.1.1",

package/src/babel/webpack-loader.js

@@ -199,7 +199,13 @@ exports.default = (0, babel_loader_1.custom)(() => {
                 // `@ampproject/remapping` source map objects but both are compatible with Webpack.
                 // This method for merging is used because it provides more accurate output
                 // and is faster while using less memory.
-                result.map = (0, remapping_1.default)([result.map, inputSourceMap], () => null);
+                result.map = {
+                    // Convert the SourceMap back to simple plain object.
+                    // This is needed because otherwise code-coverage will fail with `don't know how to turn this value into a node`
+                    // Which is thrown by Babel if it is invoked again from `istanbul-lib-instrument`.
+                    // https://github.com/babel/babel/blob/780aa48d2a34dc55f556843074b6aed45e7eabeb/packages/babel-types/src/converters/valueToNode.ts#L115-L130
+                    ...(0, remapping_1.default)([result.map, inputSourceMap], () => null),
+                };
             }
             return result;
         },

package/src/builders/extract-i18n/ivy-extract-loader.js

@@ -29,6 +29,10 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const nodePath = __importStar(require("path"));
 const load_esm_1 = require("../../utils/load-esm");
 function localizeExtractLoader(content, map) {
+    // This loader is not cacheable due to how message extraction works.
+    // Extracted messages are not part of webpack pipeline and hence they cannot be retrieved from cache.
+    // TODO: We should investigate in the future on making this deterministic and more cacheable.
+    this.cacheable(false);
     const options = this.getOptions();
     const callback = this.async();
     extract(this, content, map, options).then(() => {

package/src/webpack/utils/helpers.js

@@ -102,7 +102,7 @@ exports.isPolyfillsEntry = isPolyfillsEntry;
 function getWatchOptions(poll) {
     return {
         poll,
-        ignored: poll === undefined ? '**/$_lazy_route_resources' : 'node_modules/**',
+        ignored: poll === undefined ? '**/$_lazy_route_resources' : '**/node_modules/**',
     };
 }
 exports.getWatchOptions = getWatchOptions;