fix: eslint 10 support

Author: BenoitZugmeyer

src/index.js

@@ -57,16 +57,21 @@ function patch(module) {
   }
   Linter[LINTER_ISPATCHED_PROPERTY_NAME] = true
 
-  const verifyMethodName = Linter.prototype._verifyWithoutProcessors
-    ? "_verifyWithoutProcessors" // ESLint 6+
-    : "verify" // ESLint 5-
-  const verify = Linter.prototype[verifyMethodName]
-  Linter.prototype[verifyMethodName] = createVerifyPatch(verify)
+  // ESLint >= 8.4.0
+  if (Linter.prototype._verifyWithFlatConfigArrayAndWithoutProcessors) {
+    Linter.prototype._verifyWithFlatConfigArrayAndWithoutProcessors =
+      createVerifyWithFlatConfigPatch(
+        module,
+        Linter.prototype._verifyWithFlatConfigArrayAndWithoutProcessors
+      )
+  }
 
-  const verifyWithFlatConfig =
-    Linter.prototype._verifyWithFlatConfigArrayAndWithoutProcessors
-  Linter.prototype._verifyWithFlatConfigArrayAndWithoutProcessors =
-    createVerifyWithFlatConfigPatch(module, verifyWithFlatConfig)
+  // ESLint >= 4.7.0 .. < 10.0
+  if (Linter.prototype._verifyWithoutProcessors) {
+    Linter.prototype._verifyWithoutProcessors = createVerifyPatch(
+      Linter.prototype._verifyWithoutProcessors
+    )
+  }
 }
 
 function getLinterFromModule(module) {

src/test/plugin.js

@@ -892,42 +892,36 @@ ifVersion(">= 5", describe, "compatibility with external HTML plugins", () => {
         "@html-eslint/require-img-alt": ["error"],
       },
     })
-    assert.deepStrictEqual(
-      messages.map((message) => ({
-        ...message,
-
-        // ESLint v8.54.0 adds suggestions for the no-console rule. As we are running tests on older
-        // versions of ESLint, we need to ignore these suggestions.
-        suggestions: "(ignored)",
-      })),
-      [
-        {
-          column: 1,
-          endColumn: 13,
-          endLine: 1,
-          line: 1,
-          message: "Missing `alt` attribute at `<img>` tag",
-          messageId: "missingAlt",
-          nodeType: null,
-          ruleId: "@html-eslint/require-img-alt",
-          severity: 2,
-          suggestions: "(ignored)",
-        },
-        {
-          column: 3,
-          endColumn: 14,
-          endLine: 3,
-          line: 3,
-          message: "Unexpected console statement.",
-          messageId: "unexpected",
-          nodeType: "MemberExpression",
-          ruleId: "no-console",
-          severity: 2,
-          source: '  console.log("toto")',
-          suggestions: "(ignored)",
-        },
-      ]
-    )
+    for (const message of messages) {
+      // ESLint v8.54.0 adds suggestions for the no-console rule. As we are running tests on older
+      // versions of ESLint, we need to ignore these suggestions.
+      delete message.suggestions
+      // ESLint v10.0 removed the nodeType property
+      delete message.nodeType
+    }
+    assert.deepStrictEqual(messages, [
+      {
+        column: 1,
+        endColumn: 13,
+        endLine: 1,
+        line: 1,
+        message: "Missing `alt` attribute at `<img>` tag",
+        messageId: "missingAlt",
+        ruleId: "@html-eslint/require-img-alt",
+        severity: 2,
+      },
+      {
+        column: 3,
+        endColumn: 14,
+        endLine: 3,
+        line: 3,
+        message: "Unexpected console statement.",
+        messageId: "unexpected",
+        ruleId: "no-console",
+        severity: 2,
+        source: '  console.log("toto")',
+      },
+    ])
   })
 
   it("fix", async () => {

src/verifyWithFlatConfigPatch.js

@@ -63,7 +63,10 @@ function createVerifyWithFlatConfigPatch(eslintModule, verifyWithFlatConfig) {
         rules: {
           [PREPARE_RULE_NAME]: {
             create(context) {
-              sourceCodes.set(codePart, context.getSourceCode())
+              sourceCodes.set(
+                codePart,
+                context.sourceCode || context.getSourceCode()
+              )
               return {
                 Program(program) {
                   if (prepare) {

src/verifyWithSharedScopes.js

@@ -62,7 +62,7 @@ function verifyWithSharedScopes(codeParts, verifyCodePart, parserOptions) {
 }
 
 function markGlobalVariableAsUsed(context, program, name) {
-  const sourceCode = context.getSourceCode()
+  const sourceCode = context.sourceCode || context.getSourceCode()
 
   if (sourceCode.markVariableAsUsed) {
     sourceCode.markVariableAsUsed(name, program)
@@ -72,7 +72,7 @@ function markGlobalVariableAsUsed(context, program, name) {
 }
 
 function getGlobalScope(context, program) {
-  const sourceCode = context.getSourceCode()
+  const sourceCode = context.sourceCode || context.getSourceCode()
   if (sourceCode.getScope) {
     // eslint 9+
     return sourceCode.getScope(program)