@putout/plugin-esm 2.2.0 → 3.0.0

package/README.md

@@ -18,6 +18,7 @@ npm i putout @putout/plugin-esm -D
 
 ## Rules
 
+- ✅ [apply-export-from](#apply-export-from);
 - ✅ [declare-imports-first](#declare-imports-first);
 - ✅ [group-imports-by-source](#group-imports-by-source);
 - ✅ [merge-duplicate-imports](#merge-duplicate-imports);
@@ -31,6 +32,7 @@ npm i putout @putout/plugin-esm -D
 ```json
 {
     "rules": {
+        "esm/apply-export-from": "on",
         "esm/declare-imports-first": "on",
         "esm/group-imports-by-source": "on",
         "esm/merge-duplicate-imports": "on",
@@ -44,6 +46,30 @@ npm i putout @putout/plugin-esm -D
 }
 ```
 
+## apply-export-from
+
+> The `export` declaration is used to export values from a JavaScript module.
+>
+> (c) [MDN](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/export)
+
+Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/c9a3983d269745da89c1c7560f3b7fac/3ecb9aa6b910ce3816605bae11c8dd86bdc457e5).
+
+## ❌ Example of incorrect code
+
+```js
+import * as ns_1 from 'x';
+
+export {
+    ns_1 as ns,
+};
+```
+
+## ✅ Example of correct code
+
+```js
+export * as ns from 'x';
+```
+
 ## 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).

package/lib/apply-export-from/index.js

@@ -0,0 +1,43 @@
+import {types, operator} from 'putout';
+
+const {
+    compare,
+    getTemplateValues,
+    remove,
+} = operator;
+
+const {exportNamespaceSpecifier} = types;
+
+export const report = () => `Use 'export *' instead of 'import/export'`;
+
+const IMPORT = 'import * as __a from "__b"';
+
+export const fix = ({path, current}) => {
+    const {exported} = current.node.specifiers[0];
+    
+    current.node.source = path.node.source;
+    
+    delete current.node.local;
+    delete current.node.exported;
+    
+    current.node.specifiers = [
+        exportNamespaceSpecifier(exported),
+    ];
+    
+    remove(path);
+};
+
+export const traverse = ({push}) => ({
+    [IMPORT]: (path) => {
+        const {__a} = getTemplateValues(path, IMPORT);
+        const {name} = __a;
+        
+        for (const current of path.parentPath.get('body')) {
+            if (compare(current, `export {${name} as __b}`))
+                push({
+                    path,
+                    current,
+                });
+        }
+    },
+});

package/lib/convert-assert-to-with/index.js

@@ -1,17 +1,15 @@
-'use strict';
-
 const setImportType = (path, type) => {
     path.node.options.properties[0].key.name = type;
 };
 
-module.exports.report = () => `Use 'with' instead of 'assert'`;
+export const report = () => `Use 'with' instead of 'assert'`;
 
-module.exports.include = () => [
+export const include = () => [
     'ImportDeclaration',
     'import(__a, {assert: {type: "json"}})',
 ];
 
-module.exports.fix = (path) => {
+export const fix = (path) => {
     if (path.isImportDeclaration()) {
         delete path.node.extra.deprecatedAssertSyntax;
         return;
@@ -20,7 +18,7 @@ module.exports.fix = (path) => {
     setImportType(path, 'with');
 };
 
-module.exports.filter = (path) => {
+export const filter = (path) => {
     const {extra} = path.node;
     
     if (path.isImportDeclaration())

package/lib/declare-imports-first/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {types, operator} from 'putout';
 
-const {types, operator} = require('putout');
 const {
     isImportDeclaration,
     isExportDeclaration,
@@ -9,9 +8,9 @@ const {
 const {replaceWith} = operator;
 const isESMNode = (a) => isImportDeclaration(a) || isExportDeclaration(a);
 
-module.exports.report = () => `Declare imports first`;
+export const report = () => `Declare imports first`;
 
-module.exports.fix = ({path, importPath}) => {
+export const fix = ({path, importPath}) => {
     let prev = path;
     let preventInfiniteLoop = 500;
     
@@ -29,7 +28,7 @@ module.exports.fix = ({path, importPath}) => {
     }
 };
 
-module.exports.traverse = ({push, pathStore}) => ({
+export const traverse = ({push, pathStore}) => ({
     ImportDeclaration: (path) => {
         pathStore(path);
     },

package/lib/group-imports-by-source/index.js

@@ -1,7 +1,6 @@
-'use strict';
+import {isDeepStrictEqual} from 'node:util';
+import {types, operator} from 'putout';
 
-const {isDeepStrictEqual} = require('node:util');
-const {types, operator} = require('putout');
 const {isImportDeclaration} = types;
 
 const {
@@ -9,9 +8,9 @@ const {
     remove,
 } = operator;
 
-module.exports.report = () => `Group imports by source: 'builtins', 'external', 'hashed', 'internal'`;
+export const report = () => `Group imports by source: 'builtins', 'external', 'hashed', 'internal'`;
 
-module.exports.fix = ({grouped}) => {
+export const fix = ({grouped}) => {
     const [first, ...others] = grouped;
     const nodes = [first.node];
     
@@ -24,7 +23,7 @@ module.exports.fix = ({grouped}) => {
     replaceWithMultiple(first, nodes);
 };
 
-module.exports.traverse = ({pathStore, push}) => ({
+export const traverse = ({pathStore, push}) => ({
     ImportDeclaration: pathStore,
     Program: {
         exit(path) {

package/lib/index.js

@@ -1,15 +1,14 @@
-'use strict';
+import * as declareImportsFirst from './declare-imports-first/index.js';
+import * as groupImportsBySource from './group-imports-by-source/index.js';
+import * as removeQuotesFromImportAssertions from './remove-quotes-from-import-assertions/index.js';
+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 convertAssertToWith from './convert-assert-to-with/index.js';
+import * as applyExportFrom from './apply-export-from/index.js';
 
-const declareImportsFirst = require('./declare-imports-first');
-const groupImportsBySource = require('./group-imports-by-source');
-const removeQuotesFromImportAssertions = require('./remove-quotes-from-import-assertions');
-const sortImportsBySpecifiers = require('./sort-imports-by-specifiers');
-const removeEmptyImport = require('./remove-empty-import');
-const removeEmptyExport = require('./remove-empty-export');
-const mergeDuplicateImports = require('./merge-duplicate-imports');
-const convertAssertToWith = require('./convert-assert-to-with');
-
-module.exports.rules = {
+export const rules = {
     ...mergeDuplicateImports.rules,
     'declare-imports-first': declareImportsFirst,
     'group-imports-by-source': groupImportsBySource,
@@ -18,4 +17,5 @@ module.exports.rules = {
     'remove-empty-export': removeEmptyExport,
     'sort-imports-by-specifiers': sortImportsBySpecifiers,
     'convert-assert-to-with': convertAssertToWith,
+    'apply-export-from': applyExportFrom,
 };

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

@@ -1,9 +1,7 @@
-'use strict';
+import * as mergeDuplicateImportsJoin from './join/index.js';
+import * as mergeDuplicateImportsRename from './rename/index.js';
 
-const mergeDuplicateImportsJoin = require('./join');
-const mergeDuplicateImportsRename = require('./rename');
-
-module.exports.rules = {
+export const rules = {
     'merge-duplicate-imports-join': mergeDuplicateImportsJoin,
     'merge-duplicate-imports-rename': mergeDuplicateImportsRename,
 };

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

@@ -1,6 +1,4 @@
-'use strict';
-
-const {types, operator} = require('putout');
+import {types, operator} from 'putout';
 
 const {remove, compareAny} = operator;
 const {values} = Object;
@@ -11,9 +9,9 @@ const {
     isImportDeclaration,
 } = types;
 
-module.exports.report = () => `Avoid duplicate imports`;
+export const report = () => `Avoid duplicate imports`;
 
-module.exports.fix = ({path, imports}) => {
+export const fix = ({path, imports}) => {
     const all = [];
     
     for (const imp of imports) {
@@ -37,7 +35,7 @@ module.exports.fix = ({path, imports}) => {
     path.node.specifiers.push(...all);
 };
 
-module.exports.traverse = ({push, pathStore}) => ({
+export const traverse = ({push, pathStore}) => ({
     ImportDeclaration(path) {
         pathStore(path);
     },

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

@@ -1,12 +1,11 @@
-'use strict';
+import {types, operator} from 'putout';
 
-const {types, operator} = require('putout');
 const {rename, remove} = operator;
 const {isImportDefaultSpecifier} = types;
 
-module.exports.report = () => 'Avoid duplicate imports';
+export const report = () => 'Avoid duplicate imports';
 
-module.exports.fix = ({path, imports}) => {
+export const fix = ({path, imports}) => {
     const {name} = path.node.specifiers[0].local;
     remove(path);
     
@@ -22,7 +21,7 @@ module.exports.fix = ({path, imports}) => {
     }
 };
 
-module.exports.traverse = ({push, uplist}) => ({
+export const traverse = ({push, uplist}) => ({
     ImportDeclaration: (path) => {
         const {value} = path.node.source;
         const [specifier, ...other] = path.node.specifiers;

package/lib/remove-empty-export/index.js

@@ -1,19 +1,18 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {remove} = operator;
 
-module.exports.report = () => 'Remove empty export';
+export const report = () => 'Remove empty export';
 
-module.exports.fix = (path) => {
+export const fix = (path) => {
     remove(path);
 };
 
-module.exports.include = () => [
+export const include = () => [
     'ExportNamedDeclaration',
 ];
 
-module.exports.filter = (path) => {
+export const filter = (path) => {
     const {specifiers, declaration} = path.node;
     
     return !declaration && !specifiers.length;

package/lib/remove-empty-import/index.js

@@ -1,22 +1,21 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {remove} = operator;
 
-module.exports.report = () => `Avoid empty 'import' statement`;
+export const report = () => `Avoid empty 'import' statement`;
 
-module.exports.fix = (path) => {
+export const fix = (path) => {
     remove(path);
 };
 
 const isCSS = (a) => /\.css/.test(a);
 const isMin = (a) => /\.min\./.test(a);
 
-module.exports.include = () => [
+export const include = () => [
     'ImportDeclaration',
 ];
 
-module.exports.filter = (path, {options}) => {
+export const filter = (path, {options}) => {
     const {specifiers, source} = path.node;
     
     const {ignore = []} = options;

package/lib/remove-quotes-from-import-assertions/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => 'Remove quotes from import assertions';
 
-module.exports.report = () => 'Remove quotes from import assertions';
-
-module.exports.replace = () => ({
+export const replace = () => ({
     'import __imports from "__a" with {"type": "__b"}': 'import __imports from "__a" with {type: "__b"}',
 });

package/lib/sort-imports-by-specifiers/index.js

@@ -1,18 +1,17 @@
-'use strict';
+import {parseImportSpecifiers} from 'parse-import-specifiers';
+import {operator} from 'putout';
 
-const {parseImportSpecifiers} = require('parse-import-specifiers');
-const {operator} = require('putout');
 const {insertBefore, remove} = operator;
 
-module.exports.report = () => `Sort imports by specifiers count`;
+export const report = () => `Sort imports by specifiers count`;
 
-module.exports.fix = ({path, nextPath}) => {
+export const fix = ({path, nextPath}) => {
     const {node} = nextPath;
     remove(nextPath);
     insertBefore(path, node);
 };
 
-module.exports.traverse = ({push}) => ({
+export const traverse = ({push}) => ({
     ImportDeclaration(path) {
         const {node} = path;
         const {source, specifiers} = node;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@putout/plugin-esm",
-  "version": "2.2.0",
-  "type": "commonjs",
+  "version": "3.0.0",
+  "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin improves ability to transform ESM code",
   "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#readme",
@@ -37,7 +37,7 @@
     "esm"
   ],
   "devDependencies": {
-    "@putout/eslint-flat": "^2.0.0",
+    "@putout/eslint-flat": "^3.0.0",
     "@putout/plugin-declare": "*",
     "@putout/plugin-declare-before-reference": "*",
     "@putout/plugin-nodejs": "*",
@@ -45,23 +45,22 @@
     "@putout/plugin-reuse-duplicate-init": "*",
     "@putout/plugin-tape": "*",
     "@putout/plugin-typescript": "*",
-    "@putout/test": "^11.0.0",
+    "@putout/test": "^13.0.0",
     "c8": "^10.0.0",
     "eslint": "^9.0.0",
     "eslint-plugin-n": "^17.0.0",
-    "eslint-plugin-putout": "^24.0.0",
-    "lerna": "^6.0.1",
-    "madrun": "^10.0.0",
+    "eslint-plugin-putout": "^26.0.0",
+    "madrun": "^11.0.0",
     "montag": "^1.2.1",
     "nodemon": "^3.0.1",
-    "supertape": "^10.8.0"
+    "supertape": "^11.0.3"
   },
   "peerDependencies": {
-    "putout": ">=38"
+    "putout": ">=39"
   },
   "license": "MIT",
   "engines": {
-    "node": ">=18"
+    "node": ">=20"
   },
   "publishConfig": {
     "access": "public"