@putout/plugin-remove-duplicate-keys 8.1.0 → 9.0.0

package/README.md

@@ -1,7 +1,7 @@
 # @putout/plugin-remove-duplicate-keys [![NPM version][NPMIMGURL]][NPMURL]
 
 [NPMIMGURL]: https://img.shields.io/npm/v/@putout/plugin-remove-duplicate-keys.svg?style=flat&longCache=true
-[NPMURL]: https://npmjs.org/package/@putout/plugin-remove-duplicate-keys"npm"
+[NPMURL]: https://npmjs.org/package/@putout/plugin-remove-duplicate-keys "npm"
 
 > An object initializer is a comma-delimited list of zero or more pairs of property names and associated values of an object, enclosed in curly braces (`{}`).
 >

package/lib/remove-duplicate-keys.js

@@ -20,11 +20,14 @@ const isSpreadId = (name) => (a) => isSpreadElement(a) && isIdentifier(a.argumen
     name,
 });
 
-const isObjectPropertyId = (name, computed) => (a) => isObjectProperty(a, {
-    computed,
-}) && isIdentifier(a.key, {
-    name,
-});
+const isObjectPropertyId = (name, computed) => (a) => {
+    if (!isObjectProperty(a, {computed}))
+        return false;
+    
+    return isIdentifier(a.key, {
+        name,
+    });
+};
 
 const isMemberExpressionId = (name, computed) => (a) => isObjectProperty(a, {
     computed,
@@ -34,7 +37,19 @@ const isObjectPropertyLiteral = (value) => (a) => isObjectProperty(a) && isStrin
     value,
 });
 
-export const report = () => 'Avoid duplicate keys';
+const addQuote = (a) => `'${a}'`;
+
+export const report = ({names}) => {
+    const {length} = names;
+    const quotedNames = names
+        .reverse()
+        .map(addQuote)
+        .join(', ');
+    
+    const suffix = !length ? '' : `: ${quotedNames}`;
+    
+    return `Avoid duplicate keys${suffix}`;
+};
 
 export const fix = ({path, newProperties}) => {
     path.node.properties = newProperties;
@@ -50,6 +65,8 @@ export const traverse = ({push}) => ({
             .slice()
             .reverse();
         
+        const names = [];
+        
         for (const prop of reversed) {
             if (isObjectPattern(path) && !compare(prop.key, prop.value))
                 continue;
@@ -79,6 +96,7 @@ export const traverse = ({push}) => ({
                     
                     if (!isFirst) {
                         is = true;
+                        names.push(name);
                         continue;
                     }
                 }
@@ -90,6 +108,7 @@ export const traverse = ({push}) => ({
                 
                 if (!isFirst) {
                     is = true;
+                    names.push(value);
                     continue;
                 }
             }
@@ -123,6 +142,7 @@ export const traverse = ({push}) => ({
         if (is)
             push({
                 path,
+                names,
                 newProperties,
             });
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-remove-duplicate-keys",
-  "version": "8.1.0",
+  "version": "9.0.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds ability to find and remove duplicate keys",
@@ -33,21 +33,20 @@
     "key"
   ],
   "devDependencies": {
-    "@putout/eslint-flat": "^3.0.0",
-    "@putout/test": "^14.0.0",
+    "@putout/test": "^15.0.0",
     "c8": "^10.0.0",
-    "eslint": "^9.0.0",
+    "eslint": "^10.0.0",
     "eslint-plugin-n": "^17.0.0",
-    "eslint-plugin-putout": "^28.0.0",
-    "madrun": "^11.0.0",
+    "eslint-plugin-putout": "^31.0.0",
+    "madrun": "^13.0.0",
     "nodemon": "^3.0.1"
   },
   "peerDependencies": {
-    "putout": ">=40"
+    "putout": ">=42"
   },
   "license": "MIT",
   "engines": {
-    "node": ">=20"
+    "node": ">=22"
   },
   "publishConfig": {
     "access": "public"