feat(node): Improve non-error messages#9026
Conversation
mydea
left a comment
There was a problem hiding this comment.
This is def. an improvement already. I guess it is less likely to have minified class names in node, but still possible? But IMHO we can merge this in and still think about making this better for node if possible.
AbhiPrasad
left a comment
There was a problem hiding this comment.
it was decided to to use the class name due to these being mangled by bundling. Does this apply to node too?
yes, since node can be minified (common for serverless) and minified js has performance benefits even for server runtimes.
As an aside, I wonder if we can detect if object has an name, message, and stack and turn it into an error. Thoughts?
I guess it would already be a possible improvement to just detect |
Ok, I'll add that and some tests. It worth noting that it already attempts to parse the |
Closes #6822
This PR changes the messages reported for captured non-Error exceptions.
For the browser PR, it was decided not to use the class name due to these being mangled by bundling. Does this apply to node too?