@schematics/angular 8.0.0 → 8.0.4

package/application/index.js

@@ -137,6 +137,7 @@ function addAppToWorkspaceFile(options, appDir) {
                     main: `${sourceRoot}/main.ts`,
                     polyfills: `${sourceRoot}/polyfills.ts`,
                     tsConfig: `${projectRoot}tsconfig.app.json`,
+                    aot: !!options.enableIvy,
                     assets: [
                         `${sourceRoot}/favicon.ico`,
                         `${sourceRoot}/assets`,
@@ -203,7 +204,7 @@ function addAppToWorkspaceFile(options, appDir) {
                     scripts: [],
                 },
             },
-            lint: {
+            lint: options.minimal ? undefined : {
                 builder: workspace_models_1.Builders.TsLint,
                 options: {
                     tsConfig: [

package/component/files/__name@dasherize@if-flat__/__name@dasherize__.component.html.template

@@ -1,3 +1 @@
-<p>
-  <%= dasherize(name) %> works!
-</p>
+<p><%= dasherize(name) %> works!</p>

package/library/index.js

@@ -70,7 +70,12 @@ function addDependenciesToPackageJson() {
             {
                 type: dependencies_1.NodeDependencyType.Dev,
                 name: 'ng-packagr',
-                version: '^5.1.0',
+                version: latest_versions_1.latestVersions.ngPackagr,
+            },
+            {
+                type: dependencies_1.NodeDependencyType.Dev,
+                name: 'tsickle',
+                version: latest_versions_1.latestVersions.tsickle,
             },
             {
                 type: dependencies_1.NodeDependencyType.Dev,

package/migrations/update-8/differential-loading.js

@@ -66,25 +66,15 @@ function updateES5Projects() {
                 updateTsConfig(tree, testConfig.options.tsConfig);
             }
             const browserslistPath = core_1.join(core_1.normalize(project.root), 'browserslist');
-            if (typeof project.sourceRoot === 'string') {
-                // Move the CLI 7 style browserlist to root if it's there.
-                const srcBrowsersList = core_1.join(core_1.normalize(project.sourceRoot), 'browserslist');
-                if (tree.exists(srcBrowsersList) && !tree.exists(browserslistPath)) {
-                    // TODO: use rename instead.
-                    // This is a hacky workaround. We should be able to just rename it.
-                    // On unit tests the rename works fine but on real projects it fails with
-                    //     ERROR! browserslist does not exist..
-                    // This seems to happen because we are both renaming and then commiting an update.
-                    // But it's fine if we read/create/delete. There's a bug somewhere.
-                    // tree.rename(srcBrowsersList, browserslistPath);
-                    const content = tree.read(srcBrowsersList);
-                    if (content) {
-                        tree.create(browserslistPath, content);
-                        tree.delete(srcBrowsersList);
-                    }
-                }
+            // Move the CLI 7 style browserlist to root if it's there.
+            const sourceRoot = project.sourceRoot === 'string'
+                ? project.sourceRoot
+                : core_1.join(core_1.normalize(project.root), 'src');
+            const srcBrowsersList = core_1.join(core_1.normalize(sourceRoot), 'browserslist');
+            if (tree.exists(srcBrowsersList)) {
+                tree.rename(srcBrowsersList, browserslistPath);
             }
-            if (!tree.exists(browserslistPath)) {
+            else if (!tree.exists(browserslistPath)) {
                 tree.create(browserslistPath, browserslistContent);
             }
         }
@@ -111,8 +101,18 @@ function updateTsConfig(tree, tsConfigPath) {
     if (isExtendedConfig) {
         json_utils_1.removePropertyInAstObject(recorder, compilerOptions, 'target');
         json_utils_1.removePropertyInAstObject(recorder, compilerOptions, 'module');
+        json_utils_1.removePropertyInAstObject(recorder, compilerOptions, 'downlevelIteration');
     }
     else {
+        const downlevelIteration = json_utils_1.findPropertyInAstObject(compilerOptions, 'downlevelIteration');
+        if (!downlevelIteration) {
+            json_utils_1.insertPropertyInAstObjectInOrder(recorder, compilerOptions, 'downlevelIteration', true, 4);
+        }
+        else if (!downlevelIteration.value) {
+            const { start, end } = downlevelIteration;
+            recorder.remove(start.offset, end.offset - start.offset);
+            recorder.insertLeft(start.offset, 'true');
+        }
         const scriptTarget = json_utils_1.findPropertyInAstObject(compilerOptions, 'target');
         if (!scriptTarget) {
             json_utils_1.insertPropertyInAstObjectInOrder(recorder, compilerOptions, 'target', 'es2015', 4);

package/migrations/update-8/drop-es6-polyfills.js

@@ -84,7 +84,7 @@ function dropES2015PolyfillsFromFile(polyfillPath) {
             // no action required if no mention of core-js
             return;
         }
-        const sourceFile = ts.createSourceFile(polyfillPath, content, ts.ScriptTarget.Latest, true);
+        const sourceFile = ts.createSourceFile(polyfillPath, content.replace(/^\uFEFF/, ''), ts.ScriptTarget.Latest, true);
         const imports = sourceFile.statements
             .filter(s => s.kind === ts.SyntaxKind.ImportDeclaration);
         if (imports.length === 0) {

package/migrations/update-8/update-dependencies.js

@@ -4,41 +4,25 @@ const dependencies_1 = require("../../utility/dependencies");
 const latest_versions_1 = require("../../utility/latest-versions");
 function updateDependencies() {
     return (host) => {
-        let current = dependencies_1.getPackageJsonDependency(host, '@angular-devkit/build-angular');
-        if (current && current.version !== latest_versions_1.latestVersions.DevkitBuildAngular) {
+        const dependenciesToUpdate = {
+            '@angular-devkit/build-angular': latest_versions_1.latestVersions.DevkitBuildAngular,
+            '@angular-devkit/build-ng-packagr': latest_versions_1.latestVersions.DevkitBuildNgPackagr,
+            '@angular-devkit/build-webpack': latest_versions_1.latestVersions.DevkitBuildWebpack,
+            'zone.js': latest_versions_1.latestVersions.ZoneJs,
+            'tsickle': latest_versions_1.latestVersions.tsickle,
+            'ng-packagr': latest_versions_1.latestVersions.ngPackagr,
+            // FIXME: change to ^2.3.2 as soon as it's released with the pr208 fix
+            'web-animations-js': 'github:angular/web-animations-js#release_pr208',
+        };
+        for (const [name, version] of Object.entries(dependenciesToUpdate)) {
+            const current = dependencies_1.getPackageJsonDependency(host, name);
+            if (!current || current.version === version) {
+                continue;
+            }
             dependencies_1.addPackageJsonDependency(host, {
                 type: current.type,
-                name: '@angular-devkit/build-angular',
-                version: latest_versions_1.latestVersions.DevkitBuildAngular,
-                overwrite: true,
-            });
-        }
-        current = dependencies_1.getPackageJsonDependency(host, '@angular-devkit/build-ng-packagr');
-        if (current && current.version !== latest_versions_1.latestVersions.DevkitBuildNgPackagr) {
-            dependencies_1.addPackageJsonDependency(host, {
-                type: current.type,
-                name: '@angular-devkit/build-ng-packagr',
-                version: latest_versions_1.latestVersions.DevkitBuildNgPackagr,
-                overwrite: true,
-            });
-        }
-        current = dependencies_1.getPackageJsonDependency(host, 'zone.js');
-        if (current && current.version !== latest_versions_1.latestVersions.ZoneJs) {
-            dependencies_1.addPackageJsonDependency(host, {
-                type: current.type,
-                name: 'zone.js',
-                version: latest_versions_1.latestVersions.ZoneJs,
-                overwrite: true,
-            });
-        }
-        // FIXME: change to ^2.3.2 as soon as it's released with the pr208 fix
-        const webAnimationsJsVersion = 'github:angular/web-animations-js#release_pr208';
-        current = dependencies_1.getPackageJsonDependency(host, 'web-animations-js');
-        if (current && current.version !== webAnimationsJsVersion) {
-            dependencies_1.addPackageJsonDependency(host, {
-                type: current.type,
-                name: 'web-animations-js',
-                version: webAnimationsJsVersion,
+                name,
+                version,
                 overwrite: true,
             });
         }

package/migrations/update-8/update-lazy-module-paths.js

@@ -8,7 +8,7 @@ function* visit(directory) {
             if (entry) {
                 const content = entry.content;
                 if (content.includes('loadChildren')) {
-                    const source = ts.createSourceFile(entry.path, content.toString(), ts.ScriptTarget.Latest, true);
+                    const source = ts.createSourceFile(entry.path, content.toString().replace(/^\uFEFF/, ''), ts.ScriptTarget.Latest, true);
                     yield source;
                 }
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@schematics/angular",
-  "version": "8.0.0",
+  "version": "8.0.4",
   "description": "Schematics specific to Angular",
   "keywords": [
     "angular",
@@ -14,8 +14,8 @@
   ],
   "schematics": "./collection.json",
   "dependencies": {
-    "@angular-devkit/core": "8.0.0",
-    "@angular-devkit/schematics": "8.0.0"
+    "@angular-devkit/core": "8.0.4",
+    "@angular-devkit/schematics": "8.0.4"
   },
   "repository": {
     "type": "git",

package/pipe/files/__name@dasherize@if-flat__/__name@dasherize__.pipe.ts.template

@@ -5,7 +5,7 @@ import { Pipe, PipeTransform } from '@angular/core';
 })
 export class <%= classify(name) %>Pipe implements PipeTransform {
 
-  transform(value: any, args?: any): any {
+  transform(value: any, ...args: any[]): any {
     return null;
   }
 

package/utility/latest-versions.d.ts

@@ -13,4 +13,7 @@ export declare const latestVersions: {
     TsLib: string;
     DevkitBuildAngular: string;
     DevkitBuildNgPackagr: string;
+    DevkitBuildWebpack: string;
+    tsickle: string;
+    ngPackagr: string;
 };

package/utility/latest-versions.js

@@ -9,7 +9,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.latestVersions = {
     // These versions should be kept up to date with latest Angular peer dependencies.
-    Angular: '~8.0.0',
+    Angular: '~8.0.1',
     RxJs: '~6.4.0',
     ZoneJs: '~0.9.1',
     TypeScript: '~3.4.3',
@@ -17,4 +17,7 @@ exports.latestVersions = {
     // The versions below must be manually updated when making a new devkit release.
     DevkitBuildAngular: '~0.800.0',
     DevkitBuildNgPackagr: '~0.800.0',
+    DevkitBuildWebpack: '~0.800.0',
+    tsickle: '^0.35.0',
+    ngPackagr: '^5.1.0',
 };

package/web-worker/index.js

@@ -17,7 +17,9 @@ function addConfig(options, root, tsConfigPath) {
     return (host, context) => {
         context.logger.debug('updating project configuration.');
         // Add worker glob exclusion to tsconfig.app.json.
-        const workerGlob = 'src/**/*.worker.ts';
+        // Projects pre version 8 should to have tsconfig.app.json inside their application
+        const isInSrc = core_1.dirname(core_1.normalize(tsConfigPath)).endsWith('src');
+        const workerGlob = `${isInSrc ? '' : 'src/'}**/*.worker.ts`;
         const buffer = host.read(tsConfigPath);
         if (buffer) {
             const tsCfgAst = core_1.parseJsonAst(buffer.toString(), core_1.JsonParseMode.Loose);

package/workspace/files/tsconfig.json.template

@@ -5,10 +5,12 @@
     "outDir": "./dist/out-tsc",
     "sourceMap": true,
     "declaration": false,
+    "downlevelIteration": true,
     "module": "esnext",
     "moduleResolution": "node",
-    "emitDecoratorMetadata": true,
     "experimentalDecorators": true,
+    "module": "esnext",
+    "moduleResolution": "node",
     "importHelpers": true,
     "target": "es2015",
     "typeRoots": [