@putout/plugin-putout 11.14.0 → 12.0.1

package/README.md

@@ -20,6 +20,7 @@ npm i @putout/plugin-putout -D
         "putout/apply-processors-destructuring": "on",
         "putout/apply-async-formatter": "on",
         "putout/apply-remove": "on",
+        "putout/apply-declare": "on",
         "putout/add-args": "on",
         "putout/add-push": "on",
         "putout/convert-putout-test-to-create-test": "on",
@@ -93,6 +94,32 @@ export const fix = (path) => {
 };
 ```
 
+## apply-declare
+
+Better to use [`Declareator`](https://github.com/coderaiser/putout/tree/master/packages/engine-runner#declarator) instead of `operator.declare()`.
+Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/cf35de5e80e8f7aad866358a50c5eded/0af6142fc9c9e71ac2a2aa96cb85613dd95c9fbf).
+
+### ❌ Example of incorrect code
+
+```js
+const {operator} = require('putout');
+const {declare} = operator;
+
+module.exports = declare({
+    tryCatch: `import tryCatch from 'try-catch'`,
+    tryToCatch: `import tryToCatch from 'try-to-catch'`,
+});
+```
+
+### ✅ Example of correct code
+
+```js
+module.exports.declare = () => ({
+    tryCatch: `import tryCatch from 'try-catch'`,
+    tryToCatch: `import tryToCatch from 'try-to-catch'`,
+});
+```
+
 ## apply-async-formatter
 
 ### ❌ Example of incorrect code
@@ -444,7 +471,7 @@ This is additional tests, if you forget to test some case (from a big list of ru
 ## declare
 
 Depends on [@putout/convert-esm-to-commonjs](https://github.com/coderaiser/putout/tree/master/packages/plugin-convert-esm-to-commonjs#readme) and
-[@putout/declare-undefined-variables](https://github.com/coderaiser/putout/tree/master/packages/plugin-declare-undefined-variables#readme).
+[@putout/declare](https://github.com/coderaiser/putout/tree/master/packages/plugin-declare#readme).
 
 ### ❌ Example of incorrect code
 

package/lib/apply-declare/index.js

@@ -0,0 +1,8 @@
+'use strict';
+
+module.exports.report = () => `Use 'Declarator' instead of 'operator.declare()'`;
+
+module.exports.replace = () => ({
+    'module.exports = declare(__a)': 'module.exports.declare = () => __a',
+});
+

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

@@ -3,6 +3,8 @@
 const putout = require('putout');
 const tryCatch = require('try-catch');
 
+const noop = () => {};
+
 const {types, operator} = putout;
 const {replaceWith} = operator;
 const {
@@ -19,7 +21,7 @@ module.exports = (rootPath, key) => {
         isTS: true,
         plugins: [
             ['generate', {
-                report: () => {},
+                report: noop,
                 include: () => [
                     'Identifier',
                     'StringLiteral',

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

@@ -5,8 +5,9 @@ const tryCatch = require('try-catch');
 
 const generateCode = require('./generate-code');
 
-const {operator} = putout;
+const noop = () => {};
 
+const {operator} = putout;
 const {
     compare,
     extract,
@@ -80,7 +81,7 @@ module.exports.traverse = ({push}) => ({
                 isTS: true,
                 plugins: [
                     ['evaluate', {
-                        report: () => {},
+                        report: noop,
                         replace: () => ({
                             [key]: template,
                         }),

package/lib/declare/index.js

@@ -1,8 +1,12 @@
 'use strict';
 
-const {operator} = require('putout');
-const declarations = require('./declarations');
-const {declare} = operator;
+const types = require('./types');
+const operator = require('./operator');
 
-module.exports = declare(declarations);
+module.exports.declare = () => ({
+    template: `import {template} from 'putout'`,
+    createTest: `import {createTest} from '@putout/test'`,
+    ...operator,
+    ...types,
+});
 

package/lib/index.js

@@ -9,6 +9,7 @@ module.exports.rules = {
     ...getRule('apply-async-formatter'),
     ...getRule('apply-create-test'),
     ...getRule('apply-remove'),
+    ...getRule('apply-declare'),
     ...getRule('convert-putout-test-to-create-test'),
     ...getRule('convert-to-no-transform-code'),
     ...getRule('convert-find-to-traverse'),

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

@@ -22,13 +22,14 @@ const {
     Identifier,
     isIdentifier,
     isObjectExpression,
+    isMemberExpression,
 } = types;
 
 module.exports.report = () => 'Move require on top level';
 
 module.exports.match = () => ({
     [TEST]: ({__b}) => !isIdentifier(__b),
-    [TRANSFORM]: ({__b}) => !isIdentifier(__b) && !isObjectExpression(__b),
+    [TRANSFORM]: ({__b}) => !isIdentifier(__b) && !isObjectExpression(__b) && !isMemberExpression(__b),
 });
 
 module.exports.replace = () => ({

package/lib/replace-test-message/index.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const {types} = require('putout');
-
 const {isCallExpression} = types;
 
 module.exports.report = ({correct, operatorPath}) => {

package/package.json

@@ -1,12 +1,11 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "11.14.0",
+  "version": "12.0.1",
   "type": "commonjs",
   "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#readme",
   "main": "lib/index.js",
-  "commitType": "colon",
   "release": false,
   "tag": false,
   "changelog": false,
@@ -37,7 +36,7 @@
     "putout"
   ],
   "devDependencies": {
-    "@putout/test": "^5.0.0",
+    "@putout/test": "^6.0.0",
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
     "eslint-plugin-n": "^15.2.4",
@@ -48,7 +47,7 @@
     "nodemon": "^2.0.1"
   },
   "peerDependencies": {
-    "putout": ">=25"
+    "putout": ">=29"
   },
   "license": "MIT",
   "engines": {

package/lib/declare/declarations.js

@@ -1,12 +0,0 @@
-'use strict';
-
-const types = require('./types');
-const operator = require('./operator');
-
-module.exports = {
-    template: `import {template} from 'putout'`,
-    createTest: `import {createTest} from '@putout/test'`,
-    ...operator,
-    ...types,
-};
-