perf: don't re-download package tarball to global cache if local node_modules folder exists for package (#16005)

1 files changed, 12 insertions, 0 deletions

diff --git a/cli/npm/cache.rs b/cli/npm/cache.rs
index a733ecceb..6b841501a 100644
--- a/cli/npm/cache.rs
+++ b/cli/npm/cache.rs
@@ -199,6 +199,18 @@ impl NpmCache {
     dist: &NpmPackageVersionDistInfo,
     registry_url: &Url,
   ) -> Result<(), AnyError> {
+    self
+      .ensure_package_inner(id, dist, registry_url)
+      .await
+      .with_context(|| format!("Failed caching npm package '{}'.", id))
+  }
+
+  async fn ensure_package_inner(
+    &self,
+    id: &NpmPackageId,
+    dist: &NpmPackageVersionDistInfo,
+    registry_url: &Url,
+  ) -> Result<(), AnyError> {
     let package_folder = self.readonly.package_folder(id, registry_url);
     if package_folder.exists()
       // if this file exists, then the package didn't successfully extract