npm-groovy-lint 15.0.2 → 15.0.3-beta202502221045.0

package/lib/codenarc-factory.js

@@ -241,12 +241,30 @@ async function parseCodeNarcResult(options, codeNarcBaseDir, codeNarcJsonResult,
             if (files[fileNm] == null) {
                 files[fileNm] = { errors: [] };
             }
-            for (const parseError of fileParseErrors) {
+            for (const parseMessage of fileParseErrors) {
                 // Convert GroovyShell.parse Compilation exception error into NpmGroovyLint exception
-                let msg =
-                    parseError.cause && parseError.cause.message ? parseError.cause.message : `Unknown parsing error: ${JSON.stringify(parseError)}`;
+
+                let parseError = {
+                    cause: {
+                        message: null
+                    }
+                }
+                /* NOTE parsing messages like this
+                '/path/to/Pipeline.groovy: 2: unable to resolve class org.sm.Something\n' +
+                    ' @ line 2, column 1.\n' +
+                    '   import org.sm.Something\n' +
+                    '   ^\n',
+                 */
+                const parts = /(.*):\s(\d+):\s(.*)/s.exec(parseMessage)
+                if(!parts || parts.length < 4) {
+                    parseError.cause.message = `Unknown parsing error: ${JSON.stringify(parseMessage)}`;
+                } else {
+                    parseError.cause.line = parts[2]
+                    parseError.cause.message = parts[3]
+                }
+
                 // Remove 'unable to resolve class' error as GroovyShell.parse is called without ClassPath
-                if (msg.startsWith("unable to resolve class ")) {
+                if (parseError.cause.message.includes("unable to resolve class ")) {
                     continue;
                 }
                 // Create new error
@@ -255,15 +273,18 @@ async function parseCodeNarcResult(options, codeNarcBaseDir, codeNarcJsonResult,
                     line: parseError.cause && parseError.cause.line ? parseError.cause.line : 0,
                     rule: "NglParseError",
                     severity: "error",
-                    msg: msg,
+                    msg: parseError.cause.message,
                 };
+                // TODO would need a more complicated regex to parse for a "range"
+                // might not be really needed in this case for npm-groovy-lint
+                //
                 // 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 },
-                    };
-                }
+                //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 },
+                //    };
+                //}
 
                 files[fileNm].errors.push(errItemParse);
                 errId++;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "npm-groovy-lint",
-  "version": "15.0.2",
+  "version": "15.0.3-beta202502221045.0",
   "description": "Lint, format and auto-fix your Groovy / Jenkinsfile / Gradle files",
   "exports": "./lib/groovy-lint.js",
   "type": "module",
@@ -72,11 +72,11 @@
     "@eslint/eslintrc": "^3.1.0",
     "@eslint/js": "^9.9.1",
     "adm-zip": "^0.5.10",
-    "diff": "^5.0.0",
+    "diff": "^7.0.0",
     "eslint": "^9.0.0",
-    "globals": "^15.9.0",
+    "globals": "^16.0.0",
     "handlebars": "^4.7.8",
-    "mocha": "^10.2.0",
+    "mocha": "^11.0.0",
     "npm-run-all2": "^6.0.0",
     "nyc": "^17.0.0",
     "prettier": "^3.0.0",