fix(typescript-estree): allow writing to deprecated node properties

packages/typescript-estree/src/convert.ts

@@ -3319,23 +3319,30 @@ export class Converter {
     aliasKey: AliasKey,
     valueKey: ValueKey,
   ): Properties & Record<AliasKey, Properties[ValueKey]> {
-    let errored = false;
+    let warned = false;
 
     Object.defineProperty(node, aliasKey, {
+      configurable: true,
       get: this.options.suppressDeprecatedPropertyWarnings
         ? (): Properties[typeof valueKey] => node[valueKey]
         : (): Properties[typeof valueKey] => {
-            if (!this.options.suppressDeprecatedPropertyWarnings) {
-              if (!errored) {
-                // eslint-disable-next-line no-console
-                console.warn(
-                  `The '${aliasKey}' property is deprecated on ${node.type} nodes. Use '${valueKey}' instead. See https://typescript-eslint.io/linting/troubleshooting#the-key-property-is-deprecated-on-type-nodes-use-key-instead-warnings.`,
-                );
-                errored = true;
-              }
+            if (!warned) {
+              // eslint-disable-next-line no-console
+              console.warn(
+                `The '${aliasKey}' property is deprecated on ${node.type} nodes. Use '${valueKey}' instead. See https://typescript-eslint.io/linting/troubleshooting#the-key-property-is-deprecated-on-type-nodes-use-key-instead-warnings.`,
+              );
+              warned = true;
             }
+
             return node[valueKey];
           },
+      set(value): void {
+        Object.defineProperty(node, aliasKey, {
+          enumerable: true,
+          writable: true,
+          value,
+        });
+      },
     });
 
     return node as Properties & Record<AliasKey, Properties[ValueKey]>;

packages/typescript-estree/tests/lib/convert.test.ts

@@ -268,7 +268,7 @@ describe('convert', () => {
 
   describe('suppressDeprecatedPropertyWarnings', () => {
     const getEsCallExpression = (
-      converterOptions: ConverterOptions,
+      converterOptions?: ConverterOptions,
     ): TSESTree.CallExpression => {
       const ast = convertCode(`callee<T>();`);
       const tsCallExpression = (ast.statements[0] as ts.ExpressionStatement)
@@ -324,5 +324,22 @@ describe('convert', () => {
 
       expect(warn).not.toHaveBeenCalled();
     });
+
+    it('does not allow enumeration of deprecated properties', () => {
+      const esCallExpression = getEsCallExpression();
+
+      expect(Object.keys(esCallExpression)).not.toContain('typeParameters');
+    });
+
+    it('allows writing to the deprecated property as a new enumerable value', () => {
+      const esCallExpression = getEsCallExpression();
+
+      // eslint-disable-next-line deprecation/deprecation
+      esCallExpression.typeParameters = undefined;
+
+      // eslint-disable-next-line deprecation/deprecation
+      expect(esCallExpression.typeParameters).toBeUndefined();
+      expect(Object.keys(esCallExpression)).toContain('typeParameters');
+    });
   });
 });