one bugfix with trailing commas

Author: snewcomer

src/rules/no-duplicates.js

@@ -94,23 +94,24 @@ function getInlineTypeFix(nodes, sourceCode) {
   return fixer => {
     const fixes = [];
 
-    // if (!semver.satisfies(typescriptPkg.version, '>= 4.5')) {
-    //   throw new Error('Your version of TypeScript does not support inline type imports.');
-    // }
-
     // push to first import
     let [firstImport, ...rest] = nodes;
-    const valueImport = nodes.find((n) => n.specifiers.every((spec) => spec.importKind === 'value')) || nodes.find((n) => n.specifiers.some((spec) => spec.type === 'ImportDefaultSpecifier'));
+    // const valueImport = nodes.find((n) => n.specifiers.every((spec) => spec.importKind === 'value')) || nodes.find((n) => n.specifiers.some((spec) => spec.type === 'ImportDefaultSpecifier'));
+    const valueImport = nodes.find((n) => n.specifiers.some((spec) => spec.type === 'ImportDefaultSpecifier'));
     if (valueImport) {
       firstImport = valueImport;
       rest = nodes.filter((n) => n !== firstImport);
     }
 
     const nodeTokens = sourceCode.getTokens(firstImport);
     // we are moving the rest of the Type or Inline Type imports here.
-    const nodeClosingBrace = nodeTokens.find(token => isPunctuator(token, '}'));
-    // const preferInline = context.options[0] && context.options[0]['prefer-inline'];
+    const nodeClosingBraceIndex = nodeTokens.findIndex(token => isPunctuator(token, '}'));
+    const nodeClosingBrace = nodeTokens[nodeClosingBraceIndex];
+    const tokenBeforeClosingBrace = nodeTokens[nodeClosingBraceIndex - 1];
     if (nodeClosingBrace) {
+      if (rest.length && isComma(tokenBeforeClosingBrace)) {
+        fixes.push(fixer.remove(tokenBeforeClosingBrace));
+      }
       rest.forEach((node) => {
         // these will be all Type imports, no Value specifiers
         // then add inline type specifiers to importKind === 'type' import

tests/src/rules/no-duplicates.js

@@ -614,6 +614,23 @@ context('TypeScript', function () {
             },
           ],
         }),
+        test({
+          code: "import {type x} from './foo'; import {y} from './foo'",
+          ...parserConfig,
+          output: `import {type x, y} from './foo'; `,
+          errors: [
+            {
+              line: 1,
+              column: 22,
+              message: "'./foo' imported multiple times.",
+            },
+            {
+              line: 1,
+              column: 47,
+              message: "'./foo' imported multiple times.",
+            },
+          ],
+        }),
       ].concat(!tsVersionSatisfies('>= 4.5') || !typescriptEslintParserSatisfies('>= 5.7.0') ? [] : [
         // without prefer-inline, will dedupe with type import kind
         test({
@@ -1018,20 +1035,34 @@ context('TypeScript', function () {
             },
           ],
         }),
-        // #2834 Detect duplicates across type and regular imports
         test({
-          code: "import {AValue} from './foo'; import type {AType} from './foo'",
+          code: "import { type C, } from './foo';import {AValue, BValue, } from './foo';",
           ...parserConfig,
           options: [{ 'prefer-inline': true }],
-          output: `import {AValue,type AType} from './foo'; `,
+          output: "import { type C , AValue, BValue} from './foo';",
           errors: [
             {
               line: 1,
-              column: 22,
+              column: 25,
               message: "'./foo' imported multiple times.",
             },
             {
               line: 1,
+              column: 64,
+              message: "'./foo' imported multiple times.",
+            }
+          ],
+        }),
+        // #2834 Detect duplicates across type and regular imports
+        test({
+          code: "import {AValue} from './foo'; import type {AType} from './foo'",
+          ...parserConfig,
+          options: [{ 'prefer-inline': true }],
+          output: `import {AValue,type AType} from './foo'; `,
+          errors: [
+            {
+              line: 1,
+              column: 22,
               column: 56,
               message: "'./foo' imported multiple times.",
             },