Fix false positives for import binding in vue/valid-define-emits and vue/valid-define-props rules

Author: ota-meshi

lib/rules/valid-define-emits.js

@@ -77,11 +77,18 @@ module.exports = {
               ) {
                 if (
                   variable.defs.length &&
-                  variable.defs.every(
-                    (def) =>
-                      utils.inRange(scriptSetup.range, def.name) &&
-                      !utils.inRange(defineEmits.range, def.name)
-                  )
+                  variable.defs.every((def) => {
+                    if (def.type === 'ImportBinding') {
+                      return false
+                    }
+                    if (
+                      !utils.inRange(scriptSetup.range, def.name) ||
+                      utils.inRange(defineEmits.range, def.name)
+                    ) {
+                      return false
+                    }
+                    return true
+                  })
                 ) {
                   if (utils.withinTypeNode(node)) {
                     continue

lib/rules/valid-define-props.js

@@ -78,11 +78,18 @@ module.exports = {
               ) {
                 if (
                   variable.defs.length &&
-                  variable.defs.every(
-                    (def) =>
-                      utils.inRange(scriptSetup.range, def.name) &&
-                      !utils.inRange(defineProps.range, def.name)
-                  )
+                  variable.defs.every((def) => {
+                    if (def.type === 'ImportBinding') {
+                      return false
+                    }
+                    if (
+                      !utils.inRange(scriptSetup.range, def.name) ||
+                      utils.inRange(defineProps.range, def.name)
+                    ) {
+                      return false
+                    }
+                    return true
+                  })
                 ) {
                   if (utils.withinTypeNode(node)) {
                     continue

tests/lib/rules/valid-define-emits.js

@@ -117,6 +117,16 @@ tester.run('valid-define-emits', rule, {
       });
       </script>
       `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      import { propsDef, emitsDef } from './defs';
+
+      defineProps(propsDef);
+      defineEmits(emitsDef);
+      </script>`
     }
   ],
   invalid: [

tests/lib/rules/valid-define-props.js

@@ -120,6 +120,16 @@ tester.run('valid-define-props', rule, {
       });
       </script>
       `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      import { propsDef, emitsDef } from './defs';
+
+      defineProps(propsDef);
+      defineEmits(emitsDef);
+      </script>`
     }
   ],
   invalid: [