eslint-plugin-putout 14.9.1 → 14.12.0

package/README.md

@@ -36,6 +36,7 @@ Then configure the rules you want to use under the rules section.
 {
     "rules": {
         "putout/add-newlines-between-types-in-union": "error",
+        "putout/add-newlines-between-specifiers": "error",
         "putout/add-newline-before-function-call": "error",
         "putout/add-newline-after-function-call": "error",
         "putout/putout": "error",
@@ -84,6 +85,8 @@ Then configure the rules you want to use under the rules section.
 ### ESM
 
 - ✅ [No unresolved](/packages/eslint-plugin-putout/lib/no-unresolved#readme)
+- ✅ [Remove duplicate extension](/packages/eslint-plugin-putout/lib/remove-duplicate-extensions#readme)
+- ✅ [Add newlines between specifiers](/packages/eslint-plugin-putout/lib/add-newlines-between-specifiers#readme)
 
 ### Formatting
 

package/lib/add-newlines-between-specifiers/README.md

@@ -0,0 +1,36 @@
+# add-newlines-between-specifiers
+
+This rule aims to add newlines between specifiers.
+
+## ❌ Example of incorrect code
+
+```js
+let a;
+let b;
+let c;
+let d;
+let e;
+let f;
+
+export {a, b, c, d, e, f};
+```
+
+## ✅ Example of correct code
+
+```js
+let a;
+let b;
+let c;
+let d;
+let e;
+let f;
+
+export {
+    a,
+    b,
+    c,
+    d,
+    e,
+    f,
+};
+```

package/lib/add-newlines-between-specifiers/index.js

@@ -0,0 +1,27 @@
+'use strict';
+
+const regExp = /, +?[A-z]/g;
+
+module.exports.category = 'layout';
+module.exports.report = () => 'Add newlines between specifiers';
+
+module.exports.include = () => [
+    'ExportNamedDeclaration',
+];
+
+module.exports.fix = ({text}) => {
+    return text
+        .replace(/,\s+/g, ',\n    ')
+        .replace(/{/g, '{\n')
+        .replace(/}/g, '\n}')
+        .replace(/{\n+/g, '{\n')
+        .replace(/\n+}/g, '\n}');
+};
+
+module.exports.filter = ({text, node}) => {
+    if (node.specifiers.length < 4)
+        return false;
+    
+    return regExp.test(text);
+};
+

package/lib/array-element-newline/index.js

@@ -12,6 +12,9 @@ module.exports.report = () => 'Add newlines between array elements';
 const regexp = /['\da-zA-Z]+, ['\da-zA-Z]/;
 
 const isSupportedNode = (a) => {
+    if (!a)
+        return false;
+    
     if (a.type === 'Literal')
         return true;
     

package/lib/index.js

@@ -27,6 +27,7 @@ module.exports.rules = {
     ...getWrapRule('newline-function-call-arguments'),
     ...getWrapRule('function-declaration-paren-newline'),
     ...getWrapRule('add-newlines-between-types-in-union'),
+    ...getWrapRule('add-newlines-between-specifiers'),
     ...getWrapRule('add-newline-before-function-call'),
     ...getWrapRule('add-newline-after-function-call'),
     ...getWrapRule('remove-newline-after-default-import'),
@@ -38,6 +39,7 @@ module.exports.rules = {
     ...getWrapRule('objects-braces-inside-array'),
     ...getWrapRule('object-init'),
     ...getWrapRule('no-unresolved'),
+    ...getWrapRule('remove-duplicate-extensions'),
     ...getWrapRule('evaluate'),
     ...getWrapRule('tape-add-newline-before-assertion'),
     ...getWrapRule('tape-add-newline-between-tests'),
@@ -68,6 +70,7 @@ const recommended = {
         'putout/newline-function-call-arguments': 'error',
         'putout/function-declaration-paren-newline': 'error',
         'putout/add-newlines-between-types-in-union': 'error',
+        'putout/add-newlines-between-specifiers': 'error',
         'putout/add-newline-before-function-call': 'error',
         'putout/add-newline-after-function-call': 'error',
         'putout/remove-newline-after-default-import': 'error',
@@ -80,6 +83,7 @@ const recommended = {
         'putout/objects-braces-inside-array': 'error',
         'putout/object-init': 'error',
         'putout/no-unresolved': 'error',
+        'putout/remove-duplicate-extensions': 'error',
         'putout/evaluate': 'error',
         'putout/tape-add-newline-before-assertion': 'error',
         'putout/tape-add-newline-between-tests': 'error',

package/lib/remove-duplicate-extensions/README.md

@@ -0,0 +1,24 @@
+# remove-duplicate-extension
+
+Check if path has duplicate extension `.js.js` can be resolved and fix if cannot.
+Duplicates can happen when **IDE** like **WebStorm** inserts filename and mistakenly adds duplicate.
+
+Part of [`eslint-plugin-putout`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
+
+## ❌ Example of incorrect code
+
+```js
+import x from './y.js.js';
+export * from './y.js.js';
+export * as dir from './dir.js.js';
+export {m} from './y.js.js';
+```
+
+## ✅ Example of correct code
+
+```js
+import x from './y.js';
+export * from './y.js';
+export * as dir from './dir.js';
+export {m} from './y.js';
+```

package/lib/remove-duplicate-extensions/index.js

@@ -0,0 +1,21 @@
+'use strict';
+
+const getValue = ({source}) => source?.value;
+
+module.exports.category = 'errors';
+module.exports.report = () => 'Avoid duplicate extensions in relative imports';
+module.exports.include = () => [
+    'ImportDeclaration',
+    'ImportExpression',
+    'ExportAllDeclaration',
+    'ExportNamedDeclaration',
+];
+
+module.exports.fix = ({text}) => {
+    return text.replace('.js.js', '.js');
+};
+
+module.exports.filter = ({node}) => {
+    const value = getValue(node);
+    return /\.js\.js/.test(value);
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "14.9.1",
+  "version": "14.12.0",
   "type": "commonjs",
   "description": "eslint plugin for putout",
   "release": false,