muesli / termenv / 5694552778

58%

master: 50%

push

github

fix!: output race condition

`ColorProfile` reads the terminal environment every time the function is
called. This is inefficient. We only need to read the `$TERM`
environment variable once when we initialize the output. So instead, we cache
the value we read.

Rename `ColorProfile` to `termColorProfile` and rely on
`EnvColorProfile` to detect the profile. Ideally, we would rely on the
terminal Terminfo to detect the profile and fallback to the environment.
But that's for another day :)

Make output profile accessible through `ColorProfile`.
Use `SetColorProfile` to change the output color profile.
Use a mutex to guard output writes.
Use pointer receiver since we have a lock in the struct

Fixes: https://github.com/charmbracelet/lipgloss/pull/210
Fixes: https://github.com/muesli/termenv/issues/145

81 of 81 new or added lines in 6 files covered. (100.0%)

548 of 939 relevant lines covered (58.36%)

18.86 hits per line