@putout/plugin-putout 8.2.1 → 8.5.1

package/README.md

@@ -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"
     }
 }
 ```
@@ -395,6 +396,39 @@ 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
 
 MIT

package/lib/check-replace-code/index.js

@@ -10,6 +10,7 @@ const {operator} = putout;
 const {
     compare,
     extract,
+    compute,
 } = operator;
 
 const name = '__putout_plugin_check_replace_code';
@@ -28,7 +29,7 @@ module.exports.report = ({path, code, error}) => {
     if (error)
         return error.message;
     
-    const key = extract(path.node.key);
+    const [, key] = parseKey(path);
     const value = extract(path.node.value);
     
     return `transform mismatch: "${key}" -> "${value}" !== "${code}"`;
@@ -48,7 +49,17 @@ module.exports.traverse = ({push}) => ({
                 continue;
             
             const {node} = propertyPath;
-            const key = extract(node.key);
+            const [parseError, key] = parseKey(propertyPath);
+            
+            if (parseError) {
+                push({
+                    error: parseError,
+                    mainPath: path,
+                    path: propertyPath,
+                });
+                return;
+            }
+            
             const template = extract(node.value);
             const [generateError, keyCode] = generateCode(path, key);
             
@@ -96,3 +107,13 @@ module.exports.traverse = ({push}) => ({
     },
 });
 
+function parseKey(propertyPath) {
+    const keyPath = propertyPath.get('key');
+    const [isComputed, key] = compute(keyPath);
+    
+    if (!isComputed)
+        return [Error(`Replace key cannot be computed: '${keyPath.toString()}'`)];
+    
+    return [null, key];
+}
+

package/lib/index.js

@@ -26,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,6 +1,6 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "8.2.1",
+  "version": "8.5.1",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "putout plugin helps with plugins development",
   "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-putout",
@@ -25,6 +25,7 @@
   },
   "dependencies": {
     "fullstore": "^3.0.0",
+    "just-camel-case": "^6.0.1",
     "try-catch": "^3.0.0"
   },
   "keywords": [