@putout/plugin-putout 18.4.1 → 18.6.0

package/README.md

@@ -25,7 +25,8 @@ npm i @putout/plugin-putout -D
         "putout/apply-insert-before": "on",
         "putout/apply-insert-after": "on",
         "putout/apply-short-processors": "on",
-        "putout/apply-namaspace-specifier": "on",
+        "putout/apply-namespace-specifier": "on",
+        "putout/apply-for-of-to-track-file": "on",
         "putout/add-args": "on",
         "putout/add-push": "on",
         "putout/add-track-file": "on",
@@ -276,22 +277,32 @@ const test = createTest({
 });
 ```
 
-## apply-namaspace-specifier
+## apply-for-of-to-track-file
 
-Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/c8ccc07044299e463807773705aec583/71722e57f2b0b03eb8300ab6fe268251dc696629).
+> The **Generator** object is returned by a `generator function` and it conforms to both the iterable protocol and the `iterator` protocol.
+>
+> (c) [MDN](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Generator)
+
+`trackFile` is generator function used to count progress that can be used in [**Scanner**](https://github.com/coderaiser/putout/tree/master/packages/engine-runner#scanner).
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/5ec6c384aa6e4471aab17efb86e185f3/e749c7d649ef2da76410a5964e7c875d10538789)
 
 ### ❌ Example of incorrect code
 
 ```js
-import a from './index.js';
-import b from '../lib/index.js';
+module.exports.scan = (path, {push, trackFile}) => {
+    trackFile(path, '*.swp').map(push);
+};
 ```
 
 ### ✅ Example of correct code
 
 ```js
-import * as a from './index.js';
-import * as b from '../lib/index.js';
+module.exports.scan = (path, {push, trackFile}) => {
+    for (const file of trackFile(path, '*.swp')) {
+        push(file);
+    }
+};
 ```
 
 ## create-test

package/lib/apply-for-of-to-track-file/index.js

@@ -0,0 +1,7 @@
+'use strict';
+
+module.exports.report = () => `Use 'if condition' instead of 'ternary expression'`;
+
+module.exports.replace = () => ({
+    'trackFile(__a, __b).map(push)': 'for (const file of trackFile( __a, __b)) {push(file)}',
+});

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

@@ -9,7 +9,6 @@ const {replaceWith} = operator;
 
 const {
     ArrayPattern,
-    ObjectPattern,
     BlockStatement,
     ObjectExpression,
 } = types;
@@ -54,17 +53,11 @@ module.exports = (rootPath, key) => {
                     }
                     
                     if (name === '__array') {
-                        if (path.parentPath.isVariableDeclarator())
-                            return replaceWith(path, ArrayPattern([]));
-                        
                         if (path.parentPath.isCallExpression())
                             return replaceWith(path, ArrayPattern([]));
                         
                         if (path.parentPath.isFunction())
                             return replaceWith(path, ArrayPattern([]));
-                        
-                        if (path.parentPath.isAssignmentExpression())
-                            return replaceWith(path, ArrayPattern([]));
                     }
                     
                     if (name === '__object')
@@ -98,11 +91,14 @@ function createVarStore(path) {
 
 function objectify(path) {
     const {parentPath} = path;
-    const isVar = parentPath.isVariableDeclarator();
+    
     const isAssign = parentPath.isAssignmentExpression();
     
-    if (isVar && parentPath.get('id') === path)
-        return replaceWith(path, ObjectPattern([]));
+    if (path.parentPath.isExportDeclaration())
+        return replaceWith(path, ObjectExpression([]));
+    
+    if (path.parentPath.isCallExpression())
+        return replaceWith(path, ObjectExpression([]));
     
     if (isAssign && parentPath.get('right') === path)
         return replaceWith(path, ObjectExpression([]));

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

@@ -9,6 +9,7 @@ const {
     compare,
     extract,
     compute,
+    __json,
 } = operator;
 
 const name = '__putout_plugin_check_replace_code';
@@ -45,10 +46,11 @@ module.exports.traverse = ({push}) => ({
             return;
         
         for (const propertyPath of path.get('right.body.properties')) {
-            if (!propertyPath.get('value').isStringLiteral())
+            const template = extractValue(propertyPath);
+            
+            if (!template)
                 continue;
             
-            const {node} = propertyPath;
             const [parseError, key] = parseKey(propertyPath);
             
             if (parseError) {
@@ -61,7 +63,6 @@ module.exports.traverse = ({push}) => ({
                 return;
             }
             
-            const template = extract(node.value);
             const [generateError, keyCode] = generateCode(path, key);
             
             if (generateError) {
@@ -113,6 +114,10 @@ module.exports.traverse = ({push}) => ({
 
 function parseKey(propertyPath) {
     const keyPath = propertyPath.get('key');
+    
+    if (keyPath.isIdentifier({name: '__json'}))
+        return [null, __json];
+    
     const [isComputed, key] = compute(keyPath);
     
     if (!isComputed)
@@ -133,3 +138,15 @@ function hasMatch(path) {
     
     return false;
 }
+
+function extractValue(path) {
+    const valuePath = path.get('value');
+    
+    if (valuePath.isIdentifier({name: '__json'}))
+        return __json;
+    
+    if (valuePath.isStringLiteral())
+        return valuePath.node.value;
+    
+    return null;
+}

package/lib/index.js

@@ -39,7 +39,7 @@ 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 applyNamespaceSpecifier = require('./apply-namespace-specifier');
 const convertGetRuleToRequire = require('./convert-get-rule-to-require');
 const addIndexToImport = require('./add-index-to-import');
 const applyRename = require('./apply-rename');
@@ -47,6 +47,7 @@ const applyShortProcessors = require('./apply-short-processors');
 const addTrackFile = require('./add-track-file');
 const convertProgressToTrackFile = require('./convert-progress-to-track-file');
 const addAwaitToProgress = require('./add-await-to-progress');
+const applyForOfToTrackFile = require('./apply-for-of-to-track-file');
 
 module.exports.rules = {
     'apply-processors-destructuring': applyProcessorsDestructuring,
@@ -87,7 +88,7 @@ module.exports.rules = {
     'move-require-on-top-level': moveRequireOnTopLevel,
     includer,
     'create-test': createTest,
-    'apply-namaspace-specifier': applyNamaspaceSpecifier,
+    'apply-namespace-specifier': applyNamespaceSpecifier,
     'convert-get-rule-to-require': convertGetRuleToRequire,
     'add-index-to-import': addIndexToImport,
     'apply-rename': applyRename,
@@ -96,4 +97,5 @@ module.exports.rules = {
     'add-track-file': addTrackFile,
     'convert-progress-to-track-file': convertProgressToTrackFile,
     'add-await-to-progress': addAwaitToProgress,
+    'apply-for-of-to-track-file': applyForOfToTrackFile,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "18.4.1",
+  "version": "18.6.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with plugins development",
@@ -39,8 +39,8 @@
   "devDependencies": {
     "@putout/plugin-tape": "*",
     "@putout/test": "^8.0.0",
-    "c8": "^8.0.0",
-    "eslint": "^8.0.1",
+    "c8": "^9.0.0",
+    "eslint": "^9.0.0-alpha.0",
     "eslint-plugin-n": "^16.0.0",
     "eslint-plugin-putout": "^22.0.0",
     "lerna": "^6.0.1",