fix: hide banner for exec and explore

Author: lukekarrys

lib/npm.js

@@ -174,6 +174,9 @@ class Npm {
       progress: this.flatOptions.progress,
       json: this.config.get('json'),
       heading: this.config.get('heading'),
+      // TODO: avoid passing all of npm in. this is a quick
+      // fix to allow display to know the currently running command
+      npm: this,
     })
     process.env.COLOR = this.color ? '1' : '0'
 

lib/utils/display.js

@@ -124,6 +124,7 @@ class Display {
   #json
   #heading
   #silent
+  #npm
 
   // display streams
   #stdout
@@ -167,7 +168,9 @@ class Display {
     stdoutColor,
     timing,
     unicode,
+    npm,
   }) {
+    this.#npm = npm
     // get createSupportsColor from chalk directly if this lands
     // https://github.com/chalk/chalk/pull/600
     const [{ Chalk }, { createSupportsColor }] = await Promise.all([
@@ -273,9 +276,10 @@ class Display {
     }
 
     // HACK: if it looks like the banner and we are silent do not print it.
-    // There's no other way to do this right now :(
-    // eslint-disable-next-line max-len
-    if (this.#silent && args.length === 1 && args[0].startsWith('\n> ') && args[0].endsWith('\n')) {
+    // There's no other way to do this right now :(\
+    const isBanner = args.length === 1 && args[0].startsWith('\n> ') && args[0].endsWith('\n')
+    const hideBanner = this.#silent || ['exec', 'explore'].includes(this.#npm.command)
+    if (isBanner && hideBanner) {
       return
     }