refactor

Author: waynzh

lib/rules/no-dupe-keys.js

@@ -59,30 +59,30 @@ 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
+ * Collects all renamed props from a pattern
+ * @param {Pattern | null} pattern - The destructuring pattern
+ * @returns {Set<string>} - Set of prop names that have been renamed
  */
-function findRenamedProp(pattern, propName) {
+function collectRenamedProps(pattern) {
+  const renamedProps = new Set()
+
   if (!pattern || pattern.type !== 'ObjectPattern') {
-    return false
+    return renamedProps
   }
 
   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
+      prop.key.name !== prop.value.name
     ) {
-      return true
+      renamedProps.add(prop.key.name)
     }
   }
 
-  return false
+  return renamedProps
 }
 
 module.exports = {
@@ -142,10 +142,12 @@ module.exports = {
             node
           ]
 
+          const renamedProps = collectRenamedProps(propsNode)
+
           for (const prop of props) {
             if (!prop.propName) continue
 
-            if (propsNode && findRenamedProp(propsNode, prop.propName)) {
+            if (renamedProps.has(prop.propName)) {
               continue
             }