perf: improve isFileReadable performance (#12397)

vitejs · Mar 30, 2023 · acf3a14 · acf3a14
1 parent a59914c
commit acf3a14
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/packages/vite/src/node/utils.ts b/packages/vite/src/node/utils.ts
@@ -501,7 +501,14 @@ export function generateCodeFrame(
 
 export function isFileReadable(filename: string): boolean {
   try {
+    // The "throwIfNoEntry" is a performance optimization for cases where the file does not exist
+    if (!fs.statSync(filename, { throwIfNoEntry: false })) {
+      return false
+    }
+
+    // Check if current process has read permission to the file
     fs.accessSync(filename, fs.constants.R_OK)
+
     return true
   } catch {
     return false