deserializeError serializes/destroys properties unnecessarily

[fregante]

This module is set up in a weird way. destroyCircular is called both when serializing and deserializing and it's doing two separate things:

• removing circular references
• "serializing" all kinds of non-primitives, feature which I don’t think belongs to this module because it's a slippery slope until it's "complete"

The second part part is particularly problematic because during deserialization, this module is potentially dropping functions and all other "supported" types like buffer. This is unexpected:

deserializeError({
	name: 'Error'
	message: '💩',
	data: new Buffer([1,2,3])
});
// [Error: 💩] with `data` property of value "[object Buffer]"

I'd expect the string "[object Buffer]" when serializing an error, not during deserialization.

[fregante]

The new serialize option in destroyCircular sets this up for a simple fix, even though it would probably benefit from a rewrite.

https://github.com/sindresorhus/serialize-error/pull/73/files#diff-e727e4bdf3657fd1d798edcd6b099d6e092f8573cba266154583a746bba0f346R62