feat: warn on deprecated AST property accesses

typescript-eslint · Feb 24, 2023 · 0c7af52 · 0c7af52
1 parent cbb8fed
commit 0c7af52
Show file tree

Hide file tree

Showing 80 changed files with 948 additions and 4,061 deletions.
diff --git a/docs/architecture/TypeScript-ESTree.mdx b/docs/architecture/TypeScript-ESTree.mdx
@@ -36,6 +36,11 @@ interface ParseOptions {
    */
   comment?: boolean;
 
+  /**
+   * Whether deprecated AST properties should skip calling console.warn on accesses.
+   */
+  suppressDeprecatedPropertyWarnings?: boolean;
+
   /**
    * An array of modules to turn explicit debugging on for.
    * - 'typescript-eslint' is the same as setting the env var `DEBUG=typescript-eslint:*`

diff --git a/...rc/declaration/ClassDeclaration/fixtures/extends-type-param/snapshots/1-TSESTree-AST.shot b/...rc/declaration/ClassDeclaration/fixtures/extends-type-param/snapshots/1-TSESTree-AST.shot
@@ -56,26 +56,6 @@ Program {
           end: { column: 29, line: 1 },
         },
       },
-      superTypeParameters: TSTypeParameterInstantiation {
-        type: "TSTypeParameterInstantiation",
-        params: Array [
-          TSStringKeyword {
-            type: "TSStringKeyword",
-
-            range: [22, 28],
-            loc: {
-              start: { column: 22, line: 1 },
-              end: { column: 28, line: 1 },
-            },
-          },
-        ],
-
-        range: [21, 29],
-        loc: {
-          start: { column: 21, line: 1 },
-          end: { column: 29, line: 1 },
-        },
-      },
 
       range: [0, 32],
       loc: {

diff --git a/...claration/ClassDeclaration/fixtures/extends-type-param/snapshots/5-AST-Alignment-AST.shot b/...claration/ClassDeclaration/fixtures/extends-type-param/snapshots/5-AST-Alignment-AST.shot
@@ -41,26 +41,7 @@ exports[`AST Fixtures declaration ClassDeclaration extends-type-param AST Alignm
           },
         },
 -       superTypeArguments: TSTypeParameterInstantiation {
--         type: 'TSTypeParameterInstantiation',
--         params: Array [
--           TSStringKeyword {
--             type: 'TSStringKeyword',
--
--             range: [22, 28],
--             loc: {
--               start: { column: 22, line: 1 },
--               end: { column: 28, line: 1 },
--             },
--           },
--         ],
--
--         range: [21, 29],
--         loc: {
--           start: { column: 21, line: 1 },
--           end: { column: 29, line: 1 },
--         },
--       },
-        superTypeParameters: TSTypeParameterInstantiation {
++       superTypeParameters: TSTypeParameterInstantiation {
           type: 'TSTypeParameterInstantiation',
           params: Array [
             TSStringKeyword {

diff --git a/...lassDeclaration/fixtures/type-parameters-extends-type-param/snapshots/1-TSESTree-AST.shot b/...lassDeclaration/fixtures/type-parameters-extends-type-param/snapshots/1-TSESTree-AST.shot
@@ -66,36 +66,6 @@ Program {
           end: { column: 27, line: 1 },
         },
       },
-      superTypeParameters: TSTypeParameterInstantiation {
-        type: "TSTypeParameterInstantiation",
-        params: Array [
-          TSTypeReference {
-            type: "TSTypeReference",
-            typeName: Identifier {
-              type: "Identifier",
-              name: "T",
-
-              range: [25, 26],
-              loc: {
-                start: { column: 25, line: 1 },
-                end: { column: 26, line: 1 },
-              },
-            },
-
-            range: [25, 26],
-            loc: {
-              start: { column: 25, line: 1 },
-              end: { column: 26, line: 1 },
-            },
-          },
-        ],
-
-        range: [24, 27],
-        loc: {
-          start: { column: 24, line: 1 },
-          end: { column: 27, line: 1 },
-        },
-      },
       typeParameters: TSTypeParameterDeclaration {
         type: "TSTypeParameterDeclaration",
         params: Array [

diff --git a/...eclaration/fixtures/type-parameters-extends-type-param/snapshots/5-AST-Alignment-AST.shot b/...eclaration/fixtures/type-parameters-extends-type-param/snapshots/5-AST-Alignment-AST.shot
@@ -41,36 +41,7 @@ exports[`AST Fixtures declaration ClassDeclaration type-parameters-extends-type-
           },
         },
 -       superTypeArguments: TSTypeParameterInstantiation {
--         type: 'TSTypeParameterInstantiation',
--         params: Array [
--           TSTypeReference {
--             type: 'TSTypeReference',
--             typeName: Identifier {
--               type: 'Identifier',
--               name: 'T',
--
--               range: [25, 26],
--               loc: {
--                 start: { column: 25, line: 1 },
--                 end: { column: 26, line: 1 },
--               },
--             },
--
--             range: [25, 26],
--             loc: {
--               start: { column: 25, line: 1 },
--               end: { column: 26, line: 1 },
--             },
--           },
--         ],
--
--         range: [24, 27],
--         loc: {
--           start: { column: 24, line: 1 },
--           end: { column: 27, line: 1 },
--         },
--       },
-        superTypeParameters: TSTypeParameterInstantiation {
++       superTypeParameters: TSTypeParameterInstantiation {
           type: 'TSTypeParameterInstantiation',
           params: Array [
             TSTypeReference {

diff --git a/...ixtures/basics/fixtures/abstract-class-with-abstract-method/snapshots/1-TSESTree-AST.shot b/...ixtures/basics/fixtures/abstract-class-with-abstract-method/snapshots/1-TSESTree-AST.shot
@@ -71,26 +71,6 @@ Program {
                         end: { column: 34, line: 4 },
                       },
                     },
-                    typeParameters: TSTypeParameterInstantiation {
-                      type: "TSTypeParameterInstantiation",
-                      params: Array [
-                        TSStringKeyword {
-                          type: "TSStringKeyword",
-
-                          range: [147, 153],
-                          loc: {
-                            start: { column: 35, line: 4 },
-                            end: { column: 41, line: 4 },
-                          },
-                        },
-                      ],
-
-                      range: [146, 154],
-                      loc: {
-                        start: { column: 34, line: 4 },
-                        end: { column: 42, line: 4 },
-                      },
-                    },
 
                     range: [139, 154],
                     loc: {

diff --git a/...es/basics/fixtures/abstract-class-with-abstract-method/snapshots/5-AST-Alignment-AST.shot b/...es/basics/fixtures/abstract-class-with-abstract-method/snapshots/5-AST-Alignment-AST.shot
@@ -51,36 +51,17 @@ exports[`AST Fixtures legacy-fixtures basics abstract-class-with-abstract-method
                     typeAnnotation: TSTypeReference {
                       type: 'TSTypeReference',
 -                     typeArguments: TSTypeParameterInstantiation {
--                       type: 'TSTypeParameterInstantiation',
--                       params: Array [
--                         TSStringKeyword {
--                           type: 'TSStringKeyword',
--
--                           range: [147, 153],
--                           loc: {
--                             start: { column: 35, line: 4 },
--                             end: { column: 41, line: 4 },
--                           },
--                         },
--                       ],
--
--                       range: [146, 154],
--                       loc: {
--                         start: { column: 34, line: 4 },
--                         end: { column: 42, line: 4 },
--                       },
--                     },
-                      typeName: Identifier {
-                        type: 'Identifier',
-                        name: 'Promise',
-
-                        range: [139, 146],
-                        loc: {
-                          start: { column: 27, line: 4 },
-                          end: { column: 34, line: 4 },
-                        },
-                      },
-                      typeParameters: TSTypeParameterInstantiation {
++                     typeName: Identifier {
++                       type: 'Identifier',
++                       name: 'Promise',
++
++                       range: [139, 146],
++                       loc: {
++                         start: { column: 27, line: 4 },
++                         end: { column: 34, line: 4 },
++                       },
++                     },
++                     typeParameters: TSTypeParameterInstantiation {
                         type: 'TSTypeParameterInstantiation',
                         params: Array [
                           TSStringKeyword {
@@ -98,6 +79,16 @@ exports[`AST Fixtures legacy-fixtures basics abstract-class-with-abstract-method
                         loc: {
                           start: { column: 34, line: 4 },
                           end: { column: 42, line: 4 },
+-                       },
+-                     },
+-                     typeName: Identifier {
+-                       type: 'Identifier',
+-                       name: 'Promise',
+-
+-                       range: [139, 146],
+-                       loc: {
+-                         start: { column: 27, line: 4 },
+-                         end: { column: 34, line: 4 },
                         },
                       },
 

diff --git a/...ixtures/basics/fixtures/abstract-class-with-optional-method/snapshots/1-TSESTree-AST.shot b/...ixtures/basics/fixtures/abstract-class-with-optional-method/snapshots/1-TSESTree-AST.shot
@@ -72,26 +72,6 @@ Program {
                         end: { column: 26, line: 4 },
                       },
                     },
-                    typeParameters: TSTypeParameterInstantiation {
-                      type: "TSTypeParameterInstantiation",
-                      params: Array [
-                        TSStringKeyword {
-                          type: "TSStringKeyword",
-
-                          range: [139, 145],
-                          loc: {
-                            start: { column: 27, line: 4 },
-                            end: { column: 33, line: 4 },
-                          },
-                        },
-                      ],
-
-                      range: [138, 146],
-                      loc: {
-                        start: { column: 26, line: 4 },
-                        end: { column: 34, line: 4 },
-                      },
-                    },
 
                     range: [131, 146],
                     loc: {

diff --git a/...es/basics/fixtures/abstract-class-with-optional-method/snapshots/5-AST-Alignment-AST.shot b/...es/basics/fixtures/abstract-class-with-optional-method/snapshots/5-AST-Alignment-AST.shot
@@ -49,36 +49,17 @@ exports[`AST Fixtures legacy-fixtures basics abstract-class-with-optional-method
                     typeAnnotation: TSTypeReference {
                       type: 'TSTypeReference',
 -                     typeArguments: TSTypeParameterInstantiation {
--                       type: 'TSTypeParameterInstantiation',
--                       params: Array [
--                         TSStringKeyword {
--                           type: 'TSStringKeyword',
--
--                           range: [139, 145],
--                           loc: {
--                             start: { column: 27, line: 4 },
--                             end: { column: 33, line: 4 },
--                           },
--                         },
--                       ],
--
--                       range: [138, 146],
--                       loc: {
--                         start: { column: 26, line: 4 },
--                         end: { column: 34, line: 4 },
--                       },
--                     },
-                      typeName: Identifier {
-                        type: 'Identifier',
-                        name: 'Promise',
-
-                        range: [131, 138],
-                        loc: {
-                          start: { column: 19, line: 4 },
-                          end: { column: 26, line: 4 },
-                        },
-                      },
-                      typeParameters: TSTypeParameterInstantiation {
++                     typeName: Identifier {
++                       type: 'Identifier',
++                       name: 'Promise',
++
++                       range: [131, 138],
++                       loc: {
++                         start: { column: 19, line: 4 },
++                         end: { column: 26, line: 4 },
++                       },
++                     },
++                     typeParameters: TSTypeParameterInstantiation {
                         type: 'TSTypeParameterInstantiation',
                         params: Array [
                           TSStringKeyword {
@@ -96,6 +77,16 @@ exports[`AST Fixtures legacy-fixtures basics abstract-class-with-optional-method
                         loc: {
                           start: { column: 26, line: 4 },
                           end: { column: 34, line: 4 },
+-                       },
+-                     },
+-                     typeName: Identifier {
+-                       type: 'Identifier',
+-                       name: 'Promise',
+-
+-                       range: [131, 138],
+-                       loc: {
+-                         start: { column: 19, line: 4 },
+-                         end: { column: 26, line: 4 },
                         },
                       },
 

diff --git a/...ixtures/basics/fixtures/class-with-extends-generic-multiple/snapshots/1-TSESTree-AST.shot b/...ixtures/basics/fixtures/class-with-extends-generic-multiple/snapshots/1-TSESTree-AST.shot
@@ -85,55 +85,6 @@ Program {
           end: { column: 40, line: 3 },
         },
       },
-      superTypeParameters: TSTypeParameterInstantiation {
-        type: "TSTypeParameterInstantiation",
-        params: Array [
-          TSTypeReference {
-            type: "TSTypeReference",
-            typeName: Identifier {
-              type: "Identifier",
-              name: "C",
-
-              range: [108, 109],
-              loc: {
-                start: { column: 35, line: 3 },
-                end: { column: 36, line: 3 },
-              },
-            },
-
-            range: [108, 109],
-            loc: {
-              start: { column: 35, line: 3 },
-              end: { column: 36, line: 3 },
-            },
-          },
-          TSTypeReference {
-            type: "TSTypeReference",
-            typeName: Identifier {
-              type: "Identifier",
-              name: "D",
-
-              range: [111, 112],
-              loc: {
-                start: { column: 38, line: 3 },
-                end: { column: 39, line: 3 },
-              },
-            },
-
-            range: [111, 112],
-            loc: {
-              start: { column: 38, line: 3 },
-              end: { column: 39, line: 3 },
-            },
-          },
-        ],
-
-        range: [107, 113],
-        loc: {
-          start: { column: 34, line: 3 },
-          end: { column: 40, line: 3 },
-        },
-      },
       typeParameters: TSTypeParameterDeclaration {
         type: "TSTypeParameterDeclaration",
         params: Array [