@putout/plugin-esm 4.2.2 → 4.4.0

package/README.md

@@ -23,10 +23,12 @@ 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);
 - ✅ [sort-imports-by-specifiers](#sort-imports-by-specifiers);
+- ✅ [inline-export](#inline-export);
 
 ## File rules
 
@@ -43,6 +45,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", {
@@ -50,7 +53,8 @@ npm i putout @putout/plugin-esm -D
         }],
         "esm/sort-imports-by-specifiers": "on",
         "esm/resolve-imported-file": "off",
-        "esm/apply-namespace-of-file": "off"
+        "esm/apply-namespace-of-file": "off",
+        "esm/inline-export": "off"
     }
 }
 ```
@@ -104,6 +108,35 @@ export {
 export * as ns from 'x';
 ```
 
+### inline-export
+
+Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/##/gist/c9a3983d269745da89c1c7560f3b7fac/3ecb9aa6b910ce3816605bae11c8dd86bdc457e5).
+
+#### ❌ Example of incorrect code
+
+```js
+const stack = [];
+
+function sum(a, b) {
+    i32.add(local.get(), local.get());
+}
+
+export {
+    sum,
+    stack,
+};
+```
+
+#### ✅ Example of correct code
+
+```js
+export const stack = [];
+
+export function sum(a, b) {
+    i32.add(local.get(), local.get());
+}
+```
+
 ### declare-imports-first
 
 Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/b1c18e5d726afe4ebb69d6b7a7dda82b/8189590815a1b8adb35bb8a846e28228e3c7fadf). For **CommonJS** use [nodejs/declare-after-require](https://github.com/coderaiser/putout/tree/master/packages/plugin-nodejs#declare-after-require).
@@ -162,6 +195,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
@@ -315,6 +378,10 @@ import('foo.json', {
 
 ### apply-namespace-import-to-file
 
+The rule fixes:
+
+> `SyntaxError: The requested module './a.js' does not provide an export named 'default'`
+
 Check out in 🐊**Putout Editor**:
 
 - ✅ [`apply-namespace-import-to-file`](https://putout.cloudcmd.io/#/gist/1492d584559e5798325047de679222a0/c6a37a803b80823de1b64ab944f2427aecefb51b);
@@ -345,10 +412,6 @@ import a from './a.js';
 import * as a from './a.js';
 ```
 
-## License
-
-MIT
-
 ### resolve-imported-file
 
 Check out in 🐊**Putout Editor**:

package/lib/apply-namespace-import-to-file/index.js

@@ -5,7 +5,6 @@ import putout, {
     transform,
     operator,
 } from 'putout';
-
 import * as isESMPlugin from './is-esm/index.js';
 import * as hasExportDefaultPlugin from './has-export-default/index.js';
 import * as applyNamespaceImportPlugin from './apply-namespace-import/index.js';

package/lib/index.js

@@ -1,3 +1,4 @@
+import * as inlineExport from './inline-export/index.js';
 import * as applyNamespaceImportToFile from './apply-namespace-import-to-file/index.js';
 import * as resolveImportedFile from './resolve-imported-file/index.js';
 import * as addIndexToImport from './add-index-to-import/index.js';
@@ -8,6 +9,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 +26,6 @@ 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,
+    'inline-export': inlineExport,
 };

package/lib/inline-export/index.js

@@ -0,0 +1,67 @@
+import {types, operator} from 'putout';
+
+const {remove, replaceWith} = operator;
+const {
+    variableDeclaration,
+    isVariableDeclarator,
+    exportNamedDeclaration,
+    isFunctionDeclaration,
+    isImportSpecifier,
+    isImportDefaultSpecifier,
+} = types;
+
+export const report = () => `Inline export`;
+export const include = () => [
+    'export {__exports}',
+];
+export const exclude = () => [`export * as __a from '__b'`];
+
+export const filter = (path) => {
+    const {scope} = path;
+    const specifiers = path.get('specifiers');
+    
+    for (const spec of specifiers) {
+        const {local} = spec.node;
+        const {name} = local;
+        const binding = scope.bindings[name];
+        
+        if (!binding)
+            return false;
+        
+        const bindingPath = binding.path;
+        
+        if (isImportSpecifier(bindingPath))
+            return false;
+        
+        if (isImportDefaultSpecifier(bindingPath))
+            return false;
+    }
+    
+    return true;
+};
+
+export const fix = (path) => {
+    const {scope} = path;
+    const specifiers = path.get('specifiers');
+    
+    for (const spec of specifiers) {
+        const {local} = spec.node;
+        const {name} = local;
+        
+        const binding = scope.bindings[name];
+        const bindingPath = binding.path;
+        
+        if (isFunctionDeclaration(bindingPath)) {
+            replaceWith(bindingPath, exportNamedDeclaration(bindingPath.node));
+            continue;
+        }
+        
+        if (isVariableDeclarator(bindingPath)) {
+            const declaration = variableDeclaration('const', [bindingPath.node]);
+            replaceWith(bindingPath.parentPath, exportNamedDeclaration(declaration));
+            continue;
+        }
+    }
+    
+    remove(path);
+};

package/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,
+        });
+    },
+});

package/lib/merge-duplicate-imports/join/index.js

@@ -1,6 +1,6 @@
 import {types, operator} from 'putout';
 
-const {remove, compareAny} = operator;
+const {remove} = operator;
 const {values} = Object;
 
 const {
@@ -23,9 +23,6 @@ export const fix = ({path, imports}) => {
                 continue;
             }
             
-            if (compareAny(spec, path.node.specifiers))
-                continue;
-            
             all.push(spec);
         }
         

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-esm",
-  "version": "4.2.2",
+  "version": "4.4.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin improves ability to transform ESM code",