@angular/cli 12.0.0-rc.2 → 12.0.2

package/commands/add-impl.js

@@ -173,9 +173,9 @@ class AddCommand extends schematic_command_1.SchematicCommand {
         if (savePackage === false) {
             // Temporary packages are located in a different directory
             // Hence we need to resolve them using the temp path
-            const { status, tempPath } = await install_package_1.installTempPackage(packageIdentifier.raw, packageManager, options.registry ? [`--registry="${options.registry}"`] : undefined);
+            const { status, tempNodeModules } = await install_package_1.installTempPackage(packageIdentifier.raw, packageManager, options.registry ? [`--registry="${options.registry}"`] : undefined);
             const resolvedCollectionPath = require.resolve(path_1.join(collectionName, 'package.json'), {
-                paths: [tempPath],
+                paths: [tempNodeModules],
             });
             if (status !== 0) {
                 return status;

package/commands/config-impl.js

@@ -54,17 +54,22 @@ function parseJsonPath(path) {
     return result.filter((fragment) => fragment != null);
 }
 function normalizeValue(value) {
+    var _a, _b;
     const valueString = `${value}`.trim();
-    if (valueString === 'true') {
-        return true;
+    switch (valueString) {
+        case 'true':
+            return true;
+        case 'false':
+            return false;
+        case 'null':
+            return null;
+        case 'undefined':
+            return undefined;
     }
-    else if (valueString === 'false') {
-        return false;
-    }
-    else if (isFinite(+valueString)) {
+    if (isFinite(+valueString)) {
         return +valueString;
     }
-    return value || undefined;
+    return (_b = (_a = json_file_1.parseJson(valueString)) !== null && _a !== void 0 ? _a : value) !== null && _b !== void 0 ? _b : undefined;
 }
 class ConfigCommand extends command_1.Command {
     async run(options) {

package/commands/e2e-impl.d.ts

@@ -11,6 +11,6 @@ import { Schema as E2eCommandSchema } from './e2e';
 export declare class E2eCommand extends ArchitectCommand<E2eCommandSchema> {
     readonly target = "e2e";
     readonly multiTarget = true;
-    readonly missingTargetError = "\nCannot find \"e2e\" target for the specified project.\n\nYou should add a package that implements end-to-end testing capabilities.\n\nFor example:\n  WebdriverIO: ng add @wdio/schematics\n\nMore options will be added to the list as they become available.\n";
+    readonly missingTargetError = "\nCannot find \"e2e\" target for the specified project.\n\nYou should add a package that implements end-to-end testing capabilities.\n\nFor example:\n  Cypress: ng add @cypress/schematic\n  WebdriverIO: ng add @wdio/schematics\n\nMore options will be added to the list as they become available.\n";
     initialize(options: E2eCommandSchema & Arguments): Promise<number | void>;
 }

package/commands/e2e-impl.js

@@ -20,6 +20,7 @@ Cannot find "e2e" target for the specified project.
 You should add a package that implements end-to-end testing capabilities.
 
 For example:
+  Cypress: ng add @cypress/schematic
   WebdriverIO: ng add @wdio/schematics
 
 More options will be added to the list as they become available.

package/commands/update-impl.d.ts

@@ -12,7 +12,7 @@ export declare class UpdateCommand extends Command<UpdateCommandSchema> {
     readonly allowMissingWorkspace = true;
     private workflow;
     private packageManager;
-    initialize(): Promise<void>;
+    initialize(options: UpdateCommandSchema & Arguments): Promise<void>;
     private executeSchematic;
     /**
      * @return Whether or not the migration was performed successfully.

package/commands/update-impl.js

@@ -42,10 +42,11 @@ class UpdateCommand extends command_1.Command {
         this.allowMissingWorkspace = true;
         this.packageManager = workspace_schema_1.PackageManager.Npm;
     }
-    async initialize() {
+    async initialize(options) {
         this.packageManager = await package_manager_1.getPackageManager(this.context.root);
         this.workflow = new tools_1.NodeWorkflow(this.context.root, {
             packageManager: this.packageManager,
+            packageManagerForce: options.force,
             // __dirname -> favor @schematics/update from this package
             // Otherwise, use packages from the active workspace (migrations)
             resolvePaths: [__dirname, this.context.root],
@@ -474,6 +475,21 @@ class UpdateCommand extends command_1.Command {
             packageManager: this.packageManager,
             packages: packagesToUpdate,
         });
+        if (success) {
+            try {
+                // Remove existing node modules directory to provide a stronger guarantee that packages
+                // will be hoisted into the correct locations.
+                await fs.promises.rmdir(path.join(this.context.root, 'node_modules'), {
+                    recursive: true,
+                    maxRetries: 3,
+                });
+            }
+            catch { }
+            const result = await install_package_1.installAllPackages(this.packageManager, options.force ? ['--force'] : [], this.context.root);
+            if (result !== 0) {
+                return result;
+            }
+        }
         if (success && options.createCommits) {
             const committed = this.commit(`Angular CLI update for packages - ${packagesToUpdate.join(', ')}`);
             if (!committed) {

package/commands/update.d.ts

@@ -15,7 +15,8 @@ export interface Schema {
      */
     createCommits?: boolean;
     /**
-     * If false, will error out if installed packages are incompatible with the update.
+     * Ignore peer dependency version mismatches. Passes the `--force` flag to the package
+     * manager when installing packages.
      */
     force?: boolean;
     /**

package/commands/update.json

@@ -28,7 +28,7 @@
           }
         },
         "force": {
-          "description": "If false, will error out if installed packages are incompatible with the update.",
+          "description": "Ignore peer dependency version mismatches. Passes the `--force` flag to the package manager when installing packages.",
           "default": false,
           "type": "boolean"
         },

package/lib/config/schema.json

@@ -351,12 +351,14 @@
                   }
                 }
               },
+              "additionalProperties": false,
               "required": [
                 "builder"
               ]
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:app-shell"
@@ -378,6 +380,7 @@
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:browser"
@@ -399,6 +402,7 @@
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:dev-server"
@@ -420,6 +424,7 @@
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:extract-i18n"
@@ -441,6 +446,7 @@
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:karma"
@@ -462,6 +468,7 @@
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:protractor"
@@ -483,6 +490,7 @@
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:server"
@@ -504,6 +512,7 @@
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:tslint"
@@ -525,10 +534,15 @@
             },
             {
               "type": "object",
+              "additionalProperties": false,
               "properties": {
                 "builder": {
                   "const": "@angular-devkit/build-angular:ng-packagr"
                 },
+                "defaultConfiguration": {
+                  "type": "string",
+                  "description": "A default named configuration to use when a target configuration is not provided."
+                },
                 "options": {
                   "$ref": "#/definitions/AngularDevkitBuildAngularNgPackagrSchema"
                 },
@@ -2172,7 +2186,7 @@
         },
         "proxyConfig": {
           "type": "string",
-          "description": "Proxy configuration file."
+          "description": "Proxy configuration file. For more information, see https://angular.io/guide/build#proxying-to-a-backend-server."
         },
         "ssl": {
           "type": "boolean",

package/models/architect-command.js

@@ -252,6 +252,7 @@ class ArchitectCommand extends command_1.Command {
         }
     }
     _makeTargetSpecifier(commandOptions) {
+        var _a, _b, _c;
         let project, target, configuration;
         if (commandOptions.target) {
             [project, target, configuration] = commandOptions.target.split(':');
@@ -263,9 +264,14 @@ class ArchitectCommand extends command_1.Command {
             project = commandOptions.project;
             target = this.target;
             if (commandOptions.prod) {
+                const defaultConfig = project &&
+                    target &&
+                    ((_c = (_b = (_a = this.workspace) === null || _a === void 0 ? void 0 : _a.projects.get(project)) === null || _b === void 0 ? void 0 : _b.targets.get(target)) === null || _c === void 0 ? void 0 : _c.defaultConfiguration);
+                this.logger.warn(defaultConfig === 'production'
+                    ? 'Option "--prod" is deprecated: No need to use this option as this builder defaults to configuration "production".'
+                    : 'Option "--prod" is deprecated: Use "--configuration production" instead.');
                 // The --prod flag will always be the first configuration, available to be overwritten
                 // by following configurations.
-                this.logger.warn('Option "--prod" is deprecated: Use "--configuration production" instead.');
                 configuration = 'production';
             }
             if (commandOptions.configuration) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular/cli",
-  "version": "12.0.0-rc.2",
+  "version": "12.0.2",
   "description": "CLI tool for Angular",
   "main": "lib/cli/index.js",
   "bin": {
@@ -28,10 +28,10 @@
   },
   "homepage": "https://github.com/angular/angular-cli",
   "dependencies": {
-    "@angular-devkit/architect": "0.1200.0-rc.2",
-    "@angular-devkit/core": "12.0.0-rc.2",
-    "@angular-devkit/schematics": "12.0.0-rc.2",
-    "@schematics/angular": "12.0.0-rc.2",
+    "@angular-devkit/architect": "0.1200.2",
+    "@angular-devkit/core": "12.0.2",
+    "@angular-devkit/schematics": "12.0.2",
+    "@schematics/angular": "12.0.2",
     "@yarnpkg/lockfile": "1.1.0",
     "ansi-colors": "4.1.1",
     "debug": "4.3.1",
@@ -40,7 +40,7 @@
     "jsonc-parser": "3.0.0",
     "npm-package-arg": "8.1.2",
     "npm-pick-manifest": "6.1.1",
-    "open": "8.0.7",
+    "open": "8.0.2",
     "ora": "5.4.0",
     "pacote": "11.3.2",
     "resolve": "1.20.0",
@@ -52,12 +52,12 @@
   "ng-update": {
     "migrations": "@schematics/angular/migrations/migration-collection.json",
     "packageGroup": {
-      "@angular/cli": "12.0.0-rc.2",
-      "@angular-devkit/architect": "0.1200.0-rc.2",
-      "@angular-devkit/build-angular": "12.0.0-rc.2",
-      "@angular-devkit/build-webpack": "0.1200.0-rc.2",
-      "@angular-devkit/core": "12.0.0-rc.2",
-      "@angular-devkit/schematics": "12.0.0-rc.2"
+      "@angular/cli": "12.0.2",
+      "@angular-devkit/architect": "0.1200.2",
+      "@angular-devkit/build-angular": "12.0.2",
+      "@angular-devkit/build-webpack": "0.1200.2",
+      "@angular-devkit/core": "12.0.2",
+      "@angular-devkit/schematics": "12.0.2"
     }
   },
   "engines": {

package/src/commands/update/schematic/index.js

@@ -10,7 +10,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.angularMajorCompatGuarantee = void 0;
 const core_1 = require("@angular-devkit/core");
 const schematics_1 = require("@angular-devkit/schematics");
-const tasks_1 = require("@angular-devkit/schematics/tasks");
 const npa = require("npm-package-arg");
 const semver = require("semver");
 const npm_1 = require("./npm");
@@ -218,9 +217,7 @@ function _performUpdate(tree, context, infoMap, logger, migrateOnly) {
     const newContent = JSON.stringify(packageJson, null, 2);
     if (packageJsonContent.toString() != newContent || migrateOnly) {
         if (!migrateOnly) {
-            // If something changed, also hook up the task.
             tree.overwrite('/package.json', JSON.stringify(packageJson, null, 2));
-            context.addTask(new tasks_1.NodePackageInstallTask());
         }
         const externalMigrations = [];
         // Run the migrate schematics with the list of packages to use. The collection contains

package/utilities/install-package.d.ts

@@ -7,9 +7,10 @@
  */
 import { PackageManager } from '../lib/config/workspace-schema';
 import { NgAddSaveDepedency } from '../utilities/package-metadata';
+export declare function installAllPackages(packageManager?: PackageManager, extraArgs?: string[], cwd?: string): Promise<1 | 0>;
 export declare function installPackage(packageName: string, packageManager?: PackageManager, save?: Exclude<NgAddSaveDepedency, false>, extraArgs?: string[], cwd?: string): Promise<1 | 0>;
 export declare function installTempPackage(packageName: string, packageManager?: PackageManager, extraArgs?: string[]): Promise<{
     status: 1 | 0;
-    tempPath: string;
+    tempNodeModules: string;
 }>;
 export declare function runTempPackageBin(packageName: string, packageManager?: PackageManager, args?: string[]): Promise<number>;

package/utilities/install-package.js

@@ -7,7 +7,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.runTempPackageBin = exports.installTempPackage = exports.installPackage = void 0;
+exports.runTempPackageBin = exports.installTempPackage = exports.installPackage = exports.installAllPackages = void 0;
 const child_process_1 = require("child_process");
 const fs_1 = require("fs");
 const os_1 = require("os");
@@ -15,6 +15,39 @@ const path_1 = require("path");
 const rimraf = require("rimraf");
 const workspace_schema_1 = require("../lib/config/workspace-schema");
 const spinner_1 = require("./spinner");
+async function installAllPackages(packageManager = workspace_schema_1.PackageManager.Npm, extraArgs = [], cwd = process.cwd()) {
+    const packageManagerArgs = getPackageManagerArguments(packageManager);
+    const installArgs = [];
+    if (packageManagerArgs.installAll) {
+        installArgs.push(packageManagerArgs.installAll);
+    }
+    installArgs.push(packageManagerArgs.silent);
+    const spinner = new spinner_1.Spinner();
+    spinner.start('Installing packages...');
+    const bufferedOutput = [];
+    return new Promise((resolve, reject) => {
+        var _a, _b;
+        const childProcess = child_process_1.spawn(packageManager, [...installArgs, ...extraArgs], {
+            stdio: 'pipe',
+            shell: true,
+            cwd,
+        }).on('close', (code) => {
+            if (code === 0) {
+                spinner.succeed('Packages successfully installed.');
+                resolve(0);
+            }
+            else {
+                spinner.stop();
+                bufferedOutput.forEach(({ stream, data }) => stream.write(data));
+                spinner.fail('Package install failed, see above.');
+                reject(1);
+            }
+        });
+        (_a = childProcess.stdout) === null || _a === void 0 ? void 0 : _a.on('data', (data) => bufferedOutput.push({ stream: process.stdout, data: data }));
+        (_b = childProcess.stderr) === null || _b === void 0 ? void 0 : _b.on('data', (data) => bufferedOutput.push({ stream: process.stderr, data: data }));
+    });
+}
+exports.installAllPackages = installAllPackages;
 async function installPackage(packageName, packageManager = workspace_schema_1.PackageManager.Npm, save = true, extraArgs = [], cwd = process.cwd()) {
     const packageManagerArgs = getPackageManagerArguments(packageManager);
     const installArgs = [
@@ -86,19 +119,19 @@ async function installTempPackage(packageName, packageManager = workspace_schema
     ];
     return {
         status: await installPackage(packageName, packageManager, true, installArgs, tempPath),
-        tempPath,
+        tempNodeModules,
     };
 }
 exports.installTempPackage = installTempPackage;
 async function runTempPackageBin(packageName, packageManager = workspace_schema_1.PackageManager.Npm, args = []) {
-    const { status: code, tempPath } = await installTempPackage(packageName, packageManager);
+    const { status: code, tempNodeModules } = await installTempPackage(packageName, packageManager);
     if (code !== 0) {
         return code;
     }
     // Remove version/tag etc... from package name
     // Ex: @angular/cli@latest -> @angular/cli
     const packageNameNoVersion = packageName.substring(0, packageName.lastIndexOf('@'));
-    const pkgLocation = path_1.join(tempPath, packageNameNoVersion);
+    const pkgLocation = path_1.join(tempNodeModules, packageNameNoVersion);
     const packageJsonPath = path_1.join(pkgLocation, 'package.json');
     // Get a binary location for this package
     let binPath;
@@ -115,7 +148,7 @@ async function runTempPackageBin(packageName, packageManager = workspace_schema_
     if (!binPath) {
         throw new Error(`Cannot locate bin for temporary package: ${packageNameNoVersion}.`);
     }
-    const argv = [binPath, ...args];
+    const argv = [`'${binPath}'`, ...args];
     const { status, error } = child_process_1.spawnSync('node', argv, {
         stdio: 'inherit',
         shell: true,
@@ -146,6 +179,7 @@ function getPackageManagerArguments(packageManager) {
                 silent: '--silent',
                 saveDev: '--save-dev',
                 install: 'add',
+                installAll: 'install',
                 prefix: '--prefix',
                 noLockfile: '--no-lockfile',
             };
@@ -154,6 +188,7 @@ function getPackageManagerArguments(packageManager) {
                 silent: '--quiet',
                 saveDev: '--save-dev',
                 install: 'install',
+                installAll: 'install',
                 prefix: '--prefix',
                 noLockfile: '--no-package-lock',
             };