From 6ce04845442e9ac4257875823c8de87736962b98 Mon Sep 17 00:00:00 2001
From: Ben McCann <322311+benmccann@users.noreply.github.com>
Date: Wed, 12 Oct 2022 13:59:00 -0700
Subject: [PATCH] fix: address code review comment

---
 packages/vite/src/node/plugins/resolve.ts | 28 +++++++++++++----------
 1 file changed, 16 insertions(+), 12 deletions(-)

diff --git a/packages/vite/src/node/plugins/resolve.ts b/packages/vite/src/node/plugins/resolve.ts
index fef9de49b78f57..ba14a2132724ad 100644
--- a/packages/vite/src/node/plugins/resolve.ts
+++ b/packages/vite/src/node/plugins/resolve.ts
@@ -648,28 +648,31 @@ export function tryNodeResolve(
     basedir = nestedResolveFrom(nestedRoot, basedir, preserveSymlinks)
   }
 
+  let nearestPkg: PackageData | undefined
   let pkg: PackageData | undefined
-  let pkgId: string
 
-  const rootPkg =
-    possiblePkgIds.length &&
-    resolvePackageData(
-      possiblePkgIds[0],
+  let pkgId: string = possiblePkgIds.reverse().find((pkgId) => {
+    nearestPkg = resolvePackageData(
+      pkgId,
       basedir,
       preserveSymlinks,
       packageCache
     )!
+    return nearestPkg
+  })!
+
+  const rootPkgId = possiblePkgIds[0]
+  const rootPkg =
+    possiblePkgIds.length &&
+    resolvePackageData(rootPkgId, basedir, preserveSymlinks, packageCache)!
   if (rootPkg && rootPkg?.data?.exports) {
     pkg = rootPkg
-    pkgId = possiblePkgIds[0]
+    pkgId = rootPkgId
   } else {
-    pkgId = possiblePkgIds.reverse().find((pkgId) => {
-      pkg = resolvePackageData(pkgId, basedir, preserveSymlinks, packageCache)!
-      return pkg
-    })!
+    pkg = nearestPkg
   }
 
-  if (!pkg) {
+  if (!pkg || !nearestPkg) {
     // if import can't be found, check if it's an optional peer dep.
     // if so, we can resolve to a special id that errors only when imported.
     if (
@@ -768,7 +771,8 @@ export function tryNodeResolve(
   }
 
   const ext = path.extname(resolved)
-  const isCJS = ext === '.cjs' || (ext === '.js' && pkg.data.type !== 'module')
+  const isCJS =
+    ext === '.cjs' || (ext === '.js' && nearestPkg.data.type !== 'module')
 
   if (
     !options.ssrOptimizeCheck &&