Javascript: Implement ExceptionCollectorListener and make it default behaviour.

[surister]

Summary of the changes / Why this is an improvement

Implements #30 #36 #37 #34 to the Javascript target
Solves #2

Checklist

• Link to issue this PR refers to (if applicable):
• CLA is signed

Also adds support for typescript types, solving #52.

[surister]

The PR might be to big to manually inspect/test, fyi it's 'already tested' (Since it's almost identical to the Python target and the test pass)

I'm interested on your opinion @alexdametto of the resulting API, you can quickly check it in https://github.com/crate/cratedb-sqlparse/pull/38/files#diff-111a35f00d7bdf2e15b6a32750f253243ae44df87bcf0465a2fbfd7178db2dc1

[amotl]

Thanks a stack! I've just added a few nit suggestions about punctuation, wording, and formatting, to accept at your disposal.

[surister]

With the latest commit types generation is added, it works fine on my end.

installing the package locally in cratedb-alt-admin:

There are still some types, that are not being picked up, I do not know why, for example Metadata.withProperties. I set the jsdoc to object but any is generated. We can improve these on later iterations.

[alexdametto]

I'm interested on your opinion @alexdametto of the resulting API, you can quickly check it in https://github.com/crate/cratedb-sqlparse/pull/38/files#diff-111a35f00d7bdf2e15b6a32750f253243ae44df87bcf0465a2fbfd7178db2dc1

Amazing job. 💯 I really like it, especially the raise_exception and the exception details, these things are definitely needed in https://github.com/crate/cloud/issues/1888.

Could be possible to add also the line number of where the error occurs? I have seen that we have it inside stmt.exception.errorMessage but it would be good to have it stored in a separate field.

[surister]

I'm interested on your opinion @alexdametto of the resulting API, you can quickly check it in https://github.com/crate/cratedb-sqlparse/pull/38/files#diff-111a35f00d7bdf2e15b6a32750f253243ae44df87bcf0465a2fbfd7178db2dc1

Amazing job. 💯 I really like it, especially the raise_exception and the exception details, these things are definitely needed in crate/cloud#1888.

Could be possible to add also the line number of where the error occurs? I have seen that we have it inside stmt.exception.errorMessage but it would be good to have it stored in a separate field.

There is console.log(stmt.exception.line, stmt.exception.column)

[matriv]

Sorry for the late review, looks good! thx a lot for the effort @surister !