False positives for type-only exports in vue/no-export-in-script-setup rule

Author: ota-meshi

lib/rules/no-export-in-script-setup.js

@@ -6,6 +6,12 @@
 
 const utils = require('../utils')
 
+/**
+ * @typedef {import('@typescript-eslint/types').TSESTree.ExportAllDeclaration} TSESTreeExportAllDeclaration
+ * @typedef {import('@typescript-eslint/types').TSESTree.ExportDefaultDeclaration} TSESTreeExportDefaultDeclaration
+ * @typedef {import('@typescript-eslint/types').TSESTree.ExportNamedDeclaration} TSESTreeExportNamedDeclaration
+ */
+
 module.exports = {
   meta: {
     type: 'problem',
@@ -23,17 +29,32 @@ module.exports = {
   /** @param {RuleContext} context */
   create(context) {
     /** @param {ExportAllDeclaration | ExportDefaultDeclaration | ExportNamedDeclaration} node */
-    function report(node) {
+    function verify(node) {
+      const tsNode =
+        /** @type {TSESTreeExportAllDeclaration | TSESTreeExportDefaultDeclaration | TSESTreeExportNamedDeclaration} */ (
+          node
+        )
+      if (tsNode.exportKind === 'type') {
+        return
+      }
+      if (tsNode.type === 'ExportNamedDeclaration') {
+        if (
+          tsNode.specifiers.length > 0 &&
+          tsNode.specifiers.every((spec) => spec.exportKind === 'type')
+        ) {
+          return
+        }
+      }
       context.report({
         node,
         messageId: 'forbidden'
       })
     }
 
     return utils.defineScriptSetupVisitor(context, {
-      ExportAllDeclaration: report,
-      ExportDefaultDeclaration: report,
-      ExportNamedDeclaration: report
+      ExportAllDeclaration: verify,
+      ExportDefaultDeclaration: verify,
+      ExportNamedDeclaration: verify
     })
   }
 }

tests/lib/rules/no-export-in-script-setup.js

@@ -48,6 +48,20 @@ ruleTester.run('no-export-in-script-setup', rule, {
       let foo;
       </script>
       `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup lang="ts">
+      export { type Foo } from "foo"
+      export type Bar = {}
+      export interface Bar {}
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      parserOptions: {
+        parser: require.resolve('@typescript-eslint/parser')
+      }
     }
   ],
 
@@ -102,6 +116,34 @@ ruleTester.run('no-export-in-script-setup', rule, {
           line: 8
         }
       ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup lang="ts">
+      export const Foo = {}
+      export enum Bar {}
+      export {}
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      parserOptions: {
+        parser: require.resolve('@typescript-eslint/parser')
+      },
+      errors: [
+        {
+          message: '`<script setup>` cannot contain ES module exports.',
+          line: 3
+        },
+        {
+          message: '`<script setup>` cannot contain ES module exports.',
+          line: 4
+        },
+        {
+          message: '`<script setup>` cannot contain ES module exports.',
+          line: 5
+        }
+      ]
     }
   ]
 })