improve stylelint textRange reporting

Author: jancajthaml

cli/helpers/lint-stream.mjs

@@ -31,6 +31,7 @@ export async function run(options) {
           "**/*.git*/**",
           "**/*node_modules/**",
           "**/*build/**",
+          "**/*reports/ut-coverage/**",
           "**/*dist/**",
           "**/*lcov-report/**",
           "**/*.min.js",
@@ -134,24 +135,27 @@ export async function run(options) {
       }
 
       for (const issue of issues) {
-        formattedResults.push({
+        const chunk = {
           ruleId: issue.ruleId || "formatting",
           engineId: issue.engineId,
           severity: ["INFO", "MINOR", "CRITICAL", "BLOCKER"][issue.severity],
           type: issue.severity > 1 ? "BUG" : "CODE_SMELL",
           primaryLocation: {
             message: issue.message,
             filePath: filePath,
-            textRange: issue.line
-              ? {
-                  startLine: issue.line,
-                  endLine: issue.endLine,
-                  startColumn: issue.column - 1,
-                  endColumn: issue.endColumn - 1,
-                }
-              : undefined,
           },
-        });
+        };
+
+        if (issue.line) {
+          chunk.primaryLocation.textRange = { startLine: issue.line };
+          if (issue.endLine !== undefined && issue.column !== undefined && issue.endColumn !== undefined) {
+            chunk.primaryLocation.textRange.startColumn = issue.column;
+            chunk.primaryLocation.textRange.endColumn = issue.endColumn;
+            chunk.primaryLocation.textRange.endLine = issue.endLine;
+          }
+        }
+
+        formattedResults.push(chunk);
       }
     }
 

cli/helpers/stylelint.mjs

@@ -88,22 +88,38 @@ export async function createEngine(options) {
       }
 
       info.issues.push(
-        ...results.parseErrors.map((item) => ({
-          engineId: "stylelint",
-          message: item.text,
-          ruleId: item.rule,
-          ...item,
-          severity: 2,
-        })),
+        ...results.parseErrors.map((item) => {
+          const chunk = Object.assign({
+            engineId: "stylelint",
+            message: item.text,
+            ruleId: item.rule,
+          }, item);
+
+          delete chunk.column;
+          delete chunk.endLine;
+          delete chunk.endColumn;
+
+          chunk.severity = 2;
+
+          return chunk;
+        }),
       );
       info.issues.push(
-        ...results.warnings.map((item) => ({
-          engineId: "stylelint",
-          message: item.text,
-          ruleId: item.rule,
-          ...item,
-          severity: 1,
-        })),
+        ...results.warnings.map((item) => {
+          const chunk = Object.assign({
+            engineId: "stylelint",
+            message: item.text,
+            ruleId: item.rule,
+          }, item);
+
+          delete chunk.column;
+          delete chunk.endLine;
+          delete chunk.endColumn;
+
+          chunk.severity = 1;
+
+          return chunk;
+        }),
       );
     } catch (error) {
       info.issues.push({

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lastui/rocker",
-  "version": "0.20.40",
+  "version": "0.20.41",
   "license": "Apache-2.0",
   "author": "[email protected]",
   "homepage": "https://github.com/lastui/rocker#readme",

webpack/config/module/production.js

@@ -119,7 +119,7 @@ module.exports = merge(require("../../internal/base.js"), require("../../interna
         type: "asset/source",
       },
       {
-        test: /\.(mp3|png|jpe?g|gif|ico)$/i,
+        test: /\.(mp3|png|jpe?g|gif|ico|svg)$/i,
         dependency: { not: ["url"] },
         type: "asset/inline",
       },