eslint-plugin-putout 27.2.0 → 28.0.0

package/README.md

@@ -83,7 +83,6 @@ Then configure the rules you want to use under the rules section.
         "putout/long-properties-destructuring": "error",
         "putout/destructuring-as-function-argument": "error",
         "putout/align-spaces": "error",
-        "putout/keyword-spacing": "error",
         "putout/newline-function-call-arguments": "error",
         "putout/function-declaration-paren-newline": "error",
         "putout/remove-newline-after-default-import": "error",
@@ -139,7 +138,6 @@ Then configure the rules you want to use under the rules section.
 - ✅ [For-of multiple properties destructuring](/packages/eslint-plugin-putout/lib/for-of-multiple-properties-destructuring#readme)
 - ✅ [Long properties destructuring](/packages/eslint-plugin-putout/lib/long-properties-destructuring#readme)
 - ✅ [Destructuring as function argument](/packages/eslint-plugin-putout/lib/destructuring-as-function-argument#readme)
-- ✅ [Keyword spacing](/packages/eslint-plugin-putout/lib/keyword-spacing#readme)
 - ✅ [Newline function call arguments](/packages/eslint-plugin-putout/lib/newline-function-call-arguments#readme)
 - ✅ [Function declaration paren newline](/packages/eslint-plugin-putout/lib/function-declaration-paren-newline#readme)
 - ✅ [Remove newline between declarations](/packages/eslint-plugin-putout/lib/remove-newline-between-declarations#readme)

package/lib/index.mjs

@@ -25,7 +25,6 @@ const putoutConfig = [{
         'putout/long-properties-destructuring': 'error',
         'putout/destructuring-as-function-argument': 'error',
         'putout/align-spaces': 'error',
-        'putout/keyword-spacing': 'error',
         'putout/newline-function-call-arguments': 'error',
         'putout/function-declaration-paren-newline': 'error',
         'putout/add-newlines-between-types-in-union': 'error',

package/lib/plugin.mjs

@@ -19,7 +19,6 @@ export const rules = {
     ...getWrapRule('long-properties-destructuring'),
     ...getWrapRule('destructuring-as-function-argument'),
     ...getWrapRule('align-spaces'),
-    ...getWrapRule('keyword-spacing'),
     ...getWrapRule('newline-function-call-arguments'),
     ...getWrapRule('function-declaration-paren-newline'),
     ...getWrapRule('add-newlines-between-types-in-union'),

package/lib/ts.mjs

@@ -162,13 +162,14 @@ function convertPaddingLines([state, ...lines]) {
 
 function getOperatorLinebreak(options) {
     const [error, after, {overrides}] = options;
+    const newOverrides = assign({}, overrides);
     
-    delete overrides['='];
+    delete newOverrides['='];
     
     return [
         error,
         after, {
-            overrides,
+            overrides: newOverrides,
         },
     ];
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "27.2.0",
+  "version": "28.0.0",
   "type": "commonjs",
   "description": "ESLint plugin for 🐊Putout",
   "release": false,

package/lib/keyword-spacing/index.js

@@ -1,96 +0,0 @@
-'use strict';
-
-const {
-    isTryStatement,
-    isSwitchStatement,
-} = require('putout').types;
-
-const keywords = {
-    IfStatement: 'if',
-    ForStatement: 'for',
-    ForOfStatement: 'for',
-    ForOfStatement_: 'for await',
-};
-
-const checkNodeSpace = ({text, node}) => {
-    const {type} = node;
-    const key = keywords[type];
-    const key_ = keywords[`${type}_`];
-    
-    if (text.includes(`${key}(`))
-        return true;
-    
-    return key_ && text.includes(`${key_}(`);
-};
-
-const fixNodeSpace = ({node, text}) => {
-    const {type} = node;
-    const key = keywords[type];
-    
-    return text
-        .replace(`${key}(`, `${key} (`)
-        .replace(`for await(`, `for await (`);
-};
-
-module.exports.report = (node) => {
-    if (isTryStatement(node))
-        return 'Use spaces around "catch"';
-    
-    if (isSwitchStatement(node))
-        return 'Avoid space after "switch"';
-    
-    return `Use space after "${keywords[node.type]}"`;
-};
-
-module.exports.fix = ({node, text}) => {
-    if (isTryStatement(node))
-        return fixCatch(text);
-    
-    if (isSwitchStatement(node))
-        return fixSwitch(text);
-    
-    return fixNodeSpace({
-        node,
-        text,
-    });
-};
-
-module.exports.include = () => [
-    'TryStatement',
-    'SwitchStatement',
-    'IfStatement',
-    'ForStatement',
-    'ForOfStatement',
-];
-
-module.exports.filter = ({node, text}) => {
-    if (isTryStatement(node))
-        return checkCatch(text);
-    
-    if (isSwitchStatement(node))
-        return checkSwitch(text);
-    
-    return checkNodeSpace({
-        node,
-        text,
-    });
-};
-
-function checkCatch(text) {
-    const before = text.includes('}catch');
-    const after = text.includes('catch (');
-    const afterOptional = text.includes('catch{');
-    
-    return before || after || afterOptional;
-}
-
-const checkSwitch = (text) => text.includes('switch (');
-
-const fixCatch = (text) => text
-    .replaceAll('catch{', 'catch {')
-    .replaceAll('}catch', '} catch')
-    .replaceAll('catch (', 'catch(');
-
-function fixSwitch(text) {
-    return text.replaceAll('switch (', 'switch(');
-}