Breaking: Remove TypeAnnotation wrapper from constraint (#325)

Author: JamesHenry

lib/ast-node-types.js

@@ -121,6 +121,7 @@ module.exports = {
     TSNonNullExpression: "TSNonNullExpression",
     TSNullKeyword: "TSNullKeyword",
     TSNumberKeyword: "TSNumberKeyword",
+    TSObjectKeyword: "TSObjectKeyword",
     TSParameterProperty: "TSParameterProperty",
     TSPropertySignature: "TSPropertySignature",
     TSQualifiedName: "TSQualifiedName",
@@ -130,6 +131,7 @@ module.exports = {
     TSTypePredicate: "TSTypePredicate",
     TSTypeReference: "TSTypeReference",
     TSUnionType: "TSUnionType",
+    TSUndefinedKeyword: "TSUndefinedKeyword",
     TSVoidKeyword: "TSVoidKeyword",
 
     TypeAnnotation: "TypeAnnotation",

lib/convert.js

@@ -146,6 +146,10 @@ module.exports = function convert(config) {
             params: typeParameters.map(typeParameter => {
                 const name = nodeUtils.unescapeIdentifier(typeParameter.name.text);
 
+                const constraint = typeParameter.constraint
+                    ? convert({ node: typeParameter.constraint, parent: typeParameter, ast, additionalOptions })
+                    : null;
+
                 const defaultParameter = typeParameter.default
                     ? convert({ node: typeParameter.default, parent: typeParameter, ast, additionalOptions })
                     : typeParameter.default;
@@ -158,9 +162,7 @@ module.exports = function convert(config) {
                     ],
                     loc: nodeUtils.getLoc(typeParameter, ast),
                     name,
-                    constraint: (typeParameter.constraint)
-                        ? convertTypeAnnotation(typeParameter.constraint)
-                        : null,
+                    constraint,
                     default: defaultParameter
                 };
             })

tests/fixtures/typescript/babylon-convergence/type-parameters.src.ts

@@ -0,0 +1 @@
+function f<T extends object = object>() {}