@putout/plugin-remove-duplicate-keys 2.1.0 → 4.0.0

package/README.md

@@ -1,11 +1,13 @@
-# @putout/plugin-remove-duplicate-keys [![NPM version][NPMIMGURL]][NPMURL] [![Dependency Status][DependencyStatusIMGURL]][DependencyStatusURL]
+# @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"
-[DependencyStatusURL]: https://david-dm.org/coderaiser/putout?path=packages/plugin-remove-duplicate-keys
-[DependencyStatusIMGURL]: https://david-dm.org/coderaiser/putout.svg?path=packages/plugin-remove-duplicate-keys
 
-`putout` plugin adds ability to find and remove duplecate keys.
+> 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 (`{}`).
+>
+> (c) [MDN](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Object_initializer)
+
+🐊[**Putout**](https://github.com/coderaiser/putout) plugin adds ability to find and remove duplecate keys.
 
 ## Install
 
@@ -23,7 +25,7 @@ npm i @putout/plugin-remove-duplicate-keys
 }
 ```
 
-## ❌ Incorrect code example
+## ❌ Example of incorrect code
 
 ```js
 const a = {
@@ -33,7 +35,7 @@ const a = {
 };
 ```
 
-## ✅ Correct code Example
+## ✅ Example of correct code
 
 ```js
 const a = {

package/lib/remove-duplicate-keys.js

@@ -1,7 +1,8 @@
 'use strict';
 
-const {types} = require('putout');
+const {types, operator} = require('putout');
 const fullstore = require('fullstore');
+
 const {
     isSpreadElement,
     isIdentifier,
@@ -9,13 +10,25 @@ const {
     isStringLiteral,
 } = types;
 
-const isSpreadId = (name) => (a) => isSpreadElement(a) && isIdentifier(a.argument, {name});
-const isObjectPropertyId = (name, computed) => (a) => isObjectProperty(a, {computed}) && isIdentifier(a.key, {name});
-const isObjectPropertyLiteral = (value) => (a) => isObjectProperty(a) && isStringLiteral(a.key, {value});
+const {traverseProperties} = operator;
+
+const isSpreadId = (name) => (a) => isSpreadElement(a) && isIdentifier(a.argument, {
+    name,
+});
+
+const isObjectPropertyId = (name, computed) => (a) => isObjectProperty(a, {
+    computed,
+}) && isIdentifier(a.key, {
+    name,
+});
+
+const isObjectPropertyLiteral = (value) => (a) => isObjectProperty(a) && isStringLiteral(a.key, {
+    value,
+});
 
 const store = fullstore([]);
 
-module.exports.report = () => 'Duplicate keys should be avoided';
+module.exports.report = () => 'Avoid duplicate keys';
 
 module.exports.replace = () => ({
     __object: ({__object}) => {
@@ -27,11 +40,14 @@ module.exports.replace = () => ({
 });
 
 module.exports.match = () => ({
-    __object: ({__object}) => {
+    __object: ({__object}, path) => {
         let is = false;
         const newProperties = [];
         const {properties} = __object;
-        const reversed = properties.slice().reverse();
+        
+        const reversed = properties
+            .slice()
+            .reverse();
         
         for (const prop of reversed) {
             if (isSpreadElement(prop) && isIdentifier(prop.argument)) {
@@ -48,16 +64,19 @@ module.exports.match = () => ({
                 const {computed} = prop;
                 const {name} = prop.key;
                 
-                if (name !== prop.value.name)
-                    continue;
-                
-                const isFirst = checkIfFirst(properties, newProperties, isObjectPropertyId, name, {
-                    computed,
+                const props = traverseProperties(path, name, {
+                    firstLevel: true,
                 });
                 
-                if (!isFirst) {
-                    is = true;
-                    continue;
+                if (props.length > 1) {
+                    const isFirst = checkIfFirst(properties, newProperties, isObjectPropertyId, name, {
+                        computed,
+                    });
+                    
+                    if (!isFirst) {
+                        is = true;
+                        continue;
+                    }
                 }
             }
             
@@ -87,4 +106,3 @@ function checkIfFirst(properties, newProperties, isFn, str, {computed} = {}) {
     
     return !newLength || oldLength <= 1;
 }
-

package/package.json

@@ -1,9 +1,10 @@
 {
   "name": "@putout/plugin-remove-duplicate-keys",
-  "version": "2.1.0",
+  "version": "4.0.0",
+  "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
-  "description": "putout plugin adds ability to find and remove process.exit",
-  "homepage": "http://github.com/coderaiser/putout",
+  "description": "🐊Putout plugin adds ability to find and remove duplicate keys",
+  "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-duplicate-keys#readme",
   "main": "lib/remove-duplicate-keys.js",
   "release": false,
   "tag": false,
@@ -34,21 +35,21 @@
     "key"
   ],
   "devDependencies": {
-    "@putout/test": "^3.0.0",
-    "c8": "^7.5.0",
-    "eslint": "^8.0.0-beta.0",
-    "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-putout": "^10.0.0",
-    "lerna": "^4.0.0",
-    "madrun": "^8.0.1",
-    "nodemon": "^2.0.1"
+    "@putout/test": "^8.0.0",
+    "c8": "^8.0.0",
+    "eslint": "^8.0.1",
+    "eslint-plugin-n": "^16.0.0",
+    "eslint-plugin-putout": "^22.0.0",
+    "lerna": "^6.0.1",
+    "madrun": "^10.0.0",
+    "nodemon": "^3.0.1"
   },
   "peerDependencies": {
-    "putout": ">=9"
+    "putout": ">=34"
   },
   "license": "MIT",
   "engines": {
-    "node": ">=10"
+    "node": ">=18"
   },
   "publishConfig": {
     "access": "public"