@putout/plugin-printer 1.0.0 → 1.2.0

package/README.md

@@ -19,7 +19,8 @@ npm i @putout/plugin-printer -D
         "printer/add-args": "on",
         "printer/apply-breakline": "on",
         "printer/apply-linebreak": "on",
-        "printer/apply-computed-print": "on"
+        "printer/apply-computed-print": "on",
+        "printer/remove-args": "on"
     }
 }
 ```
@@ -80,6 +81,20 @@ print(path.get('block'));
 print('__block');
 ```
 
+## remove-args
+
+### ❌ Example of incorrect code
+
+```js
+print.indent(is);
+```
+
+### ✅ Example of correct code
+
+```js
+print.indent();
+```
+
 ## License
 
 MIT

package/lib/apply-breakline/index.js

@@ -7,6 +7,7 @@ const {
 
 const {
     compare,
+    compareAny,
     replaceWith,
     remove,
 } = operator;
@@ -16,14 +17,29 @@ module.exports.report = () => `breakline = newline + indent`;
 const next = (path) => path.parentPath.getNextSibling();
 
 module.exports.fix = (path) => {
-    remove(next(path));
-    replaceWith(path, template.ast('print.breakline()'));
+    const sibling = next(path);
+    const newNode = choose(path);
+    
+    remove(sibling);
+    replaceWith(path, newNode);
 };
 
 module.exports.filter = (path) => {
-    return compare(next(path), 'print.indent()');
+    return compareAny(next(path), [
+        'indent()',
+        'print.indent()',
+        'write.indent()',
+    ]);
 };
 
 module.exports.include = () => [
     'print.newline()',
+    'write.newline()',
 ];
+
+function choose(path) {
+    if (compare(path, 'print.newline()'))
+        return template.ast('print.breakline()');
+    
+    return template.ast('write.breakline()');
+}

package/lib/apply-linebreak/index.js

@@ -7,6 +7,7 @@ const {
 
 const {
     compare,
+    compareAny,
     replaceWith,
     remove,
 } = operator;
@@ -16,14 +17,29 @@ module.exports.report = () => `linebreak = indent + newline`;
 const prev = (path) => path.parentPath.getPrevSibling();
 
 module.exports.fix = (path) => {
-    remove(prev(path));
-    replaceWith(path, template.ast('print.linebreak()'));
+    const sibling = prev(path);
+    const newNode = choose(path);
+    
+    remove(sibling);
+    replaceWith(path, newNode);
 };
 
 module.exports.include = () => [
     'print.newline()',
+    'write.newline()',
 ];
 
 module.exports.filter = (path) => {
-    return compare(prev(path), 'print.indent()');
+    return compareAny(prev(path), [
+        'indent()',
+        'print.indent()',
+        'write.indent()',
+    ]);
 };
+
+function choose(path) {
+    if (compare(path, 'print.newline()'))
+        return template.ast('print.linebreak()');
+    
+    return template.ast('write.linebreak()');
+}

package/lib/remove-args/index.js

@@ -0,0 +1,25 @@
+'use strict';
+
+module.exports.report = () => 'Remove useless argument';
+
+module.exports.replace = () => ({
+    'print.newline(__args)': 'print.newline()',
+    'print.space(__args)': 'print.space()',
+    
+    'indent(__args)': 'indent()',
+    'indent.inc(__args)': 'indent.inc()',
+    'indent.dec(__args)': 'indent.dec()',
+    
+    'write.indent(__args)': 'write.indent()',
+    'write.indent.inc(__args)': 'write.indent.inc()',
+    'write.indent.dec(__args)': 'write.indent.dec()',
+    
+    'print.indent(__args)': 'print.indent()',
+    'print(__a, __b)': 'print(__b)',
+    'print.indent.inc(__args)': 'print.indent.inc()',
+    'print.indent.dec(__args)': 'print.indent.dec()',
+    
+    'write(__a, __b)': 'write(__b)',
+    
+    'maybe.write.space(__a, __b)': 'maybe.write.space(__a)',
+});

package/package.json

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