@putout/plugin-printer 8.16.2 → 8.17.1

package/README.md

@@ -31,6 +31,7 @@ npm i @putout/plugin-printer -D
 - ✅ [remove-legacy-test-declaration](#remove-legacy-test-declaration);
 - ✅ [remove-trailing-spaces-from-type-checker](#remove-trailing-spaces-from-type-checker);
 - ✅ [remove-useless-spaces-from-type-checker](#remove-useless-spaces-from-type-checker);
+- ✅ [remove-useless-arrow-from-type-checker](#remove-useless-arrow-from-type-checker);
 - ✅ [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);
@@ -59,6 +60,7 @@ npm i @putout/plugin-printer -D
         "printer/remove-useless-maybe": "on",
         "printer/remove-trailing-spaces-from-type-checker": "on",
         "printer/remove-useless-spaces-from-type-checker": "on",
+        "printer/remove-useless-arrow-from-type-checker": "on",
         "printer/remove-useless-colon-from-type-checker": "on",
         "printer/remove-useless-not-from-type-checker": "on",
         "printer/remove-useless-path-from-type-checker": "on",
@@ -278,6 +280,28 @@ export const beforeIf = createTypeChecker([
 ]);
 ```
 
+## remove-useless-arrow-from-type-checker
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/c12c37b793192eadb60fac332d95cc3b/3f4aa557c65897f4967712f30e51c29483feddc8).
+
+### ❌ Example of incorrect code
+
+```js
+const isTwoLongStringsInsideArray = createTypeChecker([
+    ['-: -> !', isTwoLongStrings],
+    ['+: parentPath -> -> ArrayExpression'],
+]);
+```
+
+### ✅ Example of correct code
+
+```js
+const isTwoLongStringsInsideArray = createTypeChecker([
+    ['-: -> !', isTwoLongStrings],
+    ['+: parentPath -> ArrayExpression'],
+]);
+```
+
 ## remove-useless-colon-from-type-checker
 
 Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/49a35399bd43d56bfd0221b3a0ace3b0/3ff3bdc8cb5f2a06786e9b616252d1b5f506dd73).

package/lib/add-missing-tuples-to-type-checker/index.js

@@ -36,14 +36,19 @@ export const fix = (path) => {
     const {node} = path;
     
     if (isIdentifier(path) || isCallExpression(path)) {
-        replaceWith(path, arrayExpression([stringLiteral('+'), node]));
+        replaceWith(path, arrayExpression([
+            stringLiteral('+'),
+            node,
+        ]));
         return;
     }
     
     const {value} = node;
     
     if (isStringLiteral(path) && !/^[+-]/.test(value)) {
-        replaceWith(path, arrayExpression([stringLiteral(`+: -> ${value}`)]));
+        replaceWith(path, arrayExpression([
+            stringLiteral(`+: -> ${value}`),
+        ]));
         return;
     }
     

package/lib/index.js

@@ -1,3 +1,4 @@
+import * as removeUselessArrowFromTypeChecker from './remove-useless-arrow-from-type-checker/index.js';
 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';
@@ -46,4 +47,5 @@ export const rules = {
     'remove-useless-not-from-type-checker': removeUselessNotFromTypeChecker,
     'reverse-comparison-in-type-checker': reverseComparisonInTypeChecker,
     'remove-useless-path-from-type-checker': removeUselessPathFromTypeChecker,
+    'remove-useless-arrow-from-type-checker': removeUselessArrowFromTypeChecker,
 };

package/lib/remove-args/index.js

@@ -1,4 +1,21 @@
-const checkArgs = ({__args}) => __args.length;
+import {types, operator} from 'putout';
+
+const {isImportSpecifier} = types;
+const {getBindingPath} = operator;
+
+const checkArgs = ({__args}, path) => {
+    if (!__args.length)
+        return false;
+    
+    const {name} = path.node.callee;
+    
+    const bindingPath = getBindingPath(path, name);
+    
+    if (!bindingPath)
+        return true;
+    
+    return !isImportSpecifier(bindingPath);
+};
 
 export const report = () => 'Remove useless argument';
 

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

@@ -0,0 +1,56 @@
+import {operator, types} from 'putout';
+
+const {
+    isIdentifier,
+    isCallExpression,
+} = types;
+
+const {setLiteralValue} = operator;
+
+export const report = ({path, where}) => {
+    const {value} = path.node;
+    const withSpaces = addColon(value, where);
+    
+    return `Remove useless arrow: '${value}' -> '${withSpaces}'`;
+};
+
+export const fix = ({path, where}) => {
+    const {value} = path.node;
+    setLiteralValue(path, addColon(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 = [];
+    
+    if (value.includes('-> ->'))
+        where.push('duplicate');
+    
+    return where;
+}
+
+function addColon(value, where) {
+    if (where.includes('duplicate'))
+        value = value.replace('-> ->', '->');
+    
+    return value;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-printer",
-  "version": "8.16.2",
+  "version": "8.17.1",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds support of transformations for @putout/printer",
@@ -43,13 +43,12 @@
     "@putout/plugin-remove-nested-blocks": "*",
     "@putout/test": "^15.0.0",
     "eslint": "^10.0.0",
-    "eslint-plugin-n": "^17.0.0",
     "eslint-plugin-putout": "^31.0.0",
     "madrun": "^13.0.0",
-    "montag": "^1.2.1",
+    "montag": "^2.0.0",
     "nodemon": "^3.0.1",
     "superc8": "^12.0.0",
-    "supertape": "^12.7.0"
+    "supertape": "^13.0.0"
   },
   "peerDependencies": {
     "putout": ">=42"