npm-groovy-lint 14.6.0 → 14.6.1-beta202408252305.0

package/README.md

@@ -427,6 +427,19 @@ Example
     console.log(JSON.stringify(linter.lintResult));
 ```
 
+## Troubleshooting
+
+If you have issues with MegaLinter, you can [report it on the repository](https://github.com/nvuillam/npm-groovy-lint/issues)
+
+To help reproducing, you can access advanced logs using DEBUG env variables
+
+Examples:
+
+- `DEBUG=npm-groovy-lint npm-groovy-lint ....`
+- `DEBUG=npm-groovy-lint,npm-groovy-lint-trace npm-groovy-lint ....`
+
+If you want to see what happens in CodeNarc Server, you can clone the repo and run locally `npm server:run` before running npm-groovy-lint: you'll see the live logs of the to CodeNarc Server for npm-groovy-lint.
+
 ## Contribute
 
 Contributions are very welcome !

package/lib/codenarc-caller.js

@@ -1,20 +1,23 @@
 // Call CodeNarc by server or java
-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 c = require("chalk");
+import axios from "axios";
+import * as cliProgress from "cli-progress";
+import Debug from "debug";
+const debug = Debug("npm-groovy-lint");
+const trace = Debug("npm-groovy-lint-trace");
+import { JavaCaller } from "java-caller";
+import { optionsDefinition } from "./options.js";
+import { performance } from "node:perf_hooks";
+import c from "chalk";
+import findJavaHome from "find-java-home";
+import * as path from "path";
+import { fileURLToPath } from "node:url";
 
 // Request over IPv4 because Java typically prefers it.
-const http = require("http");
+import http from "http";
 axios.defaults.httpAgent = new http.Agent({ family: 4, keepAlive: true });
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
-class CodeNarcCaller {
-    "use strict";
-
+export class CodeNarcCaller {
     args = [];
     options;
     codenarcArgs;
@@ -34,14 +37,14 @@ class CodeNarcCaller {
             minimumJavaVersion: 17,
             maximumJavaVersion: 17,
             rootPath: __dirname,
-            jar: "java/CodeNarcServer.jar"
+            jar: "java/CodeNarcServer.jar",
         },
         codeNarcJava: {
             minimumJavaVersion: 17,
             maximumJavaVersion: 17,
             rootPath: __dirname,
-            jar: "java/CodeNarcServer.jar"
-        }
+            jar: "java/CodeNarcServer.jar",
+        },
     };
 
     constructor(codenarcArgs1, serverStatus1, args1, options1, execOpts1) {
@@ -69,9 +72,9 @@ class CodeNarcCaller {
                 codeNarcExcludes: this.execOpts.codeNarcExcludes,
                 parse: this.options.parse !== false && this.execOpts.onlyCodeNarc === false,
                 fileList: this.execOpts.groovyFileName ? [this.execOpts.groovyFileName] : this.execOpts.inputFileList,
-                requestKey: this.execOpts.requestKey || null
+                requestKey: this.execOpts.requestKey || null,
             },
-            timeout: 600000
+            timeout: 600000,
         };
         trace(`CALL CodeNarcServer with ${JSON.stringify(axiosConfig, null, 2)}`);
         let response;
@@ -98,8 +101,8 @@ class CodeNarcCaller {
                     error: {
                         msg: `exception: ${e.response.data.exceptionType} message: ${e.response.data.errorMessage}`,
                         stack: e.stack,
-                        responseData: e.response.data.errorDtl
-                    }
+                        responseData: e.response.data.errorDtl,
+                    },
                 };
             } else if (e.code === "ECONNRESET") {
                 // The server was shutdown just retry.
@@ -111,13 +114,16 @@ class CodeNarcCaller {
                 // respObj.status = 'cancelledByDuplicateRequest'
                 // respObj.statusCode = 444
                 return {
-                    status: 9
+                    status: 9,
                 };
             } else {
                 console.error(
                     c.red(
-                        "CodeNarcServer unexpected error:\n" + JSON.stringify(e, null, 2) + "\n" + JSON.stringify(e.response?.data?.errorDtl, null, 2)
-                    )
+                        "CodeNarcServer unexpected error:\n" +
+                            JSON.stringify(e, null, 2) +
+                            "\n" +
+                            JSON.stringify(e.response?.data?.errorDtl, null, 2),
+                    ),
                 );
             }
             this.serverStatus = "error";
@@ -126,8 +132,8 @@ class CodeNarcCaller {
                 error: {
                     msg: e.message,
                     stack: e.stack,
-                    responseData: e.response?.data?.errorDtl
-                }
+                    responseData: e.response?.data?.errorDtl,
+                },
             };
         }
 
@@ -139,7 +145,7 @@ class CodeNarcCaller {
                 parseErrors: response.data.parseErrors,
                 codeNarcStdOut: response.data.stdout,
                 codeNarcStdErr: undefined,
-                status: 0
+                status: 0,
             };
         }
 
@@ -148,7 +154,7 @@ class CodeNarcCaller {
             return {
                 codeNarcStdOut: undefined,
                 codeNarcStdErr: undefined,
-                status: 9
+                status: 9,
             };
         }
 
@@ -165,9 +171,9 @@ class CodeNarcCaller {
                 msgDtl: {
                     parseErrors: response.data.parseErrors,
                     stdout: response.data.stdout,
-                    stderr: response.data.errorDtl
-                }
-            }
+                    stderr: response.data.errorDtl,
+                },
+            },
         };
     }
 
@@ -183,7 +189,7 @@ class CodeNarcCaller {
         if (this.execOpts.groovyFileName) {
             scriptArgs.unshift("--file", this.execOpts.groovyFileName);
         } else if (this.execOpts.inputFileList) {
-            this.execOpts.inputFileList.forEach(file => {
+            this.execOpts.inputFileList.forEach((file) => {
                 scriptArgs.unshift("--file", file);
             });
         }
@@ -194,9 +200,9 @@ class CodeNarcCaller {
             {
                 format: "[{bar}] Running CodeNarc for {duration_formatted}",
                 hideCursor: true,
-                clearOnComplete: true
+                clearOnComplete: true,
             },
-            cliProgress.Presets.shades_classic
+            cliProgress.Presets.shades_classic,
         );
         this.bar.start(10, 1);
         this.barTimer = setInterval(() => {
@@ -232,22 +238,19 @@ class CodeNarcCaller {
                     reason =
                         "It seems node.js has not been found on your computer. Please install a recent node.js: https://nodejs.org/en/download/\nIf node is already installed, make sure your PATH contains node installation folder: https://love2dev.com/blog/node-is-not-recognized-as-an-internal-or-external-command/";
                 } else {
-                    await new Promise(resolve => {
-                        require("find-java-home")(err => {
-                            if (err) {
-                                reason =
-                                    "Java is required to run npm-groovy-lint, as CodeNarc is written in Java/Groovy. Please install Java (version 8 minimum) https://www.java.com/download";
-                            }
-                            resolve();
-                        });
+                    await findJavaHome({ allowJre: true }, (err) => {
+                        if (err) {
+                            reason =
+                                "Java is required to run npm-groovy-lint, as CodeNarc is written in Java/Groovy. Please install Java (version 8 minimum) https://www.java.com/download";
+                        }
                     });
                 }
                 return {
                     codeNarcStdErr: javaResult.stderr,
                     status: 2,
                     error: {
-                        msg: `Fatal error while calling CodeNarc\n${reason}\n${javaResult.stderr}`
-                    }
+                        msg: `Fatal error while calling CodeNarc\n${reason}\n${javaResult.stderr}`,
+                    },
                 };
             }
         }
@@ -259,7 +262,7 @@ class CodeNarcCaller {
             parseErrors: response.parseErrors,
             codeNarcStdOut: javaResult.stdout,
             codeNarcStdErr: javaResult.stderr,
-            status: 0
+            status: 0,
         };
     }
 
@@ -295,16 +298,16 @@ class CodeNarcCaller {
         const start = performance.now();
         let notified = false;
         let interval;
-        await new Promise(resolve => {
+        await new Promise((resolve) => {
             interval = setInterval(() => {
                 debug(
                     `pinging CodeNarcServer at ${serverPingUri} notified: ${notified}, serverStatus: ${
                         this.serverStatus
-                    }, since: ${performance.now() - start}, maxAttemptTimeMs: ${maxAttemptTimeMs}`
+                    }, since: ${performance.now() - start}, maxAttemptTimeMs: ${maxAttemptTimeMs}`,
                 );
                 axios
                     .get(serverPingUri)
