[JS API] Throw useful exceptions on parse errors

[kripken]

wasm-opt will just fatally error on invalid module inputs, but binaryen.js
is a library and users want to get something they can handle, and see
the actual error. This PR throws a C++ exception instead, and converts
it on the JS side.

Fixes #8256

[spotandjake]

This looks great to me.

[spotandjake]

Just a nit but with #8122 I started trying to add some documentation with jsdoc how would we feel about continuing that pattern when possible?

Suggested change

	Module['readBinary'] = function(data) {
	/**
	* Creates a module from binary data.
	*
	* @param {Uint8Array} data - A Uint8Array containing a valid WebAssembly binary.
	*
	* @return {Module} The constructed wasm module.
	*
	* @throws {Error} If the input is invalid or parsing fails.
	*/
	Module['readBinary'] = function(data) {

[aheejin]

Is this to autogenerate some doc? The comment here seems kind of obvious.

[kripken]

I assume @spotandjake you are thinking of autogenerating a docs page from the jsdocs comments?

That might be useful, though I'm not sure how much.

[kripken]

Let's leave the topic of adding jsdocs for separate discussion. I'll land this for now.

[spotandjake]

Same as above:

Suggested change

	Module['parseText'] = function(text) {
	/**
	* Creates a module from Binaryen's s-expression text format (not official stack-style text format).
	*
	* @param {string} text - A string containing a WebAssembly text module (Binaryen-style WAT).
	*
	* @return {Module} The constructed wasm module.
	*
	* @throws {Error} If the input is invalid or parsing fails.
	*/
	Module['parseText'] = function(text) {

[aheejin]

Emscripten supports printing exception messages when run with -sEXCEPTION_STACK_TRACES. Can we use that without half-reimplementing this? Or do we not want the stack traces but only want the message?

[kripken]

Or do we not want the stack traces but only want the message?

Yes, this is not focused on the stack trace in the exception. This is about being able to catch an exception in JS, and to get the message. Before this PR, a parse error just shut down the entire application, with no useful message for the code calling the binaryen.js library.