putout 22.1.0 → 22.3.0

package/ChangeLog

@@ -1,3 +1,59 @@
+2021.11.30, v22.3.0
+
+feature:
+- (putout) add config validation
+- (@putout/plugin-remove-empty-pattern) improve support of nested destructuring
+- (@putout/plugin-tape) add-stop-all: add support of mockRequire
+- (@putout/plugin-remove-unused-variables) improve support of nested destructuring
+
+
+2021.11.28, v22.2.1
+
+fix:
+- (@putout/plugin-remove-constant-conditions) report: "constant condtions should not be used" -> "Avoid constant conditions"
+
+feature:
+- (package) @putout/plugin-madrun v11.0.0
+- (putout) improve ability to test from IDE
+- (@putout/plugin-madrun) drop support of putout < 22
+- (@putout/plugin-madrun) add declare
+- (eslint-plugin-putout) putout: add ability to preserve comments places
+
+
+2021.11.27, v22.2.0
+
+fix:
+- (@putout/plugin-remove-unused-variables) OptionalMemberExpression: computed property
+- (eslint-plugin-putout) array-element-newline: minimum elements: 4 -> 5
+- (eslint-plugin-putout) add-newlinew-before-function-call: trimed whitespaces on empty line
+- (eslint-plugin-putout) add-newline-before-function-call: add support of comments
+- (eslint-plugin-putout) add-newline-before-function-call: continue -> return
+
+feature:
+- (package) @putout/plugin-remove-constant-conditions v3.0.0
+- (@putout/plugin-remove-constant-conditions) drop support of node < 14
+- (@putout/plugin-remove-constant-conditions) add support of consequent not body
+- (eslint-plugin-putout) safe: disable remove-unreachable-code
+- (@putout/plugin-remove-unused-variables) improve support of OptionalMemberExpression
+- (eslint-plugin-putout) add-newline-after-function-call: add support of ArrayExpression and ObjectExpression
+- (eslint-plugin-putout) add add-newline-after-function-call
+- (@putout/eslint-config) padding-line-between-statements: add support of while
+- (eslint-plugin-putout) add add-newline-before-function-call
+- (@putout/eslint-config) add func-call-spacing (https://eslint.org/docs/rules/func-call-spacing)
+- (eslint-plugin-putout) plugin-tape-remove-newline-before-t-end: add support of trimed newline
+- (eslint-plugin-putout) add add-newline-before-t-end
+- (@putout/plugin-declare-undefined-variables) add maybeEmptyArray
+- (eslint-plugin-putout) array-element-newline: add support of Identifier
+- (@putout/eslint-config) padding-line-between-statements: add newline between for and return
+- (@putout/plugin-remove-useless-spread) add support of logical expressions
+
+
+2021.11.21, v22.1.1
+
+feature:
+- (package) yargs-parser v21.0.0
+
+
 2021.11.21, v22.1.0
 
 fix:
@@ -6,7 +62,7 @@ fix:
 
 feature:
 - (putout) *.md: disable convert-quotes-to-backticks
-- (@putout/plugin-convert-quotes-to-backtics) add suport of newlines
+- (@putout/plugin-convert-quotes-to-backtics) add support of newlines
 - (eslint-plugin-putout) objects-braces-inside-array: enable for json, disable for yaml
 
 

package/README.md

@@ -9,7 +9,7 @@
 
 ![putout](https://github.com/coderaiser/putout/blob/master/images/putout-logo.svg)
 
-🐊[`Putout`](https://github.com/coderaiser/putout) find and fix problems in your `JavaScript`, `JSX`, `Typescript`, `Flow`, `Yaml, `Json` and `Markdown`. It can:
+🐊[`Putout`](https://github.com/coderaiser/putout) find and fix problems in your `JavaScript`, `JSX`, `Typescript`, `Flow`, `Yaml, `Json`and`Markdown`. It can:
 
 - remove unused `variables`;
 - remove unused `for-of variables`;
@@ -192,7 +192,7 @@ When you need to ignore some routes no metter what, you can use `ignore` section
 
 🐊`Putout` supports `plugins`, there is two types: with prefix official `@putout/plugin-` and user plugins with prefix `putout-plugin-`. To use your plugin create plugin as `npm` package with keywords `putout`, `putout-plugin` and add it to `.putout.json`.
 
-For example if you need to `remove-something` create `putout` plugin with name `putout-plugin-remove-something` and it to `package.json`:
+For example if you need to `remove-something` create 🐊[`Putout`](https://github.com/coderaiser/putout) plugin with name `putout-plugin-remove-something` and it to `package.json`:
 
 ```json
 {

package/lib/cli/exit-codes.js

@@ -13,5 +13,6 @@ module.exports = {
     UNHANDLED: 9,
     RULLER_WITH_FIX: 10,
     RULLER_NO_FILES: 11,
+    INVALID_CONFIG: 12,
 };
 

package/lib/cli/exit-codes.mjs

@@ -10,4 +10,4 @@ export const CANNOT_LOAD_PROCESSOR = 8;
 export const UNHANDLED = 9;
 export const RULLER_WITH_FIX = 10;
 export const RULLER_NO_FILES = 11;
-
+export const INVALID_CONFIG = 12;

package/lib/cli/index.js

@@ -17,7 +17,7 @@ const {version} = require('../../package.json');
 const simport = createSimport(__filename);
 
 const {env} = process;
-const isIDE = env.TERMINAL_EMULATOR || env.TERM_PROGRAM === 'vscode';
+const isIDE = /JetBrains/.test(env.TERMINAL_EMULATOR) || env.TERM_PROGRAM === 'vscode';
 const chooseName = (name, resolvedName) => !isIDE ? name : resolvedName;
 
 const {
@@ -53,6 +53,7 @@ const {
     UNHANDLED,
     RULLER_WITH_FIX,
     RULLER_NO_FILES,
+    INVALID_CONFIG,
 } = require('./exit-codes');
 
 const cwd = process.cwd();
@@ -217,11 +218,7 @@ module.exports = async ({argv, halt, log, write, logError, readFile, writeFile})
     }
     
     const noConfig = !args.config;
-    const {
-        formatter,
-        ignore,
-        processors = defaultProcessors,
-    } = getOptions({
+    const [configError, config] = tryCatch(getOptions, {
         name: `${cwd}/*`,
         rulesdir,
         noConfig,
@@ -229,6 +226,15 @@ module.exports = async ({argv, halt, log, write, logError, readFile, writeFile})
         plugins,
     });
     
+    if (configError)
+        return exit(INVALID_CONFIG, configError);
+    
+    const {
+        formatter,
+        ignore,
+        processors = defaultProcessors,
+    } = config;
+    
     const [currentFormat, formatterOptions] = getFormatter(format || formatter, exit);
     const [error, processorRunners] = tryCatch(getProcessorRunners, processors);
     
@@ -311,7 +317,7 @@ module.exports = async ({argv, halt, log, write, logError, readFile, writeFile})
         const resolvedName = resolve(name)
             .replace(/^\./, cwd);
         
-        const options = getOptions({
+        const [configError, options] = tryCatch(getOptions, {
             name: resolvedName,
             rulesdir,
             noConfig,
@@ -319,6 +325,9 @@ module.exports = async ({argv, halt, log, write, logError, readFile, writeFile})
             plugins,
         });
         
+        if (configError)
+            return exit(INVALID_CONFIG, configError);
+        
         const {dir} = options;
         
         if (fileCache.canUseCache(name, options)) {

package/lib/parse-options/index.js

@@ -16,6 +16,7 @@ const defaultOptions = require('../../putout.json');
 const merge = require('../merge');
 const recursiveRead = require('./recursive-read');
 const applyModuleTypeRules = require('./apply-module-type-rules');
+const {validateOptions} = require('./validate-options');
 
 const home = homedir();
 
@@ -62,6 +63,8 @@ module.exports = (info = {}) => {
         mergedMatch,
     );
     
+    validateOptions(resultOptions);
+    
     return {
         ...resultOptions,
         dir,

package/lib/parse-options/validate-options/index.js

@@ -0,0 +1,42 @@
+'use strict';
+
+const Ajv = require('ajv');
+const schema = require('./schema.json');
+
+const ajv = new Ajv({
+    strict: true,
+    allowUnionTypes: true,
+});
+
+const validate = ajv.compile(schema);
+
+module.exports.validateOptions = (options) => {
+    validate(options);
+    
+    const [error] = validate.errors || [];
+    
+    if (error)
+        throw Error(`.putout.json: ${parsePath(error)}${parseAdditional(error)}${parseMessage(error)}`);
+};
+
+function parseMessage(error) {
+    return error.message.replace(',', ' or ');
+}
+
+const parseAdditional = (error) => {
+    const {additionalProperty} = error.params;
+    
+    if (!additionalProperty)
+        return '';
+    
+    return `${additionalProperty}: `;
+};
+
+const parsePath = (error) => {
+    const {instancePath} = error;
+    
+    if (!instancePath)
+        return '';
+    
+    return `${error.instancePath.slice(1)}: `;
+};

package/lib/parse-options/validate-options/schema.json

@@ -0,0 +1,33 @@
+{
+    "$schema": "http://json-schema.org/draft-07/schema#",
+    "type": "object",
+    "additionalProperties": false,
+    "properties": {
+        "parser": {
+            "type": "string"
+        },
+        "formatter": {
+            "type": ["string", "array"]
+        },
+        "processors": {
+            "type": "array",
+            "uniqueItems": true
+        },
+        "ignore": {
+            "type": "array",
+            "uniqueItems": true
+        },
+        "match": {
+            "type": "object"
+        },
+        "plugins": {
+            "description": "List of plugins to load",
+            "type": "array",
+            "uniqueItems": true
+        },
+        "rules": {
+            "description": "Names list of rules",
+            "type": "object"
+        }
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "putout",
-  "version": "22.1.0",
+  "version": "22.3.0",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊 Pluggable and configurable code transformer with built-in eslint, babel plugins and jscodeshift codemods support of js, jsx typescript, flow files, markdown, yaml and json",
   "homepage": "http://github.com/coderaiser/putout",
@@ -108,7 +108,7 @@
     "@putout/plugin-extract-sequence-expressions": "^2.0.0",
     "@putout/plugin-github": "^2.0.0",
     "@putout/plugin-gitignore": "^3.0.0",
-    "@putout/plugin-madrun": "^10.0.0",
+    "@putout/plugin-madrun": "^11.0.0",
     "@putout/plugin-merge-destructuring-properties": "^5.0.0",
     "@putout/plugin-merge-duplicate-imports": "^4.0.0",
     "@putout/plugin-merge-if-statements": "^3.0.0",
@@ -122,7 +122,7 @@
     "@putout/plugin-remove-boolean-from-assertions": "^1.0.0",
     "@putout/plugin-remove-boolean-from-logical-expressions": "^4.0.0",
     "@putout/plugin-remove-console": "^3.0.0",
-    "@putout/plugin-remove-constant-conditions": "^2.0.0",
+    "@putout/plugin-remove-constant-conditions": "^3.0.0",
     "@putout/plugin-remove-debugger": "^4.0.0",
     "@putout/plugin-remove-duplicate-case": "^1.0.0",
     "@putout/plugin-remove-duplicate-interface-keys": "^3.0.0",
@@ -178,6 +178,7 @@
     "@putout/processor-markdown": "^5.0.0",
     "@putout/processor-yaml": "^3.0.0",
     "@putout/traverse": "^4.0.0",
+    "ajv": "^8.8.2",
     "chalk": "^4.0.0",
     "ci-info": "^3.1.1",
     "debug": "^4.1.1",
@@ -196,7 +197,7 @@
     "try-catch": "^3.0.0",
     "try-to-catch": "^3.0.0",
     "wraptile": "^3.0.0",
-    "yargs-parser": "^20.0.0"
+    "yargs-parser": "^21.0.0"
   },
   "keywords": [
     "ast",

package/putout.json

@@ -108,9 +108,6 @@
         "**/build",
         "**/package-lock.json"
     ],
-    "exclude": [
-        ".md"
-    ],
     "rules": {
         "apply-array-at": "off",
         "apply-as-type-assertions": "off",