@putout/plugin-putout 25.1.0 → 25.3.0

package/README.md

@@ -493,7 +493,7 @@ export default createRenameProperty([
 ### ✅ Example of correct code
 
 ```js
-const {
+export const {
     report,
     fix,
     scan,
@@ -501,12 +501,6 @@ const {
     ...v32,
     ...v29,
 ]);
-
-export {
-    report,
-    fix,
-    scan,
-};
 ```
 
 ## apply-exports-to-add-args
@@ -522,17 +516,11 @@ export default addArgs(__args);
 ### ✅ Example of correct code
 
 ```js
-const {
+export const {
     report,
     fix,
     traverse,
 } = addArgs(__args);
-
-export {
-    report,
-    fix,
-    traverse,
-};
 ```
 
 ## apply-exports-to-match-files
@@ -550,19 +538,13 @@ export default matchFiles({
 ### ✅ Example of correct code
 
 ```js
-const {
+export const {
     report,
     fix,
     scan,
 } = matchFiles({
     '*.cjs': plugin,
 });
-
-export {
-    report,
-    fix,
-    scan,
-};
 ```
 
 ## apply-exports-to-rename-files
@@ -584,7 +566,7 @@ export default renameFiles({
 ### ✅ Example of correct code
 
 ```js
-const {
+export const {
     report,
     fix,
     scan,
@@ -595,12 +577,6 @@ const {
         return name.replace(/mjs$/, 'js');
     },
 });
-
-export {
-    report,
-    fix,
-    scan,
-};
 ```
 
 ## apply-async-formatter
@@ -660,6 +636,22 @@ const dirPath = createNestedDirectory(path, '/hello/world');
 const dirPath2 = createDirectory(path, 'world');
 ```
 
+## apply-namespace-specifier
+
+Most likely what you need is [`esm/resolve-imported-file`](https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#resolve-imported-file).
+
+### ❌ Example of incorrect code
+
+```js
+import rmUnused from '@putout/plugin-remove-unused-variables';
+```
+
+### ✅ Example of correct code
+
+```js
+import * as rmUnused from '@putout/plugin-remove-unused-variables';
+```
+
 ## apply-for-of-to-track-file
 
 > The **Generator** object is returned by a `generator function` and it conforms to both the iterable protocol and the `iterator` protocol.

package/lib/add-test-args/index.js

@@ -2,7 +2,7 @@ import {operator} from 'putout';
 
 const {addArgs} = operator;
 
-const {
+export const {
     report,
     fix,
     traverse,
@@ -28,9 +28,3 @@ const {
         'test.only("__a", async (__args) => __body)',
     ]],
 });
-
-export {
-    report,
-    fix,
-    traverse,
-};

package/lib/add-track-file/index.js

@@ -2,7 +2,7 @@ import {operator} from 'putout';
 
 const {addArgs} = operator;
 
-const {
+export const {
     report,
     fix,
     traverse,
@@ -12,9 +12,3 @@ const {
         '(__a) => __body',
     ]],
 });
-
-export {
-    report,
-    fix,
-    traverse,
-};

package/lib/apply-exports/index.js

@@ -1,8 +1,3 @@
 import {createApplyExports} from '../create-apply-exports.js';
 
-const {report, replace} = createApplyExports();
-
-export {
-    report,
-    replace,
-};
+export const {report, replace} = createApplyExports();

package/lib/apply-exports-to-add-args/index.js

@@ -1,14 +1,9 @@
 import {createApplyExports} from '../create-apply-exports.js';
 
-const {report, replace} = createApplyExports({
+export const {report, replace} = createApplyExports({
     addArgs: [
         'report',
         'fix',
         'traverse',
     ],
 });
-
-export {
-    report,
-    replace,
-};

package/lib/apply-exports-to-match-files/index.js

@@ -1,14 +1,9 @@
 import {createApplyExports} from '../create-apply-exports.js';
 
-const {report, replace} = createApplyExports({
+export const {report, replace} = createApplyExports({
     matchFiles: [
         'report',
         'fix',
         'scan',
     ],
 });
-
-export {
-    report,
-    replace,
-};

package/lib/apply-exports-to-rename-files/index.js

@@ -1,14 +1,9 @@
 import {createApplyExports} from '../create-apply-exports.js';
 
-const {report, replace} = createApplyExports({
+export const {report, replace} = createApplyExports({
     renameFiles: [
         'report',
         'fix',
         'scan',
     ],
 });
-
-export {
-    report,
-    replace,
-};

package/lib/apply-fixture-name-to-message/index.js

@@ -14,12 +14,14 @@ const NAMES = [
     ...FIXTURE,
     'no report',
     'no transform',
+    'no report after transform',
 ];
 
 export const report = () => `Apply 'fixture' name to 'message'`;
 
 export const match = () => ({
     't.noReport(__a)': check,
+    't.noReportAfterTransform(__a)': check,
     't.report(__a, __b)': check,
     't.transform(__a)': check,
     't.transform(__a, __b)': check,
@@ -32,6 +34,7 @@ export const replace = () => ({
     't.transform(__a)': transform,
     't.transform(__a, __b)': transform,
     't.noTransform(__a)': transform,
+    't.noReportAfterTransform(__a)': transform,
 });
 
 const isTest = (path) => compare(path, 'test(__a, (t) => __body)', {

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

@@ -19,8 +19,6 @@ export const traverse = ({push, listStore, pathStore}) => ({
         names: [
             '@putout/plugin-',
             './index.js',
-            '../lib/index.js',
-            '../',
         ],
     }),
     'Program': {
@@ -66,6 +64,10 @@ const createImportVisitor = ({push, names, pathStore = noop}) => ({
             return;
         
         for (const name of names) {
+            if (!value.replace('.js', '').replace('./', '')
+                .includes('/'))
+                continue;
+            
             if (value === name || value.startsWith(name) || name === 'any') {
                 push({
                     path,

package/lib/apply-report/index.js

@@ -19,6 +19,9 @@ export const report = (path) => {
     return `Use 't.${TYPES[name]}()' instead of 't.${name}()'`;
 };
 
+export const exclude = () => [
+    't.noReport(__a, __array)',
+];
 export const replace = () => ({
     't.noReport(__a, __object)': 't.noReportWithOptions(__a, __object)',
     't.noReport(__a, __b)': 't.noReport(__a)',

package/lib/convert-replace-with/index.js

@@ -1,68 +1,17 @@
-import {
-    operator,
-    template,
-    types,
-} from 'putout';
+import {operator} from 'putout';
 import fullstore from 'fullstore';
 
-const {objectProperty, identifier} = types;
-
-const {
-    replaceWith,
-    insertAfter,
-    insertBefore,
-} = operator;
+const {replaceWith} = operator;
 
 export const report = () => {
     return `Use 'operator.replaceWith()' instead of 'path.replaceWith()'`;
 };
 
-export const fix = ({path, calleePath, property, object, program, isInserted}) => {
+export const fix = ({path, calleePath, property, object}) => {
     replaceWith(calleePath, property);
-    const {bindings} = program.scope;
-    
     path.node.arguments.unshift(object);
-    
-    if (bindings.replaceWith || isInserted())
-        return;
-    
-    if (!bindings.replaceWithMultiple && !bindings.insertAfter && !isInserted()) {
-        const replaceWithAST = template.ast.fresh(`
-            const {replaceWith} = require('putout').operator;
-        `);
-        
-        const {types} = bindings;
-        const first = program.get('body.0');
-        const pathToInsert = types ? types.path.parentPath : first;
-        
-        if (types)
-            insertAfter(pathToInsert, replaceWithAST);
-        else
-            insertBefore(pathToInsert, replaceWithAST);
-        
-        isInserted(true);
-        
-        return;
-    }
-    
-    const id = identifier('replaceWith');
-    const varPath = getVarPath(bindings);
-    
-    varPath.node.id.properties.unshift(objectProperty(id, id, false, true));
 };
 
-function getVarPath(bindings) {
-    const {
-        replaceWithMultiple,
-        insertAfter,
-    } = bindings;
-    
-    if (replaceWithMultiple)
-        return replaceWithMultiple.path;
-    
-    return insertAfter.path;
-}
-
 export const traverse = ({push}) => {
     const isInserted = fullstore();
     

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

@@ -13,10 +13,11 @@ export const fix = ({path, incorrect, correct}) => {
 };
 
 const INCORRECT = {
-    TRANSFORM: /: (no transform|report|no report)/,
-    NO_TRANSFORM: /: (transform|report|no report)/,
-    REPORT: /: (no report|transform|no transform)/,
-    NO_REPORT: /: (report|transform|no transform)/,
+    TRANSFORM: /: (no report after transform|no transform|report|no report)/,
+    NO_TRANSFORM: /: (no report after transform|transform|report|no report)/,
+    REPORT: /: (no report after transform|no report|transform|no transform)/,
+    NO_REPORT: /: (no report after transform|report|transform|no transform)/,
+    NO_REPORT_AFTER_TRANSFORM: /: (report|transform|no transform|no report)/,
 };
 
 export const traverse = ({push}) => ({
@@ -45,12 +46,18 @@ export const traverse = ({push}) => ({
         incorrect: INCORRECT.NO_REPORT,
         correct: ': no report',
     }),
+    't.noReportAfterTransform(__a)': convert({
+        push,
+        incorrect: INCORRECT.NO_REPORT_AFTER_TRANSFORM,
+        correct: ': no report after transform',
+    }),
 });
 
 const convert = ({push, correct, incorrect}) => (path) => {
     const [is, messagePath] = isCorrect({
         incorrect,
         path,
+        correct,
     });
     
     if (is)
@@ -66,7 +73,7 @@ const convert = ({push, correct, incorrect}) => (path) => {
 
 const CORRECT = true;
 
-function isCorrect({path, incorrect}) {
+function isCorrect({path, correct, incorrect}) {
     const calleePath = path.findParent(isCallExpression);
     
     if (!calleePath)
@@ -78,6 +85,10 @@ function isCorrect({path, incorrect}) {
         return [CORRECT];
     
     const {value} = messagePath.node;
+    
+    if (value.includes(correct))
+        return [CORRECT];
+    
     const is = !incorrect.test(value);
     
     return [is, messagePath];

package/package.json

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