$.verbose = false; is not working as expected when using zx as a module

[bsnux]

Expected Behavior

When $.verbose = false then only output of the command should be displayed.

Example:

#!/usr/local/bin/node
import { $ } from 'zx'
$.verbose = false
await ($`uptime`)

Expected output:

9:42  up 16 days, 21 mins, 1 user, load averages: 1.63 2.00 3.2

Actual Behavior

Based on the example above:

$ uptime
9:42  up 16 days, 21 mins, 1 user, load averages: 1.63 2.00 3.2

Steps to Reproduce the Problem

1. Create a js file with the code mentioned above
2. $ node index.js

Specifications

• Version: zx version 1.2.2
• Platform: Darwin 19.6.0 Darwin Kernel Version 19.6.0

Suggested patch:

--- index.mjs
+++ index.mjs.original
@@ -53,12 +53,12 @@
     let child = exec($.prefix + cmd, options)
     let stdout = '', stderr = '', combined = ''
     child.stdout.on('data', data => {
-      process.stdout.write(data)
+      if ($.verbose) process.stdout.write(data)
       stdout += data
       combined += data
     })
     child.stderr.on('data', data => {
-      process.stderr.write(data)
+      if ($.verbose) process.stderr.write(data)
       stderr += data
       combined += data
     })

[antonmedv]

Works for me.

⌘ zx ⎇ main ❯❯❯ cat foo.mjs
#!/usr/local/bin/node
import { $ } from './index.mjs'
$.verbose = false
await ($`uptime`)
⌘ zx ⎇ main ❯❯❯ node foo.mjs
⌘ zx ⎇ main ❯❯❯

[bsnux]

That example is not displaying the output of the command. verbose should be just for the command.