refactor(@angular/cli): support copying config files to temp package directory

This change adds support for copying package manager configuration files (like `bunfig.toml`) from the project root to the temporary directory used for acquiring packages. This is particularly necessary for Bun, which does not automatically inherit configuration from parent directories when running in a separate temporary location.

(cherry picked from commit d58c714)

Author: clydin

packages/angular/cli/src/package-managers/host.ts

@@ -14,8 +14,8 @@
  */
 
 import { type SpawnOptions, spawn } from 'node:child_process';
-import { Stats } from 'node:fs';
-import { mkdtemp, readFile, readdir, rm, stat, writeFile } from 'node:fs/promises';
+import { Stats, constants } from 'node:fs';
+import { copyFile, 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';
@@ -45,6 +45,14 @@ export interface Host {
    */
   readFile(path: string): Promise<string>;
 
+  /**
+   * Copies a file from the source path to the destination path.
+   * @param src The path to the source file.
+   * @param dest The path to the destination file.
+   * @returns A promise that resolves when the copy is complete.
+   */
+  copyFile(src: string, dest: string): Promise<void>;
+
   /**
    * Creates a new, unique temporary directory.
    * @param baseDir The base directory in which to create the temporary directory.
@@ -94,6 +102,7 @@ export const NodeJS_HOST: Host = {
   stat,
   readdir,
   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-')),
   deleteDirectory: (path: string) => rm(path, { recursive: true, force: true }),

packages/angular/cli/src/package-managers/package-manager-descriptor.ts

@@ -65,6 +65,15 @@ export interface PackageManagerDescriptor {
   /** The flag to ignore peer dependency warnings/errors. */
   readonly ignorePeerDependenciesFlag?: string;
 
+  /** The configuration files used by the package manager. */
+  readonly configFiles: readonly string[];
+
+  /**
+   * Whether to copy configuration files from the project root to the temporary directory.
+   * This is necessary for package managers that do not inherit configuration from parent directories (e.g., bun).
+   */
+  readonly copyConfigFromProject?: boolean;
+
   /** A function that returns the arguments and environment variables to use a custom registry. */
   readonly getRegistryOptions?: (registry: string) => {
     args?: string[];
@@ -144,6 +153,7 @@ export const SUPPORTED_PACKAGE_MANAGERS = {
     noLockfileFlag: '--no-package-lock',
     ignoreScriptsFlag: '--ignore-scripts',
     ignorePeerDependenciesFlag: '--force',
+    configFiles: ['.npmrc'],
     getRegistryOptions: (registry: string) => ({ args: ['--registry', registry] }),
     versionCommand: ['--version'],
     listDependenciesCommand: ['list', '--depth=0', '--json=true', '--all=true'],
@@ -168,6 +178,7 @@ export const SUPPORTED_PACKAGE_MANAGERS = {
     saveDevFlag: '--dev',
     noLockfileFlag: '',
     ignoreScriptsFlag: '--mode=skip-build',
+    configFiles: ['.yarnrc.yml', '.yarnrc.yaml'],
     getRegistryOptions: (registry: string) => ({ env: { YARN_NPM_REGISTRY_SERVER: registry } }),
     versionCommand: ['--version'],
     listDependenciesCommand: ['list', '--depth=0', '--json', '--recursive=false'],
@@ -195,6 +206,7 @@ export const SUPPORTED_PACKAGE_MANAGERS = {
     saveDevFlag: '--dev',
     noLockfileFlag: '--no-lockfile',
     ignoreScriptsFlag: '--ignore-scripts',
+    configFiles: ['.yarnrc', '.npmrc'],
     getRegistryOptions: (registry: string) => ({ args: ['--registry', registry] }),
     versionCommand: ['--version'],
     listDependenciesCommand: ['list', '--depth=0', '--json'],
@@ -220,6 +232,7 @@ export const SUPPORTED_PACKAGE_MANAGERS = {
     noLockfileFlag: '--no-lockfile',
     ignoreScriptsFlag: '--ignore-scripts',
     ignorePeerDependenciesFlag: '--strict-peer-dependencies=false',
+    configFiles: ['.npmrc', 'pnpm-workspace.yaml'],
     getRegistryOptions: (registry: string) => ({ args: ['--registry', registry] }),
     versionCommand: ['--version'],
     listDependenciesCommand: ['list', '--depth=0', '--json'],
@@ -244,6 +257,8 @@ export const SUPPORTED_PACKAGE_MANAGERS = {
     saveDevFlag: '--development',
     noLockfileFlag: '', // Bun does not have a flag for this.
     ignoreScriptsFlag: '--ignore-scripts',
+    configFiles: ['bunfig.toml', '.npmrc'],
+    copyConfigFromProject: true,
     getRegistryOptions: (registry: string) => ({ args: ['--registry', registry] }),
     versionCommand: ['--version'],
     listDependenciesCommand: ['pm', 'ls', '--json'],

packages/angular/cli/src/package-managers/package-manager.ts

@@ -552,6 +552,18 @@ export class PackageManager {
     // Writing an empty package.json file beforehand prevents this.
     await this.host.writeFile(join(workingDirectory, 'package.json'), '{}');
 
+    // Copy configuration files if the package manager requires it (e.g., bun).
+    if (this.descriptor.copyConfigFromProject) {
+      for (const configFile of this.descriptor.configFiles) {
+        try {
+          const configPath = join(this.cwd, configFile);
+          await this.host.copyFile(configPath, join(workingDirectory, configFile));
+        } catch {
+          // Ignore missing config files.
+        }
+      }
+    }
+
     const flags = [options.ignoreScripts ? this.descriptor.ignoreScriptsFlag : ''].filter(
       (flag) => flag,
     );

packages/angular/cli/src/package-managers/testing/mock-host.ts

@@ -62,4 +62,8 @@ export class MockHost implements Host {
   readFile(): Promise<string> {
     throw new Error('Method not implemented.');
   }
+
+  copyFile(): Promise<void> {
+    throw new Error('Method not implemented.');
+  }
 }