@putout/plugin-printer 8.19.0 → 8.21.0

package/README.md

@@ -21,6 +21,7 @@ npm i @putout/plugin-printer -D
 - ✅ [apply-computed-print](#apply-computed-print);
 - ✅ [apply-create-test-url](#apply-create-test-url);
 - ✅ [apply-linebreak](#apply-linebreak);
+- ✅ [apply-printer-declarations](#apply-printer-declarations);
 - ✅ [apply-types](#apply-types);
 - ✅ [check-type-passed-to-type-checker](#check-type-passed-to-type-checker);
 - ✅ [check-if-success-possible-in-type-checker](#check-if-success-possible-in-type-checker);
@@ -28,6 +29,7 @@ npm i @putout/plugin-printer -D
 - ✅ [declare](#declare);
 - ✅ [remove-args](#remove-args);
 - ✅ [remove-useless-maybe](#remove-useless-maybe);
+- ✅ [remove-useless-print](#remove-useless-print);
 - ✅ [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);
@@ -51,6 +53,7 @@ npm i @putout/plugin-printer -D
         "printer/apply-linebreak": "on",
         "printer/apply-computed-print": "on",
         "printer/apply-create-test-url": "on",
+        "printer/apply-printer-declarations": "on",
         "printer/apply-types": "on",
         "printer/check-type-passed-to-type-checker": "on",
         "printer/check-if-success-possible-in-type-checker": "on",
@@ -65,6 +68,7 @@ npm i @putout/plugin-printer -D
         "printer/remove-useless-not-from-type-checker": "on",
         "printer/remove-useless-path-from-type-checker": "on",
         "printer/remove-useless-tuples-from-type-checker": "on",
+        "printer/remove-useless-print": "on",
         "printer/reverse-comparison-in-type-checker": "on"
     }
 }
@@ -86,6 +90,36 @@ print.breakline();
 print.linebreak();
 ```
 
+## apply-printer-declarations
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/2fc66c476a235eb25f47e55fd7b33042/b0371fe2dd62e8f9fe5ce1fff6d352386e69fcc6).
+
+### ❌ Example of incorrect code
+
+```js
+export const ExportNamedDeclaration = (path, printer, {indent}) => {
+    const {print, write} = printer;
+    const {leadingComments} = path.node;
+    
+    indent();
+};
+```
+
+### ✅ Example of correct code
+
+```js
+export const ExportNamedDeclaration = (path, printer) => {
+    const {leadingComments} = path.node;
+    const {
+        print,
+        write,
+        indent,
+    } = printer;
+    
+    indent();
+};
+```
+
 ## apply-types
 
 Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/8a88fec812a52e302cf7b191eca2f49e/474ca87860d2eab755373ba9dc8b12efc4a80252).
@@ -572,6 +606,32 @@ const isMoreThenMaxElementLengthInOneLine = createTypeChecker([
 -});
 ```
 
+## reverse-comparison-in-type-checker
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/be2bb33f5c7ac7b687c1fa856704bc73/e57fa5a3ded571adca03f5bdeb5a0f63a594aa81).
+
+### ❌ Example of incorrect code
+
+```js
+export const ExpressionStatement = {
+    print(path, {indent, print}) {
+        indent();
+        print('__expression');
+        print.newline();
+    },
+};
+```
+
+### ✅ Example of correct code
+
+```js
+export const ExpressionStatement = (path, {indent, print}) => {
+    indent();
+    print('__expression');
+    print.newline();
+};
+```
+
 ## License
 
 MIT

package/lib/apply-printer-declarations/index.js

@@ -0,0 +1,40 @@
+import {types, operator} from 'putout';
+
+const {superTraverse} = operator;
+const {objectProperty, identifier} = types;
+
+const {assign} = Object;
+
+const names = [
+    'indent',
+    'write',
+    'print',
+    'maybe',
+];
+
+export const report = () => `Destructure from 'printer' variable instead of argument`;
+
+export const replace = () => {
+    const visitors = {};
+    
+    for (const name of names) {
+        assign(visitors, createVisitor(name));
+    }
+    
+    return visitors;
+};
+
+const createVisitor = (name) => ({
+    [`(path, printer, {${name}}) => {__body}`]: (vars, path) => {
+        const id = identifier(String(name));
+        
+        superTraverse(path, {
+            'const __object = printer': (path) => {
+                const {properties} = path.node.declarations[0].id;
+                properties.push(objectProperty(id, id));
+            },
+        });
+        
+        return '(path, printer) => {__body}';
+    },
+});

package/lib/index.js

@@ -1,3 +1,5 @@
+import * as applyPrinterDeclarations from './apply-printer-declarations/index.js';
+import * as removeUselessPrint from './remove-useless-print/index.js';
 import * as addMissingArrowToTypeChecker from './add-missing-arrow-to-type-checker/index.js';
 import * as removeUselessArrowFromTypeChecker from './remove-useless-arrow-from-type-checker/index.js';
 import * as removeUselessPathFromTypeChecker from './remove-useless-path-from-type-checker/index.js';
@@ -25,15 +27,16 @@ import * as applyTypes from './apply-types/index.js';
 import * as removeLegacyTestDeclaration from './remove-legacy-test-declaration/index.js';
 
 export const rules = {
-    'remove-args': removeArgs,
+    'add-args': addArgs,
+    'apply-printer-declarations': applyPrinterDeclarations,
     'apply-breakline': applyBreakline,
     'apply-linebreak': applyLinebreak,
     'apply-computed-print': applyComputedPrint,
-    'add-args': addArgs,
-    declare,
     'apply-types': applyTypes,
-    'remove-legacy-test-declaration': removeLegacyTestDeclaration,
     'apply-create-test-url': applyCreateTestUrl,
+    declare,
+    'remove-legacy-test-declaration': removeLegacyTestDeclaration,
+    'remove-args': removeArgs,
     'remove-useless-maybe': removeUselessMaybe,
     'remove-trailing-spaces-from-type-checker': removeTrailingSpacesFromTypeChecker,
     'add-missing-spaces-to-type-checker': addMissingSpacesToTypeChecker,
@@ -50,4 +53,5 @@ export const rules = {
     'remove-useless-path-from-type-checker': removeUselessPathFromTypeChecker,
     'remove-useless-arrow-from-type-checker': removeUselessArrowFromTypeChecker,
     'add-missing-arrow-to-type-checker': addMissingArrowToTypeChecker,
+    'remove-useless-print': removeUselessPrint,
 };

package/lib/remove-useless-print/index.js

@@ -0,0 +1,5 @@
+export const report = () => `Avoid 'print' when it is the only method`;
+
+export const replace = () => ({
+    'const __a = {print(__args) {__body}}': 'const __a = (__args) => __body',
+});

package/package.json

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