From e267d6571fcb833676f9f09a6d6ab5bed37c0028 Mon Sep 17 00:00:00 2001
From: yeonjuan <yeonjuan93@naver.com>
Date: Tue, 3 Aug 2021 23:17:44 +0900
Subject: [PATCH] fix(eslint-plugin): [prefer-reduce-type-parameter] handle
 already existing type params

---
 .../src/rules/prefer-reduce-type-parameter.ts | 41 +++++++++++--------
 .../prefer-reduce-type-parameter.test.ts      | 17 ++++++++
 2 files changed, 42 insertions(+), 16 deletions(-)

diff --git a/packages/eslint-plugin/src/rules/prefer-reduce-type-parameter.ts b/packages/eslint-plugin/src/rules/prefer-reduce-type-parameter.ts
index 3b1af79ac6e..8656c8d2376 100644
--- a/packages/eslint-plugin/src/rules/prefer-reduce-type-parameter.ts
+++ b/packages/eslint-plugin/src/rules/prefer-reduce-type-parameter.ts
@@ -77,22 +77,31 @@ export default util.createRule({
           context.report({
             messageId: 'preferTypeParameter',
             node: secondArg,
-            fix: fixer => [
-              fixer.removeRange([
-                secondArg.range[0],
-                secondArg.expression.range[0],
-              ]),
-              fixer.removeRange([
-                secondArg.expression.range[1],
-                secondArg.range[1],
-              ]),
-              fixer.insertTextAfter(
-                callee,
-                `<${context
-                  .getSourceCode()
-                  .getText(secondArg.typeAnnotation)}>`,
-              ),
-            ],
+            fix: fixer => {
+              const fixes = [
+                fixer.removeRange([
+                  secondArg.range[0],
+                  secondArg.expression.range[0],
+                ]),
+                fixer.removeRange([
+                  secondArg.expression.range[1],
+                  secondArg.range[1],
+                ]),
+              ];
+
+              if (!callee.parent.typeParameters) {
+                fixes.push(
+                  fixer.insertTextAfter(
+                    callee,
+                    `<${context
+                      .getSourceCode()
+                      .getText(secondArg.typeAnnotation)}>`,
+                  ),
+                );
+              }
+
+              return fixes;
+            },
           });
 
           return;
diff --git a/packages/eslint-plugin/tests/rules/prefer-reduce-type-parameter.test.ts b/packages/eslint-plugin/tests/rules/prefer-reduce-type-parameter.test.ts
index a683a4ab0dc..06613b260f7 100644
--- a/packages/eslint-plugin/tests/rules/prefer-reduce-type-parameter.test.ts
+++ b/packages/eslint-plugin/tests/rules/prefer-reduce-type-parameter.test.ts
@@ -39,6 +39,23 @@ ruleTester.run('prefer-reduce-type-parameter', rule, {
     '[1, 2, 3]?.reduce<number[]>((a, s) => a.concat(s * 2), []);',
   ],
   invalid: [
+    {
+      code: `
+declare const arr: string[];
+arr.reduce<string>(acc => acc, arr.shift() as string);
+      `,
+      output: `
+declare const arr: string[];
+arr.reduce<string>(acc => acc, arr.shift());
+      `,
+      errors: [
+        {
+          messageId: 'preferTypeParameter',
+          column: 32,
+          line: 3,
+        },
+      ],
+    },
     {
       code: '[1, 2, 3].reduce((a, s) => a.concat(s * 2), [] as number[]);',
       output: '[1, 2, 3].reduce<number[]>((a, s) => a.concat(s * 2), []);',