refactor: replace duplicate code with tryStatSync

packages/vite/src/node/packages.ts

@@ -1,7 +1,12 @@
 import fs from 'node:fs'
 import path from 'node:path'
 import { createRequire } from 'node:module'
-import { createFilter, isInNodeModules, safeRealpathSync } from './utils'
+import {
+  createFilter,
+  isInNodeModules,
+  safeRealpathSync,
+  tryStatSync,
+} from './utils'
 import type { Plugin } from './plugin'
 
 let pnp: typeof import('pnpapi') | undefined
@@ -125,17 +130,17 @@ export function findNearestPackageData(
     }
 
     const pkgPath = path.join(basedir, 'package.json')
-    try {
-      if (fs.statSync(pkgPath, { throwIfNoEntry: false })?.isFile()) {
+    if (tryStatSync(pkgPath)?.isFile()) {
+      try {
         const pkgData = loadPackageData(pkgPath)
 
         if (packageCache) {
           setFnpdCache(packageCache, pkgData, basedir, originalBasedir)
         }
 
         return pkgData
-      }
-    } catch {}
+      } catch {}
+    }
 
     const nextBasedir = path.dirname(basedir)
     if (nextBasedir === basedir) break

packages/vite/src/node/utils.ts

@@ -386,6 +386,7 @@ export function isDefined<T>(value: T | undefined | null): value is T {
 
 export function tryStatSync(file: string): fs.Stats | undefined {
   try {
+    // The "throwIfNoEntry" is a performance optimization for cases where the file does not exist
     return fs.statSync(file, { throwIfNoEntry: false })
   } catch {
     // Ignore errors
@@ -501,12 +502,11 @@ 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
-    }
+  if (!tryStatSync(filename)) {
+    return false
+  }
 
+  try {
     // Check if current process has read permission to the file
     fs.accessSync(filename, fs.constants.R_OK)