change in v5: `const {inspect} = require('util');` can lead to web bundle bloat

This was added in v5 for deserializeError:

const {inspect} = require('util');

This adds about 8KB to web bundles, which can be significant, especially in projects not using deserializeError.

Possible solutions:

• Not worry about 8KB and tell developers to use v4 if bundle size is of concern.
• Separate serializeError into a separate file that can be imported without the deserializeError code.
• Separate deserializeError into a new module.
• Publish code with ES6-style import, add sideEffects: false to the package.json to enable tree shaking in the module.
• Other ideas?

[rohan-deshpande]

Thanks for this issue. I think this is a really handy lib and making it universal is a good idea! I support this approach:

Or we could find a simpler way to show the type. It’s not important. It’s just a bonus to make it easier to debug passed values that are not really deserializable as errors.

Basically the only reason for using inspect here is to be able to find out what happened right? Just for debugging purposes? I think if there is some light weight way to implement this that’d be ideal, maybe just use JSON.stringify if it’s an object?

[vladimiry]

PR placed: #27.