esc[39m could be replaced with esc[0m for easier interpretation by other programs? about cli-highlight

[felixfbecker]

Frankly I don't know what those escape sequences mean, I just use https://github.com/chalk/chalk

[christopherreay]

so, the issue is that when "your code" is returning to "default text", there are two ways of doing it

^[39m is "white"
^[0m is "default"

I suppose you have asked chalk to "go to white". I tried to have a look myself, but i couldnt see where the logic for the code was. I can have a look if you like, if you can point me in roughly the right direction

You can imagine, without ever "going to default", some tools are creating extremely nested output

[felixfbecker]

The nodes are wrapped in escape sequences here:

cli-highlight/src/index.ts

Line 17 in c075112

return ((theme as any)[token] || (DEFAULT_THEME as any)[token] || plain)(nodeData)

with the default theme being defined here:

cli-highlight/src/theme.ts

Line 294 in c075112

export const DEFAULT_THEME: Theme = {

[christopherreay]

ok, Ill have a look.

At the moment im processing the output with sed to do the same.

Thanks :)

Christopher

[sindresorhus]

^[39m is "white"

This is not correct. That code escape closes/resets a foreground color.

See: https://github.com/chalk/ansi-styles/blob/de7527a86c1cf49906b0eb32a0de1402d849ccc2/index.js#L34-L51

[christopherreay]

ok, so I spoke to chalk, they explained what [39m and [0m mean:

[39m means "reset the foreground color to default"
[0m means "reset everything to default"

So they reckon that it is the script I am using that is bugged, not the output of cli-highlight that should be changed :)

Cheers :)