perf: use throwIfNoEntry on Node 14+

chenxinyanc · chenxinyanc · commit d112e36860a0 · 2022-09-07T21:01:09.000+08:00
diff --git a/src/index.ts b/src/index.ts
@@ -243,8 +243,10 @@ function removeQuerystring(id: string) {
 
 const isFile = (path?: string | undefined): path is string => {
   try {
-    return !!path && fs.statSync(path).isFile()
+    if (!path) return false
+    return fs.statSync(path, { throwIfNoEntry: false })?.isFile() || false
   } catch {
+    // Node 12 does not support throwIfNoEntry.
     return false
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -243,8 +243,10 @@ function removeQuerystring(id: string) {`
`243`	`243`
`244`	`244`	`const isFile = (path?: string \| undefined): path is string => {`
`245`	`245`	`try {`
`246`		`- return !!path && fs.statSync(path).isFile()`
	`246`	`+ if (!path) return false`
	`247`	`+ return fs.statSync(path, { throwIfNoEntry: false })?.isFile() \|\| false`
`247`	`248`	`} catch {`
	`249`	`+ // Node 12 does not support throwIfNoEntry.`
`248`	`250`	`return false`
`249`	`251`	`}`
`250`	`252`	`}`