fix: give a default baseUrl on markdown links (#71)

app/services/markdown_renderer.ts

@@ -47,10 +47,37 @@ export class MarkdownRenderer {
     })
   }
 
+  /**
+   * Rewrite markdown link tokens to include
+   * baseUrl when there isn't one.
+   */
+  #rewrite(tokens: any[], baseUrl: string) {
+    for (const token of tokens) {
+      if (token.type === 'link_open') {
+        for (const attr of token.attrs) {
+          if (attr[0] === 'href') {
+            const value: string = attr[1]
+            try {
+              new URL(value)
+            } catch (_) {
+              attr[1] = new URL(value, `${baseUrl}/`).href
+            }
+          }
+        }
+      }
+      if (token.children !== null) {
+        this.#rewrite(token.children, baseUrl)
+      }
+    }
+  }
+
   /**
    * Render the markdown to HTML
    */
-  render(markdown: string): string {
+  render(markdown: string, baseUrl: string): string {
+    this.#renderer.core.ruler.push('baseurl', (state) => {
+      this.#rewrite(state.tokens, baseUrl)
+    })
     const unsanitized = this.#renderer.render(markdown)
     return this.#sanitize(unsanitized)
   }

app/services/packages_fetcher.ts

@@ -178,7 +178,7 @@ export class PackagesFetcher {
 
     return {
       package: this.#mergePackageStatsAndInfo(pkg, stats),
-      readme: this.#markdownRenderer.render(readme),
+      readme: this.#markdownRenderer.render(readme, pkg.github),
     }
   }
 }