fix: address review issues from PR #1213

- Use String.startsWith instead of String.includes for blog route
  filtering to avoid accidentally excluding non-blog routes that
  contain 'blog' as a substring

- Replace JsExn.throw with JsError.throwWithMessage in
  BlogArticleRoute for consistency with BlogApi.res and to get
  proper Error objects with stack traces

- Normalize path separators in MdxFile.scanDir to fix Windows
  compatibility where Node.Path.join2 produces backslashes

Author: jderochervlk

app/routes.res

@@ -35,7 +35,7 @@ let blogArticleRoutes =
 
 let mdxRoutes =
   mdxRoutes("./routes/MdxRoute.jsx")->Array.filter(r =>
-    !(r.path->Option.map(String.includes(_, "blog"))->Option.getOr(false))
+    !(r.path->Option.map(String.startsWith(_, "blog"))->Option.getOr(false))
   )
 
 let default = [

app/routes/BlogArticleRoute.res

@@ -15,7 +15,7 @@ let loader: ReactRouter.Loader.t<loaderData> = async ({request}) => {
 
   let frontmatter = switch BlogFrontmatter.decode(frontmatter) {
   | Ok(fm) => fm
-  | Error(msg) => JsExn.throw(msg)
+  | Error(msg) => JsError.throwWithMessage(msg)
   }
 
   let archived = filePath->String.includes("/archived/")

src/MdxFile.res

@@ -28,7 +28,11 @@ let rec scanDir = (baseDir, currentDir) => {
       scanDir(baseDir, fullPath)
     } else if Node.Path.extname(entry) === ".mdx" {
       // Get the relative path from baseDir
-      let relativePath = fullPath->String.replace(baseDir ++ "/", "")->String.replace(".mdx", "")
+      let relativePath =
+        fullPath
+        ->String.replaceAll("\\", "/")
+        ->String.replace(baseDir->String.replaceAll("\\", "/") ++ "/", "")
+        ->String.replace(".mdx", "")
       [relativePath]
     } else {
       []

vitest.config.mjs

@@ -10,8 +10,8 @@ export default defineConfig({
     setupFiles: ["./vitest.setup.mjs"],
     browser: {
       enabled: true,
-      provider: playwright(),
       // https://vitest.dev/config/browser/playwright
+      provider: playwright(),
       instances: [
         {
           browser: "chromium",