fix, hide pnpm output when a command is running with --json (#9275)

Author: davidfirst

scopes/dependencies/dependency-resolver/dependency-installer.ts

@@ -102,15 +102,15 @@ export class DependencyInstaller {
 
     private preferOffline?: boolean,
 
-    private installingContext: DepInstallerContext = {},
+    private installingContext: DepInstallerContext = {}
   ) {}
 
   async install(
     rootDir: string | undefined,
     rootPolicy: WorkspacePolicy,
     componentDirectoryMap: ComponentMap<string>,
     options: InstallOptions = DEFAULT_INSTALL_OPTIONS,
-    packageManagerOptions: PackageManagerInstallOptions = DEFAULT_PM_INSTALL_OPTIONS,
+    packageManagerOptions: PackageManagerInstallOptions = DEFAULT_PM_INSTALL_OPTIONS
   ) {
     const finalRootDir = rootDir ?? this.rootDir;
     if (!finalRootDir) {
@@ -131,7 +131,7 @@ export class DependencyInstaller {
       rootPolicy,
       componentDirectoryMap,
       options,
-      packageManagerOptions,
+      packageManagerOptions
     );
   }
 
@@ -141,7 +141,7 @@ export class DependencyInstaller {
     rootPolicy: WorkspacePolicy,
     componentDirectoryMap: ComponentMap<string>,
     options: InstallOptions = DEFAULT_INSTALL_OPTIONS,
-    packageManagerOptions: PackageManagerInstallOptions = DEFAULT_PM_INSTALL_OPTIONS,
+    packageManagerOptions: PackageManagerInstallOptions = DEFAULT_PM_INSTALL_OPTIONS
   ): Promise<{ dependenciesChanged: boolean }> {
     const args = {
       componentDirectoryMap,
@@ -159,7 +159,7 @@ export class DependencyInstaller {
     if (options.linkedDependencies) {
       manifests = JSON.parse(JSON.stringify(manifests));
       const linkedDependencies = JSON.parse(
-        JSON.stringify(options.linkedDependencies),
+        JSON.stringify(options.linkedDependencies)
       ) as typeof options.linkedDependencies;
       if (linkedDependencies[finalRootDir]) {
         const directDeps = new Set<string>();
@@ -188,7 +188,9 @@ export class DependencyInstaller {
         };
       });
     }
-    const hidePackageManagerOutput = !!(this.installingContext.inCapsule && process.env.VERBOSE_PM_OUTPUT !== 'true');
+    const isJsonCmd = process.argv.includes('--json') || process.argv.includes('-j');
+    const hidePackageManagerOutput =
+      Boolean(this.installingContext.inCapsule && process.env.VERBOSE_PM_OUTPUT !== 'true') || isJsonCmd;
 
     // Make sure to take other default if passed options with only one option
     const calculatedPmOpts = {
@@ -252,7 +254,7 @@ export class DependencyInstaller {
         manifests,
         componentDirectoryMap,
       },
-      calculatedPmOpts,
+      calculatedPmOpts
     );
     if (!hidePackageManagerOutput) {
       this.logger.consoleSuccess(`done ${message}`, startTime);
@@ -303,7 +305,7 @@ export class DependencyInstaller {
       rootDir,
       componentDirectoryMap.components,
       options,
-      this.installingContext,
+      this.installingContext
     );
     const manifests: Record<string, ProjectManifest> = componentDirectoryMap
       .toArray()
@@ -335,7 +337,7 @@ export class DependencyInstaller {
   private async runPrePostSubscribers(
     subscribers: PreInstallSubscriberList | PostInstallSubscriberList = [],
     type: 'pre' | 'post',
-    args: InstallArgs,
+    args: InstallArgs
   ): Promise<void> {
     const message = this.installingContext?.inCapsule
       ? `(capsule) running ${type} install subscribers in root dir ${this.rootDir}`