es-tooling / ecosystem-cleanup

A place to keep track of ongoing efforts to clean up the JS ecosystem
386 stars 2 forks source link

Replace `strip-ansi` with native `stripVTControlCharacters` #122

Open Namchee opened 1 month ago

Namchee commented 1 month ago

Overview

stripVTControlCharacters from Node core utilities provide the same functionality.

Do note that this utility function requires Node 16.11.0 or higher.

benmccann commented 1 month ago

Some big ones:

The latter two are by the same author as strip-ansi, so TBD if they'd want to swap. Worth a shot though

ziebam commented 1 month ago

Also:

43081j commented 1 month ago

The latter two are by the same author as strip-ansi, so TBD if they'd want to swap. Worth a shot though

Maybe we can go one level higher and migrate away from the packages consuming it?

Namchee commented 1 month ago

Would like to include:

Done:

In Progress:

Namchee commented 1 month ago

As for lerna, I think it's better for us to do it on nx instead since the stewardship has passed from Lerna

benmccann commented 1 month ago

I noticed today that enquirer says it has only a single dependency. It's not true, but would be if this migration were done

Namchee commented 1 month ago

I don't think we can remove strip-ansi from enquirer as enquirer sets the engine to >=8.6