Add a new function that accepts an interface instead of byte slice to diff against about jsondiff HOT 1 OPEN

There could be some concerns with that kind of interface. Yes, internally lib compares things using interface{} representation of the JSON AST as produced by the Go encoding/json lib. But I can't say lib accepts any interface{}, because that's not true. For example code inside expects you to use UseNumber() on go's json decoder, and never checks for float64 or int64 type in the interface{} typed values. Those types are replaced by json.Number.

Effectively we'll have to say something like: Here's the CompareValues(a, b interface{}, opts *Options) function, but a and b values must be a result of: d := json.NewDecoder(input); d.UseNumber(); d.Decode(). Which is a strange kind of interface to have. Not sure it's worth the effort.

Another argument against it, is that this lib is just used for testing, in tests you can encode the JSON to []byte and feed it to the lib without thinking about performance. I doubt it will make your tests drastically slower. The lib in general is not that optimized for performance anyway.

I'm not convinced about the need for such interface, but I'll leave the issue open, maybe will reconsider it one day.

And as a side note. This lib sadly is very far from the top of my maintenance effort list. I still use it in tests and it works for me, don't feel like changing anything. (This is more of a message for people who wonder why there are no commits and/or work on other issues)

from jsondiff.