Add support for TSTupleType and TSAbstractClassDeclaration in ast-node-types.js

[vivekgalatage]

What version of TypeScript are you using?
Typescript 3.0.1

What code were you trying to parse?

class Class {
  method(parameters: [number, string?, boolean?]) {
  }
}

Please take a look at the AST explorer snippet for the same
https://astexplorer.net/#/gist/e5dd1011355354fea561d02cb10e0b05/9ccb97f94ca1871a0f8917bed5b4d95402a07e13

What did you expect to happen?
The TSTupleType keyword should be in the ast-node-types.js file.

What actually happened?
Support for TSTupleType as ast-node-type is missing.

[vivekgalatage]

This also applies to TSAbstractClassDeclaration

abstract class Hello {
  static world(): string {
    return ""
  }
}

Please take a look at the AST here
https://astexplorer.net/#/gist/3b79e871a564de599f26479d58bb5cd8/33d9327299b131a23e78e5e738bef37e2bec5584

So can TSAbstractClassDeclaration be added to the ast-node-types.js as well?

[JamesHenry]

Thanks for reporting @vivekgalatage, I'm curious in what situation this surfaced as an issue for you?

[vivekgalatage]

@JamesHenry I am using the typescript as a replacement for defining the WebIDL[1] files. The type checking and language support is awesome with the typescript.

So the various spec.ts files are written in typescript which are parsed using typescript-eslint-parser which in turn relies upon ast-node-types.js.

During parsing, the tokens are passed through various switch case statements and this is where I found out these couple of missing constants in the above file.

[1] https://heycam.github.io/webidl/

[JamesHenry]

🎉 This issue has been resolved in version 8.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