From 828999a147783ee4ff9225623cd002067f929b44 Mon Sep 17 00:00:00 2001
From: chouchouji <70570907+chouchouji@users.noreply.github.com>
Date: Fri, 21 Feb 2025 10:13:24 +0000
Subject: [PATCH 1/4] perf(no-export-in-script-setup): support autofix for
 vue/no-export-in-script-setup

---
 lib/rules/no-export-in-script-setup.js | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/lib/rules/no-export-in-script-setup.js b/lib/rules/no-export-in-script-setup.js
index 66286375a..a471726db 100644
--- a/lib/rules/no-export-in-script-setup.js
+++ b/lib/rules/no-export-in-script-setup.js
@@ -20,7 +20,7 @@ module.exports = {
       categories: ['vue3-essential', 'vue2-essential'],
       url: 'https://eslint.vuejs.org/rules/no-export-in-script-setup.html'
     },
-    fixable: null,
+    fixable: [],
     schema: [],
     messages: {
       forbidden: '`<script setup>` cannot contain ES module exports.'
@@ -46,7 +46,11 @@ module.exports = {
       }
       context.report({
         node,
-        messageId: 'forbidden'
+        loc: node.loc,
+        messageId: 'forbidden',
+        fix(fixer) {
+          return fixer.remove(node)
+        }
       })
     }
 

From d5a96bece097335aa1b8ebf63b5ca437b7964e03 Mon Sep 17 00:00:00 2001
From: chouchouji <70570907+chouchouji@users.noreply.github.com>
Date: Wed, 5 Mar 2025 09:03:03 +0000
Subject: [PATCH 2/4] test(no-export-in-script-setup): update test cases

---
 tests/lib/rules/no-export-in-script-setup.js | 24 ++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/tests/lib/rules/no-export-in-script-setup.js b/tests/lib/rules/no-export-in-script-setup.js
