fix(docs): only rewrite .md links on anchor tags (#187)

The docs link plugin rewrote any href= in the rendered HTML, including
escaped markup inside code samples, so an example like <a href="./x.md">
in a code block was changed to a site route. It now only rewrites the
href of real <a> tags.

Author: erkamyaman

apps/docs/md-links.plugin.spec.ts

@@ -41,6 +41,16 @@ describe('resolveDocLinks', () => {
         '<a href="/agents/tools">c</a> <a href="https://github.com/x/README.md">d</a>',
     );
   });
+
+  it('leaves escaped markup in code and non-anchor attributes alone', () => {
+    const html =
+      '<pre><code>&lt;a href="./configuration.md"&gt;</code></pre> ' +
+      '<a data-href="./security.md" href="./security.md">a</a> <link href="./x.md">';
+    expect(resolveDocLinks(html, '/src/content/getting-started/installation')).toBe(
+      '<pre><code>&lt;a href="./configuration.md"&gt;</code></pre> ' +
+        '<a data-href="./security.md" href="/getting-started/security">a</a> <link href="./x.md">',
+    );
+  });
 });
 
 describe('mdLinksPlugin', () => {

apps/docs/md-links.plugin.ts

@@ -43,7 +43,7 @@ export function resolveMdHref(href: string, pageFile: string): string | null {
 
 export function resolveDocLinks(html: string, pageFile: string): string {
   return html.replace(
-    /(\shref=)(["'])([^"']*)\2/g,
+    /(<a\s(?:[^>]*?\s)?href=)(["'])([^"']*)\2/gi,
     (match, attr: string, quote: string, href: string) => {
       const route = resolveMdHref(href, pageFile);
       return route === null ? match : `${attr}${quote}${route}${quote}`;