refactor(@angular/cli): use canonical npm logic to determine update package version

Author: clydin

packages/angular/cli/commands/add-impl.ts

@@ -90,7 +90,7 @@ export class AddCommand extends SchematicCommand<AddCommandSchema> {
         }
       } else if (!latestManifest || (await this.hasMismatchedPeer(latestManifest))) {
         // 'latest' is invalid so search for most recent matching package
-        const versionManifests = Array.from(packageMetadata.versions.values()).filter(
+        const versionManifests = Object.values(packageMetadata.versions).filter(
           value => !prerelease(value.version),
         );
 

packages/angular/cli/commands/update-impl.ts

@@ -26,7 +26,11 @@ import {
 import { PackageTreeNode, findNodeDependencies, readPackageTree } from '../utilities/package-tree';
 import { Schema as UpdateCommandSchema } from './update';
 
-const npa = require('npm-package-arg');
+const npa = require('npm-package-arg') as (selector: string) => PackageIdentifier;
+const pickManifest = require('npm-pick-manifest') as (
+  metadata: PackageMetadata,
+  selector: string,
+) => PackageManifest;
 
 const oldConfigFileNames = ['.angular-cli.json', 'angular-cli.json'];
 
@@ -188,7 +192,7 @@ export class UpdateCommand extends Command<UpdateCommandSchema> {
     const packages: PackageIdentifier[] = [];
     for (const request of options['--'] || []) {
       try {
-        const packageIdentifier: PackageIdentifier = npa(request);
+        const packageIdentifier = npa(request);
 
         // only registry identifiers are supported
         if (!packageIdentifier.registry) {
@@ -271,8 +275,19 @@ export class UpdateCommand extends Command<UpdateCommandSchema> {
 
     this.logger.info(`Found ${Object.keys(rootDependencies).length} dependencies.`);
 
-    if (options.all || packages.length === 0) {
-      // Either update all packages or show status
+    if (options.all) {
+      // 'all' option and a zero length packages have already been checked.
+      // Add all direct dependencies to be updated
+      for (const dep of Object.keys(rootDependencies)) {
+        const packageIdentifier = npa(dep);
+        if (options.next) {
+          packageIdentifier.fetchSpec = 'next';
+        }
+
+        packages.push(packageIdentifier);
+      }
+    } else if (packages.length === 0) {
+      // Show status
       const { success } = await this.executeSchematic('@schematics/update', 'update', {
         force: options.force || false,
         next: options.next || false,
@@ -397,7 +412,7 @@ export class UpdateCommand extends Command<UpdateCommandSchema> {
 
     const requests: {
       identifier: PackageIdentifier;
-      node: PackageTreeNode | string;
+      node: PackageTreeNode;
     }[] = [];
 
     // Validate packages actually are part of the workspace
@@ -410,11 +425,7 @@ export class UpdateCommand extends Command<UpdateCommandSchema> {
       }
 
       // If a specific version is requested and matches the installed version, skip.
-      if (
-        pkg.type === 'version' &&
-        typeof node === 'object' &&
-        node.package.version === pkg.fetchSpec
-      ) {
+      if (pkg.type === 'version' && node.package.version === pkg.fetchSpec) {
         this.logger.info(`Package '${pkg.name}' is already at '${pkg.fetchSpec}'.`);
         continue;
       }
@@ -448,18 +459,34 @@ export class UpdateCommand extends Command<UpdateCommandSchema> {
       // Try to find a package version based on the user requested package specifier
       // registry specifier types are either version, range, or tag
       let manifest: PackageManifest | undefined;
-      if (requestIdentifier.type === 'version') {
-        manifest = metadata.versions.get(requestIdentifier.fetchSpec);
-      } else if (requestIdentifier.type === 'range') {
-        const maxVersion = semver.maxSatisfying(
-          Array.from(metadata.versions.keys()),
-          requestIdentifier.fetchSpec,
-        );
-        if (maxVersion) {
-          manifest = metadata.versions.get(maxVersion);
+      if (
+        requestIdentifier.type === 'version' ||
+        requestIdentifier.type === 'range' ||
+        requestIdentifier.type === 'tag'
+      ) {
+        try {
+          manifest = pickManifest(metadata, requestIdentifier.fetchSpec);
+        } catch (e) {
+          if (e.code === 'ETARGET') {
+            // If not found and next was used and user did not provide a specifier, try latest.
+            // Package may not have a next tag.
+            if (
+              requestIdentifier.type === 'tag' &&
+              requestIdentifier.fetchSpec === 'next' &&
+              !requestIdentifier.rawSpec
+            ) {
+              try {
+                manifest = pickManifest(metadata, 'latest');
+              } catch (e) {
+                if (e.code !== 'ETARGET' && e.code !== 'ENOVERSIONS') {
+                  throw e;
+                }
+              }
+            }
+          } else if (e.code !== 'ENOVERSIONS') {
+            throw e;
+          }
         }
-      } else if (requestIdentifier.type === 'tag') {
-        manifest = metadata.tags[requestIdentifier.fetchSpec];
       }
 
       if (!manifest) {
@@ -470,10 +497,7 @@ export class UpdateCommand extends Command<UpdateCommandSchema> {
         return 1;
       }
 
-      if (
-        (typeof node === 'string' && manifest.version === node) ||
-        (typeof node === 'object' && manifest.version === node.package.version)
-      ) {
+      if (manifest.version === node.package.version) {
         this.logger.info(`Package '${packageName}' is already up to date.`);
         continue;
       }

packages/angular/cli/package.json

@@ -36,6 +36,7 @@
     "ini": "1.3.5",
     "inquirer": "7.0.0",
     "npm-package-arg": "6.1.1",
+    "npm-pick-manifest": "3.0.2",
     "open": "6.4.0",
     "pacote": "9.5.8",
     "read-package-tree": "5.3.1",

packages/angular/cli/utilities/package-metadata.ts

@@ -50,7 +50,7 @@ export interface PackageManifest {
 export interface PackageMetadata {
   name: string;
   tags: { [tag: string]: PackageManifest | undefined };
-  versions: Map<string, PackageManifest>;
+  versions: Record<string, PackageManifest>;
 }
 
 let npmrc: { [key: string]: string };
@@ -179,18 +179,22 @@ export async function fetchPackageMetadata(
   const metadata: PackageMetadata = {
     name: response.name,
     tags: {},
-    versions: new Map(),
+    versions: {},
   };
 
   if (response.versions) {
     for (const [version, manifest] of Object.entries(response.versions)) {
-      metadata.versions.set(version, normalizeManifest(manifest as {}));
+      metadata.versions[version] = normalizeManifest(manifest as {});
     }
   }
 
   if (response['dist-tags']) {
+    // Store this for use with other npm utility packages
+    // tslint:disable-next-line: no-any
+    (metadata as any)['dist-tags'] = response['dist-tags'];
+
     for (const [tag, version] of Object.entries(response['dist-tags'])) {
-      const manifest = metadata.versions.get(version as string);
+      const manifest = metadata.versions[version as string];
       if (manifest) {
         metadata.tags[tag] = manifest;
       } else if (verbose) {

yarn.lock

@@ -7114,7 +7114,7 @@ npm-packlist@^1.1.12, npm-packlist@^1.1.6:
     ignore-walk "^3.0.1"
     npm-bundled "^1.0.1"
 
-npm-pick-manifest@^3.0.0:
+npm-pick-manifest@3.0.2, npm-pick-manifest@^3.0.0:
   version "3.0.2"
   resolved "https://registry.yarnpkg.com/npm-pick-manifest/-/npm-pick-manifest-3.0.2.tgz#f4d9e5fd4be2153e5f4e5f9b7be8dc419a99abb7"
   integrity sha512-wNprTNg+X5nf+tDi+hbjdHhM4bX+mKqv6XmPh7B5eG+QY9VARfQPfCEH013H5GqfNj6ee8Ij2fg8yk0mzps1Vw==