Markdoc's formatter is experimental and breaks things so use a regex

docs/markdoc/load-all.ts

@@ -7,8 +7,7 @@ export async function loadAllMarkdoc() {
   return await Promise.all(
     files.map(async file => {
       const contents = await fs.readFile(file, 'utf8');
-      const root = Markdoc.parse(contents, file);
-      return { file, root };
+      return { file, contents };
     })
   );
 }

docs/markdoc/remove-next-release-conditions.test.ts

@@ -10,14 +10,13 @@ some content
 
 {% /if %}
   `;
-  expect(removeNextReleaseConditions(Markdoc.parse(content))).toMatchInlineSnapshot(`
+  expect(removeNextReleaseConditions(content)).toMatchInlineSnapshot(`
     {
       "contents": "## Heading 1
-
     some content
 
     ## Some heading
-    ",
+      ",
       "errors": [],
     }
   `);
@@ -36,14 +35,13 @@ Content in else
 
 {% /if %}
   `;
-  expect(removeNextReleaseConditions(Markdoc.parse(content))).toMatchInlineSnapshot(`
+  expect(removeNextReleaseConditions(content)).toMatchInlineSnapshot(`
     {
       "contents": "## Heading 1
-
     some content
 
     ## Some heading
-    ",
+      ",
       "errors": [],
     }
   `);
@@ -65,16 +63,16 @@ Content in else
 
 {% /if %}
   `;
-  expect(removeNextReleaseConditions(Markdoc.parse(content))).toMatchInlineSnapshot(`
+  expect(removeNextReleaseConditions(content)).toMatchInlineSnapshot(`
     {
       "contents": "## Heading 1
 
     {% $nextRelease %}
-
+      
     some content
 
     ## Some heading
-    ",
+      ",
       "errors": [
         {
           "error": {

docs/markdoc/remove-next-release-conditions.ts

@@ -15,19 +15,18 @@ function transform(node: Node): Node[] | Node {
   return elseTag === -1 ? node.children : node.children.slice(0, elseTag);
 }
 
-export function removeNextReleaseConditions(root: Node) {
-  const transformed = new Ast.Node(
-    root.type,
-    root.attributes,
-    root.children.flatMap(transform),
-    root.tag
-  );
-  const contents = Markdoc.__EXPERIMENTAL__format(transformed);
+const pattern =
+  /{%\s+if\s+\$nextRelease\s+%}\s*([^]+?)\s*(?:{%\s+else\s+\/%}[^]*?)?{%\s+\/if\s+%}/g;
+
+export function removeNextReleaseConditions(contents: string) {
+  // ideally this would be a transform
+  // but Markdoc's formatter is experimental and as of the time of writing this
+  // doesn't seem to break some of our content
+  const newContent = contents.replace(pattern, '$1');
 
-  // we parse it again before validating so that positions are correct
-  const reparsed = Markdoc.parse(contents);
   // this will report usages of variables in case the transform here is broken
-  const errors = Markdoc.validate(reparsed, baseMarkdocConfig);
+  const parsed = Markdoc.parse(newContent);
+  const errors = Markdoc.validate(parsed, baseMarkdocConfig);
 
-  return { contents, errors };
+  return { contents: newContent, errors };
 }

docs/pages/docs/fields/virtual.md

@@ -13,36 +13,43 @@ Options:
   Defines what the Admin UI should fetch from this field, it's interpolated into a query like this:
   ```graphql
   query {
-  item(where: { id: "..." }) {
-    field${ui.query}
-  }
-}
   ```
+
+item(where: { id: "..." }) {
+field${ui.query}
+}
+
+```
+
+}
+
+```
+
 This is only needed when you your field returns a GraphQL type other than a scalar(String and etc.)
-  or an enum or you need to provide arguments to the field.
+or an enum or you need to provide arguments to the field.
 
 ```typescript
 import { config, createSchema, graphql, list } from '@keystone-6/core';
 import { virtual } from '@keystone-6/core/fields';
 
 export default config({
-  lists: {
-    SomeListName: list({
-      fields: {
-        someFieldName: virtual({
-          field: graphql.field({
-            type: graphql.String,
-            args: { something: graphql.arg({ type: graphql.Int }) },
-            resolve(item, args, context, info) {
-
-            }
-          })
-        }),
-        /* ... */
-      },
-    }),
-    /* ... */
-  },
+lists: {
+  SomeListName: list({
+    fields: {
+      someFieldName: virtual({
+        field: graphql.field({
+          type: graphql.String,
+          args: { something: graphql.arg({ type: graphql.Int }) },
+          resolve(item, args, context, info) {
+
+          }
+        })
+      }),
+      /* ... */
+    },
+  }),
   /* ... */
+},
+/* ... */
 });
 ```

docs/remove-conditionals.ts

@@ -8,8 +8,8 @@ import { removeNextReleaseConditions } from './markdoc/remove-next-release-condi
   const docs = await loadAllMarkdoc();
   const allErrors: ValidateError[] = [];
   await Promise.all(
-    docs.map(({ file, root }) => {
-      const { contents, errors } = removeNextReleaseConditions(root);
+    docs.map(({ file, contents: initialContents }) => {
+      const { contents, errors } = removeNextReleaseConditions(initialContents);
       allErrors.push(...errors);
       return fs.writeFile(file, contents, 'utf8');
     })

docs/validate-links.ts

@@ -14,7 +14,8 @@ const NON_MARKDOWN_PAGES = [
 ];
 
 (async () => {
-  const parsedFiles = (await loadAllMarkdoc()).map(({ file, root }) => {
+  const parsedFiles = (await loadAllMarkdoc()).map(({ file, contents }) => {
+    const root = Markdoc.parse(contents, file);
     const ids = new Set<string>();
     for (const node of root.walk()) {
       if (node.type === 'heading') {