@putout/plugin-putout 14.2.0 → 14.4.0

package/README.md

@@ -46,6 +46,7 @@ npm i @putout/plugin-putout -D
         "putout/convert-dirname-to-url": "on",
         "putout/convert-url-to-dirname": "on",
         "putout/convert-report-to-function": "on",
+        "putout/convert-get-rule-to-require": "on",
         "putout/create-test": "on",
         "putout/shorten-imports": "on",
         "putout/declare": "on",
@@ -711,7 +712,7 @@ const test = createTest(import.meta.url, {
 });
 ```
 
-## convert-url-to-dirname-
+## convert-url-to-dirname
 
 ### ❌ Example of incorrect code
 
@@ -749,6 +750,32 @@ module.exports.report = `'report' should be a 'function'`;
 module.exports.report = () => `'report' should be a 'function'`;
 ```
 
+## convert-get-rule-to-require
+
+- ✅ import [Nested plugins](https://github.com/coderaiser/putout/tree/master/packages/engine-loader#nested-plugin) in [**Deno** and **Browser**](https://github.com/putoutjs/bundle/);
+- ✅ easier bundle with rollup without [`dynamicRequireTargets`](https://github.com/rollup/plugins/tree/master/packages/commonjs/#dynamicrequiretargets);
+- ✅ easier to migrate to **ESM**;
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/38336fcc5b5ae6e441697e098067319c/dd98578c9554b7bd5dceee0499118f7d8216e1da).
+
+### ❌ Example of incorrect code
+
+```js
+module.exports.rules = {
+    ...getRule('remove-unused-variables'),
+};
+```
+
+### ✅ Example of correct code
+
+```js
+const removeUnusedVariables = require('./remove-unused-variables');
+
+module.exports.rules = {
+    'remove-unused-variables': removeUnusedVariables,
+};
+```
+
 ## move-require-on-top-level
 
 ### ❌ Example of incorrect code
@@ -776,10 +803,9 @@ const test = require('@putout/test')(__dirname, {
 });
 
 test('remove debugger: report', (t) => {
-    const test = require('@putout/test')(__dirname, {
+    t.transform('debugger', {
         'remove-debugger': removeDebugger,
     });
-    
     t.end();
 });
 ```

package/lib/apply-namaspace-specifier/index.js

@@ -6,7 +6,31 @@ module.exports.fix = ({first}) => {
     first.node.type = 'ImportNamespaceSpecifier';
 };
 
-module.exports.traverse = ({push}) => ({
+module.exports.traverse = ({push, listStore}) => ({
+    'export const rules = __object': listStore,
+    ...createImportVisitor({
+        push,
+        names: [
+            './index.js',
+            '../lib/index.js',
+        ],
+    }),
+    'Program': {
+        exit(path) {
+            const rules = listStore();
+            
+            if (!rules.length)
+                return;
+            
+            path.traverse(createImportVisitor({
+                push,
+                names: ['any'],
+            }));
+        },
+    },
+});
+
+const createImportVisitor = ({push, names}) => ({
     ImportDeclaration(path) {
         const {value} = path.node.source;
         const first = path.get('specifiers.0');
@@ -17,20 +41,14 @@ module.exports.traverse = ({push}) => ({
         if (first.isImportNamespaceSpecifier())
             return;
         
-        if (value === './index.js') {
-            push({
-                path,
-                first,
-            });
-            return;
-        }
-        
-        if (value === '../lib/index.js') {
-            push({
-                path,
-                first,
-            });
-            return;
+        for (const name of names) {
+            if (value === name || name === 'any') {
+                push({
+                    path,
+                    first,
+                });
+                return;
+            }
         }
     },
 });

package/lib/convert-get-rule-to-require/index.js

@@ -0,0 +1,78 @@
+'use strict';
+
+const camel = require('just-camel-case');
+const {
+    types,
+    operator,
+    template,
+} = require('putout');
+
+const {
+    replaceWith,
+    insertBefore,
+    compare,
+} = operator;
+
+const {
+    ObjectProperty,
+    Identifier,
+    StringLiteral,
+} = types;
+
+module.exports.report = () => `Use top-level 'require()' instead of '...getRule()'`;
+
+const REQUIRE = `const __a = require(__b)`;
+const createRequire = template(REQUIRE, {
+    placeholderPattern: /__/,
+});
+
+module.exports.match = () => ({
+    'getRule(__a)': ({__a}, path) => {
+        const name = __a.value;
+        
+        if (!path.parentPath.isSpreadElement())
+            return false;
+        
+        return !path.scope.getAllBindings()[name];
+    },
+});
+
+module.exports.replace = () => ({
+    'getRule(__a)': ({__a}, path) => {
+        const name = camel(__a.value);
+        const id = Identifier(name);
+        const node = ObjectProperty(__a, id);
+        
+        replaceWith(path.parentPath, node);
+        
+        const programPath = path.scope.getProgramParent().path;
+        const body = programPath.get('body');
+        const [first] = body;
+        
+        const nodeRequire = createRequire({
+            __a: id,
+            __b: StringLiteral(`./${__a.value}`),
+        });
+        
+        if (compare(first, REQUIRE)) {
+            const latest = getLatest(body.slice(1));
+            insertBefore(latest, nodeRequire);
+            
+            return path;
+        }
+        
+        programPath.node.body.unshift(nodeRequire);
+        
+        return path;
+    },
+});
+
+function getLatest(body) {
+    let path;
+    
+    for (path of body)
+        if (!compare(path, REQUIRE))
+            break;
+    
+    return path;
+}

package/lib/declare/index.js

@@ -3,9 +3,14 @@
 const types = require('./types');
 const operator = require('./operator');
 
+const getRule = (a) => ({
+    [a]: require(`./${a}`),
+});
+
 module.exports.declare = () => ({
     template: `import {template} from 'putout'`,
     createTest: `import {createTest} from '@putout/test'`,
     ...operator,
     ...types,
+    getRule: `const getRule = ${getRule.toString()};`,
 });

package/lib/index.js

@@ -1,47 +1,85 @@
 'use strict';
 
-const getRule = (a) => ({
-    [a]: require(`./${a}`),
-});
+const applyProcessorsDestructuring = require('./apply-processors-destructuring');
+const applyAsyncFormatter = require('./apply-async-formatter');
+const applyCreateTest = require('./apply-create-test');
+const applyRemove = require('./apply-remove');
+const applyInsertBefore = require('./apply-insert-before');
+const applyInsertAfter = require('./apply-insert-after');
+const applyDeclare = require('./apply-declare');
+const checkReplaceCode = require('./check-replace-code');
+const checkMatch = require('./check-match');
+const convertPutoutTestToCreateTest = require('./convert-putout-test-to-create-test');
+const convertToNoTransformCode = require('./convert-to-no-transform-code');
+const convertFindToTraverse = require('./convert-find-to-traverse');
+const convertDestructuringToIdentifier = require('./convert-destructuring-to-identifier');
+const convertNumberToNumeric = require('./convert-number-to-numeric');
+const convertReplaceWith = require('./convert-replace-with');
+const convertReplaceWithMultiple = require('./convert-replace-with-multiple');
+const convertReplaceToFunction = require('./convert-replace-to-function');
+const convertMatchToFunction = require('./convert-match-to-function');
+const convertBabelTypes = require('./convert-babel-types');
+const convertNodeToPathInGetTemplateValues = require('./convert-node-to-path-in-get-template-values');
+const convertTraverseToInclude = require('./convert-traverse-to-include');
+const convertTraverseToReplace = require('./convert-traverse-to-replace');
+const convertProcessToFind = require('./convert-process-to-find');
+const convertMethodToProperty = require('./convert-method-to-property');
+const convertAddArgumentToAddArgs = require('./convert-add-argument-to-add-args');
+const convertDirnameToUrl = require('./convert-dirname-to-url');
+const convertUrlToDirname = require('./convert-url-to-dirname');
+const convertReportToFunction = require('./convert-report-to-function');
+const replaceTestMessage = require('./replace-test-message');
+const renameOperateToOperator = require('./rename-operate-to-operator');
+const replaceOperateWithOperator = require('./replace-operate-with-operator');
+const shortenImports = require('./shorten-imports');
+const declare = require('./declare');
+const addArgs = require('./add-args');
+const addPush = require('./add-push');
+const moveRequireOnTopLevel = require('./move-require-on-top-level');
+const includer = require('./includer');
+const createTest = require('./create-test');
+const applyNamaspaceSpecifier = require('./apply-namaspace-specifier');
+const convertGetRuleToRequire = require('./convert-get-rule-to-require');
 
 module.exports.rules = {
-    ...getRule('apply-processors-destructuring'),
-    ...getRule('apply-async-formatter'),
-    ...getRule('apply-create-test'),
-    ...getRule('apply-remove'),
-    ...getRule('apply-insert-before'),
-    ...getRule('apply-insert-after'),
-    ...getRule('apply-declare'),
-    ...getRule('check-replace-code'),
-    ...getRule('check-match'),
-    ...getRule('convert-putout-test-to-create-test'),
-    ...getRule('convert-to-no-transform-code'),
-    ...getRule('convert-find-to-traverse'),
-    ...getRule('convert-destructuring-to-identifier'),
-    ...getRule('convert-number-to-numeric'),
-    ...getRule('convert-replace-with'),
-    ...getRule('convert-replace-with-multiple'),
-    ...getRule('convert-replace-to-function'),
-    ...getRule('convert-match-to-function'),
-    ...getRule('convert-babel-types'),
-    ...getRule('convert-node-to-path-in-get-template-values'),
-    ...getRule('convert-traverse-to-include'),
-    ...getRule('convert-traverse-to-replace'),
-    ...getRule('convert-process-to-find'),
-    ...getRule('convert-method-to-property'),
-    ...getRule('convert-add-argument-to-add-args'),
-    ...getRule('convert-dirname-to-url'),
-    ...getRule('convert-url-to-dirname'),
-    ...getRule('convert-report-to-function'),
-    ...getRule('replace-test-message'),
-    ...getRule('rename-operate-to-operator'),
-    ...getRule('replace-operate-with-operator'),
-    ...getRule('shorten-imports'),
-    ...getRule('declare'),
-    ...getRule('add-args'),
-    ...getRule('add-push'),
-    ...getRule('move-require-on-top-level'),
-    ...getRule('includer'),
-    ...getRule('create-test'),
-    ...getRule('apply-namaspace-specifier'),
+    'apply-processors-destructuring': applyProcessorsDestructuring,
+    'apply-async-formatter': applyAsyncFormatter,
+    'apply-create-test': applyCreateTest,
+    'apply-remove': applyRemove,
+    'apply-insert-before': applyInsertBefore,
+    'apply-insert-after': applyInsertAfter,
+    'apply-declare': applyDeclare,
+    'check-replace-code': checkReplaceCode,
+    'check-match': checkMatch,
+    'convert-putout-test-to-create-test': convertPutoutTestToCreateTest,
+    'convert-to-no-transform-code': convertToNoTransformCode,
+    'convert-find-to-traverse': convertFindToTraverse,
+    'convert-destructuring-to-identifier': convertDestructuringToIdentifier,
+    'convert-number-to-numeric': convertNumberToNumeric,
+    'convert-replace-with': convertReplaceWith,
+    'convert-replace-with-multiple': convertReplaceWithMultiple,
+    'convert-replace-to-function': convertReplaceToFunction,
+    'convert-match-to-function': convertMatchToFunction,
+    'convert-babel-types': convertBabelTypes,
+    'convert-node-to-path-in-get-template-values': convertNodeToPathInGetTemplateValues,
+    'convert-traverse-to-include': convertTraverseToInclude,
+    'convert-traverse-to-replace': convertTraverseToReplace,
+    'convert-process-to-find': convertProcessToFind,
+    'convert-method-to-property': convertMethodToProperty,
+    'convert-add-argument-to-add-args': convertAddArgumentToAddArgs,
+    'convert-dirname-to-url': convertDirnameToUrl,
+    'convert-url-to-dirname': convertUrlToDirname,
+    'convert-report-to-function': convertReportToFunction,
+    'replace-test-message': replaceTestMessage,
+    'rename-operate-to-operator': renameOperateToOperator,
+    'replace-operate-with-operator': replaceOperateWithOperator,
+    'shorten-imports': shortenImports,
+    declare,
+    'add-args': addArgs,
+    'add-push': addPush,
+    'move-require-on-top-level': moveRequireOnTopLevel,
+    includer,
+    'create-test': createTest,
+    'apply-namaspace-specifier': applyNamaspaceSpecifier,
+    'convert-get-rule-to-require': convertGetRuleToRequire,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "14.2.0",
+  "version": "14.4.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with plugins development",