perf: use throwIfNoEntry on Node 14+

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

Original file line number	Diff line number	Diff line change
`@@ -243,8 +243,9 @@ 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`	`+ return !!(path && fs.statSync(path, { throwIfNoEntry: false })?.isFile())`
`247`	`247`	`} catch {`
	`248`	`+ // Node 12 does not support throwIfNoEntry.`
`248`	`249`	`return false`
`249`	`250`	`}`
`250`	`251`	`}`