feat!: output using proc-log

BREAKING CHANGE: The existing banner is now emitted using `proc-log`
instead of `console.log`.  It is always emitted. Consuming libraries can
decide under which situations to show the banner.

Author: wraithgar

Diff for: lib/run-script-pkg.js

@@ -4,19 +4,7 @@ const packageEnvs = require('./package-envs.js')
 const { isNodeGypPackage, defaultGypInstallScript } = require('@npmcli/node-gyp')
 const signalManager = require('./signal-manager.js')
 const isServerPackage = require('./is-server-package.js')
-
-// you wouldn't like me when I'm angry...
-const bruce = (id, event, cmd, args) => {
-  let banner = id
-    ? `\n> ${id} ${event}\n`
-    : `\n> ${event}\n`
-  banner += `> ${cmd.trim().replace(/\n/g, '\n> ')}`
-  if (args.length) {
-    banner += ` ${args.join(' ')}`
-  }
-  banner += '\n'
-  return banner
-}
+const { output } = require('proc-log')
 
 const runScriptPkg = async options => {
   const {
@@ -29,8 +17,6 @@ const runScriptPkg = async options => {
     pkg,
     args = [],
     stdioString,
-    // note: only used when stdio:inherit
-    banner = true,
     // how long to wait for a process.kill signal
     // only exposed here so that we can make the test go a bit faster.
     signalTimeout = 500,
@@ -59,10 +45,17 @@ const runScriptPkg = async options => {
     return { code: 0, signal: null }
   }
 
-  if (stdio === 'inherit' && banner !== false) {
-    // we're dumping to the parent's stdout, so print the banner
-    console.log(bruce(pkg._id, event, cmd, args))
+  // consuming library decides if this is displayed or not
+  output.standard('')
+  if (pkg._id) {
+    output.standard(`> ${pkg._id} ${event}`)
+  } else {
+    output.standard(`> ${event}`)
+  }
+  if (args.length) {
+    output.standard(`> ${cmd.trim().replace(/\n/g, '\n> ')} ${args.join(' ')}`)
   }
+  output.standard('')
 
   const [spawnShell, spawnArgs, spawnOpts] = makeSpawnArgs({
     event,

Diff for: test/run-script-pkg.js

@@ -6,18 +6,17 @@ const isWindows = process.platform === 'win32'
 const emptyDir = t.testdir({})
 
 const pkill = process.kill
-const consoleLog = console.log
 
-const mockConsole = t => {
-  const logs = []
-  console.log = (...args) => logs.push(args)
-  t.teardown(() => console.log = consoleLog)
-  return logs
-}
+const output = []
+process.on('output', (level, ...args) => {
+  if (level === 'standard') {
+    output.push([...args])
+  }
+})
+t.afterEach(() => output.length = 0)
 
 t.test('run-script-pkg', async t => {
-  await t.test('do the banner when stdio is inherited, handle line breaks', async t => {
-    const logs = mockConsole(t)
+  await t.test('output with no args and a pkgid', async t => {
     spawk.spawn('sh', a => a.includes('bar\nbaz\n'))
     await runScript({
       event: 'foo',
@@ -33,34 +32,15 @@ t.test('run-script-pkg', async t => {
         scripts: {},
       },
     })
-    t.strictSame(logs, [['\n> [email protected] foo\n> bar\n> baz\n']])
-    t.ok(spawk.done())
-  })
-
-  await t.test('do not show banner when stdio is inherited, if suppressed', async t => {
-    const logs = mockConsole(t)
-    spawk.spawn('sh', a => a.includes('bar'))
-    await runScript({
-      event: 'foo',
-      path: emptyDir,
-      scriptShell: 'sh',
-      env: {
-        environ: 'value',
-      },
-      stdio: 'inherit',
-      cmd: 'bar',
-      pkg: {
-        _id: '[email protected]',
-        scripts: {},
-      },
-      banner: false,
-    })
-    t.strictSame(logs, [])
+    t.strictSame(output, [
+      [''],
+      ['> [email protected] foo'],
+      [''],
+    ])
     t.ok(spawk.done())
   })
 
-  await t.test('do the banner with no pkgid', async t => {
-    const logs = mockConsole(t)
+  await t.test('output with args and no pkgid', async t => {
     spawk.spawn('sh', a => a.includes('bar baz buzz'))
     await runScript({
       event: 'foo',
@@ -76,12 +56,16 @@ t.test('run-script-pkg', async t => {
         scripts: {},
       },
     })
-    t.strictSame(logs, [['\n> foo\n> bar baz buzz\n']])
+    t.strictSame(output, [
+      [''],
+      ['> foo'],
+      ['> bar baz buzz'],
+      [''],
+    ])
     t.ok(spawk.done())
   })
 
   await t.test('pkg has foo script', async t => {
-    const logs = mockConsole(t)
     spawk.spawn('sh', a => a.includes('bar'))
     await runScript({
       event: 'foo',
@@ -98,12 +82,15 @@ t.test('run-script-pkg', async t => {
         },
       },
     })
-    t.strictSame(logs, [])
+    t.strictSame(output, [
+      [''],
+      ['> [email protected] foo'],
+      [''],
+    ])
     t.ok(spawk.done())
   })
 
   await t.test('pkg has foo script, with args', async t => {
-    const logs = mockConsole(t)
     spawk.spawn('sh', a => a.includes('bar a b c'))
     await runScript({
       event: 'foo',
@@ -122,7 +109,12 @@ t.test('run-script-pkg', async t => {
       args: ['a', 'b', 'c'],
       binPaths: false,
     })
-    t.strictSame(logs, [])
+    t.strictSame(output, [
+      [''],
+      ['> [email protected] foo'],
+      ['> bar a b c'],
+      [''],
+    ])
     t.ok(spawk.done())
   })
 
@@ -131,7 +123,6 @@ t.test('run-script-pkg', async t => {
       'binding.gyp': 'exists',
     })
 
-    const logs = mockConsole(t)
     spawk.spawn('sh', a => a.includes('node-gyp rebuild'))
     await runScript({
       event: 'install',
@@ -146,7 +137,11 @@ t.test('run-script-pkg', async t => {
         scripts: {},
       },
     })
-    t.strictSame(logs, [])
+    t.strictSame(output, [
+      [''],
+      ['> [email protected] install'],
+      [''],
+    ])
     t.ok(spawk.done())
   })