Breaking: Support TypeScript 2.5 (fixes #368) (#369) (#370)

Author: JamesHenry

README.md

@@ -8,7 +8,7 @@ A parser that converts TypeScript into an [ESTree](https://github.com/estree/est
 
 We will always endeavor to support the latest stable version of TypeScript.
 
-The version of TypeScript currently supported by this parser is `~2.4.0`. This is reflected in the `devDependency` requirement within the package.json file, and it is what the tests will be run against. We have an open `peerDependency` requirement in order to allow for experimentation on newer/beta versions of TypeScript.
+The version of TypeScript currently supported by this parser is `~2.5.1`. This is reflected in the `devDependency` requirement within the package.json file, and it is what the tests will be run against. We have an open `peerDependency` requirement in order to allow for experimentation on newer/beta versions of TypeScript.
 
 If you use a non-supported version of TypeScript, the parser will log a warning to the console.
 

lib/ast-node-types.js

@@ -124,6 +124,7 @@ module.exports = {
     TSModuleDeclaration: "TSModuleDeclaration",
     TSNamespaceFunctionDeclaration: "TSNamespaceFunctionDeclaration",
     TSNonNullExpression: "TSNonNullExpression",
+    TSNeverKeyword: "TSNeverKeyword",
     TSNullKeyword: "TSNullKeyword",
     TSNumberKeyword: "TSNumberKeyword",
     TSObjectKeyword: "TSObjectKeyword",

lib/convert.js

@@ -528,7 +528,7 @@ module.exports = function convert(config) {
         case SyntaxKind.CatchClause:
             Object.assign(result, {
                 type: AST_NODE_TYPES.CatchClause,
-                param: convertChild(node.variableDeclaration.name),
+                param: node.variableDeclaration ? convertChild(node.variableDeclaration.name) : null,
                 body: convertChild(node.block)
             });
             break;

package.json

@@ -30,7 +30,7 @@
     "npm-license": "0.3.3",
     "shelljs": "0.7.8",
     "shelljs-nodecli": "0.1.1",
-    "typescript": "~2.4.0"
+    "typescript": "~2.5.1"
   },
   "keywords": [
     "ast",

tests/ast-alignment/spec.js

@@ -473,6 +473,7 @@ const fixturePatternsToTest = [
     "typescript/basics/class-with-generic-method-default.src.ts",
     "typescript/basics/class-with-generic-method.src.ts",
     "typescript/basics/type-guard.src.ts",
+    "typescript/basics/never-type-param.src.ts",
     {
         pattern: "typescript/basics/export-named-enum.src.ts",
         config: { babylonParserOptions: { sourceType: "module" } }

tests/fixtures/ecma-features/experimentalOptionalCatchBinding/optional-catch-binding-finally.src.js

@@ -0,0 +1 @@
+try {} catch {} finally {}

tests/fixtures/ecma-features/experimentalOptionalCatchBinding/optional-catch-binding.src.js

@@ -0,0 +1 @@
+try {} catch {}

tests/fixtures/typescript/basics/never-type-param.src.ts

@@ -0,0 +1,2 @@
+const x: X<never>;
+Observable.empty<never>();