feature: @putout/printer: apply-linebreak: add support of 'indent()', 'write.indent()'

Author: coderaiser

packages/plugin-printer/.putout.json

@@ -1,6 +1,8 @@
 {
     "printer": "putout",
-    "rules": {
-        "putout": "on"
+    "match": {
+        "*.md": {
+            "putout/declare": "off"
+        }
     }
 }

packages/plugin-printer/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

packages/plugin-printer/lib/apply-breakline/index.js

@@ -18,13 +18,10 @@ const next = (path) => path.parentPath.getNextSibling();
 
 module.exports.fix = (path) => {
     const sibling = next(path);
-    remove(sibling);
-    
-    if (compare(path, 'print.newline()'))
-        return replaceWith(path, template.ast('print.breakline()'));
+    const newNode = choose(path);
 
-    if (compare(path, 'write.newline()'))
-        return replaceWith(path, template.ast('write.breakline()'));
+    remove(sibling);
+    replaceWith(path, newNode);
 };
 
 module.exports.filter = (path) => {
@@ -39,3 +36,10 @@ 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()');
+}

packages/plugin-printer/lib/apply-linebreak/fixture/indent-fix.js

@@ -0,0 +1 @@
+write.linebreak();

packages/plugin-printer/lib/apply-linebreak/fixture/indent.js

@@ -0,0 +1,2 @@
+indent();
+write.newline();

packages/plugin-printer/lib/apply-linebreak/fixture/write-fix.js

@@ -0,0 +1 @@
+write.linebreak();

packages/plugin-printer/lib/apply-linebreak/fixture/write.js

@@ -0,0 +1,2 @@
+write.indent();
+write.newline();

packages/plugin-printer/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()');
+}

packages/plugin-printer/lib/apply-linebreak/index.spec.js

@@ -19,3 +19,13 @@ test('rule: apply-linebreak: transform', (t) => {
     t.transform('apply-linebreak');
     t.end();
 });
+
+test('rule: apply-linebreak: transform: indent', (t) => {
+    t.transform('indent');
+    t.end();
+});
+
+test('rule: apply-linebreak: transform: write', (t) => {
+    t.transform('write');
+    t.end();
+});

packages/plugin-printer/lib/remove-args/fixture/remove-args-fix.js

@@ -0,0 +1,17 @@
+print.newline();
+print.space();
+
+indent();
+indent.inc();
+indent.dec();
+
+write.indent();
+write.indent.inc();
+write.indent.dec();
+
+print.indent();
+print('hello');
+print.indent.inc();
+print.indent.dec();
+
+write('hello');