perf: support # in path only for dependencies

vitejs · Mar 17, 2023 · 4c111af · 4c111af
1 parent 1a8af8d
commit 4c111af
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/packages/vite/src/node/plugins/resolve.ts b/packages/vite/src/node/plugins/resolve.ts
@@ -474,11 +474,14 @@ function tryFsResolve(
 ): string | undefined {
   const { file, postfix } = splitFileAndPostfix(fsPath)
 
+  // Dependencies like es5-ext use `#` in their paths. We don't support `#` in user
+  // source code so we only need to perform the check for dependencies.
+  const trySharp = fsPath.includes('#') && fsPath.includes('node_modules')
+
   let res: string | undefined
 
-  // if there is a postfix, try resolving it as a complete path first (#4703)
   if (
-    postfix &&
+    trySharp &&
     (res = tryResolveFile(
       fsPath,
       '',
@@ -508,7 +511,7 @@ function tryFsResolve(
 
   for (const ext of options.extensions) {
     if (
-      postfix &&
+      trySharp &&
       (res = tryResolveFile(
         fsPath + ext,
         '',
@@ -543,7 +546,7 @@ function tryFsResolve(
   if (!tryIndex) return
 
   if (
-    postfix &&
+    trySharp &&
     (res = tryResolveFile(
       fsPath,
       '',