fix(eslint-plugin): [no-extra-parens] handle generic ts array type. (#…

…5550) Co-authored-by: Josh Goldberg <me@joshuakgoldberg.com>
typescript-eslint · Sep 4, 2022 · 0d6a190 · 0d6a190
1 parent 6634e86
commit 0d6a190
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/packages/eslint-plugin/src/rules/no-extra-parens.ts b/packages/eslint-plugin/src/rules/no-extra-parens.ts
@@ -79,7 +79,9 @@ export default util.createRule<Options, MessageIds>({
       if (
         node.arguments.length === 1 &&
         node.typeParameters?.params.some(
-          param => param.type === AST_NODE_TYPES.TSImportType,
+          param =>
+            param.type === AST_NODE_TYPES.TSImportType ||
+            param.type === AST_NODE_TYPES.TSArrayType,
         )
       ) {
         return rule({

diff --git a/packages/eslint-plugin/tests/rules/no-extra-parens.test.ts b/packages/eslint-plugin/tests/rules/no-extra-parens.test.ts
@@ -227,6 +227,17 @@ switch (foo) { case 1: case (<2>2): break; default: break; }
         },
       ],
     }),
+    ...batchedSingleLineTests({
+      code: `
+declare const f: <T>(x: T) => any
+f<(number | string)[]>(['a', 1])
+      `,
+      parserOptions: {
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+    }),
   ],
 
   invalid: [