@putout/plugin-types 1.3.0 → 1.5.0

package/README.md

@@ -20,7 +20,8 @@ npm i putout @putout/plugin-types -D
         "types/convert-typeof-to-istype": "on",
         "types/remove-useless-conversion": "on",
         "types/remove-double-negations": "on",
-        "types/remove-useless-typeof": "on"
+        "types/remove-useless-typeof": "on",
+        "types/apply-is-array": "on"
     }
 }
 ```
@@ -33,6 +34,7 @@ Based on [`@putout/operator-declare`](https://github.com/coderaiser/putout/tree/
 Supported assertions:
 
 - `isString`;
+- `isEmptyString`;
 - `isNumber`;
 - `isFn`;
 - `isBool`;
@@ -147,6 +149,46 @@ typeof typeof 'hello';
 typeof 'hello';
 ```
 
+### apply-is-array
+
+> The `Array.isArray()` method determines whether the passed value is an `Array`.
+> When checking for `Array` instance, `Array.isArray()` is preferred over `instanceof` because it works through `iframes`.
+
+#### ❌ Example of incorrect code
+
+```js
+x instanceof Array;
+```
+
+#### ✅ Example of correct code
+
+```js
+const {isArray} = Array;
+isArray(x);
+```
+
+In case of using `inline` option:
+
+```json
+{
+    "rules": {
+        "apply-is-array": ["on", {
+            "inline": true
+        }]
+    }
+}
+```
+
+`Array.isArray` will be inlined:
+
+```js
+Array.isArray(x);
+```
+
+## License
+
+MIT
+
 ## Comparison
 
 Linter | Rule | Fix

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

@@ -0,0 +1,17 @@
+'use strict';
+
+module.exports.report = () => `Use 'Array.isArray()' instead of 'instanceof'`;
+
+module.exports.replace = ({options}) => {
+    const {inline} = options;
+    
+    return {
+        '__a instanceof Array': () => {
+            if (inline)
+                return 'Array.isArray(__a)';
+            
+            return 'isArray(__a)';
+        },
+    };
+};
+

package/lib/declare/index.js

@@ -2,6 +2,7 @@
 
 module.exports.declare = () => ({
     isString: `const isString = (a) => typeof a === 'string'`,
+    isEmptyString: `const isEmptyString = (a) => !a && isString(a)`,
     isNumber: `const isNumber = (a) => typeof a === 'number'`,
     isFn: `const isFn = (a) => typeof a === 'function'`,
     isBool: `const isBool = (a) => typeof a === 'boolean'`,

package/lib/index.js

@@ -5,6 +5,7 @@ const getRule = (a) => ({
 });
 
 module.exports.rules = {
+    ...getRule('apply-is-array'),
     ...getRule('declare'),
     ...getRule('convert-typeof-to-is-type'),
     ...getRule('remove-useless-conversion'),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-types",
-  "version": "1.3.0",
+  "version": "1.5.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",
@@ -36,7 +36,7 @@
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
     "eslint-plugin-n": "^15.2.4",
-    "eslint-plugin-putout": "^16.0.0",
+    "eslint-plugin-putout": "^17.0.0",
     "lerna": "^6.0.1",
     "madrun": "^9.0.0",
     "montag": "^1.2.1",