refactor(@angular/cli): remove old package manager utilities

alan-agius4 · alan-agius4 · commit 9145f5e78045 · 2026-01-21T09:22:59.000Z
This removes the old implementation of the package-manager.
diff --git a/packages/angular/cli/src/command-builder/command-module.ts b/packages/angular/cli/src/command-builder/command-module.ts
@@ -6,9 +6,9 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { logging, schema } from '@angular-devkit/core';
+import { schema } from '@angular-devkit/core';
 import { readFileSync } from 'node:fs';
-import * as path from 'node:path';
+import { join, posix } from 'node:path';
 import type { ArgumentsCamelCase, Argv, CommandModule as YargsCommandModule } from 'yargs';
 import { Parser as yargsParser } from 'yargs/helpers';
 import { getAnalyticsUserId } from '../analytics/analytics';
@@ -17,7 +17,6 @@ import { EventCustomDimension, EventCustomMetric } from '../analytics/analytics-
 import { considerSettingUpAutocompletion } from '../utilities/completion';
 import { AngularWorkspace } from '../utilities/config';
 import { memoize } from '../utilities/memoize';
-import { PackageManagerUtils } from '../utilities/package-manager';
 import { CommandContext, CommandScope, Options, OtherOptions } from './definitions';
 import { Option, addSchemaOptionsToCommand } from './utilities/json-schema';
 
