fix support for illegal decorators in the TypeScript extractor

erik-krogh · erik-krogh · commit 6aca9b9f4949 · 2022-08-29T12:14:58.000+02:00
diff --git a/javascript/extractor/lib/typescript/src/ast_extractor.ts b/javascript/extractor/lib/typescript/src/ast_extractor.ts
@@ -202,6 +202,12 @@ 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)

Original file line number	Diff line number	Diff line change
`@@ -202,6 +202,12 @@ export function augmentAst(ast: AugmentedSourceFile, code: string, project: Proj`
`202`	`202`	`}`
`203`	`203`	`}`
`204`	`204`
	`205`	`+ // This PR: https://github.com/microsoft/TypeScript/pull/50343/ changed how illegal decorators are stored, this hack reverts it.`
	`206`	`+ // Those decorator uses are syntax errors, but people use them anyway.`
	`207`	`+ if ((node as any).illegalDecorators) {`
	`208`	`+ (node as any).decorators = (node as any).illegalDecorators;`
	`209`	`+ }`
	`210`	`+`
`205`	`211`	`if (typeChecker != null && insideConditionalTypes === 0) {`
`206`	`212`	`if (isTypedNode(node)) {`
`207`	`213`	`let contextualType = isContextuallyTypedNode(node)`