tidying up

Author: jderochervlk

app/routes/ApiRoute.res

@@ -111,8 +111,6 @@ let loader: ReactRouter.Loader.t<loaderData> = async args => {
   | _ => parseApi(await Node.Fs.readFile("./markdown-pages/docs/api/stdlib.json", "utf-8"))
   }
 
-  let stdlibToc = apiDocs->Dict.get(basePath)
-
   let toctree = groupItems(apiDocs)
 
   let data = {

app/routes/SyntaxLookupRoute.res

@@ -15,7 +15,7 @@ let convert = (mdx: Mdx.attributes): SyntaxLookup.item => {
   }
 }
 
-let loader: Loader.t<loaderData> = async ({request}) => {
+let loader: Loader.t<loaderData> = async _ => {
   let allMdx = await Shims.runWithoutLogging(() => loadAllMdx())
 
   let mdxSources =

patches/react-router-mdx+1.0.8.patch

@@ -1,13 +1,12 @@
 diff --git a/node_modules/react-router-mdx/dist/server/index.js b/node_modules/react-router-mdx/dist/server/index.js
-index 1e56a6c..5f08c1c 100644
+index 1e56a6c..b323b05 100644
 --- a/node_modules/react-router-mdx/dist/server/index.js
 +++ b/node_modules/react-router-mdx/dist/server/index.js
-@@ -59,15 +59,16 @@ import matter from "gray-matter";
+@@ -59,15 +59,15 @@ import matter from "gray-matter";
  import { compile as compileMdx } from "@mdx-js/mdx";
  import remarkFrontmatter from "remark-frontmatter";
  var compile = async (content, options) => {
 -  const compiled = await compileMdx(content, {
-+  // TODO: open a PR to add in the path!
 +  const compiled = await compileMdx({ value: content, path: options?.path }, {
      ...options ?? {},
      outputFormat: "function-body",
@@ -23,11 +22,9 @@ index 1e56a6c..5f08c1c 100644
    return attributes;
  };
 
-@@ -118,13 +119,15 @@ var loadMdx = async (request, options) => {
-   const aliases = getAliases(getOptions());
+@@ -119,12 +119,13 @@ var loadMdx = async (request, options) => {
    const path = getFilePathBasedOnUrl(request.url, paths, aliases);
    const content = await getFileContent(path);
-+  console.log({ path })
    const [mdxContent, attributes] = await Promise.all([
 -    compile(content, options),
 +    compile(content, { ...options, path }),

scripts/generate_llms.res

@@ -68,7 +68,8 @@ let createSmallFile = (content: string, filePath: string): unit => {
   Node.Fs.appendFileSync(filePath, smallContent, "utf8")
 }
 
-let createLlmsFiles = (version: string, docsDirectory: string, llmsDirectory: string): unit => {
+// TODO: post RR7 - refactor to remove the version parameter, as it's unused
+let createLlmsFiles = (_version: string, docsDirectory: string, llmsDirectory: string): unit => {
   let mdxFileTemplatePath = llmsDirectory->Node.Path.join2("template.mdx")
   let mdxFilePath = docsDirectory->Node.Path.join2("llms.mdx")
   let txtFileTemplatePath = llmsDirectory->Node.Path.join2("template.txt")

src/ApiDocs.res

@@ -128,11 +128,6 @@ module SidebarTree = {
       // This is the full path to this module, e.g. "/docs/manual/api/stdlib/array" or "/docs/manual/api/stdlib/int"
       let fullPath = `${moduleRoute}/${relativePath}`->Url.normalizePath
 
-      let parentPath = switch node.path->Array.join("/")->Url.normalizePath {
-      | "" => None
-      | path => Some(path)
-      }
-
       let isCurrentRoute = fullPath == (location.pathname :> string)
 
       let classNameActive = isCurrentRoute ? classNameActive : ""
@@ -181,12 +176,6 @@ module SidebarTree = {
       }
     }
 
-    let {pathname} = ReactRouter.useLocation()
-
-    let url = (pathname :> string)->Url.parse->Some
-
-    let onChange = evt => ()
-
     let preludeSection =
       <div className="flex justify-between text-fire font-medium items-baseline">
         <VersionSelect />
@@ -301,11 +290,6 @@ let make = (props: props) => {
 
   let toggleSidebar = () => setSidebarOpen(prev => !prev)
 
-  let title = switch props {
-  | Ok({module_: {id}}) => id
-  | _ => "API"
-  }
-
   let children = {
     open Markdown
     switch props {
@@ -402,8 +386,6 @@ module Data = {
   let getVersion = (~moduleName: string) => {
     open Node
 
-    let pathModule = Path.join([dir, `${moduleName}.json`])
-
     let moduleContent =
       Fs.readFileSync(`markdown-pages/docs/api/${moduleName}.json`)->JSON.parseOrThrow
 

src/Blog.res

@@ -257,8 +257,6 @@ let make = (~posts: array<BlogApi.post>, ~category: category): React.element =>
     </>
   }
 
-  let (isOverlayOpen, setOverlayOpen) = React.useState(() => false)
-
   <>
     <div className="mt-16 pt-2">
       <div className="text-gray-80 text-18">

src/BlogArticle.res

@@ -101,7 +101,7 @@ module BlogHeader = {
 }
 
 let make = (props: props) => {
-  let {children, isArchived, path, frontmatter} = props
+  let {children, isArchived, frontmatter} = props
 
   let archivedNote = isArchived
     ? {

src/Mdx.res

@@ -129,12 +129,16 @@ let remarkReScriptPrelude = tree => {
 let remarkLinkPlugin = (tree, vfile) => {
   visit(tree, "link", node => {
     let url = node["url"]
-    let filePath =
-      vfile["history"][0]->Option.getOrThrow(
-        ~message=`File path not found for vfile: ${JSON.stringifyAny(vfile)->Option.getOr(
-            "unknown vfile",
-          )}`,
+
+    // The dev server behaves differently than production builds, so we need to not fail here
+    let filePath = switch (vfile["history"][0], Env.dev) {
+    | (Some(path), _) => path
+    | (None, false) =>
+      JsExn.throw(
+        `File path not found for vfile: ${JSON.stringifyAny(vfile)->Option.getOr("unknown vfile")}`,
       )
+    | (None, true) => ""
+    }
 
     // Direct links to the homepage are OK
     if url == "https://rescript-lang.org" {

src/Packages.res

@@ -399,8 +399,7 @@ let make = (props: props) => {
     </Category>
   }
 
-  let location = ReactRouter.useLocation()
-  let (searchParams, setSearchParams) = ReactRouter.useSearchParams()
+  let (_, setSearchParams) = ReactRouter.useSearchParams()
 
   // On first render, the router query is undefined so we set a flag.
   let firstRenderDone = React.useRef(false)
@@ -422,7 +421,6 @@ let make = (props: props) => {
     None
   }, [state])
 
-  let (isOverlayOpen, setOverlayOpen) = React.useState(() => false)
   <>
     <Meta
       siteName="ReScript Packages"

src/Playground.res

@@ -1503,7 +1503,7 @@ let initialReContent = `Js.log("Hello Reason 3.6!");`
 
 @react.component
 let make = (~bundleBaseUrl: string, ~versions: array<string>) => {
-  let (searchParams, setSearchParams) = ReactRouter.useSearchParams()
+  let (searchParams, _) = ReactRouter.useSearchParams()
 
   let versions =
     versions