npm-groovy-lint 11.1.0 → 11.1.2-beta202310250645.0

package/README.md

@@ -16,14 +16,14 @@
 
 ## Groovy & Jenkinsfile Linter, Formatter and Auto-fixer
 
-_**New: The [article about the story of npm-groovy-lint](https://nicolas.vuillamy.fr/a-groovy-journey-to-open-source-during-covid-19-npm-groovy-lint-8d88c7eecebc), and why you should dive in open-source community !**_
+***New: The [article about the story of npm-groovy-lint](https://nicolas.vuillamy.fr/a-groovy-journey-to-open-source-during-covid-19-npm-groovy-lint-8d88c7eecebc), and why you should dive in open-source community !***
 
 Based on [CodeNarc](http://codenarc.github.io/CodeNarc/) , this out of the box package allows to **track groovy errors** and **correct a part of them**
 
 - Use option **--format** to format & prettify source code
 - Use option **--fix** to activate autofixing of fixable rules
 
-Easy to integrate in a CD/CI process (Jenkins Pipeline,CircleCI...) to lint your groovy or Jenkinsfile at each build :)
+Easy to integrate in a CI/CD process (Jenkins Pipeline,CircleCI...) to lint your groovy or Jenkinsfile at each build :)
 
 You can also use this package in :
 
@@ -45,14 +45,14 @@ Any **question**, **problem** or **enhancement request** ? Ask [**here**](https:
 
 | Parameter               | Type    | Description                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                   |
 |-------------------------|---------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
-| -o<br/> --output        | String  | Output format (txt,json,sarif,html,xml), or path to a file with one of these extensions<br/> Default: `txt`<br/> Examples:<br/> - `"txt"`<br/> - `"json"`<br/> - `"./logs/myLintResults.txt"`<br/> - `"./logs/myLintResults.sarif"`<br/> - `"./logs/myLintResults.html"`<br/> - `"./logs/myLintResults.xml"`                                                                                                                                                                                                                                                                  |
+| -o<br/> --output        | String  | Output format (txt,json,sarif,html,xml), or path to a file with one of these extensions<br/> Default: `txt`<br/> Examples:<br/> - `"txt"`<br/> - `"json"`<br/> - `"./logs/myLintResults.txt"`<br/> - `"./logs/myLintResults.sarif"`<br/> - `"./logs/myLintResults.html"`<br/> - `"./logs/myLintResults.xml"`<br/>Note: HTML and XML are directly from CodeNarc so using these formats will disable many npm-groovy-lint features                                                                                                                                              |
 | -l<br/> --loglevel      | String  | Log level (error,warning or info)<br/>Default: info                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                           |
 | --failon                | String  | Defines the error level where CLI will fail (return code = 1). error,warning,info or none. Each failure level includes the more critical ones.                                                                                                                                                                                                                                                                                                                                                                                                                                |
-| -c<br/> --config        | String  | Custom path to [GroovyLint config file](#Configuration), or preset config `recommended|recommended-jenkinsfile|all`<br/> Default: Browse current directory to find `.groovylintrc.json|js|yml|package.json` config file, or default npm-groovy-lint config if not defined.<br/>Note: command-line arguments have priority on config file properties                                                                                                                                                                                                                           |
+| -c<br/> --config        | String  | Custom path to [GroovyLint config file](#configuration), or preset config `recommended|recommended-jenkinsfile|all`<br/> Default: Browse current directory to find`.groovylintrc.json|js|yml|package.json` config file, or default npm-groovy-lint config if not defined.<br/>Note: command-line arguments have priority on config file properties                                                                                                                                                                                                                            |
 | --parse                 | Boolean | Try to compile the source code and return parse errors (since v5.7.0, default to true, use --no-parse to deactivate)                                                                                                                                                                                                                                                                                                                                                                                                                                                          |
 | --format                | Boolean | Format source code                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            |
-| --fix                   | Boolean | Automatically fix problems when possible<br/> See [Autofixable rules](#Autofixable-rules)                                                                                                                                                                                                                                                                                                                                                                                                                                                                                     |
-| -x<br/> --fixrules      | String  | Option for --fix argument: List of rule identifiers to fix (if not specified, all available fixes will be applied). See [Autofixable rules](#Autofixable-rules) <br/> Examples:<br/> - `"SpaceBeforeClosingBrace,SpaceAfterClosingBrace,UnusedImport"`<br/> - `"Indentation"`<br/>                                                                                                                                                                                                                                                                                            |
+| --fix                   | Boolean | Automatically fix problems when possible<br/> See [Auto-fixable rules](#auto-fixable-rules)                                                                                                                                                                                                                                                                                                                                                                                                                                                                                   |
+| -x<br/> --fixrules      | String  | Option for --fix argument: List of rule identifiers to fix (if not specified, all available fixes will be applied). See [Auto-fixable rules](#auto-fixable-rules) <br/> Examples:<br/> - `"SpaceBeforeClosingBrace,SpaceAfterClosingBrace,UnusedImport"`<br/> - `"Indentation"`<br/>                                                                                                                                                                                                                                                                                          |
 | --nolintafter           | Boolean | When format or fix is called, a new lint is performed after the fixes to update the returned error list. If you just want the updated source code and do not care about the error logs, use this parameter to improve performances                                                                                                                                                                                                                                                                                                                                            |
 | -r<br/> --rulesets      | String  | [RuleSet file(s)](http://codenarc.github.io/CodeNarc/codenarc-creating-ruleset.html) to use for linting, if you do not want to use recommended rules or .groovylintrc.js defined rules.<br/>If list of comma separated strings corresponding to CodeNarc rules, a RuleSet file will be dynamically generated </br>  Examples:<br/> - `"./config/codenarc/RuleSet-Custom.groovy"`<br/> - `"./path/to/my/ruleset/files"`<br/>- `Indentation{"spacesPerIndentLevel":2,"severity":"warning"},UnnecessarySemicolon,UnnecessaryGString`                                             |
 | --rulesetsoverridetype  | String  | If list of rules sent in rulesets option, defines if they replace rules defined in .groovylintrc.json, or if they are appended<br/> Values: `replaceConfig` (default), `appendConfig`                                                                                                                                                                                                                                                                                                                                                                                         |
@@ -137,6 +137,7 @@ Any **question**, **problem** or **enhancement request** ? Ask [**here**](https:
 ```shell
     npm install -g npm-groovy-lint
 ```
+
 - If you have issues with v9, install previous version with `npm install -g npm-groovy-lint@8.2.0`
 - Node.js >= 12 is required to run this package. If you can't upgrade, you can use [nvm](https://github.com/nvm-sh/nvm) to have [different node versions on your computer](https://www.sitepoint.com/quick-tip-multiple-versions-node-nvm/)
 - If you do not have java (from 8 to 14) installed on your computer npm-groovy-lint will install them for you, so the first run may be long.
@@ -152,16 +153,16 @@ Create a file named **.groovylintrc.json** in the current or any parent director
     - groovy
       - mygroovyfile.groovy
   - Jenkinsfile
-  - **.groovylintrc.json** _(do not forget the dot at the beginning of the file name)_
+  - **.groovylintrc.json** *(do not forget the dot at the beginning of the file name)*
 
-*If you are using [VsCode Groovy Lint extension](https://marketplace.visualstudio.com/items?itemName=NicolasVuillamy.vscode-groovy-lint), just use QuickFix* ***Ignore in all files*** _and it will generate groovylintrc.json file_
+*If you are using [VsCode Groovy Lint extension](https://marketplace.visualstudio.com/items?itemName=NicolasVuillamy.vscode-groovy-lint), just use QuickFix **Ignore in all files** and it will generate groovylintrc.json file.*
 
 ### Format
 
 - **extends**: Name of a base configuration ([`recommended`](https://github.com/nvuillam/npm-groovy-lint/blob/master/lib/.groovylintrc-recommended.json), [`recommended-jenkinsfile`](https://github.com/nvuillam/npm-groovy-lint/blob/master/lib/.groovylintrc-recommended-jenkinsfile.json), [`all`](https://github.com/nvuillam/npm-groovy-lint/blob/master/lib/.groovylintrc-all.json))
 - **rules**: List of rules definition, following format `"RuleSection.RuleName": ruleParameters` or `"RuleName": ruleParameters`
-  - _RuleName_: any of the **[CodeNarc rules](https://codenarc.github.io/CodeNarc/codenarc-rule-index.html)**
-  - _ruleParameters_: can be just a severity override ( `"off"`, `"error"`, `"warning"`, `"info"` ) , or a property list :
+  - *RuleName*: any of the **[CodeNarc rules](https://codenarc.github.io/CodeNarc/codenarc-rule-index.html)**
+  - *ruleParameters*: can be just a severity override ( `"off"`, `"error"`, `"warning"`, `"info"` ) , or a property list :
     - severity : off,error,warning,info
     - enabled : true (default) or false
     - any of the [rule advanced properties](https://codenarc.github.io/CodeNarc/codenarc-rule-index.html)
@@ -341,9 +342,9 @@ def variable = 1;
 - UnnecessaryToString
 - UnusedImport
 
-[Contribute](#Contribute) to add more [rules](http://codenarc.github.io/CodeNarc/codenarc-rule-index.html) fixes :)
+[Contribute](#contribute) to add more [rules](http://codenarc.github.io/CodeNarc/codenarc-rule-index.html) fixes :)
 
-## CD/CI
+## CI/CD
 
 ### Mega-Linter
 
@@ -445,9 +446,9 @@ Please follow [Contribution instructions](https://github.com/nvuillam/npm-groovy
 
 ### Contributors
 
-[<img alt="nvuillam" src="https://avatars1.githubusercontent.com/u/17500430?v=4&s=50 width=50">](https://github.com/nvuillam) | [<img alt="Dave Gallant" src="https://avatars2.githubusercontent.com/u/4519234?v=4&s=50 width=50">](https://github.com/davegallant) | [<img alt="warhod" src="https://avatars1.githubusercontent.com/u/1305176?v=4&s=50 width=50">](https://github.com/warhod) | [<img alt="pawelkopka" src="https://avatars1.githubusercontent.com/u/17784034?v=4&s=50 width=50">](https://github.com/pawelkopka) | [<img alt="docwhat" src="https://avatars1.githubusercontent.com/u/40799?v=4&s=50 width=50">](https://github.com/docwhat) | [<img alt="CatSue" src="https://avatars3.githubusercontent.com/u/26134618?v=4&s=50 width=50">](https://github.com/CatSue)
-:----------------------------------------------------------------------------------------------------------------------------:|:-----------------------------------------------------------------------------------------------------------------------------------:|:------------------------------------------------------------------------------------------------------------------------:|:---------------------------------------------------------------------------------------------------------------------------------:|:------------------------------------------------------------------------------------------------------------------------:|:------------------------------------------------------------------------------------------------------------------------:
-                                       [Nicolas Vuillamy](https://github.com/nvuillam)                                        |                                           [Dave Gallant](https://github.com/davegallant)                                            |                                          [Howard Lo](https://github.com/warhod)                                          |                                           [Pawel Kopka](https://github.com/pawelkopka)                                            |                                          [docwhat](https://github.com/docwhat)                                           |                                            [CatSue](https://github.com/CatSue)
+| [<img alt="nvuillam" src="https://avatars1.githubusercontent.com/u/17500430?v=4&s=50 width=50">](https://github.com/nvuillam) | [<img alt="Dave Gallant" src="https://avatars2.githubusercontent.com/u/4519234?v=4&s=50 width=50">](https://github.com/davegallant) | [<img alt="warhod" src="https://avatars1.githubusercontent.com/u/1305176?v=4&s=50 width=50">](https://github.com/warhod) | [<img alt="pawelkopka" src="https://avatars1.githubusercontent.com/u/17784034?v=4&s=50 width=50">](https://github.com/pawelkopka) | [<img alt="docwhat" src="https://avatars1.githubusercontent.com/u/40799?v=4&s=50 width=50">](https://github.com/docwhat) | [<img alt="CatSue" src="https://avatars3.githubusercontent.com/u/26134618?v=4&s=50 width=50">](https://github.com/CatSue) |
+|:-----------------------------------------------------------------------------------------------------------------------------:|:-----------------------------------------------------------------------------------------------------------------------------------:|:------------------------------------------------------------------------------------------------------------------------:|:---------------------------------------------------------------------------------------------------------------------------------:|:------------------------------------------------------------------------------------------------------------------------:|:-------------------------------------------------------------------------------------------------------------------------:|
+|                                        [Nicolas Vuillamy](https://github.com/nvuillam)                                        |                                           [Dave Gallant](https://github.com/davegallant)                                            |                                          [Howard Lo](https://github.com/warhod)                                          |                                           [Pawel Kopka](https://github.com/pawelkopka)                                            |                                          [docwhat](https://github.com/docwhat)                                           |                                            [CatSue](https://github.com/CatSue)                                            |
 
 ## Release notes
 

package/lib/codenarc-caller.js

@@ -2,12 +2,17 @@
 const axios = require("axios").default;
 const cliProgress = require("cli-progress");
 const debug = require("debug")("npm-groovy-lint");
+const trace = require("debug")("npm-groovy-lint-trace");
 const { JavaCaller } = require("java-caller");
 const optionsDefinition = require("./options");
 const { performance } = require("perf_hooks");
 const { getSourceLines } = require("./utils");
 const c = require("chalk");
 
+// Request over IPv4 because Java typically prefers it.
+const http = require("http");
+axios.defaults.httpAgent = new http.Agent({ family: 4, keepAlive: true });
+
 class CodeNarcCaller {
     "use strict";
 
@@ -85,16 +90,17 @@ class CodeNarcCaller {
             },
             timeout: 600000
         };
-        debug(`CALL CodeNarcServer with ${JSON.stringify(axiosConfig, null, 2)}`);
+        trace(`CALL CodeNarcServer with ${JSON.stringify(axiosConfig, null, 2)}`);
         let response;
         try {
             const startCodeNarc = performance.now();
             response = await axios.request(axiosConfig);
             this.serverStatus = "running";
             const elapsed = parseInt(performance.now() - startCodeNarc, 10);
-            debug(`CodeNarcServer call result: (${response.status}) ${elapsed}ms ${JSON.stringify(response.data || {})}`);
+            debug(`CodeNarcServer call result: (${response.status}) ${elapsed}ms`);
         } catch (e) {
             // If server not started , start it and try again
+            debug(`callCodeNarcServer code: ${e.code} error: ${e.message}`);
             if (
                 (startServerTried === false,
                 e.code && ["ECONNREFUSED", "ETIMEDOUT"].includes(e.code) && ["unknown", "running"].includes(this.serverStatus)) // running is here in case the Server auto-killed itself at its expiration time
@@ -178,7 +184,7 @@ class CodeNarcCaller {
         const scriptArgs = this.codenarcArgs;
 
         // Start progress bar
-        debug(`CALL CodeNarcJava with ${scriptArgs.join(" ")}`);
+        trace(`CALL CodeNarcJava with ${scriptArgs.join(" ")}`);
         this.bar = new cliProgress.SingleBar(
             {
                 format: "[{bar}] Running CodeNarc for {duration_formatted}",
@@ -208,7 +214,7 @@ class CodeNarcCaller {
         if ([666, 1].includes(javaResult.status)) {
             if (!secondAttempt) {
                 // If failure (missing class com.nvuillam.CodeNarcServer for example, it can happen on Linux, let's try the original org.codenarc.CodeNarc class)
-                debug(`Error calling CodeNarcServer via java: ${JSON.stringify(javaResult)}`);
+                trace(`Error calling CodeNarcServer via java: ${JSON.stringify(javaResult)}`);
                 return await this.callCodeNarcJava(true);
             } else {
                 let reason = "Reason: unknown";
@@ -272,12 +278,20 @@ class CodeNarcCaller {
             return false;
         }
 
+        // Store the process so we can stop it later.
+        this.codeNarcProcess = javaCallRes.childJavaProcess;
+
         // Poll it until it is ready
         const start = performance.now();
         let notified = false;
         let interval;
         await new Promise(resolve => {
             interval = setInterval(() => {
+                debug(
+                    `pinging CodeNarcServer at ${serverPingUri} notified: ${notified}, serverStatus: ${
+                        this.serverStatus
+                    }, since: ${performance.now() - start}, maxAttemptTimeMs: ${maxAttemptTimeMs}`
+                );
                 axios
                     .get(serverPingUri)
                     .then(response => {
@@ -291,18 +305,22 @@ class CodeNarcCaller {
                                 resolve();
                             }
                         } else if (notified === false && this.serverStatus === "unknown" && performance.now() - start > maxAttemptTimeMs) {
-                            // Timeout has been reached
-                            this.declareServerError(
-                                {
-                                    message: "Timeout after " + maxAttemptTimeMs + "\nResponse: " + JSON.stringify(response.toJSON())
-                                },
-                                interval
-                            );
+                            // Timeout has been reached.
+                            let since = performance.now() - start;
+                            debug(`Ping timeout after ${since}ms status: ${response.status}`);
+                            this.declareServerError({ message: `Timeout after ${since}ms} status: ${response.status}` }, interval);
                             resolve();
                         }
                     })
-                    .catch(() => {
-                        // Just do nothing
+                    .catch(e => {
+                        debug(`Ping code: ${e.code} message: ${e.message}`);
+                        let since = performance.now() - start;
+                        if (notified === false && this.serverStatus === "unknown" && since > maxAttemptTimeMs) {
+                            // Timeout has been reached
+                            debug(`Ping timeout after ${maxAttemptTimeMs}ms`);
+                            this.declareServerError({ message: `Timeout after ${since}ms error: ${e}` }, interval);
+                            resolve();
+                        }
                     });
             }, 400);
         });
@@ -315,8 +333,21 @@ class CodeNarcCaller {
         }
     }
 
+    // Kill CodeNarc process if running.
+    killCodeNarcProcess() {
+        if (this.codeNarcProcess) {
+            this.codeNarcProcess.kill("SIGKILL");
+            delete this.codeNarcProcess;
+            return "CodeNarcServer killed";
+        }
+        return "";
+    }
+
     // Stop polling and log error
     declareServerError(e, interval) {
+        // Kill off the process as it is not responding.
+        this.killCodeNarcProcess();
+
         this.serverStatus = "error";
         if (interval) {
             clearInterval(interval);
@@ -327,10 +358,16 @@ class CodeNarcCaller {
         console.error(c.grey(errMsg));
     }
 
-    // Kill CodeNarc server by telling it to do so
+    // Kill CodeNarc server.
     async killCodeNarcServer() {
+        // Try by process first as it's more reliable.
+        let outputString = this.killCodeNarcProcess();
+        if (outputString) {
+            return outputString;
+        }
+
+        // Process kill wasn't possible, so try sending a kill http request.
         const serverUri = this.getCodeNarcServerUri() + "/kill";
-        let outputString = "";
         try {
             const response = await axios.post(serverUri, { timeout: 5000 });
             if (response.data.status === "killed") {
@@ -339,10 +376,12 @@ class CodeNarcCaller {
                 outputString = "Error killing CodeNarcServer";
             }
         } catch (e) {
-            if (e.stack.includes("connResetException")) {
+            if (e.stack.includes("connResetException") || e.message.includes("socket hang up")) {
                 outputString = "CodeNarcServer terminated";
             } else {
-                outputString = "CodeNarcServer was not running";
+                // This should be ECONNREFUSED.
+                debug(`CodeNarcServer kill request failed: ${e}`);
+                outputString = `CodeNarcServer was not running`;
             }
         }
         return outputString;

package/lib/codenarc-factory.js

@@ -56,10 +56,12 @@ async function prepareCodeNarcCall(options) {
 
     // Define base directory
     const baseBefore = (cnPath !== "." && cnPath.startsWith("/")) || cnPath.includes(":/") || cnPath.includes(":\\") ? "" : process.cwd() + "/";
-    const codeNarcBaseDir = positionalArgs.length > 0 ?
-        (await getCodeNarcBaseDirFromFiles(positionalArgs)) :
-        cnPath !== "." ? baseBefore + cnPath.replace(/^"(.*)"$/, "$1") :
-            process.cwd();
+    const codeNarcBaseDir =
+        positionalArgs.length > 0
+            ? await getCodeNarcBaseDirFromFiles(positionalArgs)
+            : cnPath !== "."
+            ? baseBefore + cnPath.replace(/^"(.*)"$/, "$1")
+            : process.cwd();
     result.codeNarcBaseDir = path.resolve(codeNarcBaseDir);
     result.codenarcArgs.push(`-basedir=${result.codeNarcBaseDir}`);
 
@@ -144,10 +146,10 @@ async function prepareCodeNarcCall(options) {
         result.outputType = result.output.endsWith(".txt")
             ? "txt"
             : result.output.endsWith(".json")
-                ? "json"
-                : result.output.endsWith(".sarif")
-                    ? "sarif"
-                    : result.output;
+            ? "json"
+            : result.output.endsWith(".sarif")
+            ? "sarif"
+            : result.output;
         result.codenarcArgs.push(`-report=json:stdout`);
     } else if (["html", "xml"].includes(result.output.split(".").pop())) {
         result.outputType = result.output
@@ -156,11 +158,11 @@ async function prepareCodeNarcCall(options) {
             .endsWith("html")
             ? "html"
             : result.output
-                .split(".")
-                .pop()
-                .endsWith("xml")
-                ? "xml"
-                : "";
+                  .split(".")
+                  .pop()
+                  .endsWith("xml")
+            ? "xml"
+            : "";
         const ext = result.output.split(".").pop();
         result.codenarcArgs.push(`-report=${ext}:${result.output}`);
 
@@ -183,9 +185,9 @@ async function prepareCodeNarcCall(options) {
 async function getCodeNarcBaseDirFromFiles(positionalArgs) {
     // All arguments are not files
     if (!positionalArgs.every(fileOrDirOrPattern => fs.existsSync(fileOrDirOrPattern) || directoryExists(fileOrDirOrPattern))) {
-        return process.cwd()
+        return process.cwd();
     }
-    const folders = positionalArgs.map((fileOrDir) => {
+    const folders = positionalArgs.map(fileOrDir => {
         // Dir
         if (directoryExists(fileOrDir)) {
             return path.resolve(fileOrDir);
@@ -195,7 +197,7 @@ async function getCodeNarcBaseDirFromFiles(positionalArgs) {
         return path.dirname(fileAbsolute);
     });
     const baseDirFromFiles = commondir(folders);
-    return baseDirFromFiles
+    return baseDirFromFiles;
 }
 
 // Parse XML result file as js object
@@ -327,7 +329,7 @@ async function parseCodeNarcResult(options, codeNarcBaseDir, codeNarcJsonResult,
                         }
                         // Add unprecise range based on line when range function has not been defined on rule
                         else if (errItem.line > 0) {
-                            const range = evaluateRangeFromLine(errItem,  allLines);
+                            const range = evaluateRangeFromLine(errItem, allLines);
                             if (range && range.start.character > -1) {
                                 errItem.range = range;
                             }
@@ -419,8 +421,8 @@ async function buildRuleSets(options) {
                 typeof ruleFromConfig === "object"
                     ? Object.assign(ruleFromConfig, ruleOptions)
                     : Object.keys(ruleOptions).length > 0
-                        ? ruleOptions
-                        : ruleFromConfig;
+                    ? ruleOptions
+                    : ruleFromConfig;
             const ruleDef = buildCodeNarcRule(ruleName, mergedRuleConfig);
             // If rule has been sent as argument, enable it by default
             if (ruleDef.enabled === false) {
@@ -441,8 +443,8 @@ async function buildRuleSets(options) {
                     typeof ruleDef === "object"
                         ? Object.assign(ruleDef, ruleFromRuleSetsArg)
                         : Object.keys(ruleFromRuleSetsArg).length > 0
-                            ? ruleFromRuleSetsArg
-                            : ruleDef;
+                        ? ruleFromRuleSetsArg
+                        : ruleDef;
             }
             // Add in the list of rules to test , except if it is disabled
             if (!(ruleDef === "off" || ruleDef.disabled === true || ruleDef.enabled === false)) {

package/lib/groovy-lint-fix.js

@@ -2,6 +2,7 @@
 const fse = require("fs-extra");
 const cliProgress = require("cli-progress");
 const debug = require("debug")("npm-groovy-lint");
+const trace = require("debug")("npm-groovy-lint-trace");
 const os = require("os");
 const { getNpmGroovyLintRules, getFormattingRulesToAlwaysRun } = require("./groovy-lint-rules.js");
 const { evaluateVariables, getSourceLines } = require("./utils.js");
@@ -135,7 +136,7 @@ class NpmGroovyLintFix {
                 return a.rule.priority > b.rule.priority ? 1 : a.rule.priority < b.rule.priority ? -1 : 0;
             });
         }
-        debug(`Parsed fixable errors: ${JSON.stringify(this.fixableErrors)}`);
+        trace(`Parsed fixable errors: ${JSON.stringify(this.fixableErrors)}`);
     }
 
     // Add fixable error but do not add twice if scope if the full file
@@ -256,7 +257,7 @@ class NpmGroovyLintFix {
             if (strBefore instanceof RegExp || (!strBefore.includes("{{") && !strAfter.includes("{{"))) {
                 newLine = newLine.replace(strBefore, strAfter);
             } else {
-                debug("GroovyLint: missing replacement variable(s):\n" + strBefore + "\n" + strAfter + "\n" + JSON.stringify(fixableError));
+                trace(`GroovyLint: missing replacement variable(s):\n${strBefore}\n${strAfter}\n${JSON.stringify(fixableError)}`);
             }
         }
         // Function defined in rule
@@ -267,7 +268,7 @@ class NpmGroovyLintFix {
                 }
                 newLine = fix.func(newLine, evaluatedVars, fixableError);
             } catch (e) {
-                debug("ERROR: Fix function error: " + e.message + " / " + JSON.stringify(fixableError));
+                trace(`ERROR: Fix function error: ${e.message} / ${JSON.stringify(fixableError)}`);
                 throw e;
             }
         }

package/lib/groovy-lint.js

@@ -1,5 +1,6 @@
 // Imports
 const debug = require("debug")("npm-groovy-lint");
+const trace = require("debug")("npm-groovy-lint-trace");
 const fs = require("fs-extra");
 const os = require("os");
 const performance = require("perf_hooks").performance;
@@ -139,16 +140,15 @@ class NpmGroovyLint {
                     }
                 }
                 // Try to catch input from stdin. if found, use it as groovy source
-                if (this.options._ && this.options._[0] === '-') {
-                    const stdInData = fs.readFileSync(0, 'utf-8');
+                if (this.options._ && this.options._[0] === "-") {
+                    const stdInData = fs.readFileSync(0, "utf-8");
                     this.options.source = stdInData;
                     this.options._ = [];
                     this.options.sourcefilepath = this.options.sourcefilepath || process.cwd();
                     if (this.options.format || this.options.fix) {
-                        this.options.output = 'stdout';
+                        this.options.output = "stdout";
                     }
                 }
-
             } catch (err) {
                 this.status = 2;
                 this.error = {
@@ -231,7 +231,7 @@ class NpmGroovyLint {
         - Call CodeNarcServer via Http (except if --noserver)
         - Launch CodeNarcServer using com.nvuillam.CodeNarcServer, then call CodeNarcServer via Http (except if --noserver)
         - Call CodeNarc java using com.nvuillam.CodeNarcServer (without launching server)
-        - Call CodeNarc java using org.codenarc.CodeNarc 
+        - Call CodeNarc java using org.codenarc.CodeNarc
     */
     async callCodeNarc() {
         const startPerf = performance.now();
@@ -358,7 +358,7 @@ class NpmGroovyLint {
         }
         lintAgainOptions.fix = false;
         lintAgainOptions.output = "none";
-        debug(`Fix is done, lint again with options ${JSON.stringify(lintAgainOptions)}`);
+        trace(`Fix is done, lint again with options ${JSON.stringify(lintAgainOptions)}`);
         const newLinter = new NpmGroovyLint(lintAgainOptions, {
             parseOptions: false,
             origin: "lintAgainAfterFix"
@@ -469,18 +469,18 @@ class NpmGroovyLint {
     manageReturnCode() {
         if (this.status > 1) {
             // There has been a fatal error before, so there are no results
-            return ;
+            return;
         }
         const failureLevel =
             this.options.failon && this.options.failon !== "none"
                 ? this.options.failon
                 : this.options.failonerror
-                    ? "error"
-                    : this.options.failonwarning
-                        ? "warning"
-                        : this.options.failoninfo
-                            ? "info"
-                            : "none";
+                ? "error"
+                : this.options.failonwarning
+                ? "warning"
+                : this.options.failoninfo
+                ? "info"
+                : "none";
         if (failureLevel === "none") {
             return;
         }

package/lib/options.js

@@ -177,7 +177,7 @@ module.exports = optionator({
         {
             option: "serverhost",
             type: "String",
-            default: "http://localhost", //"http://" + require("ip").address(),
+            default: "http://localhost",
             description: "If use of CodeNarc server, host where is the CodeNarc server (default: localhost)"
         },
         {

package/lib/output.js

@@ -109,12 +109,12 @@ async function processOutput(outputType, output, lintResult, options, fixer = nu
         for (const fileNm of Object.keys(lintResult.files)) {
             const fileErrors = lintResult.files[fileNm].errors;
             let fileOutputString = c.underline(fileNm) + "\n";
-            let showFileInOutput = false ;
+            let showFileInOutput = false;
             for (const err of fileErrors) {
                 if (!isErrorInLogLevelScope(err.severity, options.loglevel)) {
                     continue;
                 }
-                showFileInOutput = true ;
+                showFileInOutput = true;
                 let color = "grey";
                 switch (err.severity) {
                     case "error":
@@ -150,7 +150,7 @@ async function processOutput(outputType, output, lintResult, options, fixer = nu
             }
             fileOutputString += "\n";
             if (showFileInOutput || options.verbose) {
-                outputString += fileOutputString ;
+                outputString += fileOutputString;
             }
         }
         outputString += "\nnpm-groovy-lint results in " + c.bold(lintResult.summary.totalFilesLinted) + " linted files:";

package/lib/utils.js

@@ -2,6 +2,7 @@
 "use strict";
 
 const debug = require("debug")("npm-groovy-lint");
+const trace = require("debug")("npm-groovy-lint-trace");
 const decodeHtml = require("decode-html");
 const fse = require("fs-extra");
 const os = require("os");
@@ -124,7 +125,7 @@ function evaluateRange(errItem, rule, evaluatedVars, errLine, allLines) {
 
 // Get position to highlight in sources
 function evaluateRangeFromLine(errItem, allLines) {
-    return getDefaultRange(allLines, errItem)
+    return getDefaultRange(allLines, errItem);
 }
 
 // Evaluate variables from messages
@@ -142,14 +143,14 @@ function evaluateVariables(variableDefs, msg) {
                     varDef.type && varDef.type === "number"
                         ? parseInt(value, 10)
                         : varDef.type && varDef.type === "array"
-                            ? JSON.parse(value)
-                            : value;
+                        ? JSON.parse(value)
+                        : value;
                 evaluatedVars.push({
                     name: varDef.name,
                     value: varValue
                 });
             } else {
-                debug("GroovyLint: Unable to match " + varDef.regex + " in " + msg);
+                trace(`GroovyLint: Unable to match ${varDef.regex} in ${msg}`);
             }
         }
     }

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "npm-groovy-lint",
-  "version": "11.1.0",
+  "version": "11.1.2-beta202310250645.0",
   "description": "Lint, format and auto-fix your Groovy / Jenkinsfile / Gradle files",
   "main": "index.js",
   "scripts": {
     "lint:fix": "eslint **/*.js --fix && prettier --write \"./lib/**/*.{js,jsx}\" --tab-width 4 --print-width 150",
-    "groovy:run-server-from-source": "npm run dev:kill-server && groovy -cp \"lib/java/CodeNarc-3.1.0.jar;lib/java/groovy/lib/groovy-3.0.9.jar;lib/java/groovy/lib/groovy-templates-3.0.9.jar;lib/java/groovy/lib/groovy-xml-3.0.9.jar;lib/java/groovy/lib/groovy-json-3.0.9.jar;lib/java/groovy/lib/groovy-ant-3.0.9.jar;lib/java/groovy/lib/ant-1.10.11.jar;lib/java/groovy/lib/ant-launcher-1.10.11.jar;lib/java/slf4j-api-1.7.9.jar;lib/java/log4j-slf4j-impl-2.18.0.jar;lib/java/log4j-api-2.18.0.jar;lib/java/log4j-core-2.18.0.jar;lib/java/GMetrics-2.1.0.jar\" groovy/src/main/com/nvuillam/CodeNarcServer.groovy --server",
-    "groovy:build": "npm run dev:kill-server && groovyc -cp \"./lib/java*\" --encoding utf-8 ./groovy/src/main/com/nvuillam/CodeNarcServer.groovy -d ./tmp && cd ./tmp && jar -cvfm ./../lib/java/CodeNarcServer.jar ./../MANIFEST.txt ./com/nvuillam/*.class && cd ..",
+    "groovy:run-server-from-source": "npm run dev:kill-server && groovy -cp \"lib/java/*\" groovy/src/main/com/nvuillam/CodeNarcServer.groovy --server",
+    "groovy:build": "npm run dev:kill-server && groovyc -cp \"lib/java/*\" --encoding utf-8 ./groovy/src/main/com/nvuillam/CodeNarcServer.groovy -d ./tmp && cd ./tmp && jar -cvfm ./../lib/java/CodeNarcServer.jar ./../MANIFEST.txt ./com/nvuillam/*.class && cd ..",
     "test": "npm run dev:kill-server && mocha \"test/**/*.test.js\"",
     "test:coverage": "nyc npm run test",
     "test:debug": "npm run dev:kill-server && mocha --reporter spec --inspect-brk \"test/**/*.test.js\"",
@@ -48,7 +48,7 @@
     "ansi-colors": "^4.1.1",
     "axios": "^0.21.1",
     "chalk": "^4.1.2",
-    "cli-progress": "^3.10.0",
+    "cli-progress": "^3.12.0",
     "commondir": "^1.0.1",
     "debug": "^4.1.1",
     "decode-html": "^2.0.0",
@@ -60,25 +60,25 @@
     "ip": "^1.1.5",
     "java-caller": "^2.2.4",
     "js-yaml": "^4.1.0",
-    "node-sarif-builder": "^2.0.1",
+    "node-sarif-builder": "^2.0.3",
     "optionator": "^0.8.3",
     "semver": "^7.1.3",
     "strip-json-comments": "^3.0.1",
     "uuid": "^8.2.0"
   },
   "devDependencies": {
-    "@babel/core": "^7.16.0",
-    "@babel/eslint-parser": "^7.16.3",
+    "@babel/core": "^7.23.2",
+    "@babel/eslint-parser": "^7.22.15",
     "diff": "^4.0.2",
-    "eslint": "^8.2.0",
-    "mocha": "^7.0.1",
+    "eslint": "^8.52.0",
+    "mocha": "^10.2.0",
     "nyc": "^15.1.0",
     "prettier": "^1.19.1",
     "rimraf": "^3.0.2",
     "which": "^2.0.2"
   },
   "engines": {
-    "node": ">=12.0.0"
+    "node": ">=18.0.0"
   },
   "mocha": {
     "require": [

package/CHANGELOG.md

@@ -1,619 +0,0 @@
-# Changelog
-
-## UNRELEASED
-
-## [11.1.0] 2022-10-31
-
-- Provide default range when only information available is a line number ([#248](https://github.com/nvuillam/npm-groovy-lint/issues/248))
-- New CodeNarc issues definition to calculate range in file
-  - GStringExpressionWithinString
-  - VariableName
-
-## [11.0.0] 2022-10-07
-
-- **BREAKING CHANGE**:`--failon` is now `ìnfo` by default, meaning exit code will be `1` if there is at least an info issue found. To have previous behaviour, use `--failon none`.
-- Display all files in console log only if `--verbose` is used ([#243](https://github.com/nvuillam/npm-groovy-lint/issues/243))
-
-## [10.1.0] 2022-08-15
-
-- Allow to send groovy sources as input from stdin
-- If `--format` or `--fix` option is used when source is sent as stdin, the result is output as stdout
-
-Example: `cat path/to/my/Jenkinsfile | npm-groovy-lint --format -`
-
-## [10.0.3] 2022-08-15
-
-- Do not output results summary in console logs when output is json or sarif
-- Add test methods for SARIF called by CLI
-
-## [10.0.2] 2022-08-15
-
-- Fix error when absolute files sent as positional arguments on a linux system ([#232](https://github.com/nvuillam/npm-groovy-lint/issues/232))
-- Improve performances by calculating the longest command directory to send as base path to CodeNarc
-
-## [10.0.1] 2022-08-14
-
-- Fix error when files sent as positional arguments ([#232](https://github.com/nvuillam/npm-groovy-lint/issues/232))
-
-## [10.0.0] 2022-08-13
-
-- Core
-  - Upgrade to [CodeNarc v3.1.0](https://github.com/CodeNarc/CodeNarc/blob/master/CHANGELOG.md#version-310----jun-2022)
-  - Accept list of directories / files as arguments (`--path` and `--files` become deprecated but are still usable)
-    - Examples
-      - Multiple files: `npm-groovy-lint path/to/file1.groovy path/to/file2.groovy`
-      - Directory: `npm groovy-lint path/to`
-      - Single file: `npm-groovy-lint Jenkinsfile`
-      - Ant pattern(s): `npm-groovy-lint path/to/**/*.groovy`
-  - Upgrade npm dependencies
-  - CI: Upgrade [MegaLinter](https://oxsecurity.github.io/megalinter/latest/) to v6
-  - Test classes for collecting error ranges
-
-- New error ranges rules
-  - DuplicateNumberLiteral
-  - DuplicateStringLiteral
-  - MethodParameterTypeRequired
-  - NoDef
-  - SimpleDateFormatMissingLocale
-  - SpaceInsideParenthesis
-  - UnnecessaryPublicModifier
-  - VariableTypeRequired
-
-- New fix rules
-  - SpaceAfterMethodCallName
-  - SpaceInsideParentheses
-
-- Bug Fixes
-  - Wrongly calculated ranges are no more returned
-
-## [9.5.0] 2022-04-12
-
-- Disable Amplitude anonymous stats by default (use `--insight` to enable them)
-
-## [9.4.1] 2022-01-12
-
-- Upgrade node-sarif-builder to 2.0.1 and send npm-groovy-lint version in SARIF logs
-- New range detection for rules:
-  - MethodReturnTypeRequired
-  - UnusedImport
-
-## [9.4.0] 2022-01-11
-
-- Add [SARIF](https://sarifweb.azurewebsites.net/) output format using [node-sarif-builder](https://github.com/nvuillam/node-sarif-builder)
-
-## [9.3.2] 2022-01-09
-
-- Upgrade cli-progress to avoid [colors lib boring but harmless hack](https://github.com/Marak/colors.js/issues/285)
-
-## [9.3.1] 2022-01-06
-
-- Fix issue when used as module and with file containing spaces ([VsCodeGroovyLint #137](https://github.com/nvuillam/vscode-groovy-lint/issues/137))
-
-## [9.3.0] 2021-12-29
-
-- Upgrade log4j dependencies because of [security issue](https://nvd.nist.gov/vuln/detail/CVE-2021-44832).
-
-## [9.2.0] 2021-12-23
-
-- Upgrade log4j dependencies because of (another) [security issue](https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-45105) (#196)
-
-## [9.1.0] 2021-12-14
-
-- Upgrade log4j dependencies because of [security issue](https://unit42.paloaltonetworks.com/apache-log4j-vulnerability-cve-2021-44228/) (#194)
-
-## [9.0.0] 2021-09-30
-
-- Upgrade to [CodeNarc v2.2.0](https://github.com/CodeNarc/CodeNarc/releases/tag/v2.2.0)
-- Upgrade to [Groovy v3.0.9](https://groovy-lang.org/)
-- Reorganize sources test folder
-- Use java from 8 to 14 (install it if a later version is found)
-- Add timeouts in CI jobs
-- Add colors in error logs
-
-## [8.2.0] 2021-06-11
-
-- Upgrade dependencies advised by dependabot
-- Fix Mega-Linter errors found
-
-## [8.1.0] 2020-12-14
-
-- Exclude `UnnecessaryGetter`, `FactoryMethodName`, `MethodReturnTypeRequired`, and `GStringExpressionWithinString` in `recommended-jenkinsfile` ([#140](https://github.com/nvuillam/npm-groovy-lint/pull/140)) ([Felipe Santos](https://github.com/felipecrs))
-
-## [8.0.2] 2020-11-26
-
-- Fix documentation about --verbose and --version options
-- Use GitHub action to deploy to NPM
-
-## [8.0.1] 2020-11-19
-
-- Add a test case where variable includes if in its name for fix of rule SpaceAfterIf ([Behlül Uçar](https://github.com/ucarbehlul))
-
-## [8.0.0] 2020-11-15
-
-- Upgrade to CodeNarc 2.0.0
-  - Upgrade jars
-  - Adapt Indentation rule to new behaviour
-  - Use codenarc --ruleset argument instead of temp ruleset file
-- Replace super-linter by [Mega-Linter](https://nvuillam.github.io/mega-linter/)
-- Fixes
-  - [(#127)](https://github.com/nvuillam/npm-groovy-lint/issues/127) Formatting breaks code, converting `else if` into `elseif` in some cases
-  - Apply formatting rules also for --fix mode
-  - Fix Markdown dead links
-- CI
-  - Migrate from CircleCI to GitHub Actions: now tests are on Linux, Windows & MacOs
-  - Activate spelling linter of Mega-Linter (+ `.cspell.json` file)
-  - Add test cases for format and fix using CLI
-
-## [7.6.2] 2020-09-09
-
-- Disable TrailingComma rule by default until crash is solved in [CodeNarc](https://codenarc.github.io/CodeNarc) ([#75@vscode-groovy-lint](https://github.com/nvuillam/vscode-groovy-lint/issues/75))
-
-## [7.6.0] 2020-09-08
-
-- Add GitHub Action [GitHub Super-Linter](https://github.com/marketplace/actions/super-linter) to the repository
-- Update Dockerfile to pass Docker lint rules
-
-## [7.5.5] 2020-09-05
-
-- Upgrade [java-caller](https://github.com/nvuillam/node-java-caller) to v2.2.3
-  - Fix Java 8 detection ([#101](https://github.com/nvuillam/npm-groovy-lint/issues/101))
-
-## [7.5.4] 2020-09-04
-
-- Update frameworks detection
-
-## [7.5.1] 2020-09-02
-
-- Fix [(#96)](https://github.com/nvuillam/npm-groovy-lint/issues/96) --fix adds redundant space into `${VARIABLE}` (SpaceBeforeOpeningBrace fix rule error)
-- Fix grails framework detection
-- Fix Groovy parsing parsing when multiple files
-- Add `.gvy` and `.nf` in default browsed files extensions
-
-## [7.5.0] 2020-09-02
-
-- Add anonymous framework usage stats for Groovy core Team
-
-## [7.4.3] 2020-08-29
-
-- Upgrade [java-caller](https://github.com/nvuillam/node-java-caller) to v2.2.0
-  - Fix CLASSPATH on windows in case there are spaces in paths
-
-## [7.4.2] 2020-08-26
-
-- Fix [(#90)](https://github.com/nvuillam/npm-groovy-lint/issues/90) When log level is specified number of linted files appear to be off
-
-## [7.4.1] 2020-08-23
-
-- [(#88)](https://github.com/nvuillam/npm-groovy-lint/pull/88) Fix Docker image to allow to use extra parameters (by [Howard Lo](https://github.com/warhod))
-
-## [7.4.0] 2020-08-17
-
-- [(#87)](https://github.com/nvuillam/npm-groovy-lint/pull/87) update to openjdk 11 in [official Docker image](https://hub.docker.com/r/nvuillam/npm-groovy-lint) (by [Pawel Kopka](https://github.com/pawelkopka))
-
-## [7.3.1] 2020-08-16
-
-- Add number of lines & reorganize anonymous statistics
-- Fix error type counters in anonymous statistics
-
-## [7.3.0] 2020-08-15
-
-- Allow to link to [CodeNarc RuleSet files](https://codenarc.github.io/CodeNarc/codenarc-creating-ruleset.html) from `.groovylintrc.json`, using property `"codenarcRulesets"`. Warning: doing so means that all other properties of config file will be ignored.
-
-## [7.1.1] 2020-08-11
-
-- Upgrade [java-caller](https://www.npmjs.com/package/java-caller) to v2.0.0
-
-## [7.1.0] 2020-08-10
-
-- Externalize JavaCaller class into a separate package [java-caller](https://www.npmjs.com/package/java-caller) and use it
-
-## [7.0.0] 2020-08-07
-
-- New default recommended rules (activate/deactivate/change severity)
-- Allow to call `--config recommended-jenkinsfile` to use delivered .groovylintrc-recommended-jenkinsfile.json
-
-## [6.1.1] 2020-08-04
-
-- Fix SpaceAfterComma auto-fixing rule
-
-## [6.1.0] 2020-08-04
-
-- Java 14 compatibility (Closes [#77](https://github.com/nvuillam/npm-groovy-lint/issues/7))
-
-## [6.0.0] 2020-08-03
-
-- Upgrade to [CodeNarc v1.6.1](https://github.com/CodeNarc/CodeNarc/blob/v1.6-patch-releases/CHANGELOG.md#version-161----aug-2020)
-  - Update list of rules
-  - Use new CodeNarc JSON console Output instead of temporary XML files
-  - Call CodeNarc to get its version instead of using npm-groovy-lint hardcoded value
-- Upgrade to [GMetrics v1.1](https://github.com/dx42/gmetrics/blob/master/CHANGELOG.md#version-11-may-2020)
-- Upgrade to [Groovy v3.0.5](https://groovy-lang.org/)
-- Add anonymous usage stats on types of errors found and fixed
-
-## [5.8.0] 2020-08-01
-
-- Fix & enhance anonymous statistics
-
-## [5.7.0] 2020-07-23
-
-- [(#62)](https://github.com/nvuillam/npm-groovy-lint/pull/74) Check parse error in all files when called via CLI . Closes [#69](https://github.com/nvuillam/npm-groovy-lint/issues/69)
-
-## [5.6.1] 2020-07-20
-
-Fixes:
-
-- [(#62)](https://github.com/nvuillam/npm-groovy-lint/issues/62) using a codenarc ruleset file seems to fail / groovylintrc is not codenarc compatible
-
-## [5.6.0] 2020-07-20
-
-- [(#68)](https://github.com/nvuillam/npm-groovy-lint/pull/68) Generate a [ready-to-use docker image](https://hub.docker.com/r/nvuillam/npm-groovy-lint) when publishing new npm-groovy-lint version (by [Dave Gallant](https://github.com/davegallant))
-
-## [5.5.1] 2020-07-15
-
-- Fixes
-  - [(#64)](https://github.com/nvuillam/vscode-groovy-lint/issues/64) The contents of a string gets formatted unexpectedly
-
-## [5.5.0] 2020-07-09
-
-- Allow to override java executable and options [(#54)](https://github.com/nvuillam/vscode-groovy-lint/issues/54)
-
-## [5.4.2] 2020-07-09
-
-- Use os.EOL [(#65)](https://github.com/nvuillam/npm-groovy-lint/pull/65) solving  [(#63)](https://github.com/nvuillam/npm-groovy-lint/issues/63) --fix for indentation adds CRLF line-endings to all files it touches
-
-## [5.4.1] 2020-07-01
-
-- CodeNarcServer listens to localhost only [(#59)](https://github.com/nvuillam/npm-groovy-lint/pull/59) solving [(#56)](https://github.com/nvuillam/npm-groovy-lint/issues/56)
-- Replace @analytics/segment with @amplitude/node for anonymous stats
-
-## [5.3.0] 2020-06-29
-
-- New option **--failon** , replacing `--failonerror`,`--failonwarning` and `--failoninfo`. It can take error, warning or info values (default: none). Previous options remain working but are deprecated and will be removed in a future major version
-- Update help for `--fixrules` option
-
-## [5.1.0] 2020-06-04
-
-- Install Java 8 using node-jre in case java version found is higher than Java 11 (CodeNarc compatibility is Java 8 to 11)
-
-## [5.0.3] 2020-05-30
-
-- Updated fix rules
-  - Indentation
-  - IndentationClosingBrace
-
-## [5.0.2] 2020-05-27
-
-- Avoid to apply wrong fix in case of CodeNarc false positive
-- New fix rules
-  - BlankLineBeforePackage
-- Updated fix rules
-  - BracesForIfElse
-  - BracesForMethod
-  - BracesForTryCatchFinally
-  - ClassEndsWithBlankLine
-  - ClassStartsWithBlankLine
-  - MissingBlankLineAfterImports
-  - MissingBlankLineAfterPackage
-  - UnnecessaryGroovyImport
-  - UnusedImport
-
-## [5.0.0] 2020-05-25
-
-- **BIG BANG**: Improve performances, compatibility, architecture and delivery
-  - Get rid of [jDeploy](https://github.com/shannah/jdeploy) dependency
-    - Use own **java-caller.js** for java commands
-    - Update CircleCI config to use `npm link`instead of `jdeploy install`
-  - Get rid of [request](https://github.com/request/request) dependency
-    - Use [axios](https://github.com/axios/axios) for promisified http calls
-
-## [4.14.0] 2020-05-22
-
-- Send rule configuration to fix functions
-- Add `.gradle` files in default linted files
-- Fixes:
-  - Missing number of linted files returned in summary
-  - Try to call CodeNarcJava in case there is an error with CodeNarcServer call
-
-## [4.13.0] 2020-05-20
-
-- Manage to send options for rules sent in `rulesets`: Ex: `Indentation{"spacesPerIndentLevel":2,"severity":"warning"},UnnecessarySemicolon`
-- New parameter `--rulesetsoverridetype` : If list of rules sent in rulesets option, defines if they replace rules defined in .groovylintrc.json, or if they are appended
-
-## [4.12.0] 2020-05-18
-
-- Improve performances and avoid `Unknown command: node` error by using childProcess.fork to call CodeNarcServer
-
-## [4.11.1] 2020-05-16
-
-- Detect when crash is related to "node" or "java" command not found and return a human readable error message
-
-## [4.11.0] 2020-05-13
-
-- Add CI , rule overrides and crashes in anonymous insights for debugging investigation
-- When used as a module, **never crash intentionally with throw**, so when called by module, check linter.status and linter.error instead of try/catch
-  - 0: ok
-  - 1: expected error
-  - 2: unexpected error
-  - 9: if cancelled request
-
-## [4.10.0] 2020-05-12
-
-- Update analytics to use [analytics](https://www.npmjs.com/package/analytics) & [@analytics-segment](https://github.com/DavidWells/analytics/tree/master/packages/analytics-plugin-segment) instead of [insight](https://www.npmjs.com/package/insight). If you want to disable anonymous usage stats, you can still use `--no-insight` option.
-
-## [4.9.0] 2020-05-10
-
-- Add anonymous usage statistics using [insight](https://www.npmjs.com/package/insight), in order to make new improvements based on how users use this package.
-  - Note: Analytics obviously does not receive sensitive information like your code, as you can see in [analytics.js](https://github.com/nvuillam/npm-groovy-lint/blob/master/lib/analytics.js). If you want to disable anonymous usage stats, use `--no-insight` option.
-
-## [4.8.0] 2020-05-08
-
-- New fix rules
-  - AssignmentInConditional
-  - DuplicateImport
-  - ExplicitLinkedListInstantiation
-  - InsecureRandom
-  - UnnecessaryDefInVariableDeclaration
-  - UnnecessaryDotClass
-  - UnnecessaryFinalOnPrivateMethod
-  - UnnecessaryInstantiationToGetClass
-
-- Updated fix rules
-  - BracesForForLoop: False positive triggering messy code after fixing
-  - UnnecessaryGString: Fix multiline replacements ( `"""` by `'''` )
-
-- Fixes :
-  - Launch JVM with high memory (`-Xms256m -Xmx2048m`) to improve performances on big files
-  - Increase CodeNarcServ call timeout (+ Manage ETIMEOUT as result, not only ECONNREFUSED )
-
-- Utils
-  - Allow regex in range functions
-
-## [4.7.0] 2020-05-06
-
-- New fix rules
-  - BracesForClass
-  - BracesForForLoop
-  - BracesForIfElse
-  - BracesForMethod
-  - BracesForTryCatchFinally
-  - ExplicitArrayListInstantiation
-  - MissingBlankLineAfterImports
-  - MissingBlankLineAfterPackage
-
-- Updated fix rules
-  - UnnecessaryGString: Fix replacements containing `\n` and `\r`
-
-## [4.6.0] 2020-05-01
-
-- New fix rules
-  - SpaceBeforeClosingBrace
-  - UnnecessaryDefInMethodDeclaration
-  - UnnecessaryPackageReference
-  - UnnecessaryParenthesesForMethodCallWithClosure
-
-- Updated fix rules
-  - MisorderedStaticImports: Fix `@Grapes` killer fixing rule
-  - ElseBlockBrace :issue when instruction is on the same line than `else`
-
-## [4.5.5] 2020-04-30
-
-- Fixes
-  - ignorepattern option not working [#34](https://github.com/nvuillam/npm-groovy-lint/issues/34)
-
-## [4.5.2] 2020-04-29
-
-- Expose `loadConfig()` method to load rules when npm-groovy-lint is used as a library
-- Fixes
-  - Missing temporary rulesets file missing
-  - Handle better CodeNarcServer concurrent calls
-
-## [4.5.1] 2020-04-28
-
-- Fixes
-  - Take in account user overridden indentation space (and other rules) when using --format option [#31](https://github.com/nvuillam/npm-groovy-lint/issues/31)
-
-## [4.5.0] 2020-04-24
-
-- Configuration updates ([#29](https://github.com/nvuillam/npm-groovy-lint/issues/29)):
-  - New default config "recommended-jenkinsfile". Use it with argument `--config recommended-jenkinsfile`
-  - Allow to directly target a config file name. Use it with argument `--config /my/custom/path/.groovylintrc-custom-name.json`
-  - Allow to send a string key that will be used to find config file `--config custom-name`
-- Updated fix rules:
-  - IfStatementBraces
-  - ElseStatementBraces
-
-## [4.4.1] 2020-04-16
-
-- Fixes:
-  - CodeNarcServer: Use cachedThreadPool instead of fixedThreadPool
-
-## [4.4.0] 2020-04-16
-
-- Cancel a CodeNarc Lint when a similar CodeNarcServer request is received (allowing onType mode for language servers)
-
-## [4.3.0] 2020-04-14
-
-- Allow to disable rules using comments in source in [eslint style](https://eslint.org/docs/user-guide/configuring#disabling-rules-with-inline-comments)
-
-## [4.2.0] 2020-04-13
-
-- New option **--parse**: Capability to parse source code and return compilation errors
-- New fix rules (thanks [CatSue](https://github.com/CatSue) !):
-  - SpaceAfterSemicolon
-  - SpaceAfterWhile
-
-## [4.1.0] 2020-04-12
-
-- Upgrade to [Groovy 3.0.3](https://dl.bintray.com/groovy/maven/apache-groovy-binary-3.0.3.zip)
-- Automatic generation of fixable rules list for README
-- Refactor CodeNarcServer.groovy
-
-## [4.0.0] 2020-04-09
-
-- Fix [issue](https://github.com/nvuillam/vscode-groovy-lint/issues/16) affecting performances on Linux and MacOs
-
-## [3.3.0] 2020-04-06
-
-- When formatting, always run some custom npm-groovy-lint fix rules not corresponding to CodeNarc violations
-- Return CodeNarc and Groovy versions when --version options is called
-- Fixes
-  - Lost indentation when applying some fix rules
-- Updated fix rules:
-  - IndentationClosingBraces
-  - IndentationComments
-  - SpaceAfterCatch
-  - SpaceAfterIf
-- New fix rules:
-  - ClassEndsWithBlankLine
-  - ClassStartsWithNewLine
-  - SpaceAfterFor
-  - SpaceAfterSwitch
-
-## [3.2.4] 2020-04-03
-
-- Error message in postinstall if env Node.js is lower than the minimal required (12)
-
-## [3.2.3] 2020-04-02
-
-- When If or Else brackets are fixed,trigger another rule lint & fix only with Indentation rules so CodeNarc recalculate them correctly
-- New option `nolintafter`: When format or fix is called, a new lint is performed after the fixes to update the error list. If you just want the updated source code and do not care about the error logs, use this parameter to improve performances
-- Fixes
-  - Manage correctly options `failonerror`, `failonwarning` and `failoninfo`
-  - `npm-groovy-lint -version` now returns version from package.json
-- Mocha tests updates:
-  - Add stats on calls to CodeNarc (`globalThis.codeNarcCallsCounter` and `globalThis.codeNarcCalls`, activated if `globalThis.codeNarcCallsCounter` is set to 0 before calling NmpGroovyLint)
-  - Factorize test classes common code in module helper/common.js
-  - Use a smaller groovy file for test classes when not impacting the tests quality
-
-## [3.2.2] 2020-03-31
-
-- New option **returnrules** if you want to return rules descriptions and documentation url in results
-- Use npm ci instead of npm install in CircleCI build
-
-## [3.2.1] 2020-03-29
-
-- Return rules descriptions in results
-- New option **nolintafter**: do not lint again a format or a fix, as the client prefers to request it
-- Fixes
-  - [Issue #13](https://github.com/nvuillam/npm-groovy-lint/issues/13): False positive error ClassNameSameAsFileName
-  - Sometimes returning wrong .groovylint.json config file
-
-## [3.2.0] 2020-03-26
-
-- New option "--format", allowing to reformat source code (using .groovylintrc-format.json)
-- Update default recommended rules
-
-## [3.1.3] 2020-03-22
-
-- README: Link to [Visual Studio Code Groovy Lint extension](https://marketplace.visualstudio.com/items?itemName=NicolasVuillamy.vscode-groovy-lint)
-
-## [3.1.2] 2020-03-22
-
-- New Fix rules:
-  - BlockEndsWithBlankLine
-  - BlockStartsWithBlankLine
-  - MisorderedStaticImports
-  - SpaceAfterIf
-- Fix: Update correctly the lineNb & ranges of next errors after an error has been fixed
-- Do not return rules tests if call is not from a test file
-- Fix rules:
-  - ElseBlockBraces
-  - IfStatementBraces
-  - SpaceAroundOperator
-  - UnusedImport
-
-## [3.1.1] 2020-03-20
-
-- New Fix rules:
-  - BlockEndsWithBlankLine
-  - BlockStartsWithBlankLine
-  - MisorderedStaticImports
-  - SpaceAfterIf
-- Updated Fix rules:
-  - SpaceAroundOperator
-
-## [3.1.0] 2020-03-18
-
-- Test suites: Improve reliability & logs for rule fixes tests (detected numerous bugs, now corrected)
-- Send computed range to fix functions
-- Updated Fix rules:
-  - ClosingBraceNotAlone
-  - ElseBlockBraces
-  - IfStatementBraces
-  - NoTabCharacter
-  - SpaceAfterComma
-  - SystemExit
-  - TrailingWhitespace
-  - UnnecessaryGroovyImport
-  - UnusedImport
-
-## [3.0.1] 2020-03-17
-
-- Add new test suites: errors.test.js and miscellaneous.test.js
-- Use JSON as default GroovyLint configuration file type
-- Order of fixable rules must be defined in groovy-lint-rules.js
-- Do not load rules test data except during tests
-- Do not lint again after a call to fixErrors on an existing NpmGroovyLint instance (except if lintAgainAfterFix : true is sent in options)
-- Fix: Deletion of temp RuleSite file
-- Fix: UnnecessarySemiColon rule
-- Fix: ClosingBraceNotAlone rule
-
-## [3.0.0] 2020-03-15
-
-- Local microservice "CodeNarcServer" called via Http by npm-groovy-lint, to avoid loading all groovy/java classes at each lint request. This microservice auto-kills itself after one hour idle.
-- Capability to define RuleSets in argument or js/json/yml config file formats instead of groovy/xml RuleSet format
-- Test classes for rules fix (before / after fix defined in rule definitions)
-- Add debug logs (use it by setting DEBUG env variable , ex: `DEBUG=npm-groovy-lint npm-groovy-lint args...`)
-- Update lines and ranges of other errors after a fix updated the number of lines
-- Generate automatically .groovylintrc-all.js during build
-- Split rules definition into files instead of all in a huge single file
-- Reorganize groovy-lint.js code, using codenarc-factory.js and codenarc-caller.js
-- New lib utils.js that can be used by rules definition
-- Fix: Crash when there was no error found in a file
-- Fix: Remove Promise error display in log after launching CodeNarcServer
-- Fix: Add more options exclusivity arguments rules
-- Removed "Groovy", "Jenkinsfile" and "All" identifiers for --rulesets arguments. Please now use .groovylintrc.js configuration
-
-## [2.2.0] 2020-02-28
-
-- Capability to call NpmGroovyLint with options as object, not only command line arguments
-- New option "source", allowing to call NpmGroovyLint with the groovy code as a string , not only path & files pattern
-- Run lint again after fix all errors, to get updated lintResult
-- Add gitattributes to normalized line-endings (thanks [docwhat](https://github.com/docwhat))
-- API: fix only some errors after an initial lint (better performances)
-- Return error range in files for some rules in JSON or API result (will be useful for the VsCode extension currently developed)
-- Define range function for existing rules, new fixable rules
-
-## [2.0.1] - 2020-02-21
-
-- Capability to fix errors
-  - ConsecutiveBlankLines
-    - Indentation (IfStatementBraces and ElseIfStatementBraces must be manually fixed to have correct indentation)
-    - IndentationComments (custom npm-groovy-rule triggered by Indentation rule)
-    - IndentationClosingBrace (custom npm-groovy-rule triggered by Indentation rule)
-    - NoTabCharacter
-    - SpaceAfterCatch
-    - SpaceAfterOpeningBrace
-    - SpaceAroundOperator
-    - SpaceAfterComma
-    - SpaceBeforeOpeningBrace
-    - UnnecessaryDefInFieldDeclaration
-    - UnnecessaryGString
-    - UnnecessaryPublicModifier
-    - UnnecessarySemicolon
-    - TrailingWhitespace
-- Log formatting with severity colors and summary table
-- Default recommended RuleSets for Groovy and Jenkins
-- Progress bar in console
-- More code coverage with test campaigns
-- New Capability to call NpmGroovyLint from another package (VsCode extension development in progress ^^)
-- Refactored command line arguments ( simpler, but different from CodeNarc ones : retro-compatibility with CodeNarc arguments assured if you add --codenarcargs)
-- Upgrade to CodeNarc v1.5
-- Upgrade to Groovy v3.0.1
-- Refactored documentation with detailed arguments description & examples
-
-___
-
-## Before
-
-- I wasn't serious enough to keep a changelog, sorry !