refactor(npm): remove needless resolve_nv_ref_from_pkg_req_ref on NpmResolver (#19582)

2 files changed, 6 insertions, 9 deletions

diff --git a/ext/node/lib.rs b/ext/node/lib.rs
index e77b3c089..d144d89ca 100644
--- a/ext/node/lib.rs
+++ b/ext/node/lib.rs
@@ -17,9 +17,7 @@ use deno_fs::sync::MaybeSync;
 use deno_npm::resolution::PackageReqNotFoundError;
 use deno_npm::NpmPackageId;
 use deno_semver::npm::NpmPackageNv;
-use deno_semver::npm::NpmPackageNvReference;
 use deno_semver::npm::NpmPackageReq;
-use deno_semver::npm::NpmPackageReqReference;
 use once_cell::sync::Lazy;
 
 pub mod analyze;
@@ -93,11 +91,6 @@ pub trait NpmResolver: std::fmt::Debug + MaybeSend + MaybeSync {
     req: &NpmPackageReq,
   ) -> Result<NpmPackageId, PackageReqNotFoundError>;
 
-  fn resolve_nv_ref_from_pkg_req_ref(
-    &self,
-    req_ref: &NpmPackageReqReference,
-  ) -> Result<NpmPackageNvReference, PackageReqNotFoundError>;
-
   fn in_npm_package(&self, specifier: &ModuleSpecifier) -> bool;
 
   fn in_npm_package_at_path(&self, path: &Path) -> bool {
diff --git a/ext/node/resolution.rs b/ext/node/resolution.rs
index 966491505..6db2a9655 100644
--- a/ext/node/resolution.rs
+++ b/ext/node/resolution.rs
@@ -310,9 +310,13 @@ impl NodeResolver {
     mode: NodeResolutionMode,
     permissions: &dyn NodePermissions,
   ) -> Result<Option<NodeResolution>, AnyError> {
-    let reference = self
+    let pkg_id = self
       .npm_resolver
-      .resolve_nv_ref_from_pkg_req_ref(reference)?;
+      .resolve_pkg_id_from_pkg_req(&reference.req)?;
+    let reference = NpmPackageNvReference {
+      nv: pkg_id.nv,
+      sub_path: reference.sub_path.clone(),
+    };
     self.resolve_npm_reference(&reference, mode, permissions)
   }