remove Critical dependency: the request of a dependency is an expression warnings (#3330)

* patch `@typescript/vfs` to remove `Critical dependency: the request of a dependency is an expression` warnings

* fix

* Update .changeset/dry-seas-pull.md

* aa

* prettier

Author: dimaMachina

.changeset/dry-seas-pull.md

@@ -0,0 +1,5 @@
+---
+'nextra': patch
+---
+
+remove `Critical dependency: the request of a dependency is an expression` warnings

packages/nextra/src/server/index.ts

@@ -20,6 +20,8 @@ const DEFAULT_EXTENSIONS = ['js', 'jsx', 'ts', 'tsx']
 
 const AGNOSTIC_PAGE_MAP_PATH = `.next${sep}static${sep}chunks${sep}nextra-page-map`
 
+const RE_SEP = sep === '/' ? '/' : '\\\\'
+
 const nextra: Nextra = nextraConfig => {
   const { error } = nextraConfigSchema.safeParse(nextraConfig)
   if (error) {
@@ -152,6 +154,14 @@ const nextra: Nextra = nextraConfig => {
           })
         }
 
+        const defaultLoaderOptions = [
+          options.defaultLoaders.babel,
+          {
+            loader: 'nextra/loader',
+            options: loaderOptions
+          }
+        ]
+
         rules.push(
           {
             // Match Markdown imports from non-pages. These imports have an
@@ -162,13 +172,7 @@ const nextra: Nextra = nextraConfig => {
             issuer: request =>
               (!!request && !request.includes(AGNOSTIC_PAGE_MAP_PATH)) ||
               request === null,
-            use: [
-              options.defaultLoaders.babel,
-              {
-                loader: 'nextra/loader',
-                options: loaderOptions
-              }
-            ]
+            use: defaultLoaderOptions
           },
           {
             // Match pages (imports without an issuer request).
@@ -198,15 +202,16 @@ const nextra: Nextra = nextraConfig => {
           },
           {
             // Use platform separator because /pages\/_app\./ will not work on windows
-            test: new RegExp(`pages${sep === '/' ? '/' : '\\\\'}_app\\.`),
+            test: new RegExp(`pages${RE_SEP}_app\\.`),
             issuer: request => !request,
-            use: [
-              options.defaultLoaders.babel,
-              {
-                loader: 'nextra/loader',
-                options: loaderOptions
-              }
-            ]
+            use: defaultLoaderOptions
+          },
+          {
+            test: new RegExp(
+              `@typescript${RE_SEP}vfs${RE_SEP}dist${RE_SEP}vfs\\.`
+            ),
+            issuer: request => !!request,
+            use: defaultLoaderOptions
           }
         )
 

packages/nextra/src/server/loader.ts

@@ -83,6 +83,14 @@ export async function loader(
 
   const currentPath = slash(mdxPath)
 
+  if (currentPath.includes('@typescript/vfs/dist/vfs.')) {
+    // Fixes https://github.com/microsoft/TypeScript-Website/pull/3022
+    // Fixes https://github.com/shuding/nextra/issues/3322#issuecomment-2384046618
+    return source
+      .replace(/String\.fromCharCode\(112, ?97, ?116, ?104\)/, '"path"')
+      .replace(/String\.fromCharCode\(102, ?115\)/, '"fs"')
+  }
+
   if (currentPath.includes('/pages/api/')) {
     logger.warn(
       `Ignoring ${currentPath} because it is located in the "pages/api" folder.`