@putout/plugin-putout 7.12.0 → 8.2.1

package/README.md

@@ -1,11 +1,9 @@
-# @putout/plugin-putout [![NPM version][NPMIMGURL]][NPMURL] [![Dependency Status][DependencyStatusIMGURL]][DependencyStatusURL]
+# @putout/plugin-putout [![NPM version][NPMIMGURL]][NPMURL]
 
 [NPMIMGURL]: https://img.shields.io/npm/v/@putout/plugin-putout.svg?style=flat&longCache=true
 [NPMURL]: https://npmjs.org/package/@putout/plugin-putout"npm"
-[DependencyStatusURL]: https://david-dm.org/coderaiser/putout?path=packages/plugin-putout
-[DependencyStatusIMGURL]: https://david-dm.org/coderaiser/putout.svg?path=packages/plugin-putout
 
-`putout` plugin helps with `putout` plugins development.
+🐊[`Putout`](https://github.com/coderaiser/putout) plugin helps with 🐊`Putout` plugins development.
 
 ## Install
 
@@ -19,7 +17,9 @@ npm i @putout/plugin-putout -D
 {
     "rules": {
         "putout/apply-processors-destructuring": "on",
-        "putout/add-missing-argument": "on",
+        "putout/apply-async-formatter": "on",
+        "putout/add-args": "on",
+        "putout/convert-putout-test-to-create-test": "on",
         "putout/convert-to-no-transform-code": "on",
         "putout/convert-replace-with": "on",
         "putout/convert-replace-with-multiple": "on",
@@ -30,6 +30,7 @@ npm i @putout/plugin-putout -D
         "putout/convert-traverse-to-replace": "on",
         "putout/convert-process-to-find": "on",
         "putout/convert-method-to-property": "on",
+        "putout/convert-add-argument-to-add-args": "on",
         "putout/shorten-imports": "on",
         "putout/check-replace-code": "on",
         "putout/declare": "on"
@@ -56,6 +57,49 @@ test('', async ({process}) => {
 });
 ```
 
+## apply-async-formatter
+
+### ❌ Incorrect code example
+
+```js
+test('formatter: codeframea', (t) => {
+    t.format(codeframe, 1);
+    t.end();
+});
+```
+
+### ✅ Correct code example
+
+```js
+test('formatter: codeframea', async ({format}) => {
+    await format(codeframe, 1);
+});
+```
+
+## convert-putout-test-to-create-test"
+
+Fixes results of [@putout/convert-commonjs-to-esm](https://github.com/coderaiser/putout/tree/master/packages/plugin-convert-commonjs-to-esm) work.
+
+### ❌ Incorrect code example
+
+```js
+import putoutTest from '@putout/test';
+
+const test = putoutTest(__dirname, {
+    'remove-unused-variables': rmVars,
+});
+```
+
+### ✅ Correct code Example
+
+```js
+import createTest from '@putout/test';
+
+const test = createTest(__dirname, {
+    'remove-unused-variables': rmVars,
+});
+```
+
 ## convert-to-no-transform-code
 
 ### ❌ Incorrect code example
@@ -153,7 +197,6 @@ module.exports.replace = () => ({
     'const __a = __b': ({}) => {
     },
     'const __c = __d': ({}, path) => {
-    
     },
 });
 ```
@@ -286,7 +329,8 @@ module.exports.replace = () => ({
 });
 ```
 
-There is no `fix` for this rule, it used internally to be more confident about `test coverage`, because of declaration form, transforms cannon be checked by `nyc` and `c8`, and uncovered lines can find unfixable false possitives when running on code. This additional tests, if you forget to test some case (from a big list of rules that is supported) it will be checked with this `rule` and make code more stable and transform bugs.
+There is no `fix` for this rule, it used internally to be more confident about `test coverage`, because of declaration form, transforms cannon be checked by `nyc` and `c8`, and uncovered lines can find unfixable false possitives when running on code.
+This is additional tests, if you forget to test some case (from a big list of rules that is supported) it will be checked with this `rule` and make transforms more stable.
 
 ## declare
 
@@ -311,7 +355,7 @@ compare(a, 'const __a = __b');
 isIdentifier(a);
 ```
 
-## add-missing-argument
+## add-args
 
 ### ❌ Incorrect code example
 
@@ -329,6 +373,28 @@ test('', ({comparePlaces}) => {
 });
 ```
 
+## convert-add-argument-to-add-args
+
+```js
+const {operator} = require('putout');
+const {addArgument} = operator;
+
+module.exports = addArgument({
+    t: ['t', 'test("__a", (__args) => __body)'],
+});
+```
+
+### ✅ Correct code Example
+
+```js
+const {operator} = require('putout');
+const {addArgs} = operator;
+
+module.exports = addArgs({
+    t: ['t', 'test("__a", (__args) => __body)'],
+});
+```
+
 ## License
 
 MIT

package/lib/{add-missing-argument → add-args}/index.js

@@ -1,9 +1,9 @@
 'use strict';
 
 const {operator} = require('putout');
-const {addArgument} = operator;
+const {addArgs} = operator;
 
-module.exports = addArgument({
+module.exports = addArgs({
     comparePlaces: ['{comparePlaces}', 'test("__a", (__args) => __body)'],
     compare: ['{compare}', 'test("__a", (__args) => __body)'],
 });

package/lib/apply-async-formatter/index.js

@@ -0,0 +1,47 @@
+'use strict';
+
+const {
+    operator,
+    types,
+} = require('putout');
+
+const computed = true;
+const shorthand = true;
+
+const {
+    Identifier,
+    ObjectPattern,
+    ObjectProperty,
+} = types;
+
+const {compare} = operator;
+
+module.exports.report = () => 'Use Async API to test Formatter';
+
+module.exports.replace = () => ({
+    't.format(__args)': create('format'),
+    't.noFormat(__args)': create('noFormat'),
+    't.formatMany(__args)': create('formatMany'),
+});
+
+const create = (name) => (vars, path) => {
+    const {block} = path.scope;
+    const {body} = block.body;
+    const n = body.length - 1;
+    const nameId = Identifier(name);
+    
+    block.async = true;
+    
+    block.params = [
+        ObjectPattern([
+            ObjectProperty(nameId, nameId, !computed, shorthand),
+        ]),
+    ];
+    
+    if (compare(body[n], 't.end()')) {
+        body.pop();
+    }
+    
+    return `await ${name}(__args)`;
+};
+

package/lib/convert-add-argument-to-add-args/index.js

@@ -0,0 +1,13 @@
+'use strict';
+
+module.exports.report = () => 'Use addArgs instead of addArgument';
+
+module.exports.replace = () => ({
+    'addArgument(__args)': (vars, path) => {
+        path.scope.rename('addArgument', 'addArgs');
+        path.scope.crawl();
+        
+        return path;
+    },
+});
+

package/lib/convert-destructuring-to-identifier/index.js

@@ -26,5 +26,6 @@ function findUp(path, str) {
         
         path = path.parentPath;
     }
+    
     return false;
 }

package/lib/convert-putout-test-to-create-test/index.js

@@ -0,0 +1,14 @@
+'use strict';
+
+module.exports.report = () => `Use 'createTest' instead of 'putoutTest'`;
+
+module.exports.filter = ({scope}) => !scope.bindings.createTest;
+
+module.exports.include = () => [
+    'import putoutTest from "@putout/test"',
+];
+
+module.exports.fix = (path) => {
+    path.scope.rename('putoutTest', 'createTest');
+};
+

package/lib/index.js

@@ -6,6 +6,8 @@ const getRule = (a) => ({
 
 module.exports.rules = {
     ...getRule('apply-processors-destructuring'),
+    ...getRule('apply-async-formatter'),
+    ...getRule('convert-putout-test-to-create-test'),
     ...getRule('convert-to-no-transform-code'),
     ...getRule('convert-find-to-traverse'),
     ...getRule('convert-replace-with'),
@@ -17,11 +19,12 @@ module.exports.rules = {
     ...getRule('convert-traverse-to-replace'),
     ...getRule('convert-process-to-find'),
     ...getRule('convert-method-to-property'),
+    ...getRule('convert-add-argument-to-add-args'),
     ...getRule('rename-operate-to-operator'),
     ...getRule('replace-operate-with-operator'),
     ...getRule('shorten-imports'),
     ...getRule('check-replace-code'),
     ...getRule('declare'),
-    ...getRule('add-missing-argument'),
+    ...getRule('add-args'),
 };
 

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "7.12.0",
+  "version": "8.2.1",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "putout plugin helps with plugins development",
-  "homepage": "http://github.com/coderaiser/putout",
+  "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-putout",
   "main": "lib/index.js",
   "release": false,
   "tag": false,
@@ -13,6 +13,7 @@
     "url": "git://github.com/coderaiser/putout.git"
   },
   "scripts": {
+    "wisdom": "madrun wisdom",
     "test": "madrun test",
     "watch:test": "madrun watch:test",
     "lint": "madrun lint",
@@ -33,17 +34,17 @@
     "putout"
   ],
   "devDependencies": {
-    "@putout/test": "^3.0.0",
+    "@putout/test": "^4.0.0",
     "c8": "^7.5.0",
-    "eslint": "^8.0.0-beta.0",
+    "eslint": "^8.0.1",
     "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-putout": "^10.0.0",
+    "eslint-plugin-putout": "^12.0.0",
     "lerna": "^4.0.0",
     "madrun": "^8.0.1",
     "nodemon": "^2.0.1"
   },
   "peerDependencies": {
-    "putout": ">=20.10"
+    "putout": ">=20.13"
   },
   "license": "MIT",
   "engines": {