muesli / termenv / 5694303814

58%

master: 50%

Pull #146

github

fix!: output race condition

BREAKING!

`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 we cache
the value we read.

Rename `ColorProfile` to `termColorProfile` and rely on
`EnvColorProfile` to detect the profile. Ideally, we rely on the
terminal Terminfo to detect the profile and fallback to the environment.

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

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

544 of 935 relevant lines covered (58.18%)

18.28 hits per line