@putout/plugin-types 1.5.0 → 1.7.0

package/README.md

@@ -62,9 +62,7 @@ When you want to skip some declaration use `dismiss`:
 {
     "rules": {
         "types/declare": ["on", {
-            "dismiss": [
-                "isString"
-            ]
+            "dismiss": ["isString"]
         }]
     }
 }

package/lib/apply-is-array/index.js

@@ -14,4 +14,3 @@ module.exports.replace = ({options}) => {
         },
     };
 };
-

package/lib/convert-typeof-to-is-type/index.js

@@ -1,7 +1,10 @@
 'use strict';
 
 const {operator} = require('putout');
-const {compare, getBindingPath} = operator;
+const {
+    compare,
+    getBindingPath,
+} = operator;
 
 const NAMES = {
     function: 'isFn',
@@ -12,6 +15,8 @@ const NAMES = {
     symbol: 'isSymbol',
 };
 
+const GENERAL = 'typeof __a === "__b"';
+
 const BODIES = {
     function: `typeof __a === 'function'`,
     string: `typeof __a === 'string'`,
@@ -23,7 +28,7 @@ const BODIES = {
 
 module.exports.report = () => `Use function to check type instead of 'typeof'`;
 module.exports.match = () => ({
-    'typeof __a === "__b"': ({__a, __b}, path) => {
+    [GENERAL]: ({__a, __b}, path) => {
         if (path.parentPath.isFunction())
             return false;
         
@@ -33,10 +38,7 @@ module.exports.match = () => ({
         if (isBind(path, __b.value))
             return false;
         
-        if (!getBindingPath(path, __a))
-            return false;
-        
-        return true;
+        return Boolean(getBindingPath(path, __a));
     },
 });
 
@@ -57,8 +59,10 @@ function isBind(path, name) {
     if (!bindingPath)
         return false;
     
+    if (!bindingPath.isVariableDeclarator())
+        return false;
+    
     const {body} = bindingPath.node.init;
     
     return !compare(body, fnBody);
 }
-

package/lib/declare/index.js

@@ -11,4 +11,3 @@ module.exports.declare = () => ({
     isSymbol: `const isSymbol = (a) => typeof a === 'symbol'`,
     isNull: `const isNull = (a) => !a && typeof a === 'object'`,
 });
-

package/lib/index.js

@@ -12,4 +12,3 @@ module.exports.rules = {
     ...getRule('remove-double-negations'),
     ...getRule('remove-useless-typeof'),
 };
-

package/lib/remove-double-negations/index.js

@@ -9,8 +9,9 @@ module.exports.replace = () => ({
     '!!__a ? __b : __c': '__a ? __b : __c',
     '!!__a && __b': '__a && __b',
     '!!__a.includes(__b)': '__a.includes(__b)',
+    '!!__a.startsWith(__b)': '__a.startsWith(__b)',
+    
     'while(!!__a) __b': 'while(__a) __b',
     'do __a; while(!!__b)': 'do __a; while(__b)',
     'for (__a; !!__b; __c) __d': 'for (__a; __b; __c) __d',
 });
-

package/lib/remove-useless-conversion/index.js

@@ -13,4 +13,3 @@ module.exports.replace = () => ({
     'for (; Boolean(__b);__c) __d': 'for (; __b; __c) __d',
     'for (; Boolean(__b);) __d': 'for (; __b;) __d',
 });
-

package/lib/remove-useless-typeof/index.js

@@ -5,4 +5,3 @@ module.exports.report = () => `Useless typeof should be avoided`;
 module.exports.replace = () => ({
     'typeof typeof __a': 'typeof __a',
 });
-

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-types",
-  "version": "1.5.0",
+  "version": "1.7.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds ability to transform code related to types assertions",
@@ -35,7 +35,7 @@
     "@putout/test": "^6.0.0",
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
-    "eslint-plugin-n": "^15.2.4",
+    "eslint-plugin-n": "^16.0.0",
     "eslint-plugin-putout": "^17.0.0",
     "lerna": "^6.0.1",
     "madrun": "^9.0.0",