refactor: autofix

Author: waynzh

lib/rules/no-import-compiler-macros.js

@@ -23,13 +23,6 @@ function isComma(node) {
   return node.type === 'Punctuator' && node.value === ','
 }
 
-/**
- * @param {Token} node
- */
-function isLeftCurlyBrace(node) {
-  return node.type === 'Punctuator' && node.value === '{'
-}
-
 module.exports = {
   meta: {
     type: 'problem',
@@ -73,22 +66,25 @@ module.exports = {
               name: specifier.imported.name
             },
             fix: (fixer) => {
-              const tokenAfter = sourceCode.getTokenAfter(specifier)
-              const tokenBefore = sourceCode.getTokenBefore(specifier)
-
-              const hasCommaAfter = isComma(tokenAfter)
-              const isFirstSpecifier = isLeftCurlyBrace(tokenBefore)
-
-              const codeStart = hasCommaAfter
-                ? tokenBefore.range[1]
-                : isFirstSpecifier
-                  ? specifier.range[0]
-                  : tokenBefore.range[0]
-              const codeEnd = hasCommaAfter
-                ? tokenAfter.range[1]
-                : specifier.range[1]
+              const isOnlySpecifier = node.specifiers.length === 1
+              const isLastSpecifier =
+                specifier === node.specifiers[node.specifiers.length - 1]
 
-              return fixer.removeRange([codeStart, codeEnd])
+              if (isOnlySpecifier) {
+                return fixer.remove(node)
+              } else if (isLastSpecifier) {
+                const commaToken = sourceCode.getTokenBefore(specifier, isComma)
+                return fixer.removeRange([
+                  commaToken ? commaToken.range[0] : specifier.range[0],
+                  specifier.range[1]
+                ])
+              } else {
+                const commaToken = sourceCode.getTokenAfter(specifier, isComma)
+                return fixer.removeRange([
+                  specifier.range[0],
+                  commaToken ? commaToken.range[1] : specifier.range[1]
+                ])
+              }
             }
           })
         }

tests/lib/rules/no-import-compiler-macros.js

@@ -40,16 +40,12 @@ tester.run('no-import-compiler-macros', rule, {
       filename: 'test.vue',
       code: `
       <script setup>
-      import { computed, defineProps } from 'vue'
-      import { defineEmits, ref, withDefaults } from '@vue/runtime-core'
-      import { defineExpose, watch } from '@vue/runtime-dom'
+      import { defineProps } from 'vue'
       </script>
       `,
       output: `
       <script setup>
-      import { computed } from 'vue'
-      import { ref } from '@vue/runtime-core'
-      import { watch } from '@vue/runtime-dom'
+      
       </script>
       `,
       errors: [
@@ -59,7 +55,34 @@ tester.run('no-import-compiler-macros', rule, {
             name: 'defineProps'
           },
           line: 3,
-          column: 26
+          column: 16
+        }
+      ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      import { ref, defineProps } from 'vue'
+      import { defineEmits, computed } from '@vue/runtime-core'
+      import { defineExpose, watch, withDefaults } from '@vue/runtime-dom'
+      </script>
+      `,
+      output: `
+      <script setup>
+      import { ref } from 'vue'
+      import {  computed } from '@vue/runtime-core'
+      import {  watch } from '@vue/runtime-dom'
+      </script>
+      `,
+      errors: [
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'defineProps'
+          },
+          line: 3,
+          column: 21
         },
         {
           messageId: 'noImportCompilerMacros',
@@ -72,46 +95,46 @@ tester.run('no-import-compiler-macros', rule, {
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'withDefaults'
+            name: 'defineExpose'
           },
-          line: 4,
-          column: 34
+          line: 5,
+          column: 16
         },
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'defineExpose'
+            name: 'withDefaults'
           },
           line: 5,
-          column: 16
+          column: 37
         }
       ]
     },
     {
       filename: 'test.vue',
       code: `
       <script setup>
-      import { defineProps, withDefaults, ref } from 'vue'
+      import { defineModel, defineOptions } from 'vue'
       </script>
       `,
       output: `
       <script setup>
-      import { withDefaults, ref } from 'vue'
+      import {  defineOptions } from 'vue'
       </script>
       `,
       errors: [
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'defineProps'
+            name: 'defineModel'
           },
           line: 3,
           column: 16
         },
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'withDefaults'
+            name: 'defineOptions'
           },
           line: 3,
           column: 29
@@ -122,12 +145,12 @@ tester.run('no-import-compiler-macros', rule, {
       filename: 'test.vue',
       code: `
       <script setup lang="ts">
-      import { ref as refFoo, defineProps as definePropsFoo, type computed } from '@vue/runtime-core'
+      import { ref as refFoo, defineSlots as defineSlotsFoo, type computed } from '@vue/runtime-core'
       </script>
       `,
       output: `
       <script setup lang="ts">
-      import { ref as refFoo, type computed } from '@vue/runtime-core'
+      import { ref as refFoo,  type computed } from '@vue/runtime-core'
       </script>
       `,
       languageOptions: {
@@ -139,7 +162,7 @@ tester.run('no-import-compiler-macros', rule, {
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'defineProps'
+            name: 'defineSlots'
           },
           line: 3,
           column: 31