@angular/cli 8.0.0 → 8.0.4

package/bin/postinstall/analytics-prompt.js

@@ -5,10 +5,12 @@ if ('NG_CLI_ANALYTICS' in process.env) {
   return;
 }
 
-try {
-  const analytics = require('../../models/analytics');
+(async () => {
+  try {
+    const analytics = require('../../models/analytics');
 
-  if (!analytics.hasGlobalAnalyticsConfiguration()) {
-    analytics.promptGlobalAnalytics();
-  }
-} catch (_) {}
+    if (!analytics.hasGlobalAnalyticsConfiguration()) {
+      await analytics.promptGlobalAnalytics();
+    }
+  } catch (_) {}
+})();

package/commands/build-long.md

@@ -1,14 +1,18 @@
-Uses the [webpack](https://webpack.js.org/) build tool, with default configuration options specified in the workspace configuration file (`angular.json`) or with a named alternative configuration. 
-A "production" configuration is created by default when you use the CLI to create the project, and you can use that configuration by specifying the `--prod` option.
+The command can be used to build a project of type "application" or "library".
+When used to build a library, a different builder is invoked, and only the `ts-config`, `configuration`, and `watch` options are applied.
+All other options apply only to building applications.
+
+The application builder uses the [webpack](https://webpack.js.org/) build tool, with default configuration options specified in the workspace configuration file (`angular.json`) or with a named alternative configuration.
+A "production" configuration is created by default when you use the CLI to create the project, and you can use that configuration by specifying the `--configuration="production"` or the `--prod="true"` option.
 
 The configuration options generally correspond to the command options.
-You can override individual configuration defaults by specifying the corresponding options on the command line. 
+You can override individual configuration defaults by specifying the corresponding options on the command line.
 The command can accept option names given in either dash-case or camelCase.
 Note that in the configuration file, you must specify names in camelCase.
 
 Some additional options can only be set through the configuration file,
 either by direct editing or with the `ng config` command.
-These include `assets`, `styles`, and `scripts` objects that provide runtime-global resources to include in the project. 
+These include `assets`, `styles`, and `scripts` objects that provide runtime-global resources to include in the project.
 Resources in CSS, such as images and fonts, are automatically written and fingerprinted at the root of the output folder.
 
 For further details, see [Workspace Configuration](guide/workspace-config).

package/commands/update-impl.js

@@ -8,18 +8,15 @@ Object.defineProperty(exports, "__esModule", { value: true });
  * found in the LICENSE file at https://angular.io/license
  */
 const child_process_1 = require("child_process");
+const fs = require("fs");
 const path = require("path");
 const semver = require("semver");
 const schematic_command_1 = require("../models/schematic-command");
-const find_up_1 = require("../utilities/find-up");
 const package_manager_1 = require("../utilities/package-manager");
 const package_metadata_1 = require("../utilities/package-metadata");
 const package_tree_1 = require("../utilities/package-tree");
 const npa = require('npm-package-arg');
-const oldConfigFileNames = [
-    '.angular-cli.json',
-    'angular-cli.json',
-];
+const oldConfigFileNames = ['.angular-cli.json', 'angular-cli.json'];
 class UpdateCommand extends schematic_command_1.SchematicCommand {
     constructor() {
         super(...arguments);
@@ -68,21 +65,28 @@ class UpdateCommand extends schematic_command_1.SchematicCommand {
         }
         // If not asking for status then check for a clean git repository.
         // This allows the user to easily reset any changes from the update.
-        if ((packages.length !== 0 || options.all) && !this.checkCleanGit()) {
-            this.logger.error('Repository is not clean.  Please commit or stash any changes before updating.');
-            return 2;
+        const statusCheck = packages.length === 0 && !options.all;
+        if (!statusCheck && !this.checkCleanGit()) {
+            if (options.allowDirty) {
+                this.logger.warn('Repository is not clean.  Update changes will be mixed with pre-existing changes.');
+            }
+            else {
+                this.logger.error('Repository is not clean.  Please commit or stash any changes before updating.');
+                return 2;
+            }
         }
         const packageManager = package_manager_1.getPackageManager(this.workspace.root);
         this.logger.info(`Using package manager: '${packageManager}'`);
         // Special handling for Angular CLI 1.x migrations
-        if (options.migrateOnly === undefined && options.from === undefined) {
-            if (!options.all && packages.length === 1 && packages[0].name === '@angular/cli') {
-                const oldConfigFilePath = find_up_1.findUp(oldConfigFileNames, process.cwd());
-                if (oldConfigFilePath) {
-                    options.migrateOnly = true;
-                    options.from = '1.0.0';
-                }
-            }
+        if (options.migrateOnly === undefined &&
+            options.from === undefined &&
+            !options.all &&
+            packages.length === 1 &&
+            packages[0].name === '@angular/cli' &&
+            this.workspace.configFile &&
+            oldConfigFileNames.includes(this.workspace.configFile)) {
+            options.migrateOnly = true;
+            options.from = '1.0.0';
         }
         this.logger.info('Collecting installed dependencies...');
         const packageTree = await package_tree_1.readPackageTree(this.workspace.root);
@@ -145,9 +149,37 @@ class UpdateCommand extends schematic_command_1.SchematicCommand {
                 this.logger.error('Package contains a malformed migrations field.');
                 return 1;
             }
-            // if not non-relative, add package name
-            if (migrations.startsWith('.') || migrations.startsWith('/')) {
-                migrations = path.join(packageName, migrations);
+            else if (path.posix.isAbsolute(migrations) || path.win32.isAbsolute(migrations)) {
+                this.logger.error('Package contains an invalid migrations field. Absolute paths are not permitted.');
+                return 1;
+            }
+            // Normalize slashes
+            migrations = migrations.replace(/\\/g, '/');
+            if (migrations.startsWith('../')) {
+                this.logger.error('Package contains an invalid migrations field. ' +
+                    'Paths outside the package root are not permitted.');
+                return 1;
+            }
+            // Check if it is a package-local location
+            const localMigrations = path.join(packageNode.path, migrations);
+            if (fs.existsSync(localMigrations)) {
+                migrations = localMigrations;
+            }
+            else {
+                // Try to resolve from package location.
+                // This avoids issues with package hoisting.
+                try {
+                    migrations = require.resolve(migrations, { paths: [packageNode.path] });
+                }
+                catch (e) {
+                    if (e.code === 'MODULE_NOT_FOUND') {
+                        this.logger.error('Migrations for package were not found.');
+                    }
+                    else {
+                        this.logger.error(`Unable to resolve migrations for package.  [${e.message}]`);
+                    }
+                    return 1;
+                }
             }
             return this.runSchematic({
                 collectionName: '@schematics/update',
@@ -178,21 +210,23 @@ class UpdateCommand extends schematic_command_1.SchematicCommand {
                 this.logger.info(`Package '${pkg.name}' is already at '${pkg.fetchSpec}'.`);
                 continue;
             }
-            requests.push(pkg);
+            requests.push({ identifier: pkg, node });
         }
         if (requests.length === 0) {
             return 0;
         }
+        const packagesToUpdate = [];
         this.logger.info('Fetching dependency metadata from registry...');
-        for (const requestIdentifier of requests) {
+        for (const { identifier: requestIdentifier, node } of requests) {
+            const packageName = requestIdentifier.name;
             let metadata;
             try {
                 // Metadata requests are internally cached; multiple requests for same name
                 // does not result in additional network traffic
-                metadata = await package_metadata_1.fetchPackageMetadata(requestIdentifier.name, this.logger);
+                metadata = await package_metadata_1.fetchPackageMetadata(packageName, this.logger);
             }
             catch (e) {
-                this.logger.error(`Error fetching metadata for '${requestIdentifier.name}': ` + e.message);
+                this.logger.error(`Error fetching metadata for '${packageName}': ` + e.message);
                 return 1;
             }
             // Try to find a package version based on the user requested package specifier
@@ -214,6 +248,15 @@ class UpdateCommand extends schematic_command_1.SchematicCommand {
                 this.logger.error(`Package specified by '${requestIdentifier.raw}' does not exist within the registry.`);
                 return 1;
             }
+            if ((typeof node === 'string' && manifest.version === node) ||
+                (typeof node === 'object' && manifest.version === node.package.version)) {
+                this.logger.info(`Package '${packageName}' is already up to date.`);
+                continue;
+            }
+            packagesToUpdate.push(requestIdentifier.toString());
+        }
+        if (packagesToUpdate.length === 0) {
+            return 0;
         }
         return this.runSchematic({
             collectionName: '@schematics/update',
@@ -223,18 +266,26 @@ class UpdateCommand extends schematic_command_1.SchematicCommand {
             additionalOptions: {
                 force: options.force || false,
                 packageManager,
-                packages: requests.map(p => p.toString()),
+                packages: packagesToUpdate,
             },
         });
     }
     checkCleanGit() {
         try {
             const result = child_process_1.execSync('git status --porcelain', { encoding: 'utf8', stdio: 'pipe' });
-            return result.trim().length === 0;
-        }
-        catch (_a) {
-            return true;
+            if (result.trim().length === 0) {
+                return true;
+            }
+            // Only files inside the workspace root are relevant
+            for (const entry of result.split('\n')) {
+                const relativeEntry = path.relative(path.resolve(this.workspace.root), path.resolve(entry.slice(3).trim()));
+                if (!relativeEntry.startsWith('..') && !path.isAbsolute(relativeEntry)) {
+                    return false;
+                }
+            }
         }
+        catch (_a) { }
+        return true;
     }
 }
 exports.UpdateCommand = UpdateCommand;

package/commands/update.d.ts

@@ -6,6 +6,10 @@ export interface Schema {
      * Whether to update all packages in package.json.
      */
     all?: boolean;
+    /**
+     * Whether to allow updating when the repository contains modified or untracked files.
+     */
+    allowDirty?: boolean;
     /**
      * If false, will error out if installed packages are incompatible with the update.
      */

package/commands/update.json

@@ -53,6 +53,10 @@
         "to": {
           "description": "Version up to which to apply migrations. Only available with a single package being updated, and only on migrations only. Requires from to be specified. Default to the installed version detected.",
           "type": "string"
+        },
+        "allowDirty": {
+          "description": "Whether to allow updating when the repository contains modified or untracked files.",
+          "type": "boolean"
         }
       }
     }

package/commands/version-impl.js

@@ -72,8 +72,12 @@ class VersionCommand extends command_1.Command {
         if (!__dirname.match(/node_modules/)) {
             let gitBranch = '??';
             try {
-                const gitRefName = '' + child_process.execSync('git symbolic-ref HEAD', { cwd: __dirname });
-                gitBranch = path.basename(gitRefName.replace('\n', ''));
+                const gitRefName = child_process.execSync('git rev-parse --abbrev-ref HEAD', {
+                    cwd: __dirname,
+                    encoding: 'utf8',
+                    stdio: 'pipe',
+                });
+                gitBranch = gitRefName.replace('\n', '');
             }
             catch (_a) {
             }

package/lib/config/schema.d.ts

@@ -14,6 +14,10 @@ export interface Schema {
     version: number;
 }
 export interface CliOptions {
+    /**
+     * Share anonymous usage data with the Angular Team at Google.
+     */
+    analytics?: boolean;
     /**
      * The default schematics collection to use.
      */

package/lib/config/schema.json

@@ -65,6 +65,10 @@
               "x-deprecated": true
             }
           }
+        },
+        "analytics": {
+          "type": "boolean",
+          "description": "Share anonymous usage data with the Angular Team at Google."
         }
       },
       "additionalProperties": false

package/models/architect-command.js

@@ -194,9 +194,12 @@ class ArchitectCommand extends command_1.Command {
                 logger: this.logger,
                 analytics: analytics_1.isPackageNameSafeForAnalytics(builderConf) ? this.analytics : undefined,
             });
-            const result = await run.output.toPromise();
+            const { error, success } = await run.output.toPromise();
             await run.stop();
-            return result.success ? 0 : 1;
+            if (error) {
+                this.logger.error(error);
+            }
+            return success ? 0 : 1;
         }
     }
     async runArchitectTarget(options) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular/cli",
-  "version": "8.0.0",
+  "version": "8.0.4",
   "description": "CLI tool for Angular",
   "main": "lib/cli/index.js",
   "bin": {
@@ -28,11 +28,11 @@
   },
   "homepage": "https://github.com/angular/angular-cli",
   "dependencies": {
-    "@angular-devkit/architect": "0.800.0",
-    "@angular-devkit/core": "8.0.0",
-    "@angular-devkit/schematics": "8.0.0",
-    "@schematics/angular": "8.0.0",
-    "@schematics/update": "0.800.0",
+    "@angular-devkit/architect": "0.800.4",
+    "@angular-devkit/core": "8.0.4",
+    "@angular-devkit/schematics": "8.0.4",
+    "@schematics/angular": "8.0.4",
+    "@schematics/update": "0.800.4",
     "@yarnpkg/lockfile": "1.1.0",
     "debug": "^4.1.1",
     "ini": "1.3.5",
@@ -49,10 +49,10 @@
   "ng-update": {
     "migrations": "@schematics/angular/migrations/migration-collection.json",
     "packageGroup": {
-      "@angular/cli": "8.0.0",
-      "@angular-devkit/build-angular": "0.800.0",
-      "@angular-devkit/build-ng-packagr": "0.800.0",
-      "@angular-devkit/build-webpack": "0.800.0"
+      "@angular/cli": "8.0.4",
+      "@angular-devkit/build-angular": "0.800.4",
+      "@angular-devkit/build-ng-packagr": "0.800.4",
+      "@angular-devkit/build-webpack": "0.800.4"
     }
   },
   "engines": {

package/utilities/tty.js

@@ -12,6 +12,7 @@ function isTTY() {
     if (force !== undefined) {
         return !(force === '0' || force.toUpperCase() === 'FALSE');
     }
-    return !!process.stdout.isTTY && !!process.stdin.isTTY;
+    const ci = process.env['CI'];
+    return !!process.stdout.isTTY && (!ci || ci === '0' || ci.toUpperCase() === 'FALSE');
 }
 exports.isTTY = isTTY;