eslint-plugin-putout 14.10.0 → 15.0.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",
@@ -56,7 +57,7 @@ Then configure the rules you want to use under the rules section.
         "putout/remove-empty-newline-after-import": "error",
         "putout/remove-empty-specifiers": "error",
         "putout/objects-braces-inside-array": "error",
-        "putout/object-init": "error",
+        "putout/object-property-newline": "error",
         "putout/tape-add-newline-between-tests": "error",
         "putout/tape-add-newline-before-assertion": "error",
         "putout/tape-remove-newline-before-t-end": "error"
@@ -80,11 +81,13 @@ Then configure the rules you want to use under the rules section.
 ### TypeScript
 
 - ✅ [Add newlines between types in union](/packages/eslint-plugin-putout/lib/add-newlines-between-types-in-union#readme)
+- ✅ [Object property newline](/packages/eslint-plugin-putout/lib/object-property-newline#readme)
 
 ### 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
 
@@ -108,7 +111,6 @@ Then configure the rules you want to use under the rules section.
 - ✅ [Remove empty newline after import](/packages/eslint-plugin-putout/lib/remove-empty-newline-after-import#readme)
 - ✅ [Remove empty specifiers](/packages/eslint-plugin-putout/lib/remove-empty-specifiers#readme)
 - ✅ [Objects braces inside array](/packages/eslint-plugin-putout/lib/objects-braces-inside-array#readme)
-- ✅ [Object init](/packages/eslint-plugin-putout/lib/object-init#readme)
 - ✅ [Nonblock statement body newline](/packages/eslint-plugin-putout/lib/non-block-statement-body-newline#readme)
 
 ### Safe mode

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';
+
+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}) => {
+    const regExp = /, +?[a-zA-Z]/g;
+    
+    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'),
@@ -36,7 +37,7 @@ module.exports.rules = {
     ...getWrapRule('remove-empty-newline-after-last-element'),
     ...getWrapRule('remove-empty-specifiers'),
     ...getWrapRule('objects-braces-inside-array'),
-    ...getWrapRule('object-init'),
+    ...getWrapRule('object-property-newline'),
     ...getWrapRule('no-unresolved'),
     ...getWrapRule('remove-duplicate-extensions'),
     ...getWrapRule('evaluate'),
@@ -69,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',
@@ -79,7 +81,7 @@ const recommended = {
         'putout/remove-empty-newline-after-import': 'error',
         'putout/remove-empty-specifiers': 'error',
         'putout/objects-braces-inside-array': 'error',
-        'putout/object-init': 'error',
+        'putout/object-property-newline': 'error',
         'putout/no-unresolved': 'error',
         'putout/remove-duplicate-extensions': 'error',
         'putout/evaluate': 'error',

package/lib/{object-init → object-property-newline}/README.md

@@ -1,20 +1,23 @@
-# object-init
+# object-property-newline
 
-Keep each property on separate line when initializing an object. In the same way as **ESLint** [`object-property-newline`](https://eslint.org/docs/rules/object-property-newline) but for initializing variables with [**Object Expression**](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Working_with_Objects).
+Keep each property on separate line when initializing an object. In the same way as **ESLint** [`object-property-newline`](https://eslint.org/docs/rules/object-property-newline) but for initializing variables with [**Object Expression**](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Working_with_Objects) or using `type` and `interface` keywords in **TypeScript**.
 
 Part of [**eslint-plugin-putout**](https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#rules).
 
 ## ❌ Example of incorrect code
 
-```js
+```ts
 const user = {name};
 
 module.exports = {lint: 'putout lint'};
+
+type User = {name: string};
+interface Place {message: string}
 ```
 
 ## ✅ Example of correct code
 
-```js
+```ts
 const user = {
     name,
 };
@@ -22,4 +25,12 @@ const user = {
 module.exports = {
     lint: 'putout lint',
 };
+
+type User = {
+    name: string,
+};
+
+interface Place {
+    message: string;
+}
 ```

package/lib/{object-init → object-property-newline}/index.js

@@ -3,7 +3,11 @@
 const {types} = require('putout');
 const {isCorrectLoc} = require('../common');
 
-const {isVariableDeclarator} = types;
+const {
+    isVariableDeclarator,
+    isAssignmentExpression,
+    isTSTypeAliasDeclaration,
+} = types;
 
 module.exports.category = 'destructuring';
 module.exports.report = () => 'Keep each property on separate line';
@@ -11,6 +15,8 @@ module.exports.report = () => 'Keep each property on separate line';
 module.exports.include = () => [
     `VariableDeclarator[init.type="ObjectExpression"]`,
     `AssignmentExpression[right.type="ObjectExpression"]`,
+    `TSTypeAliasDeclaration[typeAnnotation.type="TSTypeLiteral"]`,
+    `TSInterfaceDeclaration`,
 ];
 
 module.exports.filter = ({node}) => {
@@ -27,16 +33,30 @@ module.exports.filter = ({node}) => {
         return !isCorrectLoc(line, properties);
     }
     
-    const {properties} = right;
+    if (isAssignmentExpression(node)) {
+        const {properties} = right;
+        const {line} = loc.start;
+        
+        return !isCorrectLoc(line, properties);
+    }
+    
+    if (isTSTypeAliasDeclaration(node)) {
+        const {members} = node.typeAnnotation;
+        const {line} = loc.start;
+        
+        return !isCorrectLoc(line, members);
+    }
+    
+    const {body} = node.body;
     const {line} = loc.start;
     
-    return !isCorrectLoc(line, properties);
+    return !isCorrectLoc(line, body);
 };
 
 module.exports.fix = ({text}) => {
     return text
-        .replace(/,/g, ',\n')
-        .replace(/{/g, '{\n')
+        .replace(/,/g, ',\n    ')
+        .replace(/{/g, '{\n    ')
         .replace(/}/g, '\n}');
 };
 

package/lib/ts.js

@@ -7,8 +7,8 @@ const extensionRules = {
     'no-undef': 'off',
     'no-var': 'off',
     
-    'brace-style': 'off',
-    '@typescript-eslint/brace-style': ['error'],
+    // putout/object-property-newline instead
+    '@typescript-eslint/brace-style': 'off',
     
     'comma-dangle': 'off',
     '@typescript-eslint/comma-dangle': rules['comma-dangle'],

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "14.10.0",
+  "version": "15.0.0",
   "type": "commonjs",
-  "description": "eslint plugin for putout",
+  "description": "ESLint plugin for 🐊Putout",
   "release": false,
   "tag": false,
   "homepage": "https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout#readme",