From 344317ec501fa124f0c74b44035fa4516999dce6 Mon Sep 17 00:00:00 2001
From: David Sherret <dsherret@users.noreply.github.com>
Date: Thu, 23 Feb 2023 10:58:10 -0500
Subject: feat(npm): support bare specifiers from package.json in more
 subcommands and language server (#17891)

---
 cli/npm/resolution/mod.rs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'cli/npm/resolution')

diff --git a/cli/npm/resolution/mod.rs b/cli/npm/resolution/mod.rs
index 8584958b5..53284d712 100644
--- a/cli/npm/resolution/mod.rs
+++ b/cli/npm/resolution/mod.rs
@@ -292,17 +292,18 @@ impl NpmResolution {
 
   pub async fn set_package_reqs(
     &self,
-    package_reqs: HashSet<NpmPackageReq>,
+    package_reqs: Vec<NpmPackageReq>,
   ) -> Result<(), AnyError> {
     let inner = &self.0;
     // only allow one thread in here at a time
     let _permit = inner.update_semaphore.acquire().await?;
     let snapshot = inner.snapshot.read().clone();
 
+    let reqs_set = package_reqs.iter().collect::<HashSet<_>>();
     let has_removed_package = !snapshot
       .package_reqs
       .keys()
-      .all(|req| package_reqs.contains(req));
+      .all(|req| reqs_set.contains(req));
     // if any packages were removed, we need to completely recreate the npm resolution snapshot
     let snapshot = if has_removed_package {
       NpmResolutionSnapshot::default()
@@ -311,7 +312,7 @@ impl NpmResolution {
     };
     let snapshot = add_package_reqs_to_snapshot(
       &inner.api,
-      package_reqs.into_iter().collect(),
+      package_reqs,
       snapshot,
       self.0.maybe_lockfile.clone(),
     )
-- 
cgit v1.2.3