Give all the things a logger instance

[lox]

With the rise of our new --spawn param, I'd really like to have a prefix in the log output for the specific agent that console output is coming from. This is really, really hard to do with the global logger.

This moves to an instance that gets passed into anything that needs a logger. It's a lot of code and opens up a lot of spots for nil pointer errors as we rarely use constructors and tend to use direct struct instantiations.

I'd love some feels on this approach. It's been hard work and I don't super like the result, but it's probably more correct?

[sj26]

Oof, I can already see the nil pointer errors in the test results. But I agree it does seem more correct. Perhaps we need to feel this pain first to move to a better pattern? I do think this is a step in the right direction, nice work.

[lox]

Yeah, I considered a lot of other options that would be more magical, but ultimately I think simply passing in the logger instance to places where it's needed is the most "go-like".