@ericchiang let me answer here, as most of your comments are related to a single problem.

The interface I added includes the following function:

WithField(key string, value interface{}) Logger

The same function in Logrus looks like this:

WithField(key string, value interface{}) *Entry

The problem is the interface has a reference to itself in order to remove dependency on Logrus. Because of this:

• the interface must be importable by external packages in order to implement it, thus it cannot be an internal package
• Logrus itself cannot satisfy the interface, so we need a wrapper

That WithField function is used in a single place:

It's inside the library part, that's why I added the function to the interface. But I didn't really have any heuristics about the implementation, I just created and implemented the interface.

So if dex code is not supposed to be used from external packages, the logger can be moved to an internal package (although then dex code should probably be moved to internal as well).

In case WithField method can somehow be replaced, then the interface wouldn't have to reference itself, it could be moved to an internal package and logrus would satisfy the interface without a wrapper.

I vote we drop our uses of WithField to simplify the Logger interface.

Well, I'm generally in favor of structured logging, but given there is a single place where it is used right now, I'm inclined to remove it as well, at least for now.

@ericchiang I removed the wrapper type and the structural logging.

I still think if any dex code is supposed to be called externally, then the logger interface should be exported too, but if you want, I can move it under internal/.