fix(define-macros-order): skip enum declarations and statements with declare modifier (#2918)

KazariEX · FloEdelmann · web-flow · commit 305100da6b0e · 2025-08-28T10:46:48.000+08:00
Co-authored-by: Flo Edelmann &lt;git@flo-edelmann.de&gt;
diff --git a/.changeset/five-roses-peel.md b/.changeset/five-roses-peel.md
@@ -0,0 +1,5 @@
+---
+"eslint-plugin-vue": patch
+---
+
+Changed `vue/define-macros-order` to ignore enum declarations and `declare` statements
diff --git a/lib/rules/define-macros-order.js b/lib/rules/define-macros-order.js
@@ -33,6 +33,13 @@ function inScriptSetup(scriptSetup, node) {
   )
 }
 
+/**
+ * @param {ASTNode} node
+ */
+function isDeclareStatement(node) {
+  return 'declare' in node && node.declare === true
+}
+
 /**
  * @param {ASTNode} node
  */
@@ -53,6 +60,7 @@ function isUseStrictStatement(node) {
 function getTargetStatementPosition(scriptSetup, program) {
   const skipStatements = new Set([
     'ImportDeclaration',
+    'TSEnumDeclaration',
     'TSModuleDeclaration',
     'TSInterfaceDeclaration',
     'TSTypeAliasDeclaration',
@@ -65,6 +73,7 @@ function getTargetStatementPosition(scriptSetup, program) {
     if (
       inScriptSetup(scriptSetup, item) &&
       !skipStatements.has(item.type) &&
+      !isDeclareStatement(item) &&
       !isUseStrictStatement(item)
     ) {
       return index
diff --git a/tests/lib/rules/define-macros-order.js b/tests/lib/rules/define-macros-order.js
@@ -128,6 +128,8 @@ tester.run('define-macros-order', rule, {
           import { bar } from 'foo'
           declare global {}
           declare namespace Namespace {}
+          declare const foo: string
+          declare function bar(): void
           export interface Props {
             msg?: string
             labels?: string[]

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"eslint-plugin-vue": patch
 +---
++
 +Changed `vue/define-macros-order` to ignore enum declarations and `declare` statements