feat(no-irregular-whitespace): add error end positions

Author: FloEdelmann

lib/rules/no-irregular-whitespace.js

@@ -50,9 +50,9 @@ module.exports = {
    * @returns {RuleListener} AST event handlers.
    */
   create(context) {
-    // Module store of error indexes that we have found
-    /** @type {number[]} */
-    let errorIndexes = []
+    // Module store of error ranges that we have found
+    /** @type {{start: number, end: number}[]} */
+    let errorIndices = []
 
     // Lookup the `skipComments` option, which defaults to `false`.
     const options = context.options[0] || {}
@@ -74,8 +74,8 @@ module.exports = {
     function removeWhitespaceError(node) {
       const [startIndex, endIndex] = node.range
 
-      errorIndexes = errorIndexes.filter(
-        (errorIndex) => errorIndex < startIndex || endIndex <= errorIndex
+      errorIndices = errorIndices.filter(
+        (error) => error.start < startIndex || endIndex <= error.start
       )
     }
 
@@ -155,16 +155,19 @@ module.exports = {
       const source = sourceCode.getText()
       let match
       while ((match = IRREGULAR_WHITESPACE.exec(source)) !== null) {
-        errorIndexes.push(match.index)
+        errorIndices.push({
+          start: match.index,
+          end: match.index + match[0].length
+        })
       }
       while ((match = IRREGULAR_LINE_TERMINATORS.exec(source)) !== null) {
-        errorIndexes.push(match.index)
+        errorIndices.push({ start: match.index, end: match.index + 1 })
       }
     }
 
     checkForIrregularWhitespace()
 
-    if (errorIndexes.length === 0) {
+    if (errorIndices.length === 0) {
       return {}
     }
     const bodyVisitor = utils.defineTemplateBodyVisitor(context, {
@@ -212,16 +215,19 @@ module.exports = {
         const [templateStart, templateEnd] = templateBody
           ? templateBody.range
           : [0, 0]
-        errorIndexes = errorIndexes.filter(
-          (errorIndex) =>
-            (scriptStart <= errorIndex && errorIndex < scriptEnd) ||
-            (templateStart <= errorIndex && errorIndex < templateEnd)
+        errorIndices = errorIndices.filter(
+          (error) =>
+            (scriptStart <= error.start && error.start < scriptEnd) ||
+            (templateStart <= error.start && error.start < templateEnd)
         )
 
         // If we have any errors remaining, report on them
-        for (const errorIndex of errorIndexes) {
+        for (const { start, end } of errorIndices) {
           context.report({
-            loc: sourceCode.getLocFromIndex(errorIndex),
+            loc: {
+              start: sourceCode.getLocFromIndex(start),
+              end: sourceCode.getLocFromIndex(end)
+            },
             messageId: 'disallow'
           })
         }