From 4a9c8d265d0e9b393ab94eb699ba756ffe68b6ee Mon Sep 17 00:00:00 2001
From: ItMaga <m-chemurziev@ya.ru>
Date: Sat, 20 May 2023 01:39:00 +0300
Subject: [PATCH 1/2] Extend `vue/no-dupe-keys` to support `<script setup>`

---
 lib/rules/no-dupe-keys.js       |  56 +++++++++++------
 tests/lib/rules/no-dupe-keys.js | 105 ++++++++++++++++++++++++++++++++
 2 files changed, 144 insertions(+), 17 deletions(-)

diff --git a/lib/rules/no-dupe-keys.js b/lib/rules/no-dupe-keys.js
index c812c5319..94ec11c8b 100644
--- a/lib/rules/no-dupe-keys.js
+++ b/lib/rules/no-dupe-keys.js
@@ -4,10 +4,13 @@
  */
 'use strict'
 
+const { findVariable } = require('@eslint-community/eslint-utils')
 const utils = require('../utils')
 
 /**
  * @typedef {import('../utils').GroupName} GroupName
+ * @typedef {import('eslint').Scope.Variable} Variable
+ * @typedef {import('../utils').ComponentProp} ComponentProp
  */
 
 /** @type {GroupName[]} */
@@ -39,24 +42,43 @@ module.exports = {
     const options = context.options[0] || {}
     const groups = new Set([...GROUP_NAMES, ...(options.groups || [])])
 
-    return utils.executeOnVue(context, (obj) => {
-      /** @type {Set<string>} */
-      const usedNames = new Set()
-      const properties = utils.iterateProperties(obj, groups)
-
-      for (const o of properties) {
-        if (usedNames.has(o.name)) {
-          context.report({
-            node: o.node,
-            message: "Duplicated key '{{name}}'.",
-            data: {
-              name: o.name
-            }
-          })
+    return utils.compositingVisitors(
+      utils.executeOnVue(context, (obj) => {
+        const properties = utils.iterateProperties(obj, groups)
+        /** @type {Set<string>} */
+        const usedNames = new Set()
+        for (const o of properties) {
+          if (usedNames.has(o.name)) {
+            context.report({
+              node: o.node,
+              message: "Duplicated key '{{name}}'.",
+              data: {
+                name: o.name
+              }
+            })
+          }
+
+          usedNames.add(o.name)
         }
+      }),
+      utils.defineScriptSetupVisitor(context, {
+        onDefinePropsEnter(node, props) {
+          for (const prop of props) {
+            if (!prop.propName) continue
 
-        usedNames.add(o.name)
-      }
-    })
+            const variable = findVariable(context.getScope(), prop.propName)
+            if (!variable) continue
+
+            context.report({
+              node: variable.defs[0].node,
+              message: "Duplicated key '{{name}}'.",
+              data: {
+                name: prop.propName
+              }
+            })
+          }
+        }
+      })
+    )
   }
 }
diff --git a/tests/lib/rules/no-dupe-keys.js b/tests/lib/rules/no-dupe-keys.js
index 26c9f7339..e1294032f 100644
--- a/tests/lib/rules/no-dupe-keys.js
+++ b/tests/lib/rules/no-dupe-keys.js
@@ -390,6 +390,32 @@ ruleTester.run('no-dupe-keys', rule, {
           },
         }
       `
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+        defineProps({
+          foo: String,
+        })
+        const bar = 0
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser')
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup lang="ts">
+        defineProps<{
+          foo: string;
+        }>();
+
+        const bar = 0
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      parserOptions: { parser: require.resolve('@typescript-eslint/parser') }
     }
   ],
 
@@ -861,6 +887,85 @@ ruleTester.run('no-dupe-keys', rule, {
           line: 7
         }
       ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+        defineProps({
+          foo: String,
+        })
+        const foo = 0
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      errors: [
+        {
+          message: "Duplicated key 'foo'.",
+          line: 6
+        }
+      ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+        import { Foo } from './Foo.vue';
+        import baz from './baz';
+        
+        defineProps({
+          foo: String,
+          bar: String,
+          baz: String,
+        });
+
+        function foo() {
+          const baz = 'baz';
+        }
+        const bar = () => 'bar';
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      errors: [
+        {
+          message: "Duplicated key 'baz'.",
+          line: 4
+        },
+        {
+          message: "Duplicated key 'foo'.",
+          line: 12
+        },
+        {
+          message: "Duplicated key 'bar'.",
+          line: 15
+        }
+      ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup lang="ts">
+      defineProps<{
+        foo: string;
+        bar: string;
+      }>();
+
+      const foo = 'foo';
+      const bar = 'bar';
+      </script>
+      `,
+      parser: require.resolve('vue-eslint-parser'),
+      parserOptions: { parser: require.resolve('@typescript-eslint/parser') },
+      errors: [
+        {
+          message: "Duplicated key 'foo'.",
+          line: 8
+        },
+        {
+          message: "Duplicated key 'bar'.",
+          line: 9
+        }
+      ]
     }
   ]
 })

From 32f9d5cb26c584e72ab7d1d4a22c9f6011e0d6e7 Mon Sep 17 00:00:00 2001
From: ItMaga <m-chemurziev@ya.ru>
Date: Sat, 20 May 2023 02:36:41 +0300
Subject: [PATCH 2/2] ignore global scope variables

---
 lib/rules/no-dupe-keys.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/rules/no-dupe-keys.js b/lib/rules/no-dupe-keys.js
index 94ec11c8b..f959f4b66 100644
--- a/lib/rules/no-dupe-keys.js
+++ b/lib/rules/no-dupe-keys.js
@@ -67,7 +67,7 @@ module.exports = {
             if (!prop.propName) continue
 
             const variable = findVariable(context.getScope(), prop.propName)
-            if (!variable) continue
+            if (!variable || variable.defs.length === 0) continue
 
             context.report({
               node: variable.defs[0].node,