@@ -75,8 +74,8 @@ export abstract class CommandModule<T extends {} = {}> implements CommandModuleI
           ...(this.longDescriptionPath
             ? {
                 longDescriptionRelativePath: path
-                  .relative(path.join(__dirname, '../../../../'), this.longDescriptionPath)
-                  .replace(/\\/g, path.posix.sep),
+                  .relative(join(__dirname, '../../../../'), this.longDescriptionPath)
+                  .replace(/\\/g, posix.sep),
                 longDescription: readFileSync(this.longDescriptionPath, 'utf8').replace(
                   /\r\n/g,
                   '\n',
@@ -156,7 +155,7 @@ export abstract class CommandModule<T extends {} = {}> implements CommandModuleI
     return userId
       ? new AnalyticsCollector(this.context.logger, userId, {
           name: this.context.packageManager.name,
-          version: this.context.packageManager.version,
+          version: await this.context.packageManager.getVersion(),
         })
       : undefined;
   }
diff --git a/packages/angular/cli/src/command-builder/command-runner.ts b/packages/angular/cli/src/command-builder/command-runner.ts
@@ -6,19 +6,22 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { logging } from '@angular-devkit/core';
+import { JsonValue, isJsonObject, logging } from '@angular-devkit/core';
+import { readFile } from 'node:fs/promises';
+import { join } from 'node:path';
 import yargs from 'yargs';
 import { Parser as yargsParser } from 'yargs/helpers';
+import { getCacheConfig } from '../commands/cache/utilities';
 import {
   CommandConfig,
   CommandNames,
   RootCommands,
   RootCommandsAliases,
 } from '../commands/command-config';
+import { PackageManagerName, createPackageManager } from '../package-managers';
 import { colors } from '../utilities/color';
-import { AngularWorkspace, getWorkspace } from '../utilities/config';
+import { AngularWorkspace, getProjectByCwd, getWorkspace } from '../utilities/config';
 import { assertIsError } from '../utilities/error';
-import { PackageManagerUtils } from '../utilities/package-manager';
 import { VERSION } from '../utilities/version';
 import { CommandContext, CommandModuleError } from './command-module';
 import {
@@ -34,11 +37,12 @@ export async function runCommand(args: string[], logger: logging.Logger): Promis
     $0,
     _,
     help = false,
+    dryRun = false,
     jsonHelp = false,
     getYargsCompletions = false,
     ...rest
   } = yargsParser(args, {
-    boolean: ['help', 'json-help', 'get-yargs-completions'],
+    boolean: ['help', 'json-help', 'get-yargs-completions', 'dry-run'],
     alias: { 'collection': 'c' },
   });
 
@@ -60,16 +64,29 @@ export async function runCommand(args: string[], logger: logging.Logger): Promis
   }
 
   const root = workspace?.basePath ?? process.cwd();
-  const localYargs = yargs(args);
 
+  const cacheConfig = getCacheConfig(workspace);
+  const packageManager = await createPackageManager({
+    cwd: root,
+    logger,
+    dryRun,
+    tempDirectory: cacheConfig.enabled ? cacheConfig.path : undefined,
+    configuredPackageManager: await getConfiguredPackageManager(
+      root,
+      workspace,
+      globalConfiguration,
+    ),
+  });
+
+  const localYargs = yargs(args);
   const context: CommandContext = {
     globalConfiguration,
     workspace,
     logger,
     currentDirectory: process.cwd(),
     yargsInstance: localYargs,
     root,
-    packageManager: new PackageManagerUtils({ globalConfiguration, workspace, root }),
+    packageManager,
     args: {
       positional: positional.map((v) => v.toString()),
       options: {
@@ -163,3 +180,58 @@ async function getCommandsToRegister(
 
   return Promise.all(commands.map((command) => command.factory().then((m) => m.default)));
 }
+
+/**
+ * Gets the configured package manager by checking package.json, or the local and global angular.json files.
+ *
+ * @param root The root directory of the workspace.
+ * @param localWorkspace The local workspace.
+ * @param globalWorkspace The global workspace.
+ * @returns The package manager name.
+ */
+async function getConfiguredPackageManager(
+  root: string,
+  localWorkspace: AngularWorkspace | undefined,
+  globalWorkspace: AngularWorkspace,
+): Promise<PackageManagerName | undefined> {
+  let result: PackageManagerName | undefined;
+
+  try {
+    const packageJsonPath = join(root, 'package.json');
+    const pkgJson = JSON.parse(await readFile(packageJsonPath, 'utf-8')) as JsonValue;
+    result = getPackageManager(pkgJson);
+  } catch {}
+
+  if (result) {
+    return result;
+  }
+
+  if (localWorkspace) {
+    const project = getProjectByCwd(localWorkspace);
+    if (project) {
+      result = getPackageManager(localWorkspace.projects.get(project)?.extensions['cli']);
+    }
+
+    result ??= getPackageManager(localWorkspace.extensions['cli']);
+  }
+
+  result ??= getPackageManager(globalWorkspace.extensions['cli']);
+
+  return result;
+}
+
+/**
+ * Get the package manager name from a JSON value.
+ * @param source The JSON value to get the package manager name from.
+ * @returns The package manager name.
+ */
+function getPackageManager(source: JsonValue | undefined): PackageManagerName | undefined {
+  if (source && isJsonObject(source)) {
+    const value = source['packageManager'];
+    if (typeof value === 'string') {
+      return value.split('@', 1)[0] as unknown as PackageManagerName;
+    }
+  }
+
+  return undefined;
+}
diff --git a/packages/angular/cli/src/command-builder/definitions.ts b/packages/angular/cli/src/command-builder/definitions.ts
@@ -8,8 +8,8 @@
 
 import { logging } from '@angular-devkit/core';
 import type { Argv, CamelCaseKey } from 'yargs';
+import { PackageManager } from '../package-managers/package-manager';
 import { AngularWorkspace } from '../utilities/config';
-import { PackageManagerUtils } from '../utilities/package-manager';
 
 export enum CommandScope {
   /** Command can only run inside an Angular workspace. */
@@ -28,7 +28,7 @@ export interface CommandContext {
   workspace?: AngularWorkspace;
   globalConfiguration: AngularWorkspace;
   logger: logging.Logger;
-  packageManager: PackageManagerUtils;
+  packageManager: PackageManager;
   yargsInstance: Argv<{}>;
 
   /** Arguments parsed in free-from without parser configuration. */
diff --git a/packages/angular/cli/src/commands/add/cli.ts b/packages/angular/cli/src/commands/add/cli.ts
@@ -10,7 +10,7 @@ import { Listr, ListrRenderer, ListrTaskWrapper, color, figures } from 'listr2';
 import assert from 'node:assert';
 import fs from 'node:fs/promises';
 import { createRequire } from 'node:module';
-import { dirname, join, relative, resolve } from 'node:path';
+import { dirname, join } from 'node:path';
 import npa from 'npm-package-arg';
 import semver, { Range, compare, intersects, prerelease, satisfies, valid } from 'semver';
 import { Argv } from 'yargs';
@@ -25,16 +25,13 @@ import {
 } from '../../command-builder/schematics-command-module';
 import {
   NgAddSaveDependency,
-  PackageManager,
   PackageManagerError,
   PackageManifest,
   PackageMetadata,
-  createPackageManager,
 } from '../../package-managers';
 import { assertIsError } from '../../utilities/error';
 import { isTTY } from '../../utilities/tty';
 import { VERSION } from '../../utilities/version';
-import { getCacheConfig } from '../cache/utilities';
 
 class CommandError extends Error {}
 
@@ -46,7 +43,6 @@ interface AddCommandArgs extends SchematicsCommandArgs {
 }
 
 interface AddCommandTaskContext {
-  packageManager: PackageManager;
   packageIdentifier: npa.Result;
   savePackage?: NgAddSaveDependency;
   collectionName?: string;
@@ -198,7 +194,8 @@ export default class AddCommandModule
       [
         {
           title: 'Determining Package Manager',
-          task: (context, task) => this.determinePackageManagerTask(context, task),
+          task: (_context, task) =>
+            (task.output = `Using package manager: ${color.dim(this.context.packageManager.name)}`),
           rendererOptions: { persistentOutput: true },
         },
         {
@@ -309,47 +306,14 @@ export default class AddCommandModule
     }
   }
 
-  private async determinePackageManagerTask(
-    context: AddCommandTaskContext,
-    task: AddCommandTaskWrapper,
-  ): Promise<void> {
-    let tempDirectory: string | undefined;
-    const tempOptions = ['node_modules'];
-
-    const cacheConfig = getCacheConfig(this.context.workspace);
-    if (cacheConfig.enabled) {
-      const cachePath = resolve(this.context.root, cacheConfig.path);
-      if (!relative(this.context.root, cachePath).startsWith('..')) {
-        tempOptions.push(cachePath);
-      }
-    }
-
-    for (const tempOption of tempOptions) {
-      try {
-        const directory = resolve(this.context.root, tempOption);
-        if ((await fs.stat(directory)).isDirectory()) {
-          tempDirectory = directory;
-          break;
-        }
-      } catch {}
-    }
-
-    context.packageManager = await createPackageManager({
-      cwd: this.context.root,
-      logger: this.context.logger,
-      dryRun: context.dryRun,
-      tempDirectory,
-    });
-    task.output = `Using package manager: ${color.dim(context.packageManager.name)}`;
-  }
-
   private async findCompatiblePackageVersionTask(
     context: AddCommandTaskContext,
     task: AddCommandTaskWrapper,
     options: Options<AddCommandArgs>,
   ): Promise<void> {
     const { registry, verbose } = options;
-    const { packageManager, packageIdentifier } = context;
+    const { packageIdentifier } = context;
+    const { packageManager } = this.context;
     const packageName = packageIdentifier.name;
 
     assert(packageName, 'Registry package identifiers should always have a name.');
@@ -446,7 +410,8 @@ export default class AddCommandModule
       rejectionReasons: string[];
     },
   ): Promise<PackageManifest | null> {
-    const { packageManager, packageIdentifier } = context;
+    const { packageIdentifier } = context;
+    const { packageManager } = this.context;
     const { registry, verbose, rejectionReasons } = options;
     const packageName = packageIdentifier.name;
     assert(packageName, 'Package name must be defined.');
@@ -524,9 +489,12 @@ export default class AddCommandModule
 
     let manifest;
     try {
-      manifest = await context.packageManager.getManifest(context.packageIdentifier.toString(), {
-        registry,
-      });
+      manifest = await this.context.packageManager.getManifest(
+        context.packageIdentifier.toString(),
+        {
+          registry,
+        },
+      );
     } catch (e) {
       assertIsError(e);
       throw new CommandError(
@@ -585,7 +553,8 @@ export default class AddCommandModule
     options: Options<AddCommandArgs>,
   ): Promise<void> {
     const { registry } = options;
-    const { packageManager, packageIdentifier, savePackage } = context;
+    const { packageIdentifier, savePackage } = context;
+    const { packageManager } = this.context;
 
     // Only show if installation will actually occur
     task.title = 'Installing package';
diff --git a/packages/angular/cli/src/commands/update/cli.ts b/packages/angular/cli/src/commands/update/cli.ts
@@ -20,12 +20,7 @@ import {
   Options,
 } from '../../command-builder/command-module';
 import { SchematicEngineHost } from '../../command-builder/utilities/schematic-engine-host';
-import {
-  InstalledPackage,
-  PackageManager,
-  PackageManifest,
-  createPackageManager,
-} from '../../package-managers';
+import { InstalledPackage, PackageManager, PackageManifest } from '../../package-managers';
 import { colors } from '../../utilities/color';
 import { disableVersionCheck } from '../../utilities/environment-options';
 import { assertIsError } from '../../utilities/error';
@@ -168,13 +163,7 @@ export default class UpdateCommandModule extends CommandModule<UpdateCommandArgs
   }
 
   async run(options: Options<UpdateCommandArgs>): Promise<number | void> {
-    const { logger } = this.context;
-    // Instantiate the package manager
-    const packageManager = await createPackageManager({
-      cwd: this.context.root,
-      logger,
-      configuredPackageManager: this.context.packageManager.name,
-    });
+    const { logger, packageManager } = this.context;
 
     // Check if the current installed CLI version is older than the latest compatible version.
     // Skip when running `ng update` without a package name as this will not trigger an actual update.
diff --git a/packages/angular/cli/src/package-managers/host.ts b/packages/angular/cli/src/package-managers/host.ts
@@ -14,8 +14,8 @@
  */
 
 import { type SpawnOptions, spawn } from 'node:child_process';
-import { Stats, constants } from 'node:fs';
-import { copyFile, mkdtemp, readFile, readdir, rm, stat, writeFile } from 'node:fs/promises';
+import { Stats, constants, existsSync } from 'node:fs';
+import { copyFile, mkdir, mkdtemp, readFile, readdir, rm, stat, writeFile } from 'node:fs/promises';
 import { platform, tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { PackageManagerError } from './error';
@@ -24,6 +24,13 @@ import { PackageManagerError } from './error';
  * An abstraction layer for side-effectful operations.
  */
 export interface Host {
+  /**
+   * Creates a directory.
+   * @param path The path to the directory.
+   * @returns A promise that resolves when the directory is created.
+   */
+  mkdir(path: string): Promise<void>;
+
   /**
    * Gets the stats of a file or directory.
    * @param path The path to the file or directory.
@@ -101,10 +108,12 @@ export interface Host {
 export const NodeJS_HOST: Host = {
   stat,
   readdir,
+  mkdir,
   readFile: (path: string) => readFile(path, { encoding: 'utf8' }),
   copyFile: (src, dest) => copyFile(src, dest, constants.COPYFILE_FICLONE),
   writeFile,
-  createTempDirectory: (baseDir?: string) => mkdtemp(join(baseDir ?? tmpdir(), 'angular-cli-')),
+  createTempDirectory: (baseDir?: string) =>
+    mkdtemp(join(baseDir ?? tmpdir(), 'angular-cli-tmp-packages-')),
   deleteDirectory: (path: string) => rm(path, { recursive: true, force: true }),
   runCommand: async (
     command: string,
diff --git a/packages/angular/cli/src/package-managers/package-manager.ts b/packages/angular/cli/src/package-managers/package-manager.ts
diff --git a/packages/angular/cli/src/utilities/package-manager.ts b/packages/angular/cli/src/utilities/package-manager.ts
