fix(@angular/cli): update bun package manager descriptor fields

Author: clydin

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

@@ -20,7 +20,7 @@ import {
   parseNpmLikeManifest,
   parseNpmLikeMetadata,
   parseYarnClassicDependencies,
-  parseYarnLegacyManifest,
+  parseYarnClassicManifest,
   parseYarnModernDependencies,
 } from './parsers';
 
@@ -168,7 +168,7 @@ export const SUPPORTED_PACKAGE_MANAGERS = {
     getManifestCommand: ['info', '--json'],
     outputParsers: {
       listDependencies: parseYarnClassicDependencies,
-      getRegistryManifest: parseYarnLegacyManifest,
+      getRegistryManifest: parseYarnClassicManifest,
       getRegistryMetadata: parseNpmLikeMetadata,
     },
   },
@@ -202,14 +202,13 @@ export const SUPPORTED_PACKAGE_MANAGERS = {
     forceFlag: '--force',
     saveExactFlag: '--exact',
     saveTildeFlag: '', // Bun does not have a flag for tilde, it defaults to caret.
-    saveDevFlag: '--development',
+    saveDevFlag: '--dev',
     noLockfileFlag: '', // Bun does not have a flag for this.
     ignoreScriptsFlag: '--ignore-scripts',
     getRegistryOptions: (registry: string) => ({ args: ['--registry', registry] }),
     versionCommand: ['--version'],
     listDependenciesCommand: ['pm', 'ls', '--json'],
     getManifestCommand: ['pm', 'view', '--json'],
-    viewCommandFieldArgFormatter: (fields) => [...fields],
     outputParsers: {
       listDependencies: parseNpmLikeDependencies,
       getRegistryManifest: parseNpmLikeManifest,

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

@@ -156,12 +156,14 @@ export class PackageManager {
       return { stdout: '', stderr: '' };
     }
 
-    return this.host.runCommand(this.descriptor.binary, finalArgs, {
+    const commandResult = await this.host.runCommand(this.descriptor.binary, finalArgs, {
       ...runOptions,
       cwd: executionDirectory,
       stdio: 'pipe',
       env: finalEnv,
     });
+
+    return { stdout: commandResult.stdout.trim(), stderr: commandResult.stderr.trim() };
   }
 
   /**

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

@@ -274,7 +274,7 @@ export function parseNpmLikeMetadata(stdout: string, logger?: Logger): PackageMe
  * @param logger An optional logger instance.
  * @returns The package manifest object.
  */
-export function parseYarnLegacyManifest(stdout: string, logger?: Logger): PackageManifest | null {
+export function parseYarnClassicManifest(stdout: string, logger?: Logger): PackageManifest | null {
   logger?.debug(`Parsing yarn classic manifest...`);
   logStdout(stdout, logger);
 
@@ -287,5 +287,7 @@ export function parseYarnLegacyManifest(stdout: string, logger?: Logger): Packag
   const data = JSON.parse(stdout);
 
   // Yarn classic wraps the manifest in a `data` property.
-  return data.data ?? data;
+  const manifest = data.data as PackageManifest;
+
+  return manifest;
 }