wip

Author: DavidAnson

doc-build/md043.md

@@ -96,7 +96,7 @@ to the previous matching marker":
 
 The simple form shown above works for most scenarios; if multiple loops are
 needed, the value `"^^"`, `"^^^"`, etc. can be used in conjunction. To avoid
-infinite loops, matching fails if there are more than 10,000 loops.
+infinite loops, matching fails if there are more than 1,000 loops.
 
 When an error is detected, this rule outputs the line number of the first
 problematic heading (otherwise, it outputs the last line number of the file).

doc/Rules.md

@@ -1941,7 +1941,7 @@ to the previous matching marker":
 
 The simple form shown above works for most scenarios; if multiple loops are
 needed, the value `"^^"`, `"^^^"`, etc. can be used in conjunction. To avoid
-infinite loops, matching fails if there are more than 10,000 loops.
+infinite loops, matching fails if there are more than 1,000 loops.
 
 When an error is detected, this rule outputs the line number of the first
 problematic heading (otherwise, it outputs the last line number of the file).

doc/md043.md

@@ -107,7 +107,7 @@ to the previous matching marker":
 
 The simple form shown above works for most scenarios; if multiple loops are
 needed, the value `"^^"`, `"^^^"`, etc. can be used in conjunction. To avoid
-infinite loops, matching fails if there are more than 10,000 loops.
+infinite loops, matching fails if there are more than 1,000 loops.
 
 When an error is detected, this rule outputs the line number of the first
 problematic heading (otherwise, it outputs the last line number of the file).

lib/md043.mjs

@@ -1,6 +1,6 @@
 // @ts-check
 
-import { addErrorContext, addErrorDetailIf } from "../helpers/helpers.cjs";
+import { addError, addErrorContext, addErrorDetailIf } from "../helpers/helpers.cjs";
 import { getHeadingLevel, getHeadingText } from "../helpers/micromark-helpers.cjs";
 import { filterByTypesCached } from "./cache.mjs";
 
@@ -40,6 +40,7 @@ export default {
     const requiredHeadings = headings.map((heading) => headingsItemsRe.test(heading) ? heading : "[Invalid syntax]");
     const matchCase = params.config.match_case || false;
     let i = 0;
+    let loopsRemaining = 1000;
     let matchAny = false;
     let hasError = false;
     let anyHeadings = false;
@@ -62,6 +63,7 @@ export default {
               if (requiredHeadings[j] === expected) {
                 i = j + 1;
                 expected = getExpected();
+                loopsRemaining--;
                 break;
               }
             }
@@ -72,7 +74,10 @@ export default {
           }
         }
 
-        if (expected === "*") {
+        if (loopsRemaining < 0) {
+          addError(onError, heading.startLine, "Too many loops");
+          hasError = true;
+        } else if (expected === "*") {
           const nextExpected = getExpected();
           if (!headingIsExpected(actual, nextExpected, matchCase)) {
             matchAny = true;