diff options
Diffstat (limited to 'ext/node')
-rw-r--r-- | ext/node/lib.rs | 20 | ||||
-rw-r--r-- | ext/node/package_json.rs | 4 | ||||
-rw-r--r-- | ext/node/resolution.rs | 18 |
3 files changed, 21 insertions, 21 deletions
diff --git a/ext/node/lib.rs b/ext/node/lib.rs index db4fe3178..da8ca3003 100644 --- a/ext/node/lib.rs +++ b/ext/node/lib.rs @@ -30,7 +30,7 @@ pub trait NodePermissions { fn check_read(&mut self, path: &Path) -> Result<(), AnyError>; } -pub trait DenoDirNpmResolver { +pub trait RequireNpmResolver { fn resolve_package_folder_from_package( &self, specifier: &str, @@ -63,7 +63,7 @@ struct Unstable(pub bool); pub fn init<P: NodePermissions + 'static>( unstable: bool, - maybe_npm_resolver: Option<Rc<dyn DenoDirNpmResolver>>, + maybe_npm_resolver: Option<Rc<dyn RequireNpmResolver>>, ) -> Extension { Extension::builder() .js(include_js_files!( @@ -121,7 +121,7 @@ where P: NodePermissions + 'static, { let resolver = { - let resolver = state.borrow::<Rc<dyn DenoDirNpmResolver>>(); + let resolver = state.borrow::<Rc<dyn RequireNpmResolver>>(); resolver.clone() }; if resolver.ensure_read_permission(file_path).is_ok() { @@ -287,7 +287,7 @@ fn op_require_resolve_deno_dir( parent_filename: String, ) -> Option<String> { check_unstable(state); - let resolver = state.borrow::<Rc<dyn DenoDirNpmResolver>>(); + let resolver = state.borrow::<Rc<dyn RequireNpmResolver>>(); resolver .resolve_package_folder_from_package( &request, @@ -300,7 +300,7 @@ fn op_require_resolve_deno_dir( #[op] fn op_require_is_deno_dir_package(state: &mut OpState, path: String) -> bool { check_unstable(state); - let resolver = state.borrow::<Rc<dyn DenoDirNpmResolver>>(); + let resolver = state.borrow::<Rc<dyn RequireNpmResolver>>(); resolver.in_npm_package(&PathBuf::from(path)) } @@ -470,7 +470,7 @@ fn op_require_try_self( return Ok(None); } - let resolver = state.borrow::<Rc<dyn DenoDirNpmResolver>>().clone(); + let resolver = state.borrow::<Rc<dyn RequireNpmResolver>>().clone(); let pkg = resolution::get_package_scope_config( &Url::from_file_path(parent_path.unwrap()).unwrap(), &*resolver, @@ -552,7 +552,7 @@ fn op_require_resolve_exports( parent_path: String, ) -> Result<Option<String>, AnyError> { check_unstable(state); - let resolver = state.borrow::<Rc<dyn DenoDirNpmResolver>>().clone(); + let resolver = state.borrow::<Rc<dyn RequireNpmResolver>>().clone(); let pkg_path = if resolver.in_npm_package(&PathBuf::from(&modules_path)) { modules_path @@ -594,7 +594,7 @@ where state, PathBuf::from(&filename).parent().unwrap(), )?; - let resolver = state.borrow::<Rc<dyn DenoDirNpmResolver>>().clone(); + let resolver = state.borrow::<Rc<dyn RequireNpmResolver>>().clone(); resolution::get_closest_package_json( &Url::from_file_path(filename).unwrap(), &*resolver, @@ -607,7 +607,7 @@ fn op_require_read_package_scope( package_json_path: String, ) -> Option<PackageJson> { check_unstable(state); - let resolver = state.borrow::<Rc<dyn DenoDirNpmResolver>>().clone(); + let resolver = state.borrow::<Rc<dyn RequireNpmResolver>>().clone(); let package_json_path = PathBuf::from(package_json_path); PackageJson::load(&*resolver, package_json_path).ok() } @@ -624,7 +624,7 @@ where check_unstable(state); let parent_path = PathBuf::from(&parent_filename); ensure_read_permission::<P>(state, &parent_path)?; - let resolver = state.borrow::<Rc<dyn DenoDirNpmResolver>>().clone(); + let resolver = state.borrow::<Rc<dyn RequireNpmResolver>>().clone(); let pkg = PackageJson::load(&*resolver, parent_path.join("package.json"))?; if pkg.imports.is_some() { diff --git a/ext/node/package_json.rs b/ext/node/package_json.rs index ced64a1b4..95d773aeb 100644 --- a/ext/node/package_json.rs +++ b/ext/node/package_json.rs @@ -2,7 +2,7 @@ use crate::NodeModuleKind; -use super::DenoDirNpmResolver; +use super::RequireNpmResolver; use deno_core::anyhow; use deno_core::anyhow::bail; use deno_core::error::AnyError; @@ -44,7 +44,7 @@ impl PackageJson { } pub fn load( - resolver: &dyn DenoDirNpmResolver, + resolver: &dyn RequireNpmResolver, path: PathBuf, ) -> Result<PackageJson, AnyError> { resolver.ensure_read_permission(&path)?; diff --git a/ext/node/resolution.rs b/ext/node/resolution.rs index a7428fe03..52ed06116 100644 --- a/ext/node/resolution.rs +++ b/ext/node/resolution.rs @@ -15,7 +15,7 @@ use regex::Regex; use crate::errors; use crate::package_json::PackageJson; -use crate::DenoDirNpmResolver; +use crate::RequireNpmResolver; pub static DEFAULT_CONDITIONS: &[&str] = &["deno", "node", "import"]; pub static REQUIRE_CONDITIONS: &[&str] = &["require", "node"]; @@ -93,7 +93,7 @@ pub fn package_imports_resolve( referrer: &ModuleSpecifier, referrer_kind: NodeModuleKind, conditions: &[&str], - npm_resolver: &dyn DenoDirNpmResolver, + npm_resolver: &dyn RequireNpmResolver, ) -> Result<PathBuf, AnyError> { if name == "#" || name.starts_with("#/") || name.ends_with('/') { let reason = "is not a valid internal imports specifier name"; @@ -224,7 +224,7 @@ fn resolve_package_target_string( pattern: bool, internal: bool, conditions: &[&str], - npm_resolver: &dyn DenoDirNpmResolver, + npm_resolver: &dyn RequireNpmResolver, ) -> Result<PathBuf, AnyError> { if !subpath.is_empty() && !pattern && !target.ends_with('/') { return Err(throw_invalid_package_target( @@ -326,7 +326,7 @@ fn resolve_package_target( pattern: bool, internal: bool, conditions: &[&str], - npm_resolver: &dyn DenoDirNpmResolver, + npm_resolver: &dyn RequireNpmResolver, ) -> Result<Option<PathBuf>, AnyError> { if let Some(target) = target.as_str() { return Ok(Some(resolve_package_target_string( @@ -441,7 +441,7 @@ pub fn package_exports_resolve( referrer: &ModuleSpecifier, referrer_kind: NodeModuleKind, conditions: &[&str], - npm_resolver: &dyn DenoDirNpmResolver, + npm_resolver: &dyn RequireNpmResolver, ) -> Result<PathBuf, AnyError> { if package_exports.contains_key(&package_subpath) && package_subpath.find('*').is_none() @@ -592,7 +592,7 @@ pub fn package_resolve( referrer: &ModuleSpecifier, referrer_kind: NodeModuleKind, conditions: &[&str], - npm_resolver: &dyn DenoDirNpmResolver, + npm_resolver: &dyn RequireNpmResolver, ) -> Result<PathBuf, AnyError> { let (package_name, package_subpath, _is_scoped) = parse_package_name(specifier, referrer)?; @@ -656,7 +656,7 @@ pub fn package_resolve( pub fn get_package_scope_config( referrer: &ModuleSpecifier, - npm_resolver: &dyn DenoDirNpmResolver, + npm_resolver: &dyn RequireNpmResolver, ) -> Result<PackageJson, AnyError> { let root_folder = npm_resolver .resolve_package_folder_from_path(&referrer.to_file_path().unwrap())?; @@ -666,7 +666,7 @@ pub fn get_package_scope_config( pub fn get_closest_package_json( url: &ModuleSpecifier, - npm_resolver: &dyn DenoDirNpmResolver, + npm_resolver: &dyn RequireNpmResolver, ) -> Result<PackageJson, AnyError> { let package_json_path = get_closest_package_json_path(url, npm_resolver)?; PackageJson::load(npm_resolver, package_json_path) @@ -674,7 +674,7 @@ pub fn get_closest_package_json( fn get_closest_package_json_path( url: &ModuleSpecifier, - npm_resolver: &dyn DenoDirNpmResolver, + npm_resolver: &dyn RequireNpmResolver, ) -> Result<PathBuf, AnyError> { let file_path = url.to_file_path().unwrap(); let mut current_dir = file_path.parent().unwrap(); |