3a4461148 refactor(@angular-devkit/build-angular): initial copy-on-write asset processing support (#15788)

Author: Angular Builds

package.json

@@ -1,21 +1,21 @@
 {
   "name": "@angular-devkit/build-angular",
-  "version": "0.900.0-next.8+22.4749052",
+  "version": "0.900.0-next.8+14.3a44611",
   "description": "Angular Webpack Build Facade",
   "experimental": true,
   "main": "src/index.js",
   "typings": "src/index.d.ts",
   "builders": "builders.json",
   "dependencies": {
-    "@angular-devkit/architect": "github:angular/angular-devkit-architect-builds#47490527f",
-    "@angular-devkit/build-optimizer": "github:angular/angular-devkit-build-optimizer-builds#47490527f",
-    "@angular-devkit/build-webpack": "github:angular/angular-devkit-build-webpack-builds#47490527f",
-    "@angular-devkit/core": "github:angular/angular-devkit-core-builds#47490527f",
+    "@angular-devkit/architect": "github:angular/angular-devkit-architect-builds#3a4461148",
+    "@angular-devkit/build-optimizer": "github:angular/angular-devkit-build-optimizer-builds#3a4461148",
+    "@angular-devkit/build-webpack": "github:angular/angular-devkit-build-webpack-builds#3a4461148",
+    "@angular-devkit/core": "github:angular/angular-devkit-core-builds#3a4461148",
     "@babel/core": "7.6.3",
     "@babel/preset-env": "7.6.3",
-    "@ngtools/webpack": "github:angular/ngtools-webpack-builds#47490527f",
+    "@ngtools/webpack": "github:angular/ngtools-webpack-builds#3a4461148",
     "ajv": "6.10.2",
-    "autoprefixer": "9.6.4",
+    "autoprefixer": "9.6.1",
     "browserslist": "4.7.0",
     "cacache": "13.0.1",
     "caniuse-lite": "1.0.30000999",
@@ -42,7 +42,6 @@
     "postcss-loader": "3.0.0",
     "raw-loader": "3.1.0",
     "regenerator-runtime": "0.13.3",
-    "rollup": "1.21.4",
     "rxjs": "6.5.3",
     "sass": "1.23.0",
     "sass-loader": "8.0.0",
@@ -81,7 +80,7 @@
     "url": "https://github.com/angular/angular-cli.git"
   },
   "engines": {
-    "node": ">= 10.13.0",
+    "node": ">= 10.9.0",
     "npm": ">= 6.11.0",
     "pnpm": ">= 3.2.0",
     "yarn": ">= 1.13.0"

src/angular-cli-files/models/build-options.d.ts

@@ -8,7 +8,7 @@
 import { logging } from '@angular-devkit/core';
 import { ParsedConfiguration } from '@angular/compiler-cli';
 import { ScriptTarget } from 'typescript';
-import { AssetPatternClass, Budget, ExtraEntryPoint, I18NMissingTranslation, Localize, OptimizationClass, SourceMapClass } from '../../browser/schema';
+import { AssetPatternClass, Budget, ExtraEntryPoint, OptimizationClass, SourceMapClass } from '../../browser/schema';
 import { NormalizedFileReplacement } from '../../utils/normalize-file-replacements';
 export interface BuildOptions {
     optimization: OptimizationClass;
@@ -17,29 +17,26 @@ export interface BuildOptions {
     resourcesOutputPath?: string;
     aot?: boolean;
     sourceMap: SourceMapClass;
-    /** @deprecated since version 8. use sourceMap instead. */
+    /** @deprecated use sourceMap instead */
     vendorSourceMap?: boolean;
-    /** @deprecated since version 8 */
+    /** @deprecated  */
     evalSourceMap?: boolean;
     vendorChunk?: boolean;
     commonChunk?: boolean;
     baseHref?: string;
     deployUrl?: string;
     verbose?: boolean;
     progress?: boolean;
-    /** @deprecated since version 9. Use 'locales' object in the project metadata instead.*/
     i18nFile?: string;
-    /** @deprecated since version 9. No longer needed as the format will be determined automatically.*/
     i18nFormat?: string;
-    /** @deprecated since version 9. Use 'localize' instead.*/
     i18nLocale?: string;
-    localize?: Localize;
-    i18nMissingTranslation?: I18NMissingTranslation;
+    i18nMissingTranslation?: string;
     extractCss?: boolean;
     bundleDependencies?: 'none' | 'all';
     watch?: boolean;
     outputHashing?: string;
     poll?: number;
+    app?: string;
     deleteOutputPath?: boolean;
     preserveSymlinks?: boolean;
     extractLicenses?: boolean;
@@ -49,12 +46,10 @@ export interface BuildOptions {
     subresourceIntegrity?: boolean;
     serviceWorker?: boolean;
     webWorkerTsConfig?: string;
-    /** @deprecated since version 8 **/
     skipAppShell?: boolean;
     statsJson: boolean;
     forkTypeChecker: boolean;
     profile?: boolean;
-    /** @deprecated since version 8 **/
     es5BrowserSupport?: boolean;
     main: string;
     polyfills?: string;
@@ -73,7 +68,6 @@ export interface BuildOptions {
     rebaseRootRelativeCssUrls?: boolean;
     esVersionInFileName?: boolean;
     scriptTargetOverride?: ScriptTarget;
-    experimentalRollupPass?: boolean;
 }
 export interface WebpackTestOptions extends BuildOptions {
     codeCoverage?: boolean;

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

@@ -20,7 +20,6 @@ const bundle_budget_1 = require("../../plugins/bundle-budget");
 const cleancss_webpack_plugin_1 = require("../../plugins/cleancss-webpack-plugin");
 const named_chunks_plugin_1 = require("../../plugins/named-chunks-plugin");
 const scripts_webpack_plugin_1 = require("../../plugins/scripts-webpack-plugin");
-const webpack_2 = require("../../plugins/webpack");
 const find_up_1 = require("../../utilities/find-up");
 const utils_2 = require("./utils");
 const ProgressPlugin = require('webpack/lib/ProgressPlugin');
@@ -39,50 +38,10 @@ function getCommonConfig(wco) {
     }
     // tslint:disable-next-line:no-any
     const extraPlugins = [];
-    const extraRules = [];
     const entryPoints = {};
     const targetInFileName = utils_2.getEsVersionForFileName(utils_1.fullDifferential ? buildOptions.scriptTargetOverride : tsConfig.options.target, buildOptions.esVersionInFileName);
     if (buildOptions.main) {
-        const mainPath = path.resolve(root, buildOptions.main);
-        entryPoints['main'] = [mainPath];
-        if (buildOptions.experimentalRollupPass) {
-            // NOTE: the following are known problems with experimentalRollupPass
-            // - vendorChunk, commonChunk, namedChunks: these won't work, because by the time webpack
-            // sees the chunks, the context of where they came from is lost.
-            // - webWorkerTsConfig: workers must be imported via a root relative path (e.g.
-            // `app/search/search.worker`) instead of a relative path (`/search.worker`) because
-            // of the same reason as above.
-            // - loadChildren string syntax: doesn't work because rollup cannot follow the imports.
-            // Rollup options, except entry module, which is automatically inferred.
-            const rollupOptions = {};
-            // Add rollup plugins/rules.
-            extraRules.push({
-                test: mainPath,
-                // Ensure rollup loader executes after other loaders.
-                enforce: 'post',
-                use: [{
-                        loader: webpack_2.WebpackRollupLoader,
-                        options: rollupOptions,
-                    }],
-            });
-            // Rollup bundles will include the dynamic System.import that was inside Angular and webpack
-            // will emit warnings because it can't resolve it. We just ignore it.
-            // TODO: maybe use https://webpack.js.org/configuration/stats/#statswarningsfilter instead.
-            // Ignore all "Critical dependency: the request of a dependency is an expression" warnings.
-            extraPlugins.push(new webpack_1.ContextReplacementPlugin(/./));
-            // Ignore "System.import() is deprecated" warnings for the main file and js files.
-            // Might still get them if @angular/core gets split into a lazy module.
-            extraRules.push({
-                test: mainPath,
-                enforce: 'post',
-                parser: { system: true },
-            });
-            extraRules.push({
-                test: /\.js$/,
-                enforce: 'post',
-                parser: { system: true },
-            });
-        }
+        entryPoints['main'] = [path.resolve(root, buildOptions.main)];
     }
     let differentialLoadingNeeded = false;
     if (wco.buildOptions.platform !== 'server') {
@@ -431,7 +390,6 @@ function getCommonConfig(wco) {
                     enforce: 'pre',
                     ...sourceMapUseRule,
                 },
-                ...extraRules,
             ],
         },
         optimization: {

src/angular-cli-files/plugins/webpack-rollup-loader.d.ts

@@ -13,4 +13,3 @@ export { RemoveHashPlugin, RemoveHashPluginOptions } from './remove-hash-plugin'
 export { NamedLazyChunksPlugin as NamedChunksPlugin } from './named-chunks-plugin';
 export { default as PostcssCliResources, PostcssCliResourcesOptions, } from './postcss-cli-resources';
 export declare const RawCssLoader: string;
-export declare const WebpackRollupLoader: string;

src/angular-cli-files/plugins/webpack-rollup-loader.js

@@ -24,4 +24,3 @@ var postcss_cli_resources_1 = require("./postcss-cli-resources");
 exports.PostcssCliResources = postcss_cli_resources_1.default;
 const path_1 = require("path");
 exports.RawCssLoader = require.resolve(path_1.join(__dirname, 'raw-css-loader'));
-exports.WebpackRollupLoader = require.resolve(path_1.join(__dirname, 'webpack-rollup-loader'));