From 87b05ba6d72a89ba3efd0d78446c986944b51683 Mon Sep 17 00:00:00 2001
From: waynzh <waynzh19@gmail.com>
Date: Thu, 13 Feb 2025 15:07:51 +0800
Subject: [PATCH 1/9] feat: add no-import-compiler-macros rule

---
 lib/rules/no-import-compiler-macros.js       |  94 ++++++++++++++
 tests/lib/rules/no-import-compiler-macros.js | 128 +++++++++++++++++++
 2 files changed, 222 insertions(+)
 create mode 100644 lib/rules/no-import-compiler-macros.js
 create mode 100644 tests/lib/rules/no-import-compiler-macros.js

diff --git a/lib/rules/no-import-compiler-macros.js b/lib/rules/no-import-compiler-macros.js
new file mode 100644
index 000000000..17c125093
--- /dev/null
+++ b/lib/rules/no-import-compiler-macros.js
@@ -0,0 +1,94 @@
+/**
+ * @author Wayne Zhang
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+const COMPILER_MACROS = new Set([
+  'defineProps',
+  'defineEmits',
+  'defineExpose',
+  'withDefaults'
+])
+
+const VUE_MODULES = new Set(['@vue/runtime-core', '@vue/runtime-dom', 'vue'])
+
+/**
+ * @param {Token} node
+ */
+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',
+    docs: {
+      description: 'disallow importing vue compiler macros',
+      categories: undefined,
+      url: 'https://eslint.vuejs.org/rules/no-import-compiler-macros.html'
+    },
+    fixable: 'code',
+    schema: [],
+    messages: {
+      noImportCompilerMacros:
+        "'{{name}}' is a compiler macro and no longer needs to be imported from '{{source}}'."
+    }
+  },
+  /**
+   * @param {RuleContext} context
+   * @returns {RuleListener}
+   */
+  create(context) {
+    const sourceCode = context.getSourceCode()
+
+    return {
+      ImportDeclaration(node) {
+        if (node.specifiers.length === 0) return
+
+        if (VUE_MODULES.has(node.source.value)) {
+          for (const specifier of node.specifiers) {
+            if (
+              specifier.type === 'ImportSpecifier' &&
+              COMPILER_MACROS.has(specifier.imported.name)
+            ) {
+              context.report({
+                node: specifier,
+                messageId: 'noImportCompilerMacros',
+                data: {
+                  name: specifier.imported.name,
+                  source: node.source.value
+                },
+                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]
+
+                  return fixer.removeRange([codeStart, codeEnd])
+                }
+              })
+            }
+          }
+        }
+      }
+    }
+  }
+}
diff --git a/tests/lib/rules/no-import-compiler-macros.js b/tests/lib/rules/no-import-compiler-macros.js
new file mode 100644
index 000000000..018052db7
--- /dev/null
+++ b/tests/lib/rules/no-import-compiler-macros.js
@@ -0,0 +1,128 @@
+/**
+ * @author Wayne Zhang
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+const RuleTester = require('../../eslint-compat').RuleTester
+const rule = require('../../../lib/rules/no-import-compiler-macros')
+
+const tester = new RuleTester({
+  languageOptions: {
+    parser: require('vue-eslint-parser'),
+    ecmaVersion: 2020,
+    sourceType: 'module'
+  }
+})
+
+tester.run('no-import-compiler-macros', rule, {
+  valid: [
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      import { ref } from 'vue'
+      import { someFunction } from '@vue/runtime-core'
+      </script>
+      `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script>
+      import { defineProps } from 'some-other-package'
+      </script>
+      `
+    }
+  ],
+  invalid: [
+    {
+      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'
+      </script>
+      `,
+      output: `
+      <script setup>
+      import { computed } from 'vue'
+      import { ref } from '@vue/runtime-core'
+      import { watch } from '@vue/runtime-dom'
+      </script>
+      `,
+      errors: [
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'defineProps',
+            source: 'vue'
+          },
+          line: 3,
+          column: 26
+        },
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'defineEmits',
+            source: '@vue/runtime-core'
+          },
+          line: 4,
+          column: 16
+        },
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'withDefaults',
+            source: '@vue/runtime-core'
+          },
+          line: 4,
+          column: 34
+        },
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'defineExpose',
+            source: '@vue/runtime-dom'
+          },
+          line: 5,
+          column: 16
+        }
+      ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      import { defineProps, withDefaults, ref } from 'vue'
+      </script>
+      `,
+      output: `
+      <script setup>
+      import { withDefaults, ref } from 'vue'
+      </script>
+      `,
+      errors: [
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'defineProps',
+            source: 'vue'
+          },
+          line: 3,
+          column: 16
+        },
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'withDefaults',
+            source: 'vue'
+          },
+          line: 3,
+          column: 29
+        }
+      ]
+    }
+  ]
+})

From d692fdd89f8f3cd64913434b7b5c2de1a773c1be Mon Sep 17 00:00:00 2001
From: waynzh <waynzh19@gmail.com>
Date: Thu, 13 Feb 2025 15:08:22 +0800
Subject: [PATCH 2/9] feat: add docs

---
 docs/rules/index.md                     |  2 ++
 docs/rules/no-import-compiler-macros.md | 48 +++++++++++++++++++++++++
 lib/index.js                            |  1 +
 3 files changed, 51 insertions(+)
 create mode 100644 docs/rules/no-import-compiler-macros.md

diff --git a/docs/rules/index.md b/docs/rules/index.md
index f170f8031..eb816a64e 100644
--- a/docs/rules/index.md
+++ b/docs/rules/index.md
@@ -233,6 +233,7 @@ For example:
 | [vue/no-deprecated-model-definition] | disallow deprecated `model` definition (in Vue.js 3.0.0+) | :bulb: | :warning: |
 | [vue/no-duplicate-attr-inheritance] | enforce `inheritAttrs` to be set to `false` when using `v-bind="$attrs"` |  | :hammer: |
 | [vue/no-empty-component-block] | disallow the `<template>` `<script>` `<style>` block to be empty | :wrench: | :hammer: |
+| [vue/no-import-compiler-macros] | disallow importing vue compiler macros | :wrench: | :warning: |
 | [vue/no-multiple-objects-in-class] | disallow passing multiple objects in an array to class |  | :hammer: |
 | [vue/no-potential-component-option-typo] | disallow a potential typo in your component property | :bulb: | :hammer: |
 | [vue/no-ref-object-reactivity-loss] | disallow usages of ref objects that can lead to loss of reactivity |  | :warning: |
@@ -467,6 +468,7 @@ The following rules extend the rules provided by ESLint itself and apply them to
 [vue/no-expose-after-await]: ./no-expose-after-await.md
 [vue/no-extra-parens]: ./no-extra-parens.md
 [vue/no-implicit-coercion]: ./no-implicit-coercion.md
+[vue/no-import-compiler-macros]: ./no-import-compiler-macros.md
 [vue/no-invalid-model-keys]: ./no-invalid-model-keys.md
 [vue/no-irregular-whitespace]: ./no-irregular-whitespace.md
 [vue/no-lifecycle-after-await]: ./no-lifecycle-after-await.md
diff --git a/docs/rules/no-import-compiler-macros.md b/docs/rules/no-import-compiler-macros.md
new file mode 100644
index 000000000..902903d3e
--- /dev/null
+++ b/docs/rules/no-import-compiler-macros.md
@@ -0,0 +1,48 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/no-import-compiler-macros
+description: disallow importing vue compiler macros
+---
+
+# vue/no-import-compiler-macros
+
+> disallow importing vue compiler macros
+
+- :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> _**This rule has not been released yet.**_ </badge>
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fix-problems) can automatically fix some of the problems reported by this rule.
+
+## :book: Rule Details
+
+This rule disallow importing vue compiler macros.
+
+<eslint-code-block fix :rules="{'vue/no-import-compiler-macros': ['error']}">
+
+```vue
+<script setup>
+/* ✗ BAD */
+import { defineProps, withDefaults } from 'vue'
+</script>
+```
+
+</eslint-code-block>
+
+<eslint-code-block fix :rules="{'vue/no-import-compiler-macros': ['error']}">
+
+```vue
+<script setup>
+/* ✓ GOOD */
+import { ref } from 'vue'
+</script>
+```
+
+</eslint-code-block>
+
+## :wrench: Options
+
+Nothing.
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/no-import-compiler-macros.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/no-import-compiler-macros.js)
diff --git a/lib/index.js b/lib/index.js
index ecb8ce5e1..7b407efe1 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -127,6 +127,7 @@ const plugin = {
     'no-expose-after-await': require('./rules/no-expose-after-await'),
     'no-extra-parens': require('./rules/no-extra-parens'),
     'no-implicit-coercion': require('./rules/no-implicit-coercion'),
+    'no-import-compiler-macros': require('./rules/no-import-compiler-macros'),
     'no-invalid-model-keys': require('./rules/no-invalid-model-keys'),
     'no-irregular-whitespace': require('./rules/no-irregular-whitespace'),
     'no-lifecycle-after-await': require('./rules/no-lifecycle-after-await'),

From 8c4c5c6aaa3f9d68c0b7be5f73a92cb9e7f79ae9 Mon Sep 17 00:00:00 2001
From: Wayne Zhang <waynzh19@gmail.com>
Date: Thu, 13 Feb 2025 19:51:34 +0800
Subject: [PATCH 3/9] Update lib/rules/no-import-compiler-macros.js

Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
---
 lib/rules/no-import-compiler-macros.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/rules/no-import-compiler-macros.js b/lib/rules/no-import-compiler-macros.js
index 17c125093..4267b2c61 100644
--- a/lib/rules/no-import-compiler-macros.js
+++ b/lib/rules/no-import-compiler-macros.js
@@ -31,7 +31,7 @@ module.exports = {
   meta: {
     type: 'problem',
     docs: {
-      description: 'disallow importing vue compiler macros',
+      description: 'disallow importing Vue compiler macros',
       categories: undefined,
       url: 'https://eslint.vuejs.org/rules/no-import-compiler-macros.html'
     },

From ce38578e9ab9be16fc3e925a504a50a011fb38c4 Mon Sep 17 00:00:00 2001
From: waynzh <waynzh19@gmail.com>
Date: Thu, 20 Feb 2025 16:02:52 +0800
Subject: [PATCH 4/9] Update docs

---
 docs/rules/no-import-compiler-macros.md | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/docs/rules/no-import-compiler-macros.md b/docs/rules/no-import-compiler-macros.md
index 902903d3e..fddeea21c 100644
--- a/docs/rules/no-import-compiler-macros.md
+++ b/docs/rules/no-import-compiler-macros.md
@@ -42,6 +42,12 @@ import { ref } from 'vue'
 
 Nothing.
 
+## :books: Further Reading
+
+- [defineProps() & defineEmits()]
+
+[defineProps() & defineEmits()]: https://vuejs.org/api/sfc-script-setup.html#defineprops-defineemits
+
 ## :mag: Implementation
 
 - [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/no-import-compiler-macros.js)

From eb0d6d2ba2de798c35a6bfb4172bd2d6aefd0cc5 Mon Sep 17 00:00:00 2001
From: waynzh <waynzh19@gmail.com>
Date: Thu, 20 Feb 2025 16:03:34 +0800
Subject: [PATCH 5/9] refactor

---
 lib/rules/no-import-compiler-macros.js       | 68 +++++++++++---------
 tests/lib/rules/no-import-compiler-macros.js | 46 +++++++++----
 2 files changed, 70 insertions(+), 44 deletions(-)

diff --git a/lib/rules/no-import-compiler-macros.js b/lib/rules/no-import-compiler-macros.js
index 4267b2c61..06fad31ef 100644
--- a/lib/rules/no-import-compiler-macros.js
+++ b/lib/rules/no-import-compiler-macros.js
@@ -8,7 +8,10 @@ const COMPILER_MACROS = new Set([
   'defineProps',
   'defineEmits',
   'defineExpose',
-  'withDefaults'
+  'withDefaults',
+  'defineModel',
+  'defineOptions',
+  'defineSlots'
 ])
 
 const VUE_MODULES = new Set(['@vue/runtime-core', '@vue/runtime-dom', 'vue'])
@@ -39,7 +42,7 @@ module.exports = {
     schema: [],
     messages: {
       noImportCompilerMacros:
-        "'{{name}}' is a compiler macro and no longer needs to be imported from '{{source}}'."
+        "'{{name}}' is a compiler macro and doesn't need to be imported."
     }
   },
   /**
@@ -53,40 +56,41 @@ module.exports = {
       ImportDeclaration(node) {
         if (node.specifiers.length === 0) return
 
-        if (VUE_MODULES.has(node.source.value)) {
-          for (const specifier of node.specifiers) {
-            if (
-              specifier.type === 'ImportSpecifier' &&
-              COMPILER_MACROS.has(specifier.imported.name)
-            ) {
-              context.report({
-                node: specifier,
-                messageId: 'noImportCompilerMacros',
-                data: {
-                  name: specifier.imported.name,
-                  source: node.source.value
-                },
-                fix: (fixer) => {
-                  const tokenAfter = sourceCode.getTokenAfter(specifier)
-                  const tokenBefore = sourceCode.getTokenBefore(specifier)
+        if (!VUE_MODULES.has(node.source.value)) return
 
-                  const hasCommaAfter = isComma(tokenAfter)
-                  const isFirstSpecifier = isLeftCurlyBrace(tokenBefore)
+        for (const specifier of node.specifiers) {
+          if (
+            specifier.type !== 'ImportSpecifier' ||
+            !COMPILER_MACROS.has(specifier.imported.name)
+          ) {
+            continue
+          }
+
+          context.report({
+            node: specifier,
+            messageId: 'noImportCompilerMacros',
+            data: {
+              name: specifier.imported.name
+            },
+            fix: (fixer) => {
+              const tokenAfter = sourceCode.getTokenAfter(specifier)
+              const tokenBefore = sourceCode.getTokenBefore(specifier)
 
-                  const codeStart = hasCommaAfter
-                    ? tokenBefore.range[1]
-                    : isFirstSpecifier
-                      ? specifier.range[0]
-                      : tokenBefore.range[0]
-                  const codeEnd = hasCommaAfter
-                    ? tokenAfter.range[1]
-                    : specifier.range[1]
+              const hasCommaAfter = isComma(tokenAfter)
+              const isFirstSpecifier = isLeftCurlyBrace(tokenBefore)
 
-                  return fixer.removeRange([codeStart, codeEnd])
-                }
-              })
+              const codeStart = hasCommaAfter
+                ? tokenBefore.range[1]
+                : isFirstSpecifier
+                  ? specifier.range[0]
+                  : tokenBefore.range[0]
+              const codeEnd = hasCommaAfter
+                ? tokenAfter.range[1]
+                : specifier.range[1]
+
+              return fixer.removeRange([codeStart, codeEnd])
             }
-          }
+          })
         }
       }
     }
diff --git a/tests/lib/rules/no-import-compiler-macros.js b/tests/lib/rules/no-import-compiler-macros.js
index 018052db7..79b6d0e0a 100644
--- a/tests/lib/rules/no-import-compiler-macros.js
+++ b/tests/lib/rules/no-import-compiler-macros.js
@@ -56,8 +56,7 @@ tester.run('no-import-compiler-macros', rule, {
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'defineProps',
-            source: 'vue'
+            name: 'defineProps'
           },
           line: 3,
           column: 26
@@ -65,8 +64,7 @@ tester.run('no-import-compiler-macros', rule, {
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'defineEmits',
-            source: '@vue/runtime-core'
+            name: 'defineEmits'
           },
           line: 4,
           column: 16
@@ -74,8 +72,7 @@ tester.run('no-import-compiler-macros', rule, {
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'withDefaults',
-            source: '@vue/runtime-core'
+            name: 'withDefaults'
           },
           line: 4,
           column: 34
@@ -83,8 +80,7 @@ tester.run('no-import-compiler-macros', rule, {
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'defineExpose',
-            source: '@vue/runtime-dom'
+            name: 'defineExpose'
           },
           line: 5,
           column: 16
@@ -107,8 +103,7 @@ tester.run('no-import-compiler-macros', rule, {
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'defineProps',
-            source: 'vue'
+            name: 'defineProps'
           },
           line: 3,
           column: 16
@@ -116,13 +111,40 @@ tester.run('no-import-compiler-macros', rule, {
         {
           messageId: 'noImportCompilerMacros',
           data: {
-            name: 'withDefaults',
-            source: 'vue'
+            name: 'withDefaults'
           },
           line: 3,
           column: 29
         }
       ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup lang="ts">
+      import { ref as refFoo, defineProps as definePropsFoo, type computed } from '@vue/runtime-core'
+      </script>
+      `,
+      output: `
+      <script setup lang="ts">
+      import { ref as refFoo, type computed } from '@vue/runtime-core'
+      </script>
+      `,
+      languageOptions: {
+        parserOptions: {
+          parser: require.resolve('@typescript-eslint/parser')
+        }
+      },
+      errors: [
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'defineProps'
+          },
+          line: 3,
+          column: 31
+        }
+      ]
     }
   ]
 })

