Refactored

[Piterden]

No description provided.

[Piterden]

Why do you need this check?

[kumavis]

i dont remember why its there, though its certainly possible to pass in a value that does not have a toString method. Fallback should likely be String(chunks)

[jimmywarting]

suggested change:

import { Writable } from 'node:stream'

export class BrowserStdout extends Writable {
  constructor (opts) {
    super(opts)
    const label = opts.label ?? 'stdout'
    this.log = label === false 
      ? console.log
      : console.log.bind(null, label + ':')
  }

  _write (chunks, encoding, cb) {
    this.log(chunks.toString ? chunks.toString() : chunks)
    cb()
  }
}

[johnsnelson]

suggested change to @jimmywarting's suggested change:

import { Writable } from 'node:stream'

export class BrowserStdout extends Writable {
  constructor (opts) {
    super(opts)
    const label = opts.label ?? 'stdout'
    this.log = label === false 
      ? console.log

     : console.log.bind(null, label + ': ') //added space after colon.

}

_write (chunks, encoding, cb) {
this.log(chunks.toString ? chunks.toString() : chunks)
cb()
}
}

Not having that space there would drive me crazy.