-                    .then(response => {
+                    .then((response) => {
                         if (response.status === 200) {
                             // Server is correctly started, as he replied to the ping request
                             this.serverStatus = "running";
@@ -322,7 +325,7 @@ class CodeNarcCaller {
                             resolve();
                         }
                     })
-                    .catch(e => {
+                    .catch((e) => {
                         debug(`Ping code: ${e.code} message: ${e.message}`);
                         let since = performance.now() - start;
                         if (notified === false && this.serverStatus === "unknown" && since > maxAttemptTimeMs) {
@@ -399,15 +402,15 @@ class CodeNarcCaller {
         const serverPingUri = this.getCodeNarcServerUri() + "/ping";
 
         let interval;
-        await new Promise(resolve => {
+        await new Promise((resolve) => {
             interval = setInterval(() => {
                 debug(`pinging CodeNarcServer at ${serverPingUri} serverStatus: ${this.serverStatus}`);
                 axios
                     .get(serverPingUri)
-                    .then(response => {
+                    .then((response) => {
                         debug(`ping response: ${response.status}`);
                     })
-                    .catch(e => {
+                    .catch((e) => {
                         debug(`Ping code: ${e.code} message: ${e.message}`);
                         clearInterval(interval);
                         resolve();
@@ -438,9 +441,7 @@ class CodeNarcCaller {
         try {
             return JSON.parse(response);
         } catch (e) {
-            return { err: `Unable to parse ${response}` };
+            return { err: `Unable to parse ${response}: ${e.message}` };
         }
     }
 }
-
-module.exports = CodeNarcCaller;

package/lib/codenarc-factory.js

@@ -1,21 +1,21 @@
 // Shared functions
-"use strict";
-
-const debug = require("debug")("npm-groovy-lint");
-const commondir = require("commondir");
-const fs = require("fs-extra");
-const os = require("os");
-const path = require("path");
-const { getConfigFileName } = require("./config.js");
-const { collectDisabledBlocks, isFilteredError } = require("./filter.js");
-const { getNpmGroovyLintRules } = require("./groovy-lint-rules.js");
-const { evaluateRange, evaluateRangeFromLine, evaluateVariables, getSourceLines, normalizeNewLines } = require("./utils.js");
 
+import Debug from "debug";
+const debug = Debug("npm-groovy-lint");
+import commondir from "commondir";
+import fs from "fs-extra";
+import * as os from "os";
+import * as path from "path";
+import { getConfigFileName } from "./config.js";
+import { collectDisabledBlocks, isFilteredError } from "./filter.js";
+import { getNpmGroovyLintRules } from "./groovy-lint-rules.js";
+import { evaluateRange, evaluateRangeFromLine, evaluateVariables, getSourceLines, normalizeNewLines } from "./utils.js";
+import { fileURLToPath } from "url";
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
 ////////////////////////////
 // Build codenarc options //
 ////////////////////////////
 
-const npmGroovyLintRules = getNpmGroovyLintRules();
 const CODENARC_TMP_FILENAME_BASE = "codeNarcTmpDir_";
 const CODENARC_WWW_BASE = "https://codenarc.github.io/CodeNarc";
 
@@ -57,7 +57,7 @@ async function prepareCodeNarcCall(options) {
             cnFiles = ["**/" + tmpFileNm];
         }
 
-        await fs.writeFile(result.tmpGroovyFileName, normalizeNewLines(options.source)).catch(err => {
+        await fs.writeFile(result.tmpGroovyFileName, normalizeNewLines(options.source)).catch((err) => {
             throw new Error(`Unable to write temp file: ${err}`); // Ensure we have a stack trace.
         });
 
@@ -89,17 +89,17 @@ async function prepareCodeNarcCall(options) {
         const filePatterns =
             options.ext && options.ext.length > 0
                 ? // Convert extensions with or without leading slash into an ant pattern.
-                  options.ext.map(ext => ext.replace(/^\.?/u, "*."))
+                  options.ext.map((ext) => ext.replace(/^\.?/u, "*."))
                 : defaultFilesPattern;
         const sourceFiles = [];
         const includes = [];
-        positionalArgs.filter(Boolean).forEach(pathname => {
+        positionalArgs.filter(Boolean).forEach((pathname) => {
             const absolutePath = path.resolve(".", pathname);
             const relativePath = path.relative(codeNarcBaseDir, absolutePath);
             if (directoryExists(absolutePath)) {
                 // Directory: convert into ant patterns.
                 const patternBase = antPath(relativePath) + "**/";
-                includes.push(...filePatterns.map(filePattern => patternBase + filePattern));
+                includes.push(...filePatterns.map((filePattern) => patternBase + filePattern));
             } else if (fs.existsSync(absolutePath)) {
                 // File: add to file / patterns list.
                 const file = antPath(relativePath);
@@ -127,7 +127,7 @@ async function prepareCodeNarcCall(options) {
         result.codeNarcIncludes = cnFiles;
     } else {
         // If files not sent, use defaultFilesPattern, guessed from options.rulesets value
-        const filePatterns = defaultFilesPattern.map(filePattern => `**/${filePattern}`);
+        const filePatterns = defaultFilesPattern.map((filePattern) => `**/${filePattern}`);
         result.codenarcArgs.push(`-includes=${filePatterns.join(",")}`);
         result.codeNarcIncludes = filePatterns;
     }
@@ -149,23 +149,13 @@ 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
-            .split(".")
-            .pop()
-            .endsWith("html")
-            ? "html"
-            : result.output
-                  .split(".")
-                  .pop()
-                  .endsWith("xml")
-            ? "xml"
-            : "";
+        result.outputType = result.output.split(".").pop().endsWith("html") ? "html" : result.output.split(".").pop().endsWith("xml") ? "xml" : "";
         const ext = result.output.split(".").pop();
         result.codenarcArgs.push(`-report=${ext}:${result.output}`);
 
@@ -178,7 +168,7 @@ async function prepareCodeNarcCall(options) {
         const errMsg = `Output not managed: ${result.output}. (For now, only output formats are txt and json in console, and html and xml as files)`;
         console.error(errMsg);
         result.error = {
-            msg: errMsg
+            msg: errMsg,
         };
     }
     return result;
@@ -197,10 +187,10 @@ function antPath(path) {
 // Calculate longest base dir by analyzing the list of files
 async function getCodeNarcBaseDirFromFiles(positionalArgs) {
     // All arguments are not files
-    if (!positionalArgs.every(fileOrDirOrPattern => fs.existsSync(fileOrDirOrPattern) || directoryExists(fileOrDirOrPattern))) {
+    if (!positionalArgs.every((fileOrDirOrPattern) => fs.existsSync(fileOrDirOrPattern) || directoryExists(fileOrDirOrPattern))) {
         return process.cwd();
     }
-    const folders = positionalArgs.map(fileOrDir => {
+    const folders = positionalArgs.map((fileOrDir) => {
         // Dir
         if (directoryExists(fileOrDir)) {
             return path.resolve(fileOrDir);
@@ -221,10 +211,11 @@ async function parseCodeNarcResult(options, codeNarcBaseDir, codeNarcJsonResult,
         return {
             status: 2,
             error: {
-                msg: errMsg
-            }
+                msg: errMsg,
+            },
         };
     }
+    const npmGroovyLintRules = await getNpmGroovyLintRules();
     const result = { summary: {} };
 
     // Parse main result
@@ -264,13 +255,13 @@ async function parseCodeNarcResult(options, codeNarcBaseDir, codeNarcJsonResult,
                     line: parseError.cause && parseError.cause.line ? parseError.cause.line : 0,
                     rule: "NglParseError",
                     severity: "error",
-                    msg: msg
+                    msg: msg,
                 };
                 // Add range if provided
                 if (parseError.cause && parseError.cause.startColumn) {
                     errItemParse.range = {
                         start: { line: parseError.cause.startLine, character: parseError.cause.startColumn },
-                        end: { line: parseError.cause.endLine, character: parseError.cause.endColumn }
+                        end: { line: parseError.cause.endLine, character: parseError.cause.endColumn },
                     };
                 }
 
@@ -310,7 +301,7 @@ async function parseCodeNarcResult(options, codeNarcBaseDir, codeNarcJsonResult,
                     rule: violation.ruleName,
                     severity:
                         violation.priority === 1 ? "error" : violation.priority === 2 ? "warning" : violation.priority === 3 ? "info" : "unknown",
-                    msg: violation.message ? violation.message : ""
+                    msg: violation.message ? violation.message : "",
                 };
                 errItem.msg = tmpGroovyFileNameReplace ? errItem.msg.replace(tmpGroovyFileNameReplace, "") : errItem.msg;
 
@@ -371,7 +362,7 @@ async function parseCodeNarcResult(options, codeNarcBaseDir, codeNarcJsonResult,
             // Add description from CodeNarc
             rules[ruleName] = { description: ruleDef.description };
             // Try to build codenarc url (ex: https://codenarc.github.io/CodeNarc/codenarc-rules-basic.html#bitwiseoperatorinconditional-rule )
-            const matchRules = grooylintrcAllRules.filter(ruleNameX => ruleNameX.split(".")[1] === ruleName);
+            const matchRules = grooylintrcAllRules.filter((ruleNameX) => ruleNameX.split(".")[1] === ruleName);
             if (matchRules && matchRules[0]) {
                 const ruleCategory = matchRules[0].split(".")[0];
                 const ruleDocUrl = `${CODENARC_WWW_BASE}/codenarc-rules-${ruleCategory}.html#${ruleName.toLowerCase()}-rule`;
@@ -389,7 +380,7 @@ async function buildRuleSets(options) {
     // If RuleSet files has already been created, or is groovy file, return it
     if (options.rulesets && (options.rulesets.endsWith(".groovy") || options.rulesets.endsWith(".xml"))) {
         const rulesetSplits = options.rulesets.split(",");
-        const normalizedRulesets = rulesetSplits.map(rulesetFile => {
+        const normalizedRulesets = rulesetSplits.map((rulesetFile) => {
             const fullFile = path.resolve(rulesetFile);
             // Encode file name so CodeNarc understands it
             if (fs.existsSync(fullFile)) {
@@ -416,8 +407,8 @@ async function buildRuleSets(options) {
         !options.rulesets.includes("/") &&
         !options.rulesets.includes("\\")
     ) {
-        let ruleList = options.rulesets.split(/(,(?!"))/gm).filter(elt => elt !== ",");
-        ruleSetsDef = ruleList.map(ruleFromArgument => {
+        let ruleList = options.rulesets.split(/(,(?!"))/gm).filter((elt) => elt !== ",");
+        ruleSetsDef = ruleList.map((ruleFromArgument) => {
             let ruleName;
             let ruleOptions = {};
             if (ruleFromArgument.includes("{")) {
@@ -434,8 +425,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) {
@@ -449,15 +440,15 @@ async function buildRuleSets(options) {
         for (const ruleName of Object.keys(options.rules)) {
             let ruleDef = options.rules[ruleName];
             // If rule has been overridden in argument, set it on top of config file properties
-            const ruleFromRuleSetsArgPos = ruleSetsDef.findIndex(ruleDef => ruleDef.ruleName === ruleName);
+            const ruleFromRuleSetsArgPos = ruleSetsDef.findIndex((ruleDef) => ruleDef.ruleName === ruleName);
             if (ruleFromRuleSetsArgPos > -1) {
                 const ruleFromRuleSetsArg = ruleSetsDef[ruleFromRuleSetsArgPos];
                 ruleDef =
                     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)) {
@@ -528,4 +519,4 @@ function directoryExists(resolvedPath) {
     }
 }
 
-module.exports = { prepareCodeNarcCall, parseCodeNarcResult };
+export { prepareCodeNarcCall, parseCodeNarcResult };

package/lib/config.js

@@ -1,18 +1,20 @@
 // Configuration file management
-"use strict";
 
-const debug = require("debug")("npm-groovy-lint");
-const fse = require("fs-extra");
-const importFresh = require("import-fresh");
-const path = require("path");
-const stripComments = require("strip-json-comments");
+import Debug from "debug";
+const debug = Debug("npm-groovy-lint");
+import fs from "fs-extra";
+import importFresh from "import-fresh";
+import * as path from "path";
+import stripJsonComments from "strip-json-comments";
+import { fileURLToPath } from "url";
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 const defaultConfigLintFileName = ".groovylintrc-recommended.json";
 const allConfigLintFileName = ".groovylintrc-all.json";
 
 const NPM_GROOVY_LINT_CONSTANTS = {
     CodeNarcVersion: "2.2.0",
-    GroovyVersion: "3.0.9"
+    GroovyVersion: "3.0.9",
 };
 
 const configLintFilenames = [
@@ -22,7 +24,7 @@ const configLintFilenames = [
     ".groovylintrc.yml",
     ".groovylintrc.yaml",
     ".groovylintrc",
-    "package.json"
+    "package.json",
 ];
 
 const configExtensions = ["json", "js", "cjs", "yml", "yaml", "groovylintrc"];
@@ -45,7 +47,7 @@ async function loadConfig(startPathOrFile, mode = "lint", sourcefilepath, fileNa
         configUser = await loadConfigFromFile(startPathOrFile);
     } else if (startPathOrFile.match(/^[a-zA-Z\d-_]+$/) && mode !== "format") {
         // Sent string: find a corresponding file name
-        fileNames = configExtensions.map(ext => `.groovylintrc-${startPathOrFile}.${ext}`);
+        fileNames = configExtensions.map((ext) => `.groovylintrc-${startPathOrFile}.${ext}`);
         configFilePath = await getConfigFileName(sourcefilepath || process.cwd(), sourcefilepath, fileNames, "");
         configUser = await loadConfigFromFile(configFilePath);
     } else {
@@ -66,7 +68,7 @@ async function loadConfig(startPathOrFile, mode = "lint", sourcefilepath, fileNa
         // Set ruleSet file if found from config file
         configUser.rulesets = configUser.codenarcRulesets
             .split(",")
-            .map(rulesetFile => path.resolve(path.dirname(configFilePath) + "/" + rulesetFile))
+            .map((rulesetFile) => path.resolve(path.dirname(configFilePath) + "/" + rulesetFile))
             .join(",");
     }
 
@@ -117,7 +119,7 @@ async function getConfigFileName(startPathOrFile, sourcefilepath, fileNames = co
     // Find one of the config file formats are the root of the linted file (if source is sent with sourcefilepath)
     if ([".", process.cwd()].includes(startPathOrFile) && sourcefilepath) {
         try {
-            const stat = await fse.lstat(sourcefilepath);
+            const stat = await fs.lstat(sourcefilepath);
             const dir = stat.isDirectory() ? sourcefilepath : path.parse(sourcefilepath).dir;
             configFilePath = await findConfigInPath(dir, fileNames);
         } catch (e) {
@@ -127,7 +129,7 @@ async function getConfigFileName(startPathOrFile, sourcefilepath, fileNames = co
     // Find one of the config file formats at the root of the project or at upper directory levels
     if (configFilePath == null) {
         try {
-            const stat = await fse.lstat(startPathOrFile);
+            const stat = await fs.lstat(startPathOrFile);
             const dir = stat.isDirectory ? startPathOrFile : path.parse(startPathOrFile).dir;
             configFilePath = await findConfigInPath(dir, fileNames);
         } catch (e) {
@@ -154,13 +156,14 @@ async function getConfigFileName(startPathOrFile, sourcefilepath, fileNames = co
 async function findConfigInPath(directoryPath, configFilenamesIn) {
     for (const filename of configFilenamesIn) {
         const filePath = path.join(directoryPath, filename);
-        if (await fse.exists(filePath)) {
+        if (await fs.exists(filePath)) {
             if (filename === "package.json") {
                 try {
                     await loadPackageJSONConfigFile(filePath);
                     return filePath;
                 } catch (error) {
                     /* ignore */
+                    debug("Error loading JSON config file: " + error.message);
                 }
             } else {
                 return filePath;
@@ -217,15 +220,15 @@ async function loadJSConfigFile(filePath) {
 // JSON format
 async function loadJSONConfigFile(filePath) {
     try {
-        const fileContent = await readFile(filePath);
-        return JSON.parse(stripComments(fileContent));
+        const fileContent = await fs.readFile(filePath);
+        return JSON.parse(stripJsonComments(fileContent.toString()));
     } catch (e) {
         debug(`Error reading JSON file: ${filePath}`);
         e.message = `Cannot read config file: ${filePath}\nError: ${e.message}`;
         e.messageTemplate = "failed-to-read-json";
         e.messageData = {
             path: filePath,
-            message: e.message
+            message: e.message,
         };
         throw e;
     }
@@ -234,7 +237,7 @@ async function loadJSONConfigFile(filePath) {
 // YAML format
 async function loadYAMLConfigFile(filePath) {
     // lazy load YAML to improve performance when not used
-    const yaml = require("js-yaml");
+    const yaml = await import("js-yaml");
 
     try {
         // empty YAML file can be null, so always use
@@ -264,7 +267,7 @@ async function loadPackageJSONConfigFile(filePath) {
 
 // Read file
 async function readFile(filePath) {
-    const fileContent = await fse.readFile(filePath, "utf8");
+    const fileContent = await fs.readFile(filePath, "utf8");
     return fileContent.replace(/^\ufeff/u, "");
 }
 
@@ -278,4 +281,4 @@ async function shortenRuleNames(rules) {
     return shortenedRules;
 }
 
-module.exports = { NPM_GROOVY_LINT_CONSTANTS, loadConfig, getConfigFileName, overriddenRules };
+export { NPM_GROOVY_LINT_CONSTANTS, loadConfig, getConfigFileName, overriddenRules };