feature: @putout/plugin-esm: merge-declaration-with-export: add

Author: coderaiser

packages/plugin-coverage/lib/coverage.js

@@ -1,7 +1,8 @@
 import {operator} from 'putout';
 
 const {ignore, __json} = operator;
-const {
+
+export const {
     match,
     replace,
     report,
@@ -12,9 +13,3 @@ const {
         '**/*.config.*',
     ],
 });
-
-export {
-    match,
-    replace,
-    report,
-};

packages/plugin-eslint/lib/convert-rc-to-flat/index.js

@@ -3,7 +3,8 @@ import * as rcToFlat from './rc-to-flat/index.js';
 import * as declare from '../declare/index.js';
 
 const {matchFiles} = operator;
-const {
+
+export const {
     report,
     fix,
     scan,
@@ -20,9 +21,3 @@ const {
         ],
     },
 });
-
-export {
-    report,
-    fix,
-    scan,
-};

packages/plugin-esm/README.md

@@ -23,6 +23,7 @@ npm i putout @putout/plugin-esm -D
 - ✅ [declare-imports-first](#declare-imports-first);
 - ✅ [group-imports-by-source](#group-imports-by-source);
 - ✅ [merge-duplicate-imports](#merge-duplicate-imports);
+- ✅ [merge-declaration-with-export](#merge-declaration-with-export);
 - ✅ [remove-quotes-from-import-assertions](#remove-quotes-from-import-assertions);
 - ✅ [remove-empty-import](#remove-empty-import);
 - ✅ [remove-empty-export](#remove-empty-export);
@@ -43,6 +44,7 @@ npm i putout @putout/plugin-esm -D
         "esm/declare-imports-first": "on",
         "esm/group-imports-by-source": "on",
         "esm/merge-duplicate-imports": "on",
+        "esm/merge-declaration-with-export": "on",
         "esm/remove-quotes-from-import-assertions": "on",
         "esm/remove-empty-export": "on",
         "esm/remove-empty-import": ["on", {
@@ -162,6 +164,36 @@ import ss from '../../bb/ss.js';
 const c = 5;
 ```
 
+### merge-declaration-with-export
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/3fad517d76942d0a1f51d7f58a2799af/7a052ab18a31fa382228d6513c412be37091cfb8).
+
+#### ❌ Example of incorrect code
+
+```js
+const {
+    report,
+    fix,
+    scan,
+} = createRemoveFiles(['*.swp', '*.swo']);
+
+export {
+    report,
+    fix,
+    scan,
+};
+```
+
+##### ✅ Example of correct code
+
+```js
+export const {
+    report,
+    fix,
+    scan,
+} = createRemoveFiles(['*.swp', '*.swo']);
+```
+
 ### merge-duplicate-imports
 
 #### join

packages/plugin-esm/lib/index.js

@@ -8,6 +8,7 @@ import * as sortImportsBySpecifiers from './sort-imports-by-specifiers/index.js'
 import * as removeEmptyImport from './remove-empty-import/index.js';
 import * as removeEmptyExport from './remove-empty-export/index.js';
 import * as mergeDuplicateImports from './merge-duplicate-imports/index.js';
+import * as mergeDeclarationWithExport from './merge-declaration-with-export/index.js';
 import * as convertAssertToWith from './convert-assert-to-with/index.js';
 import * as applyExportFrom from './apply-export-from/index.js';
 
@@ -24,4 +25,5 @@ export const rules = {
     'sort-imports-by-specifiers': sortImportsBySpecifiers,
     'resolve-imported-file': ['off', resolveImportedFile],
     'apply-namespace-import-to-file': ['off', applyNamespaceImportToFile],
+    'merge-declaration-with-export': mergeDeclarationWithExport,
 };

packages/plugin-esm/lib/merge-declaration-with-export/fixture/import.js

@@ -0,0 +1,5 @@
+import matchErrors from 'abc';
+
+export {
+    matchErrors,
+}

packages/plugin-esm/lib/merge-declaration-with-export/fixture/merge-declaration-with-export-fix.js

@@ -0,0 +1,7 @@
+export const {
+    report,
+    fix,
+    scan,
+} = createRemoveFiles(['*.swp', '*.swo']);
+
+export const x = () => {};

packages/plugin-esm/lib/merge-declaration-with-export/fixture/merge-declaration-with-export.js

@@ -0,0 +1,13 @@
+const {
+    report,
+    fix,
+    scan,
+} = createRemoveFiles(['*.swp', '*.swo']);
+
+export {
+    report,
+    fix,
+    scan,
+};
+
+export const x = () => {};

packages/plugin-esm/lib/merge-declaration-with-export/index.js

@@ -0,0 +1,58 @@
+import {operator, types} from 'putout';
+
+const {remove, replaceWith} = operator;
+const {
+    exportNamedDeclaration,
+    isExportSpecifier,
+    isVariableDeclarator,
+} = types;
+
+export const report = () => `Use 'if condition' instead of 'ternary expression'`;
+
+export const fix = ({path, bindingPath}) => {
+    const {parentPath} = bindingPath;
+    const {node} = parentPath;
+    
+    replaceWith(parentPath, exportNamedDeclaration(node));
+    remove(path);
+};
+export const traverse = ({push}) => ({
+    ExportNamedDeclaration(path) {
+        const {specifiers} = path.node;
+        
+        if (!specifiers.length)
+            return;
+        
+        const bindingPaths = [];
+        
+        for (const spec of specifiers) {
+            if (!isExportSpecifier(spec))
+                return;
+            
+            const {local} = spec;
+            
+            const {name} = local;
+            const binding = path.scope.bindings[name];
+            
+            if (!binding)
+                return;
+            
+            const {path: bindingPath} = binding;
+            
+            if (!isVariableDeclarator(bindingPath))
+                return;
+            
+            if (!bindingPaths.length) {
+                bindingPaths.push(bindingPath);
+                continue;
+            }
+        }
+        
+        const [bindingPath] = bindingPaths;
+        
+        push({
+            path,
+            bindingPath,
+        });
+    },
+});

packages/plugin-esm/lib/merge-declaration-with-export/index.spec.js

@@ -0,0 +1,23 @@
+import {createTest} from '@putout/test';
+import * as plugin from './index.js';
+
+const test = createTest(import.meta.url, {
+    plugins: [
+        ['merge-declaration-with-export', plugin],
+    ],
+});
+
+test('esm: merge-declaration-with-export: report', (t) => {
+    t.report('merge-declaration-with-export', `Use 'if condition' instead of 'ternary expression'`);
+    t.end();
+});
+
+test('esm: merge-declaration-with-export: transform', (t) => {
+    t.transform('merge-declaration-with-export');
+    t.end();
+});
+
+test('esm: merge-declaration-with-export: no report: import', (t) => {
+    t.noReport('import');
+    t.end();
+});

packages/plugin-esm/lib/merge-duplicate-imports/join/fixture/duplicate-specifier-fix.js

@@ -5,9 +5,6 @@ import {
     types,
 } from 'putout';
 
-{
-    const {replaceWith} = operator;
-}
 traverse(filesystem, {
     StringLiteral(path) {
         if (isIdentifier(path.node.id, {name: 'b'}))