stdout/stderr with 'inherit' + transform should not capture output

[bcoughlan]

#121 implemented the ability to transform stdout/stderr while using inherit. However the output is captured, like pipe mode, which is a source of memory leaks for long running processes.

Example:

import { execa } from 'execa';

async function main() {
    await execa({
        stdout: [function* (line) {
            yield `[test] ${line}`;
        }, 'inherit'],
        stderr: [function* (line) {
            yield `[test] ${line}`;
        }, 'inherit'],
    })('sh', ['ls', '-la', '/bad'])
}

main()

Expected: thrown error does not contain stdio or stderr properties.

Actual: output includes...

  stderr: '[test] /bin/ls: /bin/ls: cannot execute binary file',
  stdio: [
    undefined,
    '',
    '[test] /bin/ls: /bin/ls: cannot execute binary file'
  ],

[ehmicky]

Hi @bcoughlan,

Thanks for reaching out.

To avoid capturing output, the buffer option must be set to false.

[bcoughlan]

Thank you so much, that worked. Apologies for the noise.