fix(flaws): ignore casing of text fragments (#9882)

* refactor(flaws): extract checkHash() * fix(flaws): ignore fragment directives in hashs
mdn · Oct 26, 2023 · fc51e08 · fc51e08
1 parent 58b33b9
commit fc51e08
Showing 1 changed file with 23 additions and 20 deletions.
diff --git a/build/flaws/broken-links.ts b/build/flaws/broken-links.ts
@@ -151,6 +151,26 @@ export function getBrokenLinksFlaws(
     });
   }
 
+  function checkHash(
+    hash: string,
+    a: cheerio.Cheerio<cheerio.Element>,
+    href: string
+  ) {
+    if (hash.startsWith(":~:")) {
+      // Ignore fragment directives.
+      return;
+    }
+    if (hash !== hash.toLowerCase()) {
+      addBrokenLink(
+        a,
+        checked.get(href),
+        href,
+        href.replace(`#${hash}`, `#${hash.toLowerCase()}`),
+        "Anchor not lowercase"
+      );
+    }
+  }
+
   $("a[href]").each((i, element) => {
     const a = $(element);
     let href = a.attr("href");
@@ -347,30 +367,13 @@ export function getBrokenLinksFlaws(
           href,
           found.url + absoluteURL.search + absoluteURL.hash.toLowerCase()
         );
-      } else if (
-        hrefSplit.length > 1 &&
-        hrefSplit[1] !== hrefSplit[1].toLowerCase()
-      ) {
+      } else if (hrefSplit.length > 1) {
         const hash = hrefSplit[1];
-        addBrokenLink(
-          a,
-          checked.get(href),
-          href,
-          href.replace(`#${hash}`, `#${hash.toLowerCase()}`),
-          "Anchor not lowercase"
-        );
+        checkHash(hash, a, href);
       }
     } else if (href.startsWith("#")) {
       const hash = href.split("#")[1];
-      if (hash !== hash.toLowerCase()) {
-        addBrokenLink(
-          a,
-          checked.get(href),
-          href,
-          href.replace(`#${hash}`, `#${hash.toLowerCase()}`),
-          "Anchor not lowercase"
-        );
-      }
+      checkHash(hash, a, href);
     }
   });