From 781692c4236ff9b61427e25b590f60be67ed50de Mon Sep 17 00:00:00 2001
From: waynzh <waynzh19@gmail.com>
Date: Thu, 20 Feb 2025 19:28:25 +0800
Subject: [PATCH 6/9] refactor: autofix

---
 lib/rules/no-import-compiler-macros.js       | 40 ++++++-------
 tests/lib/rules/no-import-compiler-macros.js | 61 ++++++++++++++------
 2 files changed, 60 insertions(+), 41 deletions(-)

diff --git a/lib/rules/no-import-compiler-macros.js b/lib/rules/no-import-compiler-macros.js
index 06fad31ef..f3478483d 100644
--- a/lib/rules/no-import-compiler-macros.js
+++ b/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]
+                ])
+              }
             }
           })
         }
diff --git a/tests/lib/rules/no-import-compiler-macros.js b/tests/lib/rules/no-import-compiler-macros.js
index 79b6d0e0a..1e31f216e 100644
--- a/tests/lib/rules/no-import-compiler-macros.js
+++ b/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,18 +95,18 @@ 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
         }
       ]
     },
@@ -91,19 +114,19 @@ tester.run('no-import-compiler-macros', rule, {
       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
@@ -111,7 +134,7 @@ tester.run('no-import-compiler-macros', rule, {
         {
           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

From f539191b87da4891b3e412db6bd085a203055e20 Mon Sep 17 00:00:00 2001
From: waynzh <waynzh19@gmail.com>
Date: Thu, 20 Feb 2025 19:37:05 +0800
Subject: [PATCH 7/9] feat: add test

---
 docs/rules/index.md                          |  2 +-
 docs/rules/no-import-compiler-macros.md      |  4 +--
 tests/lib/rules/no-import-compiler-macros.js | 30 +++++++++++++++++++-
 3 files changed, 32 insertions(+), 4 deletions(-)

diff --git a/docs/rules/index.md b/docs/rules/index.md
index eb816a64e..4fea47058 100644
--- a/docs/rules/index.md
+++ b/docs/rules/index.md
@@ -233,7 +233,7 @@ For example:
 | [vue/no-deprecated-model-definition] | disallow deprecated `model` definition (in Vue.js 3.0.0+) | :bulb: | :warning: |
 | [vue/no-duplicate-attr-inheritance] | enforce `inheritAttrs` to be set to `false` when using `v-bind="$attrs"` |  | :hammer: |
 | [vue/no-empty-component-block] | disallow the `<template>` `<script>` `<style>` block to be empty | :wrench: | :hammer: |
-| [vue/no-import-compiler-macros] | disallow importing vue compiler macros | :wrench: | :warning: |
+| [vue/no-import-compiler-macros] | disallow importing Vue compiler macros | :wrench: | :warning: |
 | [vue/no-multiple-objects-in-class] | disallow passing multiple objects in an array to class |  | :hammer: |
 | [vue/no-potential-component-option-typo] | disallow a potential typo in your component property | :bulb: | :hammer: |
 | [vue/no-ref-object-reactivity-loss] | disallow usages of ref objects that can lead to loss of reactivity |  | :warning: |
diff --git a/docs/rules/no-import-compiler-macros.md b/docs/rules/no-import-compiler-macros.md
index fddeea21c..ed968daae 100644
--- a/docs/rules/no-import-compiler-macros.md
+++ b/docs/rules/no-import-compiler-macros.md
@@ -2,12 +2,12 @@
 pageClass: rule-details
 sidebarDepth: 0
 title: vue/no-import-compiler-macros
-description: disallow importing vue compiler macros
+description: disallow importing Vue compiler macros
 ---
 
 # vue/no-import-compiler-macros
 
-> disallow importing vue compiler macros
+> disallow importing Vue compiler macros
 
 - :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> _**This rule has not been released yet.**_ </badge>
 - :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fix-problems) can automatically fix some of the problems reported by this rule.
diff --git a/tests/lib/rules/no-import-compiler-macros.js b/tests/lib/rules/no-import-compiler-macros.js
index 1e31f216e..d74a268a0 100644
--- a/tests/lib/rules/no-import-compiler-macros.js
+++ b/tests/lib/rules/no-import-compiler-macros.js
@@ -21,7 +21,7 @@ tester.run('no-import-compiler-macros', rule, {
       filename: 'test.vue',
       code: `
       <script setup>
-      import { ref } from 'vue'
+      import { ref, computed } from 'vue'
       import { someFunction } from '@vue/runtime-core'
       </script>
       `
@@ -59,6 +59,34 @@ tester.run('no-import-compiler-macros', rule, {
         }
       ]
     },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      import {
+        ref,
+        defineProps
+      } from 'vue'
+      </script>
+      `,
+      output: `
+      <script setup>
+      import {
+        ref
+      } from 'vue'
+      </script>
+      `,
+      errors: [
+        {
+          messageId: 'noImportCompilerMacros',
+          data: {
+            name: 'defineProps'
+          },
+          line: 5,
+          column: 9
+        }
+      ]
+    },
     {
       filename: 'test.vue',
       code: `

From 8b746bb14169a08931bb6436585f564b1247f5eb Mon Sep 17 00:00:00 2001
From: waynzh <waynzh19@gmail.com>
Date: Thu, 20 Feb 2025 19:38:03 +0800
Subject: [PATCH 8/9] refactor

---
 lib/rules/no-import-compiler-macros.js | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/rules/no-import-compiler-macros.js b/lib/rules/no-import-compiler-macros.js
index f3478483d..d7e916a65 100644
--- a/lib/rules/no-import-compiler-macros.js
+++ b/lib/rules/no-import-compiler-macros.js
@@ -47,9 +47,8 @@ module.exports = {
 
     return {
       ImportDeclaration(node) {
-        if (node.specifiers.length === 0) return
-
-        if (!VUE_MODULES.has(node.source.value)) return
+        if (node.specifiers.length === 0 || !VUE_MODULES.has(node.source.value))
+          return
 
         for (const specifier of node.specifiers) {
           if (

From 3c7654d038bd4199d5327f6f9b1fc05c088b0186 Mon Sep 17 00:00:00 2001
From: waynzh <waynzh19@gmail.com>
Date: Thu, 20 Feb 2025 19:45:52 +0800
Subject: [PATCH 9/9] rename

---
 lib/rules/no-import-compiler-macros.js | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/lib/rules/no-import-compiler-macros.js b/lib/rules/no-import-compiler-macros.js
index d7e916a65..693a9f33b 100644
--- a/lib/rules/no-import-compiler-macros.js
+++ b/lib/rules/no-import-compiler-macros.js
@@ -72,16 +72,24 @@ module.exports = {
               if (isOnlySpecifier) {
                 return fixer.remove(node)
               } else if (isLastSpecifier) {
-                const commaToken = sourceCode.getTokenBefore(specifier, isComma)
+                const precedingComma = sourceCode.getTokenBefore(
+                  specifier,
+                  isComma
+                )
                 return fixer.removeRange([
-                  commaToken ? commaToken.range[0] : specifier.range[0],
+                  precedingComma ? precedingComma.range[0] : specifier.range[0],
                   specifier.range[1]
                 ])
               } else {
-                const commaToken = sourceCode.getTokenAfter(specifier, isComma)
+                const subsequentComma = sourceCode.getTokenAfter(
+                  specifier,
+                  isComma
+                )
                 return fixer.removeRange([
                   specifier.range[0],
-                  commaToken ? commaToken.range[1] : specifier.range[1]
+                  subsequentComma
+                    ? subsequentComma.range[1]
+                    : specifier.range[1]
                 ])
               }
             }