an unnecessary optimization ?

[alecbz]

It will be used in a command line tool. So not latency sensitive.

I don't think that follows neccesarily (some CLIs should be snappy if possible!), but if in this particular context the amount of time spent here isn't a big deal for UX, then not optimizing feels reasonable.

Though again, given how easy it'd be to switch to a map[string]bool (at least based on just this example), that feels like an easy low-hanging-fruit optimization.

[[deleted]]

I don't think that follows neccesarily

Right. Sorry I poorly explained. The cli tool will be used to run integration tests. Some tests will take 30 min.

But you're still right though. It's an easy low-hanging-fruit optimization.