@angular-devkit/build-angular 0.1000.3 → 0.1000.4

package/package.json

@@ -1,16 +1,16 @@
 {
   "name": "@angular-devkit/build-angular",
-  "version": "0.1000.3",
+  "version": "0.1000.4",
   "description": "Angular Webpack Build Facade",
   "experimental": true,
   "main": "src/index.js",
   "typings": "src/index.d.ts",
   "builders": "builders.json",
   "dependencies": {
-    "@angular-devkit/architect": "0.1000.3",
-    "@angular-devkit/build-optimizer": "0.1000.3",
-    "@angular-devkit/build-webpack": "0.1000.3",
-    "@angular-devkit/core": "10.0.3",
+    "@angular-devkit/architect": "0.1000.4",
+    "@angular-devkit/build-optimizer": "0.1000.4",
+    "@angular-devkit/build-webpack": "0.1000.4",
+    "@angular-devkit/core": "10.0.4",
     "@babel/core": "7.9.6",
     "@babel/generator": "7.9.6",
     "@babel/plugin-transform-runtime": "7.9.6",
@@ -18,8 +18,8 @@
     "@babel/runtime": "7.9.6",
     "@babel/template": "7.8.6",
     "@jsdevtools/coverage-istanbul-loader": "3.0.3",
-    "@ngtools/webpack": "10.0.3",
-    "ajv": "6.12.2",
+    "@ngtools/webpack": "10.0.4",
+    "ajv": "6.12.3",
     "autoprefixer": "9.8.0",
     "babel-loader": "8.1.0",
     "browserslist": "^4.9.1",

package/src/angular-cli-files/models/webpack-configs/common.js

@@ -422,6 +422,13 @@ function getCommonConfig(wco) {
                     test: /[\/\\]@angular[\/\\]core[\/\\].+\.js$/,
                     parser: { system: true },
                 },
+                {
+                    // Mark files inside `rxjs/add` as containing side effects.
+                    // If this is fixed upstream and the fixed version becomes the minimum
+                    // supported version, this can be removed.
+                    test: /[\/\\]rxjs[\/\\]add[\/\\].+\.js$/,
+                    sideEffects: true,
+                },
                 {
                     test: /\.m?js$/,
                     exclude: [/[\/\\](?:core-js|\@babel|tslib)[\/\\]/, /(ngfactory|ngstyle)\.js$/],
@@ -440,6 +447,7 @@ function getCommonConfig(wco) {
                                         cacheIdentifier: JSON.stringify({
                                             buildAngular: require('../../../../package.json').version,
                                         }),
+                                        sourceType: 'unambiguous',
                                         presets: [
                                             [
                                                 require.resolve('@babel/preset-env'),

package/src/angular-cli-files/utilities/stats.js

@@ -79,10 +79,11 @@ function statsWarningsToString(json, statsConfig) {
     const y = (x) => colors ? bold(yellow(x)) : x;
     const warnings = [...json.warnings];
     if (json.children) {
-        warnings.push(...json.children.map((c) => c.warnings));
+        warnings.push(...json.children
+            .map((c) => c.warnings)
+            .reduce((a, b) => [...a, ...b], []));
     }
     return rs('\n' + warnings
-        .filter(m => !!m)
         .map((warning) => `${warning}`)
         .filter((warning) => !ERRONEOUS_WARNINGS.some((erroneous) => erroneous.test(warning)))
         .map((warning) => y(`WARNING in ${warning}`))
@@ -95,10 +96,11 @@ function statsErrorsToString(json, statsConfig) {
     const r = (x) => colors ? bold(red(x)) : x;
     const errors = [...json.errors];
     if (json.children) {
-        errors.push(...json.children.map((c) => c.errors));
+        errors.push(...json.children
+            .map((c) => c.errors)
+            .reduce((a, b) => [...a, ...b], []));
     }
     return rs('\n' + errors
-        .filter(m => !!m)
         .map((error) => r(`ERROR in ${error}`))
         .join('\n\n'));
 }

package/src/utils/load-translations.js

@@ -14,15 +14,35 @@ async function createTranslationLoader() {
     const { parsers, diagnostics } = await importParsers();
     return (path) => {
         const content = fs.readFileSync(path, 'utf8');
+        const unusedParsers = new Map();
         for (const [format, parser] of Object.entries(parsers)) {
-            if (parser.canParse(path, content)) {
-                const result = parser.parse(path, content);
+            const analysis = analyze(parser, path, content);
+            if (analysis.canParse) {
+                const translationBundle = parser.parse(path, content, analysis.hint);
                 const integrity = 'sha256-' + crypto_1.createHash('sha256').update(content).digest('base64');
-                return { format, translation: result.translations, diagnostics, integrity };
+                return { format, translation: translationBundle.translations, diagnostics, integrity };
             }
+            else {
+                unusedParsers.set(parser, analysis);
+            }
+        }
+        const messages = [];
+        for (const [parser, analysis] of unusedParsers.entries()) {
+            messages.push(analysis.diagnostics.formatDiagnostics(`*** ${parser.constructor.name} ***`));
         }
-        throw new Error('Unsupported translation file format.');
+        throw new Error(`Unsupported translation file format in ${path}. The following parsers were tried:\n` + messages.join('\n'));
     };
+    // TODO: `parser.canParse()` is deprecated; remove this polyfill once we are sure all parsers provide the `parser.analyze()` method.
+    // tslint:disable-next-line: no-any
+    function analyze(parser, path, content) {
+        if (parser.analyze !== undefined) {
+            return parser.analyze(path, content);
+        }
+        else {
+            const hint = parser.canParse(path, content);
+            return { canParse: hint !== false, hint, diagnostics };
+        }
+    }
 }
 exports.createTranslationLoader = createTranslationLoader;
 async function importParsers() {