Nx: @nrwl/workspace:run-commands crashes silently when processing a large quantity of log output

[james-gibson]

Current Behavior

@nrwl/workspace:run-commands crashes when given too much output text

Expected Behavior

@nrwl/workspace:run-commands should not crash when given too much output text

If @nrwl/workspace:run-commands must crash, it should provide more debugging information.

Steps to Reproduce

git clone [email protected]:james-gibson/nx-run-commands-mem-test.git
cd nx-run-commands-mem-test
npm i
nx start memtest

See https://github.com/james-gibson/nx-run-commands-mem-test

Failure Logs

$ nx start memtest

> nx run memtest:start 
Starting fake compilation

> nx run memtest:prepare 
Starting fake prepare mode
Finishing fake prepare mode

> nx run memtest:hot 

> nx run memtest:watch 

> nx run memtest:serve 
Starting fake watch mode
Serving

The quick brown fox jumped over the dogs back
The quick brown fox jumped over the dogs back
The quick brown fox jumped over the dogs back

...

The quick brown fox jumped over the dogs back
The quick brown fox jumped over the dogs back
The quick brown fox jumped over the dogs back
Warning: @nrwl/run-commands command "nx watch memtest" exited with non-zero status code

Environment

>  NX  Report complete - copy this into the issue template

  nx : Not Found
  @nrwl/angular : Not Found
  @nrwl/cli : 10.0.13
  @nrwl/cypress : Not Found
  @nrwl/eslint-plugin-nx : Not Found
  @nrwl/express : Not Found
  @nrwl/jest : Not Found
  @nrwl/linter : Not Found
  @nrwl/nest : Not Found
  @nrwl/next : Not Found
  @nrwl/node : Not Found
  @nrwl/react : Not Found
  @nrwl/schematics : Not Found
  @nrwl/tao : 10.0.13
  @nrwl/web : Not Found
  @nrwl/workspace : 10.0.13
  typescript : 3.9.7

[jeffbcross]

Thanks for the reproduction, I've asked the Nx core team to take a look.

[vsavkin]

Thanks for submitting the issue.

Fixed it here: https://github.com/nrwl/nx/commit/f96377b74e7a6332e6f3005a9a6fa811571a8a0b

It will be part of 10.1, which we are going to release tomorrow.

[myspivey]

Thanks @vsavkin for the quick fix! Yall are incredible!

[FrozenPandaz]

Closing as it's fixed in master. Please stay tuned for the fix to be released!