0.3.7 is broken

[zloirock]

After updating implicit cli-table dependency from 0.3.6 that works fine to 0.3.7, I have:

How to reproduce it - copy https://github.com/zloirock/core-js/tree/1aaa9b1144b444e9fa1376e2f1d72c34fb99b83e and run npm run r.

[zloirock]

https://github.com/zloirock/core-js/runs/4240992349?check_suite_focus=true

[dmeeks91]

Can confirm that I am seeing the same error.

This line will throw an error if typeof stripped != "string"
Looks like PR #149 introduced a scenario where this constraint is not met. I'm guessing this is when the bug was introduced.

I haven't studied the code enough to determine if the preferred solution is to type check in the strlen utility function or revert the recently merged PR.

[pomek]

I face the same issue. In my case, I decided to cast all numbers I pass into the table to strings. It helped.

[adrianosmond]

var stripped = ("" + str).replace(code,'');
was changed to
var stripped = stripAnsi(str);

so numbers aren't getting converted to strings before stripping them. Maybe
var stripped = stripAnsi("" + str);
would be a friendlier, more backwards compatible approach?

[dreinert1994]

This problem is currently crashing my build pipeline as it is a dependency of a dependency for us. I put a PR to try and resolve the issue, hoping to get it reviewed and merged ASAP

[ljharb]

See #160 - any inclusion of strip-ansi v6 is a breaking change, because it drops support for node < 12, and the v0.3 line claims support for node 0.2+.