@putout/plugin-printer 8.14.0 → 8.16.0

package/README.md

@@ -34,6 +34,8 @@ npm i @putout/plugin-printer -D
 - ✅ [remove-useless-colon-from-type-checker](#remove-useless-colon-from-type-checker);
 - ✅ [remove-useless-tuples-from-type-checker](#remove-useless-tuples-from-type-checker);
 - ✅ [remove-useless-not-from-type-checker](#remove-useless-not-from-type-checker);
+- ✅ [remove-useless-path-from-type-checker](#remove-useless-path-from-type-checker);
+- ✅ [reverse-comparison-in-type-checker](#reverse-comparison-in-type-checker);
 
 ## Config
 
@@ -59,7 +61,9 @@ npm i @putout/plugin-printer -D
         "printer/remove-useless-spaces-from-type-checker": "on",
         "printer/remove-useless-colon-from-type-checker": "on",
         "printer/remove-useless-not-from-type-checker": "on",
-        "printer/remove-useless-tuples-from-type-checker": "on"
+        "printer/remove-useless-path-from-type-checker": "on",
+        "printer/remove-useless-tuples-from-type-checker": "on",
+        "printer/reverse-comparison-in-type-checker": "on"
     }
 }
 ```
@@ -318,6 +322,28 @@ export const beforeIf = createTypeChecker([
 ]);
 ```
 
+## remove-useless-path-from-type-checker
+
+Selector always starts from `path`, so it can be `parentPath.`, or `node.`, but never `path.`, sinse it is self reference.
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/ef8e1dc1036a4a2fcc99b32c3dc661e2/bc43a082dc9cb5ed8a8afe6964174f0d1be2e1c1).
+
+### ❌ Example of incorrect code
+
+```js
+export const beforeIf = createTypeChecker([
+    ['-: path.parentPath -> ', isCoupleLines],
+]);
+```
+
+### ✅ Example of correct code
+
+```js
+export const beforeIf = createTypeChecker([
+    ['-: parentPath -> ', isCoupleLines],
+]);
+```
+
 ## remove-useless-tuples-from-type-checker
 
 Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/f53f386b607115e0a96f7e294e34761e/cefd72aba4fcade59bbf8faa59739bac5c2fe057).
@@ -483,6 +509,32 @@ export const isNewlineAfterComma = createTypeChecker([
 ]);
 ```
 
+## reverse-comparison-in-type-checker
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/ae2335d32022de240e918ce077144767/f25c5a11f746fe532f81adede991286cb77d1b11).
+
+### ❌ Example of incorrect code
+
+```js
+const isMoreThenMaxElementLengthInOneLine = createTypeChecker([
+    ['-: node.elements.length -> !', '<', 2],
+    ['-: node.elements.length -> !', '>', 2],
+    ['-: node.elements.length -> !', '>=', 2],
+    ['-: node.elements.length -> !', '<=', 2],
+]);
+```
+
+### ✅ Example of correct code
+
+```js
+const isMoreThenMaxElementLengthInOneLine = createTypeChecker([
+    ['-: node.elements.length', '>=', 2],
+    ['-: node.elements.length', '<=', 2],
+    ['-: node.elements.length', '<', 2],
+    ['-: node.elements.length', '>', 2],
+]);
+```
+
 ## remove-legacy-test-declaration
 
 ```diff

package/lib/index.js

@@ -1,3 +1,5 @@
+import * as removeUselessPathFromTypeChecker from './remove-useless-path-from-type-checker/index.js';
+import * as reverseComparisonInTypeChecker from './reverse-comparison-in-type-checker/index.js';
 import * as removeUselessNotFromTypeChecker from './remove-useless-not-from-type-checker/index.js';
 import * as removeUselessColonFromTypeChecker from './remove-useless-colon-from-type-checker/index.js';
 import * as addMissingColonToTypeChecker from './add-missing-colon-to-type-checker/index.js';
@@ -42,4 +44,6 @@ export const rules = {
     'add-missing-colon-to-type-checker': addMissingColonToTypeChecker,
     'remove-useless-colon-from-type-checker': removeUselessColonFromTypeChecker,
     'remove-useless-not-from-type-checker': removeUselessNotFromTypeChecker,
+    'reverse-comparison-in-type-checker': reverseComparisonInTypeChecker,
+    'remove-useless-path-from-type-checker': removeUselessPathFromTypeChecker,
 };

package/lib/remove-useless-path-from-type-checker/index.js

@@ -0,0 +1,70 @@
+import {operator, types} from 'putout';
+
+const {
+    isIdentifier,
+    isCallExpression,
+} = types;
+
+const {setLiteralValue} = operator;
+
+export const report = ({path, where}) => {
+    const {value} = path.node;
+    const withoutNot = removePath(value, where);
+    
+    return `Avoid useless 'path': '${value}' -> '${withoutNot}'`;
+};
+
+export const fix = ({path, where}) => {
+    const {value} = path.node;
+    setLiteralValue(path, removePath(value, where));
+};
+export const traverse = ({push}) => ({
+    StringLiteral(path) {
+        const call = path.find(isCallExpression);
+        
+        if (!call)
+            return;
+        
+        if (!isIdentifier(call.node.callee, {name: 'createTypeChecker'}))
+            return;
+        
+        const {value} = path.node;
+        const where = createWhere(value);
+        
+        if (where.length)
+            push({
+                path,
+                where,
+            });
+    },
+});
+
+function createWhere(value) {
+    const where = [];
+    const includes = value.includes.bind(value);
+    const endsWith = value.endsWith.bind(value);
+    const push = where.push.bind(where);
+    
+    const tools = {
+        push,
+        includes,
+        endsWith,
+    };
+    
+    if (includes('path.'))
+        tracePath(tools);
+    
+    return where;
+}
+
+function tracePath({push, includes}) {
+    if (includes('path.'))
+        push('path');
+}
+
+function removePath(value, where) {
+    if (where.includes('path'))
+        value = value.replace('path.', '');
+    
+    return value;
+}

package/lib/reverse-comparison-in-type-checker/index.js

@@ -0,0 +1,32 @@
+import {operator} from 'putout';
+
+const CMP = {
+    '<': '>=',
+    '>': '<=',
+    '<=': '>',
+    '>=': '<',
+};
+
+const OPERATORS = Object.keys(CMP);
+
+const {setLiteralValue} = operator;
+
+export const report = () => `Reverse comparison`;
+
+export const match = () => ({
+    '["__a", "__b", __c]': ({__a, __b}) => {
+        if (!OPERATORS.includes(__b.value))
+            return false;
+        
+        return __a.value.endsWith('!');
+    },
+});
+
+export const replace = () => ({
+    '["__a", "__b", __c]': ({__a, __b}, path) => {
+        setLiteralValue(__a, __a.value.slice(0, -5));
+        setLiteralValue(__b, CMP[__b.value]);
+        
+        return path;
+    },
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-printer",
-  "version": "8.14.0",
+  "version": "8.16.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds support of transformations for @putout/printer",