fix(no-dupe-keys): detect props destructure rename

Author: waynzh

lib/rules/no-dupe-keys.js

@@ -58,6 +58,33 @@ function isInsideInitializer(node, references) {
   )
 }
 
+/**
+ * Find if the prop is renamed in the pattern
+ * @param {Pattern} pattern - The destructuring pattern
+ * @param {string} propName - The original prop name
+ * @returns {boolean} - True if the prop is renamed
+ */
+function findRenamedProp(pattern, propName) {
+  if (!pattern || pattern.type !== 'ObjectPattern') {
+    return false
+  }
+
+  for (const prop of pattern.properties) {
+    if (prop.type !== 'Property') continue
+
+    if (
+      prop.key.type === 'Identifier' &&
+      prop.key.name === propName &&
+      prop.value.type === 'Identifier' &&
+      prop.value.name !== propName
+    ) {
+      return true
+    }
+  }
+
+  return false
+}
+
 module.exports = {
   meta: {
     type: 'problem',
@@ -118,6 +145,10 @@ module.exports = {
           for (const prop of props) {
             if (!prop.propName) continue
 
+            if (propsNode && findRenamedProp(propsNode, prop.propName)) {
+              continue
+            }
+
             const variable = findVariable(
               utils.getScope(context, node),
               prop.propName

tests/lib/rules/no-dupe-keys.js

@@ -466,7 +466,7 @@ ruleTester.run('no-dupe-keys', rule, {
     {
       filename: 'test.vue',
       code: `
-      <script setup></script>
+      <script setup>
       const {foo,bar} = defineProps(['foo', 'bar'])
       </script>
       `,
@@ -475,7 +475,7 @@ ruleTester.run('no-dupe-keys', rule, {
     {
       filename: 'test.vue',
       code: `
-      <script setup></script>
+      <script setup>
       const {foo=42,bar='abc'} = defineProps(['foo', 'bar'])
       </script>
       `,
@@ -500,6 +500,17 @@ ruleTester.run('no-dupe-keys', rule, {
         parser: require('vue-eslint-parser'),
         parserOptions: { parser: require.resolve('@typescript-eslint/parser') }
       }
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      const { foo: renamedFoo, bar: renamedBar } = defineProps(['foo', 'bar'])
+      const foo = 42
+      const bar = 'hello'
+      </script>
+      `,
+      languageOptions: { parser: require('vue-eslint-parser') }
     }
   ],
 
@@ -1105,6 +1116,24 @@ ruleTester.run('no-dupe-keys', rule, {
           line: 5
         }
       ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      const { foo: renamedFoo } = defineProps(['foo', 'bar'])
+      const foo = 'foo'
+      const bar = 'bar'
+      </script>
+      `,
+      languageOptions: { parser: require('vue-eslint-parser') },
+      errors: [
+        {
+          message:
+            "Duplicate key 'bar'. May cause name collision in script or template tag.",
+          line: 5
+        }
+      ]
     }
   ]
 })