don't modify the TypeScript nodes, instead forward illegalDecorators to the java extractor

Author: erik-krogh

javascript/extractor/lib/typescript/src/ast_extractor.ts

@@ -202,12 +202,6 @@ export function augmentAst(ast: AugmentedSourceFile, code: string, project: Proj
             }
         }
 
-        // This PR: https://github.com/microsoft/TypeScript/pull/50343/ changed how illegal decorators are stored, this hack reverts it.
-        // Those decorator uses are syntax errors, but people use them anyway.
-        if ((node as any).illegalDecorators) {
-            (node as any).decorators = (node as any).illegalDecorators;
-        }
-
         if (typeChecker != null && insideConditionalTypes === 0) {
             if (isTypedNode(node)) {
                 let contextualType = isContextuallyTypedNode(node)

javascript/extractor/lib/typescript/src/main.ts

@@ -241,7 +241,7 @@ const astProperties: string[] = [
     "constructor",
     "declarationList",
     "declarations",
-    "decorators",
+    "illegalDecorators",
     "default",
     "delete",
     "dotDotDotToken",

javascript/extractor/src/com/semmle/ts/extractor/TypeScriptASTConverter.java

@@ -1151,7 +1151,7 @@ private Node convertEnumDeclaration(JsonObject node, SourceLocation loc) throws
             loc,
             hasModifier(node, "ConstKeyword"),
             hasModifier(node, "DeclareKeyword"),
-            convertChildrenNotNull(node, "decorators"),
+            convertChildrenNotNull(node, "illegalDecorators"), // as of https://github.com/microsoft/TypeScript/pull/50343/ the property is called `illegalDecorators` instead of `decorators`
             convertChild(node, "name"),
             convertChildren(node, "members"));
     attachSymbolInformation(enumDeclaration, node);