eslint-plugin-putout 11.7.0 → 11.11.0

package/README.md

@@ -5,7 +5,7 @@
 [CoverageURL]: https://coveralls.io/github/coderaiser/putout?branch=master
 [CoverageIMGURL]: https://coveralls.io/repos/coderaiser/putout/badge.svg?branch=master&service=github
 
-[ESLint](https://eslint.org) plugin for putout with built-in rules from [@putout/eslint-config](https://github.com/coderaiser/eslint-plugin-putout/tree/master/packages/eslint-config).
+[`ESLint`](https://eslint.org) plugin for 🐊[`Putout`](https://github.com/coderaiser/putout) with built-in rules from [@putout/eslint-config](https://github.com/coderaiser/eslint-plugin-putout/tree/master/packages/eslint-config).
 
 ## Installation
 
@@ -36,6 +36,7 @@ Then configure the rules you want to use under the rules section.
 {
     "rules": {
         "putout/putout": "error",
+        "putout/array-element-newline": "error",
         "putout/single-property-destructuring": "error",
         "putout/multiple-properties-destructuring": "error",
         "putout/long-properties-destructuring": "error",
@@ -47,6 +48,7 @@ Then configure the rules you want to use under the rules section.
         "putout/add-newlines-between-types-in-union": "error",
         "putout/remove-newline-after-default-import": "error",
         "putout/remove-newline-from-empty-object": "error",
+        "putout/remove-empty-newline-before-first-specifier": "error",
         "putout/remove-empty-newline-after-last-specifier": "error",
         "putout/objects-braces-inside-array": "error",
         "putout/object-init": "error"
@@ -57,6 +59,7 @@ Then configure the rules you want to use under the rules section.
 ## Supported Rules
 
 - [putout](lib/putout)
+- [Array element newline](/packages/eslint-plugin-putout/lib/array-element-newline)
 - [Single property destructuring](/packages/eslint-plugin-putout/lib/single-property-destructuring)
 - [Multiple properties destructuring](/packages/eslint-plugin-putout/lib/multiple-properties-destructuring)
 - [For-of multiple properties destructuring](/packages/eslint-plugin-putout/lib/for-of-multiple-properties-destructuring)
@@ -69,6 +72,7 @@ Then configure the rules you want to use under the rules section.
 - [Add newlines between types in union](/packages/eslint-plugin-putout/lib/add-newlines-between-types-in-union)
 - [Remove newline after default import](/packages/eslint-plugin-putout/lib/remove-newline-after-default-import)
 - [Remove newline from empty object](/packages/eslint-plugin-putout/lib/remove-newline-from-empty-object)
+- [Remove empty newline before first specifier](/packages/eslint-plugin-putout/lib/remove-empty-newline-before-first-specifier)
 - [Remove empty newline after last specifier](/packages/eslint-plugin-putout/lib/remove-empty-newline-after-last-specifier)
 - [Objects braces inside array](/packages/eslint-plugin-putout/lib/objects-braces-inside-array)
 - [Object init](/packages/eslint-plugin-putout/lib/object-init)

package/lib/array-element-newline/README.md

@@ -0,0 +1,28 @@
+# Add new lines between types in union (add-newlines-between-types-in-union)
+
+Because [`array-element-newline`](https://eslint.org/docs/rules/array-element-newline) requires [`array-bracket-newline`](https://eslint.org/docs/rules/array-bracket-newline).
+And `array-bracket-newline` and conflicts with [`object-braces-inside-array`](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout/lib/objects-braces-inside-array).
+
+## Rule Details
+
+This rule aims to add newlines between array elements.
+
+Examples of **incorrect** code for this rule:
+
+```js
+const a = [1, 2, 3, 4, 5, 6, 7];
+```
+
+Examples of **correct** code for this rule:
+
+```js
+const a = [
+    1,
+    2,
+    3,
+    4,
+    5,
+    6,
+    7,
+];
+```

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

@@ -0,0 +1,56 @@
+'use strict';
+
+const {types} = require('putout');
+const {
+    isMemberExpression,
+    isVariableDeclarator,
+} = types;
+
+module.exports.category = 'array';
+module.exports.report = () => 'Add newlines between array elements';
+
+const regexp = /['\da-zA-Z]+, ['\da-zA-Z]/;
+
+const isSupportedNode = (a) => {
+    if (a.type === 'Literal')
+        return true;
+    
+    if (a.type === 'Identifier')
+        return true;
+    
+    return false;
+};
+
+module.exports.filter = ({text, node}) => {
+    if (isMemberExpression(node.parent))
+        return false;
+    
+    const supported = node.elements
+        .every(isSupportedNode);
+    
+    if (!supported)
+        return false;
+    
+    if (!isVariableDeclarator(node.parent))
+        return false;
+    
+    if (node.elements.length < 4)
+        return false;
+    
+    if (regexp.test(text))
+        return true;
+    
+    return false;
+};
+
+module.exports.fix = ({text}) => {
+    return text
+        .replace(/\[/g, '[\n')
+        .replace(/\]/g, '\n]')
+        .replace(/,/g, ',\n');
+};
+
+module.exports.include = () => [
+    'ArrayExpression',
+];
+

package/lib/index.js

@@ -16,6 +16,7 @@ const getWrapRule = (a) => ({
 });
 
 module.exports.rules = {
+    ...getWrapRule('array-element-newline'),
     ...getWrapRule('single-property-destructuring'),
     ...getWrapRule('multiple-properties-destructuring'),
     ...getWrapRule('for-of-multiple-properties-destructuring'),
@@ -28,6 +29,7 @@ module.exports.rules = {
     ...getWrapRule('add-newlines-between-types-in-union'),
     ...getWrapRule('remove-newline-after-default-import'),
     ...getWrapRule('remove-newline-from-empty-object'),
+    ...getWrapRule('remove-empty-newline-before-first-specifier'),
     ...getWrapRule('remove-empty-newline-after-last-specifier'),
     ...getWrapRule('objects-braces-inside-array'),
     ...getWrapRule('object-init'),
@@ -48,6 +50,7 @@ const recommended = {
         'no-debugger': 'off',
         'no-unused-vars': 'off',
         
+        'putout/array-element-newline': 'error',
         'putout/single-property-destructuring': 'error',
         'putout/multiple-properties-destructuring': 'error',
         'putout/for-of-multiple-properties-destructuring': 'error',
@@ -60,6 +63,7 @@ const recommended = {
         'putout/add-newlines-between-types-in-union': 'error',
         'putout/remove-newline-after-default-import': 'error',
         'putout/remove-newline-from-empty-object': 'error',
+        'putout/remove-empty-newline-before-first-specifier': 'error',
         'putout/remove-empty-newline-after-last-specifier': 'error',
         'putout/objects-braces-inside-array': 'error',
         'putout/object-init': 'error',

package/lib/json.js

@@ -11,7 +11,6 @@ module.exports = [{
         'no-undef': 'off',
         'eol-last': 'off',
         'no-multi-spaces': 'off',
-        'putout/objects-braces-inside-array': 'off',
     },
 }, {
     files: 'package.json',

package/lib/keyword-spacing/README.md

@@ -21,15 +21,15 @@ try {
 } catch(error) {
 }
 
-if (a) {
+if(a) {
 }
 
-for (i = 0; i < n; i++) {}
+for(i = 0; i < n; i++) {}
 
-for (x of y) {}
+for(x of y) {}
 
 async () => {
-    for await (x of y) {}
+    for await(x of y) {}
 };
 ```
 
@@ -37,7 +37,7 @@ Examples of **correct** code for this rule:
 
 ```js
 try {
-} catch {
+}catch {
 }
 
 try {

package/lib/object-init/README.md

@@ -7,13 +7,9 @@ In the same way as eslint [object-property-newline](https://github.com/coderaise
 Examples of **incorrect** code for this rule:
 
 ```js
-const user = {
-    name,
-};
+const user = {name};
 
-module.exports = {
-    lint: 'putout lint',
-};
+module.exports = {lint: 'putout lint'};
 ```
 
 Examples of **correct** code for this rule:

package/lib/remove-empty-newline-before-first-specifier/README.md

@@ -0,0 +1,35 @@
+# Remove empty new line before first specifier(remove-empty-newline-before-first-specifier)
+
+## Rule Details
+
+This rule aims to remove empty newline before first specifier.
+
+Examples of **incorrect** code for this rule:
+
+```js
+import {
+    
+    a,
+    b,
+} from 'y';
+
+push({
+    
+    a,
+    b,
+});
+```
+
+Examples of **correct** code for this rule:
+
+```js
+import {
+    a,
+    b,
+} from 'y';
+
+push({
+    a,
+    b,
+});
+```

package/lib/remove-empty-newline-before-first-specifier/index.js

@@ -0,0 +1,21 @@
+'use strict';
+
+module.exports.category = 'import';
+module.exports.report = () => 'Remove newline before first specifier';
+
+const regExp = /{\n(\s+)?\n/;
+
+module.exports.filter = ({text}) => {
+    return regExp.test(text);
+};
+
+module.exports.fix = ({text}) => {
+    return text
+        .replace(regExp, '{\n');
+};
+
+module.exports.include = () => [
+    'ImportDeclaration',
+    'ObjectExpression',
+];
+

package/lib/yaml.js

@@ -4,6 +4,7 @@ module.exports = [{
     files: '*.{yml,yaml}{json}',
     rules: {
         'comma-spacing': 'off',
+        'putout/objects-braces-inside-array': 'off',
     },
 }];
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "11.7.0",
+  "version": "11.11.0",
   "description": "eslint plugin for putout",
   "release": false,
   "tag": false,