index 8703dc74a..d9dcaef13 100644
--- a/tests/lib/rules/no-export-in-script-setup.js
+++ b/tests/lib/rules/no-export-in-script-setup.js
@@ -92,6 +92,13 @@ ruleTester.run('no-export-in-script-setup', rule, {
       export * from 'foo'
       export default {}
       export class A {}
+      </script>
+      `,
+      output: `
+      <script setup>
+      
+      
+      
       </script>
       `,
       errors: [
@@ -119,6 +126,16 @@ ruleTester.run('no-export-in-script-setup', rule, {
       export * from 'foo'
       export default {}
       export class A {}
+      </script>
+      `,
+      output: `
+      <script>
+      let foo;
+      </script>
+      <script setup>
+      
+      
+      
       </script>
       `,
       errors: [
@@ -143,6 +160,13 @@ ruleTester.run('no-export-in-script-setup', rule, {
       export const Foo = {}
       export enum Bar {}
       export {}
+      </script>
+      `,
+      output: `
+      <script setup lang="ts">
+      
+      
+      
       </script>
       `,
       languageOptions: {

From b765d027d31d747230ea399d6cc3b5e00b054b66 Mon Sep 17 00:00:00 2001
From: chouchouji <70570907+chouchouji@users.noreply.github.com>
Date: Wed, 5 Mar 2025 09:09:18 +0000
Subject: [PATCH 3/4] fix(no-export-in-script-setup): update fixable

---
 lib/rules/no-export-in-script-setup.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/rules/no-export-in-script-setup.js b/lib/rules/no-export-in-script-setup.js
index a471726db..fdad63d30 100644
--- a/lib/rules/no-export-in-script-setup.js
+++ b/lib/rules/no-export-in-script-setup.js
@@ -20,7 +20,7 @@ module.exports = {
       categories: ['vue3-essential', 'vue2-essential'],
       url: 'https://eslint.vuejs.org/rules/no-export-in-script-setup.html'
     },
-    fixable: [],
+    fixable: 'code',
     schema: [],
     messages: {
       forbidden: '`<script setup>` cannot contain ES module exports.'

From b92dc0a5758f638bb6fec16019cc9c078d202dc6 Mon Sep 17 00:00:00 2001
From: chouchouji <1305974212@qq.com>
Date: Wed, 5 Mar 2025 21:01:36 +0800
Subject: [PATCH 4/4] refactor: optimize code

---
 lib/rules/no-export-in-script-setup.js       |  31 ++++--
 tests/lib/rules/no-export-in-script-setup.js | 102 +++++++++++++------
 2 files changed, 89 insertions(+), 44 deletions(-)

diff --git a/lib/rules/no-export-in-script-setup.js b/lib/rules/no-export-in-script-setup.js
index fdad63d30..98d41ae38 100644
--- a/lib/rules/no-export-in-script-setup.js
+++ b/lib/rules/no-export-in-script-setup.js
@@ -20,7 +20,7 @@ module.exports = {
       categories: ['vue3-essential', 'vue2-essential'],
       url: 'https://eslint.vuejs.org/rules/no-export-in-script-setup.html'
     },
-    fixable: 'code',
+    fixable: null,
     schema: [],
     messages: {
       forbidden: '`<script setup>` cannot contain ES module exports.'
@@ -28,8 +28,11 @@ module.exports = {
   },
   /** @param {RuleContext} context */
   create(context) {
-    /** @param {ExportAllDeclaration | ExportDefaultDeclaration | ExportNamedDeclaration} node */
-    function verify(node) {
+    /**
+     * @param {ExportAllDeclaration | ExportDefaultDeclaration | ExportNamedDeclaration} node
+     * @param {SourceLocation} loc
+     */
+    function verify(node, loc) {
       const tsNode =
         /** @type {TSESTreeExportAllDeclaration | TSESTreeExportDefaultDeclaration | TSESTreeExportNamedDeclaration} */ (
           node
@@ -46,18 +49,24 @@ module.exports = {
       }
       context.report({
         node,
-        loc: node.loc,
-        messageId: 'forbidden',
-        fix(fixer) {
-          return fixer.remove(node)
-        }
+        loc,
+        messageId: 'forbidden'
       })
     }
 
     return utils.defineScriptSetupVisitor(context, {
-      ExportAllDeclaration: verify,
-      ExportDefaultDeclaration: verify,
-      ExportNamedDeclaration: verify
+      ExportAllDeclaration: (node) => verify(node, node.loc),
+      ExportDefaultDeclaration: (node) => verify(node, node.loc),
+      ExportNamedDeclaration: (node) => {
+        // export let foo = 'foo', export class Foo {}, export function foo() {}
+        if (node.declaration) {
+          verify(node, context.getSourceCode().getFirstToken(node).loc)
+        }
+        // export { foo }, export { foo } from 'bar'
+        else {
+          verify(node, node.loc)
+        }
+      }
     })
   }
 }
diff --git a/tests/lib/rules/no-export-in-script-setup.js b/tests/lib/rules/no-export-in-script-setup.js
index d9dcaef13..bf24b8682 100644
--- a/tests/lib/rules/no-export-in-script-setup.js
+++ b/tests/lib/rules/no-export-in-script-setup.js
@@ -92,27 +92,62 @@ ruleTester.run('no-export-in-script-setup', rule, {
       export * from 'foo'
       export default {}
       export class A {}
-      </script>
-      `,
-      output: `
-      <script setup>
-      
-      
-      
+      export const test = '123'
+      export function foo() {}
+      const a = 1
+      export { a }
+      export { fao } from 'bar'
       </script>
       `,
       errors: [
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 3
+          line: 3,
+          endLine: 3,
+          column: 7,
+          endColumn: 26
+        },
+        {
+          message: '`<script setup>` cannot contain ES module exports.',
+          line: 4,
+          endLine: 4,
+          column: 7,
+          endColumn: 24
+        },
+        {
+          message: '`<script setup>` cannot contain ES module exports.',
+          line: 5,
+          endLine: 5,
+          column: 7,
+          endColumn: 13
+        },
+        {
+          message: '`<script setup>` cannot contain ES module exports.',
+          line: 6,
+          endLine: 6,
+          column: 7,
+          endColumn: 13
         },
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 4
+          line: 7,
+          endLine: 7,
+          column: 7,
+          endColumn: 13
         },
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 5
+          line: 9,
+          endLine: 9,
+          column: 7,
+          endColumn: 19
+        },
+        {
+          message: '`<script setup>` cannot contain ES module exports.',
+          line: 10,
+          endLine: 10,
+          column: 7,
+          endColumn: 32
         }
       ]
     },
@@ -126,30 +161,29 @@ ruleTester.run('no-export-in-script-setup', rule, {
       export * from 'foo'
       export default {}
       export class A {}
-      </script>
-      `,
-      output: `
-      <script>
-      let foo;
-      </script>
-      <script setup>
-      
-      
-      
       </script>
       `,
       errors: [
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 6
+          line: 6,
+          endLine: 6,
+          column: 7,
+          endColumn: 26
         },
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 7
+          line: 7,
+          endLine: 7,
+          column: 7,
+          endColumn: 24
         },
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 8
+          line: 8,
+          endLine: 8,
+          column: 7,
+          endColumn: 13
         }
       ]
     },
@@ -160,13 +194,6 @@ ruleTester.run('no-export-in-script-setup', rule, {
       export const Foo = {}
       export enum Bar {}
       export {}
-      </script>
-      `,
-      output: `
-      <script setup lang="ts">
-      
-      
-      
       </script>
       `,
       languageOptions: {
@@ -178,15 +205,24 @@ ruleTester.run('no-export-in-script-setup', rule, {
       errors: [
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 3
+          line: 3,
+          endLine: 3,
+          column: 7,
+          endColumn: 13
         },
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 4
+          line: 4,
+          endLine: 4,
+          column: 7,
+          endColumn: 13
         },
         {
           message: '`<script setup>` cannot contain ES module exports.',
-          line: 5
+          line: 5,
+          endLine: 5,
+          column: 7,
+          endColumn: 16
         }
       ]
     }