Return log messages rather than use a WriterT

[ChrisCoffey]

There's no reason to rely on a WriterT in the core application. It simply hides that some functions are accumulating logs. Instead, introduce a MonadLogger constraint that more clearly indicates the intent.