fix(getMessages): Emit Error object in "error" event

[frobinsonj]

Didn't really know how to title this, apologies if I got it wrong.

Error event err param should be type Error - https://github.com/abalabahaha/eris/blob/dev/index.d.ts#L711

[DonovanDMC]

It would probably be better to construct the error object, then overwrite the stack
Or even just override the message of the caught error then emit it rather than both making a new error object & emitting the original stack

[DonovanDMC]

Suggested change

	this.emit("error", new Error(`Error creating message from channel messages\n${err.stack}\n${JSON.stringify(messages)}`));
	err.message = `Error creating message from channel messages\n${JSON.stringify(messages)}`;
	this.emit("error", err);

[DonovanDMC]

options.cause was added in node 16.9.0, we're currently on v10 (v14 at the latest), so we cannot use it

[frobinsonj]

options.cause was added in node 16.9.0, we're currently on v10 (v14 at the latest), so we cannot use it

Went with your original suggestion. Thanks.

[abalabahaha]

Setting .message doesn't affect .stack. TBH I don't think the full JSON belongs in the error message. warn (also not great) and debug events are used to log JSON for other parsing errors

[DonovanDMC]

We can slice the first line off the stack and remake it with the new message

or we can just ignore the original stack and construct a new error

[bsian03]

Any other alternatives to this? Current solution (although error might be hard to read) is the best one imo