diff options
author | David Sherret <dsherret@users.noreply.github.com> | 2023-02-22 20:16:16 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-22 20:16:16 -0500 |
commit | ddc350780d6743be3794896e3987361df9f2cde7 (patch) | |
tree | f267d2756ddd7c1ffef8e4c080e0530e542aed3f /cli/proc_state.rs | |
parent | a31d8869ea6c0651109523963a23f16101ce7e64 (diff) |
fix(npm): resolve node_modules dir relative to package.json instead of cwd (#17885)
Diffstat (limited to 'cli/proc_state.rs')
-rw-r--r-- | cli/proc_state.rs | 24 |
1 files changed, 7 insertions, 17 deletions
diff --git a/cli/proc_state.rs b/cli/proc_state.rs index 9f1e7320c..2070b14c6 100644 --- a/cli/proc_state.rs +++ b/cli/proc_state.rs @@ -224,9 +224,7 @@ impl ProcState { let npm_resolver = NpmPackageResolver::new_with_maybe_lockfile( npm_cache.clone(), api, - cli_options - .resolve_local_node_modules_folder() - .with_context(|| "Resolving local node_modules folder.")?, + cli_options.node_modules_dir_path(), cli_options.get_npm_resolution_snapshot(), lockfile.as_ref().cloned(), ) @@ -329,11 +327,7 @@ impl ProcState { self.file_fetcher.clone(), root_permissions, dynamic_permissions, - self - .options - .resolve_local_node_modules_folder() - .with_context(|| "Resolving local node_modules folder.")? - .map(|path| ModuleSpecifier::from_file_path(path).unwrap()), + self.options.node_modules_dir_specifier(), ); let maybe_imports = self.options.to_maybe_imports()?; let graph_resolver = self.resolver.as_graph_resolver(); @@ -632,25 +626,21 @@ impl ProcState { } /// Creates the default loader used for creating a graph. - pub fn create_graph_loader(&self) -> Result<cache::FetchCacher, AnyError> { - Ok(cache::FetchCacher::new( + pub fn create_graph_loader(&self) -> cache::FetchCacher { + cache::FetchCacher::new( self.emit_cache.clone(), self.file_fetcher.clone(), PermissionsContainer::allow_all(), PermissionsContainer::allow_all(), - self - .options - .resolve_local_node_modules_folder() - .with_context(|| "Resolving local node_modules folder.")? - .map(|path| ModuleSpecifier::from_file_path(path).unwrap()), - )) + self.options.node_modules_dir_specifier(), + ) } pub async fn create_graph( &self, roots: Vec<ModuleSpecifier>, ) -> Result<deno_graph::ModuleGraph, AnyError> { - let mut cache = self.create_graph_loader()?; + let mut cache = self.create_graph_loader(); self.create_graph_with_loader(roots, &mut cache).await } |