fix: pass options to parse instead of using global setOptions, which can affect tests

Author: chrispcampbell

packages/docs-builder/src/gen-html.spec.ts

@@ -130,7 +130,7 @@ describe('convertMarkdownToHtml', () => {
       '<p>This is -CO<sub>2</sub>-</p>\n'
     )
     expect(convertMarkdownToHtml(undefined, '# This is CO2')).toBe(
-      '<h1 id="this-is-co2">This is CO<sub>2</sub></h1>\n'
+      '<h1>This is CO<sub>2</sub></h1>\n'
     )
     expect(convertMarkdownToHtml(undefined, '> This is _CO2_')).toBe(
       '<blockquote>\n<p>This is <em>CO<sub>2</sub></em></p>\n</blockquote>\n'

packages/docs-builder/src/gen-html.ts

@@ -595,7 +595,9 @@ export function convertMarkdownToHtml(context: Context, md: string): string {
   })
 
   // Parse the Markdown into HTML
-  return marked.parse(md)
+  return marked.parse(md, {
+    headerIds: false
+  })
 }
 
 /**

packages/docs-builder/src/parse.ts

@@ -60,11 +60,6 @@ export function parseMarkdownPageContent(
   relPath: string,
   origMarkdownWithFrontmatter: string
 ): MarkdownPage {
-  // Configure marked.js
-  marked.setOptions({
-    headerIds: false
-  })
-
   // Separate frontmatter from the content
   const origMarkdownSeparated = matter(origMarkdownWithFrontmatter)
   const origMarkdown = origMarkdownSeparated.content