@putout/plugin-putout 8.0.0 → 8.3.0

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/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",
@@ -33,7 +33,8 @@ npm i @putout/plugin-putout -D
         "putout/convert-add-argument-to-add-args": "on",
         "putout/shorten-imports": "on",
         "putout/check-replace-code": "on",
-        "putout/declare": "on"
+        "putout/declare": "on",
+        "putout/move-require-on-top-level": "on"
     }
 }
 ```
@@ -57,6 +58,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
@@ -154,7 +198,6 @@ module.exports.replace = () => ({
     'const __a = __b': ({}) => {
     },
     'const __c = __d': ({}, path) => {
-    
     },
 });
 ```
@@ -287,7 +330,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
 
@@ -352,6 +396,38 @@ module.exports = addArgs({
 });
 ```
 
+## move-require-on-top-level
+
+### ❌ Incorrect code example
+
+```js
+const test = require('@putout/test')(__dirname, {
+    'remove-debugger': require('..'),
+});
+
+test('remove debugger: report', (t) => {
+    t.transform('debugger', {
+        'remove-debugger': require('..'),
+    });
+    t.end();
+});
+```
+
+### ✅ Correct code Example
+
+```js
+const removeDebugger = require('..');
+const test = require('@putout/test')(__dirname, {
+    'remove-debugger': removeDebugger,
+});
+
+test('remove debugger: report', (t) => {
+    const test = require('@putout/test')(__dirname, {
+        'remove-debugger': removeDebugger,
+    });
+    t.end();
+});
+```
 
 ## License
 

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/check-replace-code/index.js

@@ -48,6 +48,10 @@ module.exports.traverse = ({push}) => ({
                 continue;
             
             const {node} = propertyPath;
+            
+            if (node.computed)
+                return false;
+            
             const key = extract(node.key);
             const template = extract(node.value);
             const [generateError, keyCode] = generateCode(path, key);

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'),
@@ -24,5 +26,6 @@ module.exports.rules = {
     ...getRule('check-replace-code'),
     ...getRule('declare'),
     ...getRule('add-args'),
+    ...getRule('move-require-on-top-level'),
 };
 

package/lib/move-require-on-top-level/index.js

@@ -0,0 +1,76 @@
+'use strict';
+
+const justCamelCase = require('just-camel-case');
+const {
+    types,
+    template,
+} = require('putout');
+
+const TEST = `
+    const test = require('@putout/test')(__dirname, {
+        __a: __b
+    });
+`;
+
+const TRANSFORM = `
+    t.transform(__c, {
+        __a: __b
+    });
+`;
+
+const {
+    Identifier,
+    isIdentifier,
+} = types;
+
+module.exports.report = () => 'Move require on top level';
+
+module.exports.match = () => ({
+    [TEST]: ({__b}) => !isIdentifier(__b),
+    [TRANSFORM]: ({__b}) => !isIdentifier(__b),
+});
+
+module.exports.replace = () => ({
+    [TEST]: (vars, path) => {
+        const name = declareRequire(vars, path);
+        const {__a} = vars;
+        const value = __a.value || __a.name;
+        
+        return `
+            const test = require('@putout/test')(__dirname, {
+                '${value}': ${name},
+            });
+        `;
+    },
+    [TRANSFORM]: (vars, path) => {
+        const name = declareRequire(vars, path);
+        const {__a} = vars;
+        const value = __a.value || __a.name;
+        
+        return `
+            t.transform(__c, {
+                '${value}': ${name},
+            });
+    `;
+    },
+});
+
+const buildRequire = template(`const NAME = REQUIRE`);
+
+function declareRequire({__a, __b}, path) {
+    const shortName = (__a.value || __a.name).split('/').pop();
+    const name = justCamelCase(shortName);
+    const requireNode = buildRequire({
+        NAME: Identifier(name),
+        REQUIRE: __b,
+    });
+    
+    if (path.scope.hasBinding(name))
+        return name;
+    
+    const programPath = path.scope.getProgramParent().path;
+    programPath.node.body.unshift(requireNode);
+    
+    return name;
+}
+

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "8.0.0",
+  "version": "8.3.0",
   "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",
@@ -24,6 +25,7 @@
   },
   "dependencies": {
     "fullstore": "^3.0.0",
+    "just-camel-case": "^6.0.1",
     "try-catch": "^3.0.0"
   },
   "keywords": [
@@ -33,11 +35,11 @@
     "putout"
   ],
   "devDependencies": {
-    "@putout/test": "^3.0.0",
+    "@putout/test": "^4.0.0",
     "c8": "^7.5